Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Yegor Yefremov <yegor_sub1@visionsystems.de>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH 1/2] Add package acl
Date: Tue, 28 Jun 2011 10:35:55 +0200	[thread overview]
Message-ID: <4E09926B.40203@visionsystems.de> (raw)
In-Reply-To: <20110621124002.Horde.FlRsXGms4ABOANdyT8bFvAA@www.home.zuerker.org>

Am 21.06.2011 19:40, schrieb Heiko Zuerker:
> 
> Quoting Yegor Yefremov <yegor_sub1@visionsystems.de>:
> 
>> Am 31.05.2011 14:47, schrieb Heiko Zuerker:
>>>
>>> Quoting Daniel Mack <zonque@gmail.com>:
>>>
>>>> On Mon, May 30, 2011 at 9:02 PM, Thomas Petazzoni
>>>> <thomas.petazzoni@free-electrons.com> wrote:
>>>>> On Mon, 30 May 2011 20:16:29 +0200
>>>>> Daniel Mack <zonque@gmail.com> wrote:
>>>>>
>>>>>> +define ACL_INSTALL_TARGET_CMDS
>>>>>> +     $(INSTALL) -m 0755 -D $(@D)/getfacl/.libs/getfacl $(TARGET_DIR)/usr/bin/
>>>>>> +     $(INSTALL) -m 0755 -D $(@D)/setfacl/.libs/setfacl $(TARGET_DIR)/usr/bin/
>>>>>> +     $(INSTALL) -m 0644 -D $(@D)/libacl/.libs/libacl.so* $(STAGING_DIR)/usr/lib/
>>>>>> +endef
>>>>>> +
>>>>>> +define ACL_INSTALL_STAGING_CMDS
>>>>>> +     test -d $(STAGING_DIR)/usr/include/acl/ || mkdir $(STAGING_DIR)/usr/include/acl/
>>>>>> +     $(INSTALL) -m 0644 -D $(@D)/include/*.h $(STAGING_DIR)/usr/include/acl/
>>>>>> +     $(SED) s,sys/acl.h,acl/acl.h, -i $(STAGING_DIR)/usr/include/acl/libacl.h
>>>>>> +endef
>>>>>
>>>>> Any reason to not use the default "make install" behaviour of the
>>>>> package ? (And ditto for the attr package).
>>>>
>>>> The libtool crap failed for cross compilation in these cases, so these
>>>> workarounds are unfortunately necessary.
>>>
>>> Actually this isn't necessary, please see my version of the patches.
>>>
>>
>> Any update on this issue?
> 
> I don't believe that either version of the package has been added.

I just sent new patch combined of yours and Daniels. Please review.

Best regards,
Yegor

  reply	other threads:[~2011-06-28  8:35 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-05-30 18:16 [Buildroot] Add two more packages Daniel Mack
2011-05-30 18:16 ` [Buildroot] [PATCH 1/2] Add package acl Daniel Mack
2011-05-30 18:54   ` Heiko Zuerker
2011-05-30 19:02   ` Thomas Petazzoni
2011-05-31  9:06     ` Daniel Mack
2011-05-31 12:47       ` Heiko Zuerker
2011-05-31 16:18         ` Daniel Mack
2011-06-21 10:14         ` Yegor Yefremov
2011-06-21 17:40           ` Heiko Zuerker
2011-06-28  8:35             ` Yegor Yefremov [this message]
2011-05-30 18:16 ` [Buildroot] [PATCH 2/2] Add package attr Daniel Mack

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4E09926B.40203@visionsystems.de \
    --to=yegor_sub1@visionsystems.de \
    --cc=buildroot@busybox.net \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox