Could reproduce the problem.

Fedora release 23 (Twenty Three)
{{{
Smartmatch is experimental at /usr/share/dirsrv/updates/52updateAESplugin.pl line 58, <STDIN> line 2.
Error: not applying update '/usr/share/dirsrv/updates/50fixNsState.pl'. Error: Can't locate bigint.pm in @INC (you may need to install the bigint module) (@INC contains: /usr/lib64/dirsrv/perl /usr/local/lib64/perl5 /usr/local/share/perl5 /usr/lib64/perl5/vendor_perl /usr/share/perl5/vendor_perl /usr/lib64/perl5 /usr/share/perl5 .) at /usr/share/dirsrv/updates/50fixNsState.pl line 7, <STDIN> line 2.
BEGIN failed--compilation aborted at /usr/share/dirsrv/updates/50fixNsState.pl line 7, <STDIN> line 2.
Compilation failed in require at (eval 16) line 1, <STDIN> line 2.
}}}

rpm -qa perl | egrep -i bigint

"bigint" is sub-packaged since perl 5.22.0-347.
* Wed Jul 08 2015 Petr Pisar ppisar@redhat.com - 4:5.22.0-347
- Sub-package Math-BigInt
{{{
Note: the current perl version on FC23:
perl-5.22.0-349.fc23.x86_64
Math::BigInt:
perl-Math-BigInt-1.9997-349.fc23.noarch
There is no bigint in the package any more.

The current perl version of FC22:
perl-5.20.3-328.fc22.x86_64
bigint and BigInt are both in the main package perl.
$ rpm -qf /usr/share/perl5/bigint.pm
perl-5.20.3-328.fc22.x86_64
$ rpm -qf /usr/share/perl5/Math/BigInt.pm
perl-5.20.3-328.fc22.x86_64
}}}

git patch file (master)
0001-Ticket-48348-Running-usr-sbin-setup-ds.pl-fails-with.patch

{{{
7 use bigint;
7 use Math::BigInt;
}}}
Will this work on all platforms, or do we need to do some sort of conditional check e.g.
{{{
use Socket;
$sockVersion = Socket->VERSION;
if ($sockVersion >= 2.000) {
import Socket qw ( :addrinfo inet_ntoa
unpack_sockaddr_in unpack_sockaddr_in6
AF_INET INADDR_ANY
PF_INET SO_REUSEADDR SOCK_STREAM SOL_SOCKET );
} elsif (eval {require Socket6; 1}) {
import Socket6 qw (getaddrinfo getnameinfo unpack_sockaddr_in6);
}
}}}
how we conditionally require Socket or Socket6 depending on what platform we are using.

I'm thinking to have the patch on the version 1.3.4 and newer.

1.3.4 is on F22, on which I verified the patch. I have no plan to update 1.3.3 on F21.

Do we still need the condition?

I verified it works with perl-5.10.1-141 on epel6.

ok - if it works on el6, that's early enough

Ack, works for me (el7)

Reviewed by Rich and William (Thank you!!)

Pushed to master:
a534583..65362db master -> master
commit dfc9ad0

Pushed to 389-ds-base-1.3.4:
a4c0a9e..c0d35f7 389-ds-base-1.3.4 -> 389-ds-base-1.3.4
commit 9cf6284

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

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.