diff mbox

package/leafnode2: fix news user check

Message ID 1419615491-8238-1-git-send-email-romain.naour@openwide.fr
State Accepted
Headers show

Commit Message

Romain Naour Dec. 26, 2014, 5:38 p.m. UTC
--enable-runas-user use 'news' as default but the configure stop
if news doesn't exist on the build host.
Use 'root' while cross-compiling

Signed-off-by: Romain Naour <romain.naour@openwide.fr>
---
 package/leafnode2/leafnode2.mk | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

Comments

Thomas Petazzoni Dec. 27, 2014, 4:13 p.m. UTC | #1
Dear Romain Naour,

On Fri, 26 Dec 2014 18:38:11 +0100, Romain Naour wrote:
> --enable-runas-user use 'news' as default but the configure stop
> if news doesn't exist on the build host.
> Use 'root' while cross-compiling
> 
> Signed-off-by: Romain Naour <romain.naour@openwide.fr>
> ---
>  package/leafnode2/leafnode2.mk | 6 +++++-
>  1 file changed, 5 insertions(+), 1 deletion(-)

Applied, thanks. Here, leafnode2 does not start properly at boot time,
it asks for a fully qualified host name in /etc/hosts. It would be good
to add a default config file + init script to make leafnode2 work at
least somewhat out of the box.

Thanks!

Thomas
Romain Naour Dec. 27, 2014, 8:52 p.m. UTC | #2
Le 27/12/2014 17:13, Thomas Petazzoni a écrit :
> Dear Romain Naour,
>
> On Fri, 26 Dec 2014 18:38:11 +0100, Romain Naour wrote:
>> --enable-runas-user use 'news' as default but the configure stop
>> if news doesn't exist on the build host.
>> Use 'root' while cross-compiling
>>
>> Signed-off-by: Romain Naour <romain.naour@openwide.fr>
>> ---
>>  package/leafnode2/leafnode2.mk | 6 +++++-
>>  1 file changed, 5 insertions(+), 1 deletion(-)
> Applied, thanks. Here, leafnode2 does not start properly at boot time,
> it asks for a fully qualified host name in /etc/hosts. It would be good
> to add a default config file + init script to make leafnode2 work at
> least somewhat out of the box.
>
> Thanks!
>
> Thomas
Honestly, I haven't runtime tested leadnode2.
I just checked why the build has stopped during a random build.

The configure script run the command: id -u news:
a=`$ID -u $RUNAS_USER`
if test -z "$a" ; then
    as_fn_error $? "user $RUNAS_USER does not exist. Please add this user before building leafnode.
                  Most systems have man adduser or man useradd to tell you how to do this." "$LINENO" 5
else
...

This test is used to print a warning because it expect a native
build:
WARNING: user $RUNAS_USER cannot be used to run leafnode for security reasons.
Be sure to configure run_as_user to an unprivileged user, else leafnode will not run."

I'm adding Bernd to cc since it use leadnode2.

Best regards,
Romain
diff mbox

Patch

diff --git a/package/leafnode2/leafnode2.mk b/package/leafnode2/leafnode2.mk
index 3bc1c68..0fc2a8d 100644
--- a/package/leafnode2/leafnode2.mk
+++ b/package/leafnode2/leafnode2.mk
@@ -14,9 +14,13 @@  LEAFNODE2_DEPENDENCIES = host-pcre pcre
 LEAFNODE2_CONF_ENV = \
 	PCRECONFIG="$(STAGING_DIR)/usr/bin/pcre-config"
 
+# --enable-runas-user use 'news' as default but the configure stop
+# if news doesn't exist on the build host.
+# Use 'root' while cross-compiling 
 LEAFNODE2_CONF_OPTS = \
 	--sysconfdir=/etc/leafnode2 \
-	--enable-spooldir=/var/spool/news
+	--enable-spooldir=/var/spool/news \
+	--enable-runas-user=root
 
 # Leafnode2 needs the host version of b_sortnl during
 # compilation. Instead of creating a seperate host package and