<?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>11592</bug_id>
          
          <creation_ts>2017-05-30 10:24:10 +0000</creation_ts>
          <short_desc>Add QA check for uppercase recipe names</short_desc>
          <delta_ts>2017-09-15 01:56:42 +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>OE-Core</product>
          <component>core</component>
          <version>2.3</version>
          <rep_platform>x86</rep_platform>
          <op_sys>Multiple</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>Medium</priority>
          <bug_severity>enhancement</bug_severity>
          <target_milestone>2.4 M3</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Paul Eggleton">bluelightning</reporter>
          <assigned_to name="yeoh ee peng">ee.peng.yeoh</assigned_to>
          <cc>costamagnagianfranco</cc>
    
    <cc>dl9pf</cc>
    
    <cc>meta.mr.watcher</cc>
    
    <cc>meta.watcher</cc>
    
    <cc>sgw</cc>
    
    <cc>zuloch</cc>
          
          
          <cf_os>---</cf_os>
          <cf_regression_type>---</cf_regression_type>
          
          <cf_docchange>Yes (doc changes required)</cf_docchange>

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>73563</commentid>
    <comment_count>0</comment_count>
    <who name="Paul Eggleton">bluelightning</who>
    <bug_when>2017-05-30 10:24:10 +0000</bug_when>
    <thetext>Since we disabled uppercase characters in overrides a few releases ago, uppercase characters in recipe names (and for that matter, distro and machine names) cannot be supported due to their reliance upon overrides including the name. We should have a QA check and produce an error when an uppercase character is used in PN instead of leaving the user to figure it out from weird behaviour that will otherwise be the result.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>75107</commentid>
    <comment_count>1</comment_count>
    <who name="Lorenzo Chianura">zuloch</who>
    <bug_when>2017-07-18 13:23:24 +0000</bug_when>
    <thetext>IMHO this bug should have a higher priority, simply because:

- there is no place from where one could retrieve the information
  (talking about recipe&apos;s names in lowercase)

- from my own experience using uppercase characters in recipe&apos;s names
  will lead to any kind of problems: I was in charge of porting an entire
  layer from yocto 1.8 to yocto 2.2.1 and it took a LOT of time 
  (and lots of IRC sessions) to find out that all of my problems were
  caused by uppercase characters in recipe&apos;s names.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>75851</commentid>
    <comment_count>2</comment_count>
    <who name="yeoh ee peng">ee.peng.yeoh</who>
    <bug_when>2017-08-15 07:29:59 +0000</bug_when>
    <thetext>As suggested by Paul, I am currently working on enhancement inside insane.bbclass to check for package name which is not lowercase and provide warning message.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>76487</commentid>
    <comment_count>3</comment_count>
    <who name="yeoh ee peng">ee.peng.yeoh</who>
    <bug_when>2017-09-04 02:16:17 +0000</bug_when>
    <thetext>Patch to check recipe uppercase in insane.bbclass submitted to openembedded-core and it was accepted.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>76688</commentid>
    <comment_count>4</comment_count>
    <who name="Gianfranco">costamagnagianfranco</who>
    <bug_when>2017-09-08 14:33:15 +0000</bug_when>
    <thetext>WARNING: build/../sources/meta-foo/recipes-extended/socketcan/Can_git.bb: QA Issue: PN: Can is upper case, this can result in unexpected behavior. [uppercase-pn]--:--:--


I can confirm the fix works flawlessly</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>76876</commentid>
    <comment_count>5</comment_count>
    <who name="yeoh ee peng">ee.peng.yeoh</who>
    <bug_when>2017-09-15 01:56:42 +0000</bug_when>
    <thetext>Pushed commit to mailing list for review:

http://lists.openembedded.org/pipermail/openembedded-core/2017-August/141516.html</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>