On 2016-02-12 15:34, shraptor wrote:
> On 2016-02-12 10:50, Didier Kryn wrote:
>> Le 12/02/2016 02:54, Rainer Weikusat a écrit :
>>> Didier Kryn <kryn@???> writes:
>>>
>>>> Le 11/02/2016 17:04, Rainer Weikusat a écrit :
>>>>> Didier Kryn <kryn@???> writes:
>>>>>
>>>>> [...]
>>>>>
>>>>>>> It should be the name of a shell capable of running Bourne/
>>>>>>> standard
>>>>>>> shell scripts. But this may not work if the /bin/dash in the
>>>>>>> original
>>>>>>> script was there for a reason, ie, it was using dash features.
>>>>>>>
>>>>>> As I already wrote, vdev was working well with busybox's
>>>>>> ash.,
>>>>>> replacing 'dash' with 'sh' in the shebang.
>>>>>>
>>>>>> If the question is why Jude replaced /bin/sh with /bin/dash
>>>>>> in the
>>>>>> middle of the development, I think it was to make sure to not
>>>>>> invoke
>>>>>> bash. But (sorry for the repetition) I used to modify the shebang
>>>>>> everytime I tested a new version, and there was never any issue
>>>>>> with
>>>>>> the shell.
>>>>> There is no question here. *If* the script in question uses dash
>>>>> spuriously, ie, it doesn't use features specific to dash but is
>>>>> actually
>>>>> a Bourne shell script, replacing /bin/dash with /bin/sh should be
>>>>> fine. If not, stuff is going to break sooner or later, either
>>>>> because
>>>>> /bin/sh isn't really dash (eg, someone might use bash for that) or
>>>>> because of difference between the busybox and Debian (d)ash forks.
>>>>>
>>>> There shouldn't be any "feature specific to dash", by
>>>> construction.
>>> There are, "by construction". Eg, dash supports local, the POSIX
>>> /bin/sh
>>> doesn't.
>> Then it seems Jude's scripts don't use that feature, and they
>> shouldn't.
>
>
> If you check *.sh in vdev/vdevd/helpers/LINUX on git they use local
>
I dug up some very old correspondence I had with Jude regarding why he
uses dash
I said
"however I dont have dash on my system so I
had to symlink it to bash to make vdev run
vdevd/helpers/LINUX/*.sh files"
Here is what he said then 2015-05-22 06:25
"That should be temporary. I'm currently going through the scripts to
make sure they work with Devuan's system shell (dash). When I create
the "-release" branch, I will switch back to #!/bin/sh."
/Scooby