[Buildroot] [PATCH] New packages: acl and attr

[Buildroot] [PATCH] New packages: acl and attr

[Yegor Yefremov]

In addition to providing the acl and extended attributes functionality,
these new packages satisfy newer udev versions, so you can build all extras.

Signed-off-by: Heiko Zuerker <smiley73@users.sourceforge.net>
Signed-off-by: Daniel Mack <zonque@gmail.com>
Signed-off-by: Yegor Yefremov <yegorslists@googlemail.com>
---
 package/Config.in      |    2 ++
 package/acl/Config.in  |   11 +++++++++++
 package/acl/acl.mk     |   18 ++++++++++++++++++
 package/attr/Config.in |    7 +++++++
 package/attr/attr.mk   |   16 ++++++++++++++++
 5 files changed, 54 insertions(+)
 create mode 100644 package/acl/Config.in
 create mode 100644 package/acl/acl.mk
 create mode 100644 package/attr/Config.in
 create mode 100644 package/attr/attr.mk

Index: b/package/Config.in
===================================================================
--- a/package/Config.in
+++ b/package/Config.in
@@ -487,6 +487,8 @@
 endmenu
 
 menu "System tools"
+source "package/acl/Config.in"
+source "package/attr/Config.in"
 if BR2_PACKAGE_BUSYBOX_SHOW_OTHERS
 source "package/bootutils/Config.in"
 endif
Index: b/package/acl/Config.in
===================================================================
--- /dev/null
+++ b/package/acl/Config.in
@@ -0,0 +1,11 @@
+config BR2_PACKAGE_ACL
+	bool "acl"
+	select BR2_PACKAGE_ATTR
+	help
+	  POSIX Access Control Lists, which are used to define more
+	  fine-grained discretionary access rights for files and
+	  directories.
+	  This package also provides libacl.
+
+comment "acl requires attr to be built"
+        depends on !BR2_PACKAGE_ATTR
Index: b/package/acl/acl.mk
===================================================================
--- /dev/null
+++ b/package/acl/acl.mk
@@ -0,0 +1,18 @@
+#############################################################
+#
+# acl
+#
+#############################################################
+
+ACL_VERSION = 2.2.49
+ACL_SOURCE := acl_$(ACL_VERSION).orig.tar.gz
+ACL_SITE = http://ftp.debian.org/debian/pool/main/a/acl
+ACL_INSTALL_STAGING = YES
+ACL_DEPENDENCIES = attr
+ACL_CONF_OPT = --enable-gettext=no
+
+
+ACL_INSTALL_STAGING_OPT = prefix=$(STAGING_DIR)/usr exec_prefix=$(TARGET_DIR)/usr PKG_DEVLIB_DIR=$(STAGING_DIR)/usr/lib install-dev install-lib
+ACL_INSTALL_TARGET_OPT = prefix=$(TARGET_DIR)/usr exec_prefix=$(TARGET_DIR)/usr install install-lib
+
+$(eval $(call AUTOTARGETS,package,acl))
Index: b/package/attr/Config.in
===================================================================
--- /dev/null
+++ b/package/attr/Config.in
@@ -0,0 +1,7 @@
+config BR2_PACKAGE_ATTR
+	bool "attr"
+	help
+	  Extended attributes are name:value pairs associated permanently with
+	  files and directories, similar to the environment strings associated
+	  with a process
+	  This package also provides libattr.
Index: b/package/attr/attr.mk
===================================================================
--- /dev/null
+++ b/package/attr/attr.mk
@@ -0,0 +1,16 @@
+#############################################################
+#
+# attr
+#
+#############################################################
+
+ATTR_VERSION = 2.4.46
+ATTR_SOURCE := attr_$(ATTR_VERSION).orig.tar.gz
+ATTR_SITE = http://ftp.debian.org/debian/pool/main/a/attr
+ATTR_INSTALL_STAGING = YES
+ATTR_CONF_OPT = --enable-gettext=no
+
+ATTR_INSTALL_STAGING_OPT = prefix=$(STAGING_DIR)/usr exec_prefix=$(STAGING_DIR)/usr PKG_DEVLIB_DIR=$(STAGING_DIR)/usr/lib install-dev install-lib
+ATTR_INSTALL_TARGET_OPT = prefix=$(TARGET_DIR)/usr exec_prefix=$(TARGET_DIR)/usr install install-lib
+
+$(eval $(call AUTOTARGETS,package,attr))

[Buildroot] [PATCH] New packages: acl and attr

[Thomas Petazzoni]

Hello,

Le Tue, 28 Jun 2011 10:34:52 +0200,
Yegor Yefremov <yegor_sub1@visionsystems.de> a ?crit :

This:

> +ACL_INSTALL_STAGING_OPT = prefix=$(STAGING_DIR)/usr exec_prefix=$(TARGET_DIR)/usr PKG_DEVLIB_DIR=$(STAGING_DIR)/usr/lib install-dev install-lib
> +ACL_INSTALL_TARGET_OPT = prefix=$(TARGET_DIR)/usr exec_prefix=$(TARGET_DIR)/usr install install-lib

and this:

> +ATTR_INSTALL_STAGING_OPT = prefix=$(STAGING_DIR)/usr exec_prefix=$(STAGING_DIR)/usr PKG_DEVLIB_DIR=$(STAGING_DIR)/usr/lib install-dev install-lib
> +ATTR_INSTALL_TARGET_OPT = prefix=$(TARGET_DIR)/usr exec_prefix=$(TARGET_DIR)/usr install install-lib

requires justification. It apparently looks strange and unnecessary. So
if it is actually needed, a comment explaining why would be useful.

Best regards,

Thomas
-- 
Thomas Petazzoni, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com

[Buildroot] [PATCH] New packages: acl and attr

[Yegor Yefremov]

> This:
>
>> +ACL_INSTALL_STAGING_OPT = prefix=$(STAGING_DIR)/usr exec_prefix=$(TARGET_DIR)/usr PKG_DEVLIB_DIR=$(STAGING_DIR)/usr/lib install-dev install-lib
>> +ACL_INSTALL_TARGET_OPT = prefix=$(TARGET_DIR)/usr exec_prefix=$(TARGET_DIR)/usr install install-lib
> and this:
>
>> +ATTR_INSTALL_STAGING_OPT = prefix=$(STAGING_DIR)/usr exec_prefix=$(STAGING_DIR)/usr PKG_DEVLIB_DIR=$(STAGING_DIR)/usr/lib install-dev install-lib
>> +ATTR_INSTALL_TARGET_OPT = prefix=$(TARGET_DIR)/usr exec_prefix=$(TARGET_DIR)/usr install install-lib
> requires justification. It apparently looks strange and unnecessary. So
> if it is actually needed, a comment explaining why would be useful.

These passages are needed because of non-conform installation routine (as far as I can see). Otherwise installation script would try to install the files to /usr/ directly.

Best regards,
Yegor

[Buildroot] [PATCH] New packages: acl and attr

[Thomas Petazzoni]

Le Tue, 28 Jun 2011 15:09:27 +0200,
Yegor Yefremov <yegor_sub1@visionsystems.de> a ?crit :

> These passages are needed because of non-conform installation routine
> (as far as I can see). Otherwise installation script would try to
> install the files to /usr/ directly.

This should not happen with autotools-based packages. Are you sure ?

Thomas
-- 
Thomas Petazzoni, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com

[Buildroot] [PATCH] New packages: acl and attr

[Yegor Yefremov]

Am 28.06.2011 15:24, schrieb Thomas Petazzoni:
> Le Tue, 28 Jun 2011 15:09:27 +0200,
> Yegor Yefremov <yegor_sub1@visionsystems.de> a ?crit :
>
>> These passages are needed because of non-conform installation routine
>> (as far as I can see). Otherwise installation script would try to
>> install the files to /usr/ directly.
> This should not happen with autotools-based packages. Are you sure ?

I know it shouldn't, but this is the behavior I observe.

Yegor

[Buildroot] [PATCH] New packages: acl and attr

[Daniel Nyström]

2011/6/28 Yegor Yefremov <yegor_sub1@visionsystems.de>:
> Am 28.06.2011 15:24, schrieb Thomas Petazzoni:
>> Le Tue, 28 Jun 2011 15:09:27 +0200,
>> Yegor Yefremov <yegor_sub1@visionsystems.de> a ?crit :
>>
>>> These passages are needed because of non-conform installation routine
>>> (as far as I can see). Otherwise installation script would try to
>>> install the files to /usr/ directly.
>> This should not happen with autotools-based packages. Are you sure ?
>
> I know it shouldn't, but this is the behavior I observe.

It's weird indeed:
http://git.savannah.gnu.org/cgit/attr.git/tree/configure.in
http://git.savannah.gnu.org/cgit/attr.git/tree/Makefile
http://git.savannah.gnu.org/cgit/attr.git/tree/include/builddefs.in

[Buildroot] [PATCH] New packages: acl and attr

[Yegor Yefremov]

>>>> These passages are needed because of non-conform installation routine
>>>> (as far as I can see). Otherwise installation script would try to
>>>> install the files to /usr/ directly.
>>> This should not happen with autotools-based packages. Are you sure ?
>> I know it shouldn't, but this is the behavior I observe.
> It's weird indeed:
> http://git.savannah.gnu.org/cgit/attr.git/tree/configure.in
> http://git.savannah.gnu.org/cgit/attr.git/tree/Makefile
> http://git.savannah.gnu.org/cgit/attr.git/tree/include/builddefs.in

Could we apply the patch as is, so we can move on with udev update? Meanwhile I'll try to contact the maintainers of acl and attr and discuss the issue with them.

Yegor

[Buildroot] [PATCH] New packages: acl and attr

[Yegor Yefremov]

>>>>> These passages are needed because of non-conform installation routine
>>>>> (as far as I can see). Otherwise installation script would try to
>>>>> install the files to /usr/ directly.
>>>> This should not happen with autotools-based packages. Are you sure ?
>>> I know it shouldn't, but this is the behavior I observe.
>> It's weird indeed:
>> http://git.savannah.gnu.org/cgit/attr.git/tree/configure.in
>> http://git.savannah.gnu.org/cgit/attr.git/tree/Makefile
>> http://git.savannah.gnu.org/cgit/attr.git/tree/include/builddefs.in
> 
> Could we apply the patch as is, so we can move on with udev update? Meanwhile I'll try to contact the maintainers of acl and attr and discuss the issue with them.

This is a discussion with attr and acl devs: http://lists.gnu.org/archive/html/acl-devel/2011-07/threads.html#00001

Yegor