<?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>5043</bug_id>
          
          <creation_ts>2013-08-23 13:09:25 +0000</creation_ts>
          <short_desc>Extend the layer creation documentation with own bblayers sample creation</short_desc>
          <delta_ts>2017-06-15 16:13:54 +0000</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>9</classification_id>
          <classification>Documentation</classification>
          <product>General Docs</product>
          <component>docs-general</component>
          <version>1.5</version>
          <rep_platform>x86</rep_platform>
          <op_sys>Multiple</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>OBSOLETE</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>Low</priority>
          <bug_severity>enhancement</bug_severity>
          <target_milestone>Future</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>lpapp</cc>
    
    <cc>poky.bs.watcher</cc>
    
    <cc>poky.watcher</cc>
    
    <cc>ross.burton</cc>
    
    <cc>srifenbark</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>35884</commentid>
    <comment_count>0</comment_count>
    <who name="Laszlo Papp">lpapp</who>
    <bug_when>2013-08-23 13:09:25 +0000</bug_when>
    <thetext>The current documentation does not contain much information about how to add bblayers.conf samples to the own bsp/distribution layer so that when the developers of the project source the open embedded init script, those files get automatically generated with the right content without the developers, CI, and so forth doing it manually all the time when there is a fresh build.

Looking at the ./scripts/oe-setup-builddir script, there is this line which seems to be responsible for it: TEMPLATECONF=${TEMPLATECONF:-meta-yocto/conf}

This variable is undocumented. Furthermore, I have been told on IRC there is also a OECORELAYERCONF.

This is confusing now, and it would be nice to clarify with the documentation how to create your own layer including the sample for this.

What is happening currently is that my sample is not taken into account, but the $buildfolder/conf/bblayers.conf is generated based upon the meta-yocto layer as it is the default apparently for the TEMPLATECONF.

When it is created like that as a fallback, I am getting the following error message which should not happen if I could override all this easily with a proper purposeful documentation:

Pseudo is not present but is required, building this first before the main build
ERROR:  OE-core&apos;s config sanity checker detected a potential misconfiguration.
    Either fix the cause of this error or at your own risk disable the checker (see sanity.conf).
    Following is the list of potential problems / advisories:

    Please set a valid MACHINE in your local.conf or environment
Your version of bblayers.conf was generated from an older/newer version of bblayers.conf.sample and there have been updates made to this file. Please compare the two files and merge any changes before continuing.
Matching the version numbers will remove this message.
&quot;meld conf/bblayers.conf /home/lpapp/Projects/Yocto/poky-dylan-9.0.1/meta*/conf/bblayers.conf.sample&quot; is a good way to visualise the changes.
DISTRO &apos;foo&apos; not found. Please set a valid DISTRO in your local.conf

ERROR: Execution of event handler &apos;check_sanity_eventhandler&apos; failed</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>35889</commentid>
    <comment_count>1</comment_count>
    <who name="Laszlo Papp">lpapp</who>
    <bug_when>2013-08-23 14:33:30 +0000</bug_when>
    <thetext>I think the problem is that with those two variables that they are necessary for the open embedded init script, meanwhile the rest of the configuration would probably be in the local.conf.

Hence, it might make more sense to have an option for this in there. If such an option already exists, I am just unaware of it, and it would be nice to get documented.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>35890</commentid>
    <comment_count>2</comment_count>
    <who name="Laszlo Papp">lpapp</who>
    <bug_when>2013-08-23 14:41:01 +0000</bug_when>
    <thetext>Of course, that would not be enough itself as in, bitbake would need to check for that particular variable if it needs to rerun and update the generated file accordingly based on the desired sample.

local.conf is a usual way out there to check the same file into the repository so things just work out of the box for the developers, and also for the CI.

That is the reason why I am leaning towards a local.conf entry. This would not of course have too much impact on people who would not like to use this feature as they would just skip the setting of such a variable.

Sure, there would be a minor overhead for its check, but most likely not much noticeable. Of course, it would also possible for everyone to build a small script on top of the open embedded init script, but then it is getting farther from the vanilla operation somewhat, so I would not mind the local.conf alternative respectively.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>36044</commentid>
    <comment_count>3</comment_count>
    <who name="Jessica">jessica.zhang</who>
    <bug_when>2013-08-29 14:42:14 +0000</bug_when>
    <thetext>This may tide to we need to enhance our layer template</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>36508</commentid>
    <comment_count>4</comment_count>
    <who name="Scott Rifenbark">srifenbark</who>
    <bug_when>2013-09-10 22:47:24 +0000</bug_when>
    <thetext>Hi, 

I need to wait to see what changes are made to layer template as noted by Jessica.

Thanks, 
Scott</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>36919</commentid>
    <comment_count>5</comment_count>
    <who name="Richard Purdie">richard.purdie</who>
    <bug_when>2013-09-18 22:41:08 +0000</bug_when>
    <thetext>We certainly need to make it more easily available for layers to customize the wrapping of OE-Core. The meta-yocto layer currently does this for its template files but its a bit ugly and we should complete the work started a long time ago to finally resolve this.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>74229</commentid>
    <comment_count>6</comment_count>
    <who name="Richard Purdie">richard.purdie</who>
    <bug_when>2017-06-15 15:30:22 +0000</bug_when>
    <thetext>I believe we did change the way this works so that layers could be extended. If there are specific issues that still need to be addressed please open specific bugs for those.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>74240</commentid>
    <comment_count>7</comment_count>
    <who name="Scott Rifenbark">srifenbark</who>
    <bug_when>2017-06-15 16:13:54 +0000</bug_when>
    <thetext>Great - I am turning the doc flag off then. 

Thanks, 
Scott</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>