Re: shebang fix to scripts/baculabackupreport.in

Dan Langille <[email protected]> Tue, 30 Mar 2021 12:19:38 -0400

Newsgroups	gmane.comp.sysutils.backup.bacula.devel
Message-ID	<[email protected]>

Oh.

I think adding another dependency is unwise, but it appears to be a done-deal.

I suppose something was valuable in /bin/dash which was not available in /bin/bash.

I'll add another dependency to the package.

-
Dan Langille - BSDCan / PGCon
[email protected]


> On Mar 30, 2021, at 12:11 PM, Eric Bollengier via Bacula-devel <[email protected]> wrote:
> 
> Hello,
> 
> The program is written in DASH, it may work if you replace it with Bash,
> but the two shells are not compatible.
> 
> In the old time, /bin/sh was pointing to bash, but bash is way more
> powerful than the regular /bin/sh. Then, Ubuntu guys have replaced
> /bin/sh -> bash by /bin/sh -> dash for performance reasons (the result
> was quite painful for all shell writers), and it's quite easy now to
> write a shell script on Redhat using /bin/sh that will not work on
> Ubuntu. At least, it's clear.
> 
> Best Regards,
> Eric
> 
> On 3/30/21 18:03, Martin Simmons wrote:
>> /bin/dash is another shell, which is typically used as the system shell on
>> Ubuntu and Debian.
>> 
>> I'm not sure which repo you are looking at, but my copy has /bin/dash in that
>> file
>> 
>> __Martin
>> 
>> 
>>>>>>> On Mon, 29 Mar 2021 14:23:59 -0400, Dan Langille said:
>>> 
>>> I tried to create a pull request for you, but couldn't find this code within the cloned repo.
>>> 
>>> I checked out Branch-11.0 but the error was not there.
>>> 
>>> The tarball for 11.0.2 contains this in bacula-11.0.2/scripts/baculabackupreport.in
>>> 
>>> #!/bin/dash
>>> 
>>> Perhaps that's meant to be #!/bin/bash
>>> 
> 
> 
> _______________________________________________
> Bacula-devel mailing list
> [email protected]
> https://lists.sourceforge.net/lists/listinfo/bacula-devel

_______________________________________________
Bacula-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/bacula-devel

signature.asc (application/pgp-signature, 658 B)