New packages: acl and attr

PATCH New Yegor Yefremov 3 participants 8 messages Jun 28, 2011 – Jul 5, 2011

Share & tools

Yegor Yefremov

author

2011-06-28 08:34 UTC #0 lore raw

PATCH 1/1 New packages: acl and attr

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 <[email protected]>

Signed-off-by Daniel Mack <[email protected]>

Signed-off-by Yegor Yefremov <[email protected]>

---
 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

package/Config.in +2 -0

Index: 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

package/acl/Config.in new file +11 -0

Index: 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

package/acl/acl.mk new file +18 -0

Index: 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))

package/attr/Config.in new file +7 -0

Index: 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.

package/attr/attr.mk new file +16 -0

Index: 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))

Thomas Petazzoni

2011-06-28 08:48 UTC #1 lore raw

PATCH 1/1 New packages: acl and attr

Hello,

Le Tue, 28 Jun 2011 10:34:52 +0200,
Yegor Yefremov <[email protected]> a ?crit :

This:

package/acl/acl.mk line 16

Quoted 2 lines

package/acl/acl.mk new file +2 -0

+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:

package/attr/attr.mk line 14

Quoted 2 lines

package/attr/attr.mk new file +2 -0

+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

Signature · 4 lines

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

Yegor Yefremov

author

2011-06-28 13:09 UTC #2 lore raw

PATCH 1/1 New packages: acl and attr

Quoted 11 lines

This:
package/acl/acl.mk line 16
Quoted 2 lines
package/acl/acl.mk new file +2 -0
+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:
package/attr/attr.mk line 14
Quoted 2 lines
package/attr/attr.mk new file +2 -0
+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

Thomas Petazzoni

2011-06-28 13:24 UTC #3 lore raw

PATCH 1/1 New packages: acl and attr

Le Tue, 28 Jun 2011 15:09:27 +0200,
Yegor Yefremov <[email protected]> a ?crit :

Quoted 3 lines

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

Signature · 4 lines

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

Yegor Yefremov

author

2011-06-28 13:29 UTC #4 lore raw

PATCH 1/1 New packages: acl and attr

Am 28.06.2011 15:24, schrieb Thomas Petazzoni:

Quoted 7 lines

Le Tue, 28 Jun 2011 15:09:27 +0200,
Yegor Yefremov <[email protected]> a ?crit :
Quoted 3 lines
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

Daniel Nyström

2011-06-28 13:41 UTC #5 lore raw

PATCH 1/1 New packages: acl and attr

2011/6/28 Yegor Yefremov <[email protected]>:

Quoted 10 lines

Am 28.06.2011 15:24, schrieb Thomas Petazzoni:
Quoted 7 lines
Le Tue, 28 Jun 2011 15:09:27 +0200,
Yegor Yefremov <[email protected]> a ?crit :
Quoted 3 lines
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

Yegor Yefremov

author

2011-07-01 10:48 UTC #6 lore raw

PATCH 1/1 New packages: acl and attr

Quoted 10 lines

Quoted 6 lines
Quoted 5 lines
Quoted 4 lines
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

Yegor Yefremov

author

2011-07-05 14:13 UTC #7 lore raw

PATCH 1/1 New packages: acl and attr

Quoted 4 lines

Quoted 4 lines
Quoted 4 lines
Quoted 4 lines
Quoted 4 lines
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.

Quoted 8 lines

Quoted 6 lines
Quoted 2 lines
Quoted 1 line
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