<?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>9668</bug_id>
          
          <creation_ts>2016-05-24 10:05:33 +0000</creation_ts>
          <short_desc>libarchive doesn&apos;t respect disable-acl</short_desc>
          <delta_ts>2016-06-06 04:21:57 +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>multimedia</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></status_whiteboard>
          <keywords></keywords>
          <priority>Medium</priority>
          <bug_severity>normal</bug_severity>
          <target_milestone>2.2</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Gary Thomas">gary</reporter>
          <assigned_to name="Maxin B. John">maxin.john</assigned_to>
          <cc>meta.mr.watcher</cc>
    
    <cc>meta.watcher</cc>
    
    <cc>randy.macleod</cc>
          
          
          <cf_os>---</cf_os>
          <cf_regression_type>---</cf_regression_type>
          
          <cf_docchange>Don&apos;t know</cf_docchange>

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>62414</commentid>
    <comment_count>0</comment_count>
    <who name="Gary Thomas">gary</who>
    <bug_when>2016-05-24 10:05:33 +0000</bug_when>
    <thetext>Even without acl in PACKAGECONFIG (hence --disable-acl), the configure script still checkks for acl.h/libacl.  This can lead to a broken build if acl was present during an initial run but removed later and the package simply rebuilt without rerunning configure.

From the config.log:
configure:13342: checking acl/libacl.h usability
configure:13342: arm-amltd-linux-gnueabi-gcc  -march=armv7ve -marm -mfpu=neon  -mfloat-abi=softfp -mcpu=cortex-a7 --sysroot=/local/p7619_2016-02-23/tmp/sysroots/my_target -c  -O2 -pipe -g -feliminate-unused-debug-types -fdebug-prefix-map=/local/p7619_2016-02-23/tmp/work/cortexa7-neon-amltd-linux-gnueabi/libarchive/3.1.2-r0=/usr/src/debug/libarchive/3.1.2-r0 -fdebug-prefix-map=/local/p7619_2016-02-23/tmp/sysroots/x86_64-linux= -fdebug-prefix-map=/local/p7619_2016-02-23/tmp/sysroots/teton-p7618=   -I/local/pXXXX_2016-02-23/tmp/work/cortexa7-neon-amltd-linux-gnueabi/libarchive/3.1.2-r0/extra-includes conftest.c &gt;&amp;5
conftest.c:67:24: fatal error: acl/libacl.h: No such file or directory
 #include &lt;acl/libacl.h&gt;


Tested with poky master:c7e614c438706fb3ed7520b4990ebb3973366942</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>62471</commentid>
    <comment_count>1</comment_count>
    <who name="Randy MacLeod">randy.macleod</who>
    <bug_when>2016-05-26 14:49:02 +0000</bug_when>
    <thetext>Apparently not easy to reproduce so being treated as a medium priority defect.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>62597</commentid>
    <comment_count>2</comment_count>
      <attachid>3209</attachid>
    <who name="Maxin B. John">maxin.john</who>
    <bug_when>2016-06-02 13:31:12 +0000</bug_when>
    <thetext>Created attachment 3209
add check for acl/libacl.h

Hi Gary,

This build failure is hard to re-create. However, based on the configure.ac code, header check exists only for &lt;sys/acl.h&gt;. It should also include check for &lt;acl/libacl.h&gt; which contains additional API in linux.

ie:
if test &quot;x$enable_acl&quot; != &quot;xno&quot;; then
    AC_CHECK_HEADERS([sys/acl.h])
    AC_CHECK_LIB([acl],[acl_get_file])
    AC_CHECK_FUNCS([acl_create_entry acl_init acl_set_fd acl_set_fd_np acl_set_file])
 

I have attached a patch to fix it. Is it possible for you to verify that this failure still occurs on poky (master branch) ?

Please note that libarchive was upgraded to version 3.2.0 recently</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>62598</commentid>
    <comment_count>3</comment_count>
    <who name="Maxin B. John">maxin.john</who>
    <bug_when>2016-06-02 13:35:55 +0000</bug_when>
    <thetext>Please apply this patch on top of poky (master) and check if we can recreate this error.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>62627</commentid>
    <comment_count>4</comment_count>
    <who name="Maxin B. John">maxin.john</who>
    <bug_when>2016-06-03 13:15:48 +0000</bug_when>
    <thetext>Submitted an upstream pull request.
https://github.com/libarchive/libarchive/pull/712</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>62641</commentid>
    <comment_count>5</comment_count>
    <who name="Gary Thomas">gary</who>
    <bug_when>2016-06-04 07:07:13 +0000</bug_when>
    <thetext>(In reply to comment #3)
&gt; Please apply this patch on top of poky (master) and check if we can recreate
&gt; this error.

I was able to get my build to break again and then applied this patch.  It now builds correctly, thanks</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>62646</commentid>
    <comment_count>6</comment_count>
    <who name="Maxin B. John">maxin.john</who>
    <bug_when>2016-06-06 04:21:57 +0000</bug_when>
    <thetext>http://git.yoctoproject.org/cgit/cgit.cgi/poky/commit/?id=5481363ac7f000d624c264fb27b27d851d2be311</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>3209</attachid>
            <date>2016-06-02 13:31:12 +0000</date>
            <delta_ts>2016-06-02 13:31:12 +0000</delta_ts>
            <desc>add check for acl/libacl.h</desc>
            <filename>0001-libarchive-respect-disable-acl-configuration-option.patch</filename>
            <type>text/plain</type>
            <size>2481</size>
            <attacher name="Maxin B. John">maxin.john</attacher>
            
              <data encoding="base64">RnJvbSBkMWNkODM2YzU2NDQ0ODI4M2YyYjRiNjIwOTIwMjdhNGFiMzllZDRjIE1vbiBTZXAgMTcg
MDA6MDA6MDAgMjAwMQpGcm9tOiAiTWF4aW4gQi4gSm9obiIgPG1heGluLmpvaG5AaW50ZWwuY29t
PgpEYXRlOiBUaHUsIDIgSnVuIDIwMTYgMTY6Mjg6MTUgKzAzMDAKU3ViamVjdDogW1BBVENIXSBs
aWJhcmNoaXZlOiByZXNwZWN0IGRpc2FibGUtYWNsIGNvbmZpZ3VyYXRpb24gb3B0aW9uCgpBZGQg
YWRkaXRpb25hbCBjaGVjayBmb3IgPGFjbC9saWJhY2wuaD4gZmlsZQoKU2lnbmVkLW9mZi1ieTog
TWF4aW4gQi4gSm9obiA8bWF4aW4uam9obkBpbnRlbC5jb20+Ci0tLQogLi4uMS1jb25maWd1cmUu
YWMtYWRkLWNoZWNrLWZvci1hY2wtbGliYWNsLmgucGF0Y2ggfCAyNyArKysrKysrKysrKysrKysr
KysrKysrCiAuLi4vbGliYXJjaGl2ZS9saWJhcmNoaXZlXzMuMi4wLmJiICAgICAgICAgICAgICAg
ICB8ICAxICsKIDIgZmlsZXMgY2hhbmdlZCwgMjggaW5zZXJ0aW9ucygrKQogY3JlYXRlIG1vZGUg
MTAwNjQ0IG1ldGEvcmVjaXBlcy1leHRlbmRlZC9saWJhcmNoaXZlL2xpYmFyY2hpdmUvMDAwMS1j
b25maWd1cmUuYWMtYWRkLWNoZWNrLWZvci1hY2wtbGliYWNsLmgucGF0Y2gKCmRpZmYgLS1naXQg
YS9tZXRhL3JlY2lwZXMtZXh0ZW5kZWQvbGliYXJjaGl2ZS9saWJhcmNoaXZlLzAwMDEtY29uZmln
dXJlLmFjLWFkZC1jaGVjay1mb3ItYWNsLWxpYmFjbC5oLnBhdGNoIGIvbWV0YS9yZWNpcGVzLWV4
dGVuZGVkL2xpYmFyY2hpdmUvbGliYXJjaGl2ZS8wMDAxLWNvbmZpZ3VyZS5hYy1hZGQtY2hlY2st
Zm9yLWFjbC1saWJhY2wuaC5wYXRjaApuZXcgZmlsZSBtb2RlIDEwMDY0NAppbmRleCAwMDAwMDAw
Li44MWQ0ZGE5Ci0tLSAvZGV2L251bGwKKysrIGIvbWV0YS9yZWNpcGVzLWV4dGVuZGVkL2xpYmFy
Y2hpdmUvbGliYXJjaGl2ZS8wMDAxLWNvbmZpZ3VyZS5hYy1hZGQtY2hlY2stZm9yLWFjbC1saWJh
Y2wuaC5wYXRjaApAQCAtMCwwICsxLDI3IEBACitGcm9tIDRhOTE1OTVmMDI3MDEwNDY5MDQ5YWRk
M2U1NGVmZjM0YjgzYTUxNmUgTW9uIFNlcCAxNyAwMDowMDowMCAyMDAxCitGcm9tOiAiTWF4aW4g
Qi4gSm9obiIgPG1heGluLmpvaG5AaW50ZWwuY29tPgorRGF0ZTogVGh1LCAyIEp1biAyMDE2IDE2
OjA4OjAyICswMzAwCitTdWJqZWN0OiBbUEFUQ0hdIGNvbmZpZ3VyZS5hYzogYWRkIGNoZWNrIGZv
ciBhY2wvbGliYWNsLmgKKworZW5hYmxlLWFjbCBjaGVjayBzaG91bGQgYWxzbyBpbmNsdWRlIHRo
ZSBhY2wvbGliYWNsLmggY2hlY2suCisKK1NpZ25lZC1vZmYtYnk6IE1heGluIEIuIEpvaG4gPG1h
eGluLmpvaG5AaW50ZWwuY29tPgorLS0tCisgY29uZmlndXJlLmFjIHwgMSArCisgMSBmaWxlIGNo
YW5nZWQsIDEgaW5zZXJ0aW9uKCspCisKK2RpZmYgLS1naXQgYS9jb25maWd1cmUuYWMgYi9jb25m
aWd1cmUuYWMKK2luZGV4IGNiNjk0M2YuLjNkOGY5NzggMTAwNjQ0CistLS0gYS9jb25maWd1cmUu
YWMKKysrKyBiL2NvbmZpZ3VyZS5hYworQEAgLTY3Myw2ICs2NzMsNyBAQCBBQ19BUkdfRU5BQkxF
KFthY2xdLAorIAkJW0VuYWJsZSBBQ0wgc3VwcG9ydCAoZGVmYXVsdDogY2hlY2spXSkpCisgCisg
aWYgdGVzdCAieCRlbmFibGVfYWNsIiAhPSAieG5vIjsgdGhlbgorKyAgIEFDX0NIRUNLX0hFQURF
UlMoW2FjbC9saWJhY2wuaF0pCisgICAgQUNfQ0hFQ0tfSEVBREVSUyhbc3lzL2FjbC5oXSkKKyAg
ICBBQ19DSEVDS19MSUIoW2FjbF0sW2FjbF9nZXRfZmlsZV0pCisgICAgQUNfQ0hFQ0tfRlVOQ1Mo
W2FjbF9jcmVhdGVfZW50cnkgYWNsX2luaXQgYWNsX3NldF9mZCBhY2xfc2V0X2ZkX25wIGFjbF9z
ZXRfZmlsZV0pCistLSAKKzIuNC4wCisKZGlmZiAtLWdpdCBhL21ldGEvcmVjaXBlcy1leHRlbmRl
ZC9saWJhcmNoaXZlL2xpYmFyY2hpdmVfMy4yLjAuYmIgYi9tZXRhL3JlY2lwZXMtZXh0ZW5kZWQv
bGliYXJjaGl2ZS9saWJhcmNoaXZlXzMuMi4wLmJiCmluZGV4IGUxZjAyMDIuLmNkZTYyNDYgMTAw
NjQ0Ci0tLSBhL21ldGEvcmVjaXBlcy1leHRlbmRlZC9saWJhcmNoaXZlL2xpYmFyY2hpdmVfMy4y
LjAuYmIKKysrIGIvbWV0YS9yZWNpcGVzLWV4dGVuZGVkL2xpYmFyY2hpdmUvbGliYXJjaGl2ZV8z
LjIuMC5iYgpAQCAtMzEsNiArMzEsNyBAQCBQQUNLQUdFQ09ORklHW2x6b10gPSAiLS13aXRoLWx6
bzIsLS13aXRob3V0LWx6bzIsbHpvLCIKIFBBQ0tBR0VDT05GSUdbbmV0dGxlXSA9ICItLXdpdGgt
bmV0dGxlLC0td2l0aG91dC1uZXR0bGUsbmV0dGxlLCIKIAogU1JDX1VSSSA9ICJodHRwOi8vbGli
YXJjaGl2ZS5vcmcvZG93bmxvYWRzL2xpYmFyY2hpdmUtJHtQVn0udGFyLmd6IFwKKyAgICAgICAg
ICAgZmlsZTovLzAwMDEtY29uZmlndXJlLmFjLWFkZC1jaGVjay1mb3ItYWNsLWxpYmFjbC5oLnBh
dGNoIFwKICAgICAgICAgICAgIgogCiBTUkNfVVJJW21kNXN1bV0gPSAiNTk2MjEwYjVhOWMyZmY3
NGEwZjdjYTM0ODM4ZDY1NWYiCi0tIAoyLjQuMAoK
</data>

          </attachment>
      

    </bug>

</bugzilla>