GNU bug report logs - #40581
[wip-hurd-vm] uptime from coreutils looks for /bin/w

Previous Next

To reply to this bug, email your comments to 40581 AT debbugs.gnu.org.

Toggle the display of automated, internal messages from the tracker.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Efraim Flashner <efraim <at> flashner.co.il>
To: bug-guix <at> gnu.org
Subject: [wip-hurd-vm] uptime from coreutils looks for /bin/w
Date: Sun, 12 Apr 2020 21:30:58 +0300

[Message part 1 (text/plain, inline)]

I fired up the hurd VM and ran 'uptime' and it complained that it
couldn't find /bin/w. 'w' was in the path.


-- 
Efraim Flashner   <efraim <at> flashner.co.il>   אפרים פלשנר
GPG key = A28B F40C 3E55 1372 662D  14F7 41AA E7DC CA3D 8351
Confidentiality cannot be guaranteed on emails sent or received unencrypted

[signature.asc (application/pgp-signature, inline)]

Message #8 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Rene <pacoon <at> protonmail.com>
To: "bug-guix <at> gnu.org" <bug-guix <at> gnu.org>
Cc: "janneke <at> gnu.org" <janneke <at> gnu.org>
Subject: bug#40581: [wip-hurd-vm] uptime from coreutils looks for /bin/w
Date: Mon, 13 Apr 2020 03:54:48 +0000

[Message part 1 (text/plain, inline)]

Hello,


Debian project currently uses `/bin/w` from the hurd package, so I propose the attached patch.


Rene

[0001-gnu-hurd-Fix-references-to-bin-w.patch (text/x-patch, attachment)]

Message #11 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Jan Nieuwenhuizen <janneke <at> gnu.org>
To: Rene <pacoon <at> protonmail.com>
Cc: "bug-guix <at> gnu.org" <bug-guix <at> gnu.org>
Subject: Re: bug#40581: [wip-hurd-vm] uptime from coreutils looks for /bin/w
Date: Mon, 13 Apr 2020 09:11:48 +0200

Rene writes:

Hello Rene, Efraim,

> Debian project currently uses `/bin/w` from the hurd package, so I
> propose the attached patch.

LGTM!

> @@ -398,6 +399,9 @@ boot, since this cannot be done from GNU/Linux."
>                 (substitute* '("startup/startup.c" "init/init.c" "config/ttys")
>                   (("/libexec/")
>                    (string-append out "/libexec/")))
> +               (substitute* '("utils/uptime.sh")
> +                 (("/bin/w")
> +                  (string-append out "/bin/w")))
>                 (substitute* "daemons/console-run.c"
>                   (("/hurd/")
>                    (string-append out "/hurd/")))

Looking at this again it looks fine; following the pattern we created.
This now makes me wonder whether upstream Hurd could use a patch for
${bindir} and ${libexecdir}.  Possibly even for `/hurd'.

What do you all think?

Greetings,
janneke

-- 
Jan Nieuwenhuizen <janneke <at> gnu.org> | GNU LilyPond http://lilypond.org
Freelance IT http://JoyofSource.com | Avatar® http://AvatarAcademy.com

Message #14 received at 40581 <at> debbugs.gnu.org (full text, mbox):

From: Ludovic Courtès <ludo <at> gnu.org>
To: Jan Nieuwenhuizen <janneke <at> gnu.org>
Cc: Rene <pacoon <at> protonmail.com>, 40581 <at> debbugs.gnu.org
Subject: Re: bug#40581: [wip-hurd-vm] uptime from coreutils looks for /bin/w
Date: Mon, 13 Apr 2020 12:44:35 +0200

Hey ho!

Jan Nieuwenhuizen <janneke <at> gnu.org> skribis:

> Rene writes:
>
> Hello Rene, Efraim,
>
>> Debian project currently uses `/bin/w` from the hurd package, so I
>> propose the attached patch.
>
> LGTM!
>
>> @@ -398,6 +399,9 @@ boot, since this cannot be done from GNU/Linux."
>>                 (substitute* '("startup/startup.c" "init/init.c" "config/ttys")
>>                   (("/libexec/")
>>                    (string-append out "/libexec/")))
>> +               (substitute* '("utils/uptime.sh")
>> +                 (("/bin/w")
>> +                  (string-append out "/bin/w")))
>>                 (substitute* "daemons/console-run.c"
>>                   (("/hurd/")
>>                    (string-append out "/hurd/")))
>
> Looking at this again it looks fine; following the pattern we created.

Yup, LGTM!

> This now makes me wonder whether upstream Hurd could use a patch for
> ${bindir} and ${libexecdir}.  Possibly even for `/hurd'.
>
> What do you all think?

Yes, I think this prefix issue should be fixed upstream.

Ludo’.

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.