[Buildroot] [PATCH 1/2] package/nginx: fix a typo

[Buildroot] [PATCH 1/2] package/nginx: fix a typo

[Noé Rubinstein]

Signed-off-by: No? Rubinstein <nrubinstein@aldebaran.com>
---
 package/nginx/nginx.mk | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/package/nginx/nginx.mk b/package/nginx/nginx.mk
index 32a337c..8e23bd8 100644
--- a/package/nginx/nginx.mk
+++ b/package/nginx/nginx.mk
@@ -19,7 +19,7 @@ NGINX_CONF_OPTS = \
 
 # www-data user and group are used for nginx. Because these user and group
 # are already set by buildroot, it is not necessary to redefine them.
-# See system/skeleton/passwd
+# See system/skeleton/etc/passwd
 #   username: www-data    uid: 33
 #   groupname: www-data   gid: 33
 #
-- 
2.1.4

[Buildroot] [PATCH 2/2] package/python-web2py: fix permissions

[Noé Rubinstein]

The PYTHON_WEB2PY_PERMISSIONS table was using the user and group names
instead of the uid and gid, causing makedevs to retrieve the uid and gid
from the host system.

Signed-off-by: No? Rubinstein <nrubinstein@aldebaran.com>
---
 package/python-web2py/python-web2py.mk | 10 +++++++++-
 1 file changed, 9 insertions(+), 1 deletion(-)

diff --git a/package/python-web2py/python-web2py.mk b/package/python-web2py/python-web2py.mk
index 313114a..db9ed8d 100644
--- a/package/python-web2py/python-web2py.mk
+++ b/package/python-web2py/python-web2py.mk
@@ -32,8 +32,16 @@ define PYTHON_WEB2PY_INSTALL_INIT_SYSTEMD
 		$(TARGET_DIR)/etc/systemd/system/multi-user.target.wants/web2py.service
 endef
 
+# www-data user and group are used for web2py. Because these user and group
+# are already set by buildroot, it is not necessary to redefine them.
+# See system/skeleton/etc/passwd
+#   username: www-data    uid: 33
+#   groupname: www-data   gid: 33
+#
+# So, we just need to create the directories used by web2py with the right
+# ownership.
 define PYTHON_WEB2PY_PERMISSIONS
-	/var/www/web2py  r  750  www-data  www-data  -  -  -  -  -
+	/var/www/web2py  r  750  33  33  -  -  -  -  -
 endef
 
 $(eval $(generic-package))
-- 
2.1.4

[Buildroot] [PATCH 1/2] package/nginx: fix a typo

[Thomas Petazzoni]

Dear No? Rubinstein,

On Wed,  4 Nov 2015 19:46:06 +0100, No? Rubinstein wrote:
> Signed-off-by: No? Rubinstein <nrubinstein@aldebaran.com>
> ---
>  package/nginx/nginx.mk | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

Both patches applied, thanks!

For the second patch, it would be good if someone could modify makedevs
to not accept user/group specified as strings, since resolving them
from the host point of view doesn't make any sense.

Thanks!

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

[Buildroot] [PATCH 1/2] package/nginx: fix a typo

[Arnout Vandecappelle]

On 04-11-15 21:49, Thomas Petazzoni wrote:
> Dear No? Rubinstein,
> 
> On Wed,  4 Nov 2015 19:46:06 +0100, No? Rubinstein wrote:
>> Signed-off-by: No? Rubinstein <nrubinstein@aldebaran.com>
>> ---
>>  package/nginx/nginx.mk | 2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> Both patches applied, thanks!
> 
> For the second patch, it would be good if someone could modify makedevs
> to not accept user/group specified as strings, since resolving them
> from the host point of view doesn't make any sense.

 Or better yet, use the target passwd and group files...

 Regards,
 Arnout


-- 
Arnout Vandecappelle                          arnout at mind be
Senior Embedded Software Architect            +32-16-286500
Essensium/Mind                                http://www.mind.be
G.Geenslaan 9, 3001 Leuven, Belgium           BE 872 984 063 RPR Leuven
LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle
GPG fingerprint:  7493 020B C7E3 8618 8DEC 222C 82EB F404 F9AC 0DDF