<?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>3158</bug_id>
          
          <creation_ts>2012-09-24 02:37:18 +0000</creation_ts>
          <short_desc>xserver-xorg-config_0.1.bbappend  bsp FILESPATH not set in a robust manner</short_desc>
          <delta_ts>2012-10-02 16:37:18 +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>BSPs</product>
          <component>bsps-meta-intel</component>
          <version>unspecified</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>patch submitted to yocto/meta-intel mailing lists for review</status_whiteboard>
          <keywords></keywords>
          <priority>Medium</priority>
          <bug_severity>normal</bug_severity>
          <target_milestone>1.3</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Blloyd">blloyd</reporter>
          <assigned_to name="Darren Hart">dvhart</assigned_to>
          <cc>bluelightning</cc>
    
    <cc>tom.zanussi</cc>
          
          
          <cf_os>---</cf_os>
          <cf_regression_type>---</cf_regression_type>
          
          <cf_docchange>---</cf_docchange>

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>25556</commentid>
    <comment_count>0</comment_count>
    <who name="Blloyd">blloyd</who>
    <bug_when>2012-09-24 02:37:18 +0000</bug_when>
    <thetext>This issue is known to affect the following items:

yocto-bsp
via the file
poky/scripts/lib/bsp/substrate/target/arch/i386/recipes-graphics/xorg-xserver/{{ if xserver == &quot;y&quot;: }} xserver-xf86-config_0.1.bbappend

As well as all 
xserver_xf86-config_0.1.bbappend files from meta-intel

A 
cat */recipes-graphics/xorg-xserver/xserver-xf86-config_0.1.bbappend
from the meta-intel directory shows all bbappend files followed this format.

The issue has been noted against both denzil and master branches.

The problem can be noted when you have more than one bsp configured to be searched, as the last one searched will be the one searched for files, even if the machine is from a previous bsp.
While kernel configuration effectively does not have an affect when machine is not selected, this bbappend does have an unexpected side effect.

FILESPATH =. &quot;${@base_set_filespath([&quot;${THISDIR}/${PN}&quot;], d)}:&quot;
uses delayed expansion.

Adding the line
FILESPATH := &quot;${FILESPATH}&quot;
immediately after the previous line seems to cause the immediate expansion of THISDIR needed to ensure THISDIR references the correct location, as that is commonly used in bbappend files to point the the directory that bbappend is in.

Alternately, a less likely to collide variable name should be selected.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>25572</commentid>
    <comment_count>1</comment_count>
    <who name="Paul Eggleton">bluelightning</who>
    <bug_when>2012-09-24 08:39:54 +0000</bug_when>
    <thetext>Does FILESEXTRAPATHS_prepend := &quot;${THISDIR}/${PN}:&quot; as I suggested on IRC not work?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>25611</commentid>
    <comment_count>2</comment_count>
    <who name="Blloyd">blloyd</who>
    <bug_when>2012-09-24 15:23:08 +0000</bug_when>
    <thetext>When I tried Paul Eggleton&apos;s suggestion, it did not appear to work for me.  It&apos;s possible I did something wrong, but when check FILESPATH and for which xorg.conf was included showed an intel video driver xorg.conf.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>25963</commentid>
    <comment_count>3</comment_count>
    <who name="Tom Zanussi">tom.zanussi</who>
    <bug_when>2012-09-29 23:52:26 +0000</bug_when>
    <thetext>I was able to reproduce this and have sent patches that fixes the problem for me to both the poky list for yocto-bsp and the yocto list for the meta-intel problems. i.e. with multiple BSP layers for qemux86, arm and crownbay, the correct xorg.conf was picked up where before it wasn&apos;t.

The problem was that though the other .bbappends were fixed up to use the FILESEXTRAPATHS_prepend method, the xserver-xf86-config bbappends for some reason weren&apos;t.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>26058</commentid>
    <comment_count>4</comment_count>
    <who name="Tom Zanussi">tom.zanussi</who>
    <bug_when>2012-10-02 16:37:18 +0000</bug_when>
    <thetext>Fixed in master:

poky/master: 50ac6e8785c167ea4aa4601fd690ef783151853d
meta-intel/master: 50ac6e8785c167ea4aa4601fd690ef783151853d</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>