Re: [Bacula-devel] shebang fix to scripts/baculabackupreport.in

2021-03-30 Thread Martin Simmons
>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

Re: [Bacula-devel] shebang fix to scripts/baculabackupreport.in

2021-03-30 Thread Dan Langille
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,

Re: [Bacula-devel] shebang fix to scripts/baculabackupreport.in

2021-03-30 Thread Eric Bollengier via Bacula-devel
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 perf

Re: [Bacula-devel] shebang fix to scripts/baculabackupreport.in

2021-03-30 Thread Martin Simmons
/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,

[Bacula-devel] shebang fix to scripts/baculabackupreport.in

2021-03-29 Thread Dan Langille
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 -- D