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 '.' admin_scripts.patch
Sure - patch attached.
that patch isn't enough, here's a new version
new patch with other bashisms fixed admin_scripts.diff
This patch is for 389-ds-base, not admin server. Set to needs triage...
and more bashisms fixed fix-bashisms
I've fixed the Debian packaging to just use bash instead of sh, maybe that's easiest here too.
git patch file (master) 0001-Ticket-47511-bashisms-in-389-ds-base-admin-scripts.patch
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.
Thanks!
Pushed to master: a799c46..2ce7a73 master -> master commit 2ce7a73
Pushed to 389-ds-base-1.3.4: 29c669e..4924591 389-ds-base-1.3.4 -> 389-ds-base-1.3.4 commit 4924591
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: - https://github.com/389ds/389-ds-base/issues/848
If you want to receive further updates on the issue, please navigate to the github issue and click on subscribe button.
subscribe
Thank you for understanding. We apologize for all inconvenience.
Metadata Update from @spichugi: - Issue close_status updated to: wontfix (was: Fixed)
Login to comment on this ticket.