>From what Eric said, the script starts with #!/bin/dash to document the fact
that it doesn't use bash-specific extensions.  Since dash is supposed to be a
POSIX-compliant shell, it should be safe to change this to /bin/sh in the
FreeBSD package.  Even if you add a dependency on dash, you will have to have
a patch to change the shebang to point to /usr/local/bin/dash.

__Martin


>>>>> On Tue, 30 Mar 2021 12:19:38 -0400, Dan Langille said:
> 
> 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
> d...@langille.org
> 
> 
> > On Mar 30, 2021, at 12:11 PM, Eric Bollengier via Bacula-devel 
> > <bacula-devel@lists.sourceforge.net> 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
> > Bacula-devel@lists.sourceforge.net
> > https://lists.sourceforge.net/lists/listinfo/bacula-devel
> 
> 


_______________________________________________
Bacula-devel mailing list
Bacula-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/bacula-devel

Reply via email to