[Buildroot] [PATCH] Add /var/www to device table
Romain Izard
romain.izard.pro at gmail.com
Mon Jul 2 08:33:24 UTC 2012
On 2012-06-25, spdawson at gmail.com <spdawson at gmail.com> wrote:
> From: Simon Dawson <spdawson at gmail.com>
>
> The /var/www directory is listed in /etc/passwd in the skeleton target
> filesystem as the home directory of the www-data user (uid 33).
>
> In the final target filesystem, /var/www should be owned by www-data. This
> is important for the lighttpd package, for example.
>
> Signed-off-by: Simon Dawson <spdawson at gmail.com>
Shouldn't it be changed by using a LIGHTTPD_PERMISSIONS variable,
instead of modifying the common device table ? This makes the
relationship more evident.
See the following example:
---
diff --git a/package/lighttpd/lighttpd.mk b/package/lighttpd/lighttpd.mk
index b8aa310..34f59b2 100644
--- a/package/lighttpd/lighttpd.mk
+++ b/package/lighttpd/lighttpd.mk
@@ -62,4 +62,9 @@ define LIGHTTPD_UNINSTALL_TARGET_CMDS
rm -rf $(TARGET_DIR)/usr/lib/lighttpd
endef
+#<name> <type> <mode> <uid> <gid> <major> <minor> <start> <inc> <count>
+define LIGHTTPD_PERMISSIONS
+var/www d 755 33 33 - - - - -
+endef
+
$(eval $(call AUTOTARGETS))
--
Romain Izard
More information about the buildroot
mailing list