The shell scripts in 389-ds-base/ldap/admin/src/scripts use 'source' to source common scripts; the 'source' keyword is bash-specific (or c-shell, if memory serves). The interpreter is set to /bin/sh, which is not guaranteed to be bash (and at least on Debian 7.1 it is dash).
The 'source' keyword can be replaced by '.', which should work.
Can you provide patches?
patch to replace 'source' with '.'
Sure - patch attached.
that patch isn't enough, here's a new version
new patch with other bashisms fixed
This patch is for 389-ds-base, not admin server. Set to needs triage...
and more bashisms fixed
I've fixed the Debian packaging to just use bash instead of sh, maybe that's easiest here too.
git patch file (master)
I merged the patchesadmin_scripts.diff and fix-bashisms provided by tjaalton into one 0001-Ticket-47511-bashisms-in-389-ds-base-admin-scripts.patch with a couple of minor fixes.
Reviewed and tested by nhosoi.
Pushed to master:
a799c46..2ce7a73 master -> master
Pushed to 389-ds-base-1.3.4:
29c669e..4924591 389-ds-base-1.3.4 -> 389-ds-base-1.3.4
Ticket has been cloned to Bugzilla: https://bugzilla.redhat.com/show_bug.cgi?id=1252207
Metadata Update from @nhosoi:
- Issue assigned to nhosoi
- Issue set to the milestone: 1.3.4 backlog
389-ds-base is moving from Pagure to Github. This means that new issues and pull requests
will be accepted only in 389-ds-base's github repository.
This issue has been cloned to Github and is available here:
If you want to receive further updates on the issue, please navigate to the github issue
and click on subscribe button.
Thank you for understanding. We apologize for all inconvenience.
Metadata Update from @spichugi:
- Issue close_status updated to: wontfix (was: Fixed)
to comment on this ticket.