Bug#826214: Bug#913247: Please provide a C implementation of /lib/init/init-d-script

2018-12-01 Thread Mert Dirik
On Sun, Dec 2, 2018 at 1:00 AM Michael Biebl wrote: > > Hi Mert! > > Am 29.11.18 um 14:44 schrieb Mert Dirik: > > I've created a merge request for 40-systemd at > > https://salsa.debian.org/systemd-team/systemd/merge_requests/19 . > > Reviews and comments are welcomed. > > > > If this is fully app

Bug#826214: Bug#913247: Please provide a C implementation of /lib/init/init-d-script

2018-12-01 Thread Mert Dirik
On Sat, Dec 1, 2018 at 9:48 PM Thorsten Glaser wrote: > > On Sat, 1 Dec 2018, Dmitry Bogatov wrote: > > > AFAIK, in shell [ "${foo}" ] is equal to [ -n "${foo}" ]. > > Not always / portably. > > > I recommend > > test -n "$foo" > > for POSIX (which is equivalent to [ -n "$foo" ] but better

Bug#826214: Bug#913247: Please provide a C implementation of /lib/init/init-d-script

2018-12-01 Thread Michael Biebl
Am 01.12.18 um 23:31 schrieb Thorsten Glaser: > This will also make it easier to switch to a binary > implementation later. Ok, I take this as a positive signal. So far this has been rejected outright. Regards, Michael -- Why is it that all of the instruments seeking intelligent life in the uni

Bug#826214: Bug#913247: Please provide a C implementation of /lib/init/init-d-script

2018-12-01 Thread Michael Biebl
Am 01.12.18 um 23:40 schrieb Thorsten Glaser: > On Sat, 1 Dec 2018, Michael Biebl wrote: > >> If there is a binary implementation, then you would be using >> #!/lib/init/init-d-script > > Yes, but it would *also* still work with env(1). But it would only be an unnecessary redirection. That said

Bug#826214: Bug#913247: Please provide a C implementation of /lib/init/init-d-script

2018-12-01 Thread Thorsten Glaser
On Sat, 1 Dec 2018, Michael Biebl wrote: > If there is a binary implementation, then you would be using > #!/lib/init/init-d-script Yes, but it would *also* still work with env(1). bye, //mirabilos -- >> Why don't you use JavaScript? I also don't like enabling JavaScript in > Because I use lynx

Bug#826214: Bug#913247: Please provide a C implementation of /lib/init/init-d-script

2018-12-01 Thread Michael Biebl
Am 01.12.18 um 23:31 schrieb Thorsten Glaser: > On Sat, 1 Dec 2018, Michael Biebl wrote: > >> I'd also be interested to know why >> >> #!/usr/bin/env /lib/init/init-d-script >> is preferred over >> #!/bin/sh /lib/init/init-d-script >> given that init-d-script is *no* C implementation. > > That’s

Bug#826214: Bug#913247: Please provide a C implementation of /lib/init/init-d-script

2018-12-01 Thread Thorsten Glaser
On Sat, 1 Dec 2018, Michael Biebl wrote: > I'd also be interested to know why > > #!/usr/bin/env /lib/init/init-d-script > is preferred over > #!/bin/sh /lib/init/init-d-script > given that init-d-script is *no* C implementation. That’s easy: this way, the shebang at the start of /lib/init/init-

Bug#826214: Bug#913247: Please provide a C implementation of /lib/init/init-d-script

2018-12-01 Thread Michael Biebl
Am 01.12.18 um 23:04 schrieb Michael Biebl: > Am 01.12.18 um 23:00 schrieb Michael Biebl: >> I'm still convinced, that fixing this properly in init-d-script would be >> the much better solution. If init-d-script was implemented in C, it >> could be used as a shebang/interpreter on non-Linux and it

Bug#826214: Bug#913247: Please provide a C implementation of /lib/init/init-d-script

2018-12-01 Thread Michael Biebl
Am 01.12.18 um 23:00 schrieb Michael Biebl: > I'm still convinced, that fixing this properly in init-d-script would be > the much better solution. If init-d-script was implemented in C, it > could be used as a shebang/interpreter on non-Linux and it would be > possible to have $0 be set properly. W

Bug#826214: Bug#913247: Please provide a C implementation of /lib/init/init-d-script

2018-12-01 Thread Michael Biebl
Hi Mert! Am 29.11.18 um 14:44 schrieb Mert Dirik: > I've created a merge request for 40-systemd at > https://salsa.debian.org/systemd-team/systemd/merge_requests/19 . > Reviews and comments are welcomed. > > If this is fully applied, systemd redirection will work correctly for > all init-d-script

Bug#826214: Bug#913247: Please provide a C implementation of /lib/init/init-d-script

2018-12-01 Thread Thorsten Glaser
On Sat, 1 Dec 2018, Dmitry Bogatov wrote: > AFAIK, in shell [ "${foo}" ] is equal to [ -n "${foo}" ]. Not always / portably. I recommend test -n "$foo" for POSIX (which is equivalent to [ -n "$foo" ] but better legible and making it clear that test is an external command, not a langua

Bug#826214: Bug#913247: Please provide a C implementation of /lib/init/init-d-script

2018-12-01 Thread Dmitry Bogatov
[2018-11-29 16:44] Mert Dirik > I've created a merge request for 40-systemd at > https://salsa.debian.org/systemd-team/systemd/merge_requests/19 . > Reviews and comments are welcomed. > > If this is fully applied, systemd redirection will work correctly for > all init-d-script scripts (using in

Bug#826214: Bug#913247: Please provide a C implementation of /lib/init/init-d-script

2018-11-29 Thread Mert Dirik
On Fri, 23 Nov 2018 01:24:07 +0100 Michael Biebl wrote: > Hi > > On Thu, 22 Nov 2018 16:01:33 + Ian Jackson > wrote: > > > To the systemd maintainers: will you have time to look at this, and > > make the appropriate change, soon ? If not then one of us could > > probably prepare a patch, if

Bug#826214: Bug#913247: Please provide a C implementation of /lib/init/init-d-script

2018-11-22 Thread Mert Dirik
On 11/23/18, Michael Biebl wrote: > Hi > > On Thu, 22 Nov 2018 16:01:33 + Ian Jackson > wrote: > >> To the systemd maintainers: will you have time to look at this, and >> make the appropriate change, soon ? If not then one of us could >> probably prepare a patch, if that would be helpful. >

Bug#826214: Bug#913247: Please provide a C implementation of /lib/init/init-d-script

2018-11-22 Thread Michael Biebl
Hi On Thu, 22 Nov 2018 16:01:33 + Ian Jackson wrote: > To the systemd maintainers: will you have time to look at this, and > make the appropriate change, soon ? If not then one of us could > probably prepare a patch, if that would be helpful. A patch for 40-systemd would be highly apprecia

Bug#826214: Bug#913247: Please provide a C implementation of /lib/init/init-d-script

2018-11-22 Thread Ian Jackson
Control: retitle -1 Please would 40-systemd honour __init_d_script_name Control: reassign -1 src:systemd Mert Dirik writes ("Re: Bug#826214: Bug#913247: Please provide a C implementation of /lib/init/init-d-script"): > "__init_d_script_name" variable is set inside init-d-s

Bug#826214: Bug#913247: Please provide a C implementation of /lib/init/init-d-script

2018-11-22 Thread Mert Dirik
On 11/22/18, Ian Jackson wrote: > Mert Dirik writes ("Re: Bug#913247: Please provide a C implementation of > /lib/init/init-d-script"): >> On 11/22/18, Ian Jackson wrote: >> > So I think this would be fixed if /lib/init/init-d-script detected >> > this sit

Bug#826214: Bug#913247: Please provide a C implementation of /lib/init/init-d-script

2018-11-22 Thread Ian Jackson
Mert Dirik writes ("Re: Bug#913247: Please provide a C implementation of /lib/init/init-d-script"): > On 11/22/18, Ian Jackson wrote: > > So I think this would be fixed if /lib/init/init-d-script detected > > this situation and set $0 to the original script name (which

Bug#913247: Please provide a C implementation of /lib/init/init-d-script

2018-11-22 Thread Mert Dirik
On 11/22/18, Ian Jackson wrote: > Mert Dirik writes ("Re: Bug#913247: Please provide a C implementation of > /lib/init/init-d-script"): >> On 11/22/18, Ian Jackson wrote: >> > I don't know what `systemd redirection' is. Why does it not work ? >> &g

Bug#826214: Bug#913247: Please provide a C implementation of /lib/init/init-d-script

2018-11-22 Thread Ian Jackson
Mert Dirik writes ("Re: Bug#913247: Please provide a C implementation of /lib/init/init-d-script"): > On 11/22/18, Ian Jackson wrote: > > I don't know what `systemd redirection' is. Why does it not work ? > > Can it be fixed ? > > To sum it up, when

Bug#913247: Please provide a C implementation of /lib/init/init-d-script

2018-11-22 Thread Mert Dirik
On 11/22/18, Ian Jackson wrote: > Mert Dirik writes ("Bug#913247: Please provide a C implementation of > /lib/init/init-d-script"): >> I noticed the new snippet in the man page of experimental version uses >> the >> #! /usr/bin/env /lib/init/init-d-script &

Bug#913247: Please provide a C implementation of /lib/init/init-d-script

2018-11-22 Thread Ian Jackson
Mert Dirik writes ("Bug#913247: Please provide a C implementation of /lib/init/init-d-script"): > I noticed the new snippet in the man page of experimental version uses the > #! /usr/bin/env /lib/init/init-d-script > shebang, which doesn't work with systemd redirec

Bug#913247: Please provide a C implementation of /lib/init/init-d-script

2018-11-21 Thread Mert Dirik
On Tue, 13 Nov 2018 20:22:16 + Dmitry Bogatov wrote: > > [Adding systemd maintainers into thread] > (We are discussing #826214) > > [2018-11-11 22:37] Michael Biebl > > > #!/lib/init/init-d-script? > > > > It doesn't work with #!/lib/init/init-d-script either. > > > > Only > > =

Bug#913247: Please provide a C implementation of /lib/init/init-d-script

2018-11-13 Thread Dmitry Bogatov
[Adding systemd maintainers into thread] (We are discussing #826214) [2018-11-11 22:37] Michael Biebl > > #!/lib/init/init-d-script? > > It doesn't work with #!/lib/init/init-d-script either. > > Only > > #!/bin/sh > > if [ true != "$INIT_D_SCRIPT_SOURCED" ] ; th

Bug#913247: Please provide a C implementation of /lib/init/init-d-script

2018-11-11 Thread Michael Biebl
Am 11.11.18 um 22:08 schrieb Dmitry Bogatov: > > [2018-11-09 22:16] Michael Biebl >> Just tried the >> #!/usr/bin/env /lib/init/init-d-script >> proposal. >> Seem like this breaks the systemctl integration again. >> See https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=826214 > > Wierd. I have n

Bug#913247: Please provide a C implementation of /lib/init/init-d-script

2018-11-11 Thread Dmitry Bogatov
[2018-11-09 22:16] Michael Biebl > Just tried the > #!/usr/bin/env /lib/init/init-d-script > proposal. > Seem like this breaks the systemctl integration again. > See https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=826214 Wierd. I have no systemd box to check, but are you sure it works with #!

Bug#913247: Please provide a C implementation of /lib/init/init-d-script

2018-11-09 Thread Michael Biebl
Am 09.11.18 um 21:01 schrieb Dmitry Bogatov: > > [2018-11-08 19:06] Michael Biebl >> Am 08.11.18 um 18:54 schrieb Ian Jackson: >> >>> env(1) would be helpful here. If your daemon is on /usr anyway then >>> it doesn't matter that it isn't in /bin. >> >> /usr is mounted by the initramfs nowadays,

Bug#913247: Please provide a C implementation of /lib/init/init-d-script

2018-11-09 Thread Dmitry Bogatov
[2018-11-08 19:06] Michael Biebl > Am 08.11.18 um 18:54 schrieb Ian Jackson: > > > env(1) would be helpful here. If your daemon is on /usr anyway then > > it doesn't matter that it isn't in /bin. > > /usr is mounted by the initramfs nowadays, so this wouldn't be an issue > in practice. I mad

Bug#913247: Please provide a C implementation of /lib/init/init-d-script

2018-11-08 Thread Michael Biebl
Am 08.11.18 um 18:54 schrieb Ian Jackson: > Michael Biebl writes ("Bug#913247: Please provide a C implementation of > /lib/init/init-d-script"): >> Package: sysvinit-utils >> Version: 2.88dsf-59.11 >> Severity: normal >> File: /lib/init/init-d-script >

Bug#913247: Please provide a C implementation of /lib/init/init-d-script

2018-11-08 Thread Michael Biebl
Am 08.11.18 um 18:54 schrieb Ian Jackson: > env(1) would be helpful here. If your daemon is on /usr anyway then > it doesn't matter that it isn't in /bin. /usr is mounted by the initramfs nowadays, so this wouldn't be an issue in practice. -- Why is it that all of the instruments seeking inte

Bug#913247: Please provide a C implementation of /lib/init/init-d-script

2018-11-08 Thread Ian Jackson
Michael Biebl writes ("Bug#913247: Please provide a C implementation of /lib/init/init-d-script"): > Package: sysvinit-utils > Version: 2.88dsf-59.11 > Severity: normal > File: /lib/init/init-d-script ... > This is such an ugly construct that so far I did not want to us

Bug#913247: Please provide a C implementation of /lib/init/init-d-script

2018-11-08 Thread Michael Biebl
Package: sysvinit-utils Version: 2.88dsf-59.11 Severity: normal File: /lib/init/init-d-script Hi! The skeleton file using init-d-script contains: ''' #!/bin/sh # kFreeBSD do not accept scripts as interpreters, using #!/bin/sh and sourcing. if [ true != "$INIT_D_SCRIPT_SOURCED" ] ; then set