Update ldapsh port to 2.00. Make use of new SUB_FILES variable. Use newly created Makefile.PL. Blah.
Build package on 4.x fails: ===> Building package for ldapsh-2.00,1 Creating package /usr/ports/packages/All/ldapsh-2.00,1.tgz Registering depends: p5-perl-ldap-0.32.02 p5-Convert-ASN1-0.18 p5-Date-Manip-5.42a p5-Digest-MD5-2.33 p5-Digest-1.08 p5-Getopt-Long-2.34 p5-IO-Socket-SSL-0.96 p5-URI-1.35 p5-Net-SSLeay-1.25 p5-ParseLex-2.15 p5-ReadLine-Perl-1.0203 p5-Term-ReadKey-2.21 p5-URI-1.34 p5-MIME-Base64-3.05 p5-XML-SAX-Base-1.04 p5-Authen-SASL-2.08. Creating gzip'd tar ball in '/usr/ports/packages/All/ldapsh-2.00,1.tgz' tar: lib/perl5/5.00503/man/man3/Net\:\:LDAP\:\:Config.3.gz: Cannot stat: No such file or directory tar: Error exit delayed from previous errors pkg_create: make_dist: tar command failed with code 512 *** Error code 1 Stop in /usr/ports/net/ldapsh. -- Sem.
Sergey Matveychuk wrote: > Build package on 4.x fails: > ===> Building package for ldapsh-2.00,1 > Creating package /usr/ports/packages/All/ldapsh-2.00,1.tgz > Registering depends: p5-perl-ldap-0.32.02 p5-Convert-ASN1-0.18 > p5-Date-Manip-5.42a p5-Digest-MD5-2.33 p5-Digest-1.08 > p5-Getopt-Long-2.34 p5-IO-Socket-SSL-0.96 p5-URI-1.35 p5-Net-SSLeay-1.25 > p5-ParseLex-2.15 p5-ReadLine-Perl-1.0203 p5-Term-ReadKey-2.21 > p5-URI-1.34 p5-MIME-Base64-3.05 p5-XML-SAX-Base-1.04 p5-Authen-SASL-2.08. > Creating gzip'd tar ball in '/usr/ports/packages/All/ldapsh-2.00,1.tgz' > tar: lib/perl5/5.00503/man/man3/Net\:\:LDAP\:\:Config.3.gz: Cannot stat: > No such file or directory > tar: Error exit delayed from previous errors > pkg_create: make_dist: tar command failed with code 512 > *** Error code 1 > > Stop in /usr/ports/net/ldapsh. Works here on 4.x with perl5.8. Do you have these lines before package building ? Installing /usr/local/man/man1/ldapsh.1 Installing /usr/local/lib/perl5/5.8.5/man/man3/Net::LDAP::Config.3 Installing /usr/local/lib/perl5/5.8.5/man/man3/Net::LDAP::Shell.3 Installing /usr/local/lib/perl5/5.8.5/man/man3/Net::LDAP::Shell::Parse.3 Installing /usr/local/lib/perl5/5.8.5/man/man3/Net::LDAP::Shell::Util.3
===> Building for ldapsh-2.00,1 mkdir blib mkdir blib/lib cp lib/Net/LDAP/Shell/Commands/edit.pm blib/lib/Net/LDAP/Shell/Commands/edit.pm cp lib/Net/LDAP/Shell/Commands/help.pm blib/lib/Net/LDAP/Shell/Commands/help.pm cp lib/Net/LDAP/Shell/Commands/new.pm blib/lib/Net/LDAP/Shell/Commands/new.pm cp lib/Net/LDAP/Shell/Parser.pm blib/lib/Net/LDAP/Shell/Parser.pm cp lib/Net/LDAP/Config.pm blib/lib/Net/LDAP/Config.pm cp lib/Net/LDAP/Shell.pm blib/lib/Net/LDAP/Shell.pm cp lib/Net/LDAP/Shell/Schema.pm blib/lib/Net/LDAP/Shell/Schema.pm cp lib/Net/LDAP/Shell/Commands/clone.pm blib/lib/Net/LDAP/Shell/Commands/clone.pm cp lib/Net/LDAP/Shell/Commands/commands.pm blib/lib/Net/LDAP/Shell/Commands/commands.pm cp lib/Net/LDAP/Shell/makefile blib/lib/Net/LDAP/Shell/makefile cp lib/Net/LDAP/Shell/grammar.yp blib/lib/Net/LDAP/Shell/grammar.yp cp lib/Net/LDAP/Shell/Commands/search.pm blib/lib/Net/LDAP/Shell/Commands/search.pm cp lib/Net/LDAP/Shell/Commands/cat.pm blib/lib/Net/LDAP/Shell/Commands/cat.pm cp lib/Net/LDAP/Shell/Complete.pm blib/lib/Net/LDAP/Shell/Complete.pm cp lib/Net/LDAP/Shell/Parse.pm blib/lib/Net/LDAP/Shell/Parse.pm cp lib/Net/LDAP/Shell/Util.pm blib/lib/Net/LDAP/Shell/Util.pm cp lib/Net/LDAP/Shell/Commands/stub.pm blib/lib/Net/LDAP/Shell/Commands/stub.pm cp lib/Net/LDAP/Shell/Desc.pm blib/lib/Net/LDAP/Shell/Desc.pm cp lib/Net/LDAP/Shell/Commands/ls.pm blib/lib/Net/LDAP/Shell/Commands/ls.pm cp lib/Net/LDAP/Shell/Commands/rm.pm blib/lib/Net/LDAP/Shell/Commands/rm.pm mkdir blib/arch mkdir blib/arch/auto mkdir blib/arch/auto/ldapsh mkdir blib/lib/auto/ldapsh mkdir blib/man1 mkdir blib/man3 mkdir blib/script cp ldapsh blib/script/ldapsh /usr/bin/perl5.00503 -I/usr/libdata/perl/5.00503/mach -I/usr/libdata/perl/5.00503 -MExtUtils::MakeMaker -e "MY->fixin(shift)" blib/script/ldapsh /usr/bin/pod2man: bad option in paragraph 21 of ldapsh: ``-D'' should be [CB]<-D> /usr/bin/pod2man: bad option in paragraph 35 of ldapsh: ``-D'' should be [CB]<-D> Manifying blib/man3/Net::LDAP::Shell::Parse.3 Manifying blib/man3/Net::LDAP::Shell::Util.3 Manifying blib/man1/ldapsh.1 Manifying blib/man3/Net::LDAP::Shell.3 -- Sem.
Sergey Matveychuk wrote: > /usr/bin/perl5.00503 -I/usr/libdata/perl/5.00503/mach > -I/usr/libdata/perl/5.00503 -MExtUtils::MakeMaker -e "MY->fixin(shift)" > blib/script/ldapsh > /usr/bin/pod2man: bad option in paragraph 21 of ldapsh: ``-D'' should be > [CB]<-D> > /usr/bin/pod2man: bad option in paragraph 35 of ldapsh: ``-D'' should be > [CB]<-D> Ok, so maybe I should set dependency on perl >= 5.[68], don't know exactly. I'll test it soon. Thanks for pointing that out.
Florent Thoumie wrote: > Sergey Matveychuk wrote: > >> /usr/bin/perl5.00503 -I/usr/libdata/perl/5.00503/mach >> -I/usr/libdata/perl/5.00503 -MExtUtils::MakeMaker -e >> "MY->fixin(shift)" blib/script/ldapsh >> /usr/bin/pod2man: bad option in paragraph 21 of ldapsh: ``-D'' should >> be [CB]<-D> >> /usr/bin/pod2man: bad option in paragraph 35 of ldapsh: ``-D'' should >> be [CB]<-D> > > > Ok, so maybe I should set dependency on perl >= 5.[68], don't know exactly. > > I'll test it soon. Thanks for pointing that out. But if the problem is one file, may be we can make a conditional add it in plist? Something like: MAN3=... .if ${PERL_LEVEL} < 500600 MAN3+=Net::LDAP::Config.3 .endif -- Sem.
Sergey Matveychuk wrote: > But if the problem is one file, may be we can make a conditional add it > in plist? > Something like: > MAN3=... > .if ${PERL_LEVEL} < 500600 > MAN3+=Net::LDAP::Config.3 > .endif Are you sure that's only one file ? There are 4 manpages for section 3. So I guess it'll break for the next three. I'm not a perl guru and I don't know if we can consider these manpages as optional since what we want to install is ldapsh (and not these perl modules). If we decide that they're optional, something like what you wrote would be good (replacing '<' with '>' and assuming that only Net::LDAP::Config.3 is missing).
Sergey Matveychuk wrote: > But if the problem is one file, may be we can make a conditional add it > in plist? > Something like: > MAN3=... > .if ${PERL_LEVEL} < 500600 > MAN3+=Net::LDAP::Config.3 > .endif I'm not sure we should consider these manpages as optional. If we do, something like what you wrote would be good (replacing '<' with '>' and assuming that only Net::LDAP::Config.3 is missing).
Sorry for the second mail, I thought the first was lost.
Sergey Matveychuk wrote: > Yes, I'm sure. It works, but it's a hack of course. We should have some > fix. Or we can mark the port as only for perl 5.6+ but it's not good > idea because just one file. > > BTW, the port miss dependence on textproc/p5-PodParser as I can see on > 4.x with system perl. Ok, I'll try to see what I can do to remove this dependency later. In the meantime, such a trick will be good. I was wrong when I said I tested it on 4.x since I upgraded my machine some weeks ago. I'll set up a jail to avoid these kind of problems.
Responsible Changed From-To: freebsd-ports-bugs->sem Assign to sem, who demonstrated interest in this PR already.
State Changed From-To: open->closed Committed, thanks!