nfs-utils does NOT rdepend on bash

nfs-utils does NOT rdepend on bash

[Mike Looijmans]

To my surprise, a megabyte sized package "bash" was installed on target 
when I installed nfs-utils to start the NFS server.

Commit 664ae3dc52fd7fc8c6f64e6cf5e70f97dedd332d claims the package needs 
bash.

It's simply not true. It does not require bash at all, it runs just fine 
with busybox's ash shell for example.

I don't understand the reasoning behind that commit. There's nothing in 
nfs-utils that really needs bash, it runs fine without it.



Also, there is an error in the startup script, I can submit a patch for 
that if you like. The script is needlessly complex too.


-- 
Mike Looijmans

Re: nfs-utils does NOT rdepend on bash

[ChenQi]

On 04/20/2014 01:44 AM, Mike Looijmans wrote:
> To my surprise, a megabyte sized package "bash" was installed on 
> target when I installed nfs-utils to start the NFS server.
>
> Commit 664ae3dc52fd7fc8c6f64e6cf5e70f97dedd332d claims the package 
> needs bash.
>
> It's simply not true. It does not require bash at all, it runs just 
> fine with busybox's ash shell for example.
>

How about the start-statd script?
It begins with '#!/bin/bash -p'.
If you de-bash it, I think we can remove the dependency.

//Chen Qi

> I don't understand the reasoning behind that commit. There's nothing 
> in nfs-utils that really needs bash, it runs fine without it.
>
>
>
> Also, there is an error in the startup script, I can submit a patch 
> for that if you like. The script is needlessly complex too.
>
>

Re: nfs-utils does NOT rdepend on bash

[Mike Looijmans]

On 04/22/2014 08:02 AM, ChenQi wrote:
> On 04/20/2014 01:44 AM, Mike Looijmans wrote:
>> To my surprise, a megabyte sized package "bash" was installed on target when
>> I installed nfs-utils to start the NFS server.
>>
>> Commit 664ae3dc52fd7fc8c6f64e6cf5e70f97dedd332d claims the package needs bash.
>>
>> It's simply not true. It does not require bash at all, it runs just fine
>> with busybox's ash shell for example.
>>
>
> How about the start-statd script?
> It begins with '#!/bin/bash -p'.
> If you de-bash it, I think we can remove the dependency.

I'll take a look, shouldn't be hard. And it is not required for the server 
anyway, so if the bashism can't be easily removed, I think start-statd should 
be moved into another package so that only that part will draw in the bash 
dependency.

I'll try to create separate patches, one "de-bash" patch, and one to fix and 
simplify the server startup script.



>> I don't understand the reasoning behind that commit. There's nothing in
>> nfs-utils that really needs bash, it runs fine without it.
>>
>>
>>
>> Also, there is an error in the startup script, I can submit a patch for that
>> if you like. The script is needlessly complex too.
>>
>>
>



Met vriendelijke groet / kind regards,

Mike Looijmans

TOPIC Embedded Systems
Eindhovenseweg 32-C, NL-5683 KH Best
Postbus 440, NL-5680 AK Best
Telefoon: (+31) (0) 499 33 69 79
Telefax:  (+31) (0) 499 33 69 70
E-mail: mike.looijmans@topic.nl
Website: www.topic.nl

Please consider the environment before printing this e-mail

Bezoek ons op 7 en 8 mei tijdens High-Tech Systems Den Bosch (stand 24)
http://www.hightechsystems.nl