#48783 Syntax error in ldap/admin/src/scripts/ns-accountstatus.pl.in
Closed: wontfix None Opened 8 years ago by nhosoi.

Opening a ticket on behalf of Andrey Cherepanov "cas@altlinux.ru":
On 03/31/2016 03:46 AM, Andrey Cherepanov wrote:

Our build system detects syntax error in ldap/admin/src/scripts/ns-accountstatus.pl.in:

String found where operator expected at /usr/src/tmp/389-ds-base-buildroot/usr/sbin/ns-accountstatus.pl line 34, near "STDERR " [-i] [-g seconds]\n\n""
(Do you need to predeclare STDERR?)
syntax error at /usr/src/tmp/389-ds-base-buildroot/usr/sbin/ns-accountstatus.pl line 34, near "print"

See attached patch to fix it.

Small note: ldap/admin/src/scripts/db2index.in use Bash4 syntax construction (|&), but use interpreter /bin/sh
Not all Linux distributives have /bin/sh as symlink to /bin/bash4


commit 694a0b613d0c2115fe383ec7dcc52f2ab2cc4af3
Total 7 (delta 5), reused 0 (delta 0)
To ssh://git.fedorahosted.org/git/389/ds.git
16445ac..10787a0 master -> master

Reviewed and commited. Thanks for the patch Andrey!

Metadata Update from @firstyear:
- Issue set to the milestone: 1.3.5.2

7 years ago

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/1843

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)

3 years ago

Login to comment on this ticket.

Metadata