<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugzilla.yoctoproject.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.6"
          urlbase="https://bugzilla.yoctoproject.org/"
          
          maintainer="it-coreprojects-helpdesk@linuxfoundation.org"
>

    <bug>
          <bug_id>4905</bug_id>
          
          <creation_ts>2013-07-25 07:03:22 +0000</creation_ts>
          <short_desc>No proper location diagnostic for tab/space issues</short_desc>
          <delta_ts>2014-03-26 17:41:21 +0000</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>7</classification_id>
          <classification>Build System, Metadata &amp; Runtime</classification>
          <product>BitBake</product>
          <component>bitbake</component>
          <version>unspecified</version>
          <rep_platform>x86</rep_platform>
          <op_sys>Multiple</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>WONTFIX</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>Medium</priority>
          <bug_severity>normal</bug_severity>
          <target_milestone>1.6</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Laszlo Papp">lpapp</reporter>
          <assigned_to name="Richard Purdie">richard.purdie</assigned_to>
          <cc>jessica.zhang</cc>
    
    <cc>poky.bs.watcher</cc>
    
    <cc>poky.watcher</cc>
    
    <cc>ross.burton</cc>
          
          
          <cf_os>---</cf_os>
          <cf_regression_type>---</cf_regression_type>
          
          <cf_docchange>No (bug/feature does not impact docs)</cf_docchange>

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>34838</commentid>
    <comment_count>0</comment_count>
    <who name="Laszlo Papp">lpapp</who>
    <bug_when>2013-07-25 07:03:22 +0000</bug_when>
    <thetext>If you come from denzil to for instance dylan, you might get weird issues about tab/space as the enforcement was introduced in the meantime, perhaps even for good.

However, there are two issues in here:

1) You will get an error report for your recipe even if the error is inside the .inc required!

2) You will not get proper location, just a file name. It would need more fine tuning for better user experience.

I have been told the parser design is not prepared for it. That does not mean it should not be fixed. It just means that fixing the bad design might take longer.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>34843</commentid>
    <comment_count>1</comment_count>
    <who name="Ross Burton">ross.burton</who>
    <bug_when>2013-07-25 08:00:10 +0000</bug_when>
    <thetext>(2) isn&apos;t true, you get a function name.  Generally the problem isn&apos;t specific to any particular line but the entire function.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>34845</commentid>
    <comment_count>2</comment_count>
    <who name="Laszlo Papp">lpapp</who>
    <bug_when>2013-07-25 08:14:03 +0000</bug_when>
    <thetext>I do not consider a function proper. Especially when code gets pasted from somewhere in a more fine-tuned manner than a function.

I do consider it true.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>41790</commentid>
    <comment_count>3</comment_count>
    <who name="Richard Purdie">richard.purdie</who>
    <bug_when>2014-03-26 17:41:21 +0000</bug_when>
    <thetext>The issue is that the function can consist of expanded data. We therefore can&apos;t check at the point the function is first created since it can be subsequently modified.

We do have variable tracking code however we default to not running with it since it has significant performance overhead. We therefore have to make a design choice between the usability aspect and performance.

The cases where people run into that error are comparatively rare, most people have migrated from denzil at this point. We therefore chose to keep the performance and won&apos;t be able to fix this issue any time soon.</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>