NetBSD source update | 1 Jan 08:34 2007
Picon

daily pkgsrc CVS update output


Updating pkgsrc tree:
? pkgsrc/README-IPv6.html
? pkgsrc/README-all.html
? pkgsrc/net/wget/work
P pkgsrc/devel/monotone/Makefile
U pkgsrc/doc/CHANGES-2007
P pkgsrc/doc/TODO
P pkgsrc/mk/check/check-portability.sh
P pkgsrc/mk/check/check-subr.awk
P pkgsrc/print/lilypond/Makefile

Killing core files:

Updating pkgsrc-2006Q3 pkgsrc tree (/ftp/pub/pkgsrc/pkgsrc-2006Q3):
cvs update: warning: cannot open /cvsroot/CVSROOT/val-tags read/write: Permission denied

Georg Schwarz | 1 Jan 17:10 2007
Picon

best practise of adding dependency on getopt.h?

I discovered that a package (wip/weplab) requires getopt.h to compile  
which is not present on Solaris 7.
I added the following lines to the Makefile:

.if !exists(/usr/include/getopt.h)
.include "../../devel/libgetopt/buildlink3.mk"
LIBS+=  ${BUILDLINK_LDADD.getopt:Q}
.endif

is this the right way to do it? Should I in addition evaluate some  
"Prefer pkgsrc" variable and, if that one is set, include pkgsrc's  
libgetopt?

A more general question: should BUILDLINK_LDADD.* be added to LIBS or  
to LDFLAGS?

--

-- 
Georg Schwarz    http://home.pages.de/~schwarz/
georg.schwarz <at> freenet.de  +49 151 11559652

Christian Biere | 1 Jan 23:35 2007
Picon
Picon

Re: best practise of adding dependency on getopt.h?

Georg Schwarz wrote:
> I discovered that a package (wip/weplab) requires getopt.h to compile  
> which is not present on Solaris 7.

Does it need getopt.h or does it simply include the header file
without checking for its existence? Why is the file used at all?

--

-- 
Christian

Georg Schwarz | 1 Jan 23:37 2007
Picon

Re: best practise of adding dependency on getopt.h?


Am 01.01.2007 um 23:35 schrieb Christian Biere:

> Does it need getopt.h or does it simply include the header file
> without checking for its existence? Why is the file used at all?

in its main.c it simply has
#include <getopt.h>

--

-- 
Georg Schwarz    http://home.pages.de/~schwarz/
georg.schwarz <at> freenet.de  +49 151 11559652

Christian Biere | 1 Jan 23:46 2007
Picon
Picon

Re: best practise of adding dependency on getopt.h?

Georg Schwarz wrote:
> 
> Am 01.01.2007 um 23:35 schrieb Christian Biere:
> 
> >Does it need getopt.h or does it simply include the header file
> >without checking for its existence? Why is the file used at all?
> 
> in its main.c it simply has
> #include <getopt.h>

Yes, I've just seen this. Does GCC silently provide getopt_long()?
Otherwise you need the additional library anyway.

--

-- 
Christian

Georg Schwarz | 2 Jan 00:32 2007
Picon

Re: best practise of adding dependency on getopt.h?


Am 01.01.2007 um 23:46 schrieb Christian Biere:

> Yes, I've just seen this. Does GCC silently provide getopt_long()?
> Otherwise you need the additional library anyway.

yes, I think so, and unconditionally adding

.include "../../devel/libgetopt/buildlink3.mk"
LIBS+=  ${BUILDLINK_LDADD.getopt:Q}

seems like the best way to me.

--

-- 
Georg Schwarz    http://home.pages.de/~schwarz/
georg.schwarz <at> freenet.de  +49 151 11559652

NetBSD source update | 2 Jan 08:32 2007
Picon

daily pkgsrc CVS update output


Updating pkgsrc tree:
? pkgsrc/README-IPv6.html
? pkgsrc/README-all.html
? pkgsrc/net/wget/work
P pkgsrc/devel/wide/Makefile
P pkgsrc/doc/TODO
P pkgsrc/emulators/vmware/Makefile
P pkgsrc/emulators/vmware3/Makefile
P pkgsrc/math/gsl/distinfo
U pkgsrc/math/gsl/patches/patch-ae
U pkgsrc/math/gsl/patches/patch-af

Killing core files:

Updating pkgsrc-2006Q3 pkgsrc tree (/ftp/pub/pkgsrc/pkgsrc-2006Q3):
cvs update: warning: cannot open /cvsroot/CVSROOT/val-tags read/write: Permission denied

Joerg Sonnenberger | 2 Jan 10:58 2007
Picon

Re: best practise of adding dependency on getopt.h?

On Mon, Jan 01, 2007 at 05:10:02PM +0100, Georg Schwarz wrote:
> I discovered that a package (wip/weplab) requires getopt.h to compile  
> which is not present on Solaris 7.

Please check *first* whether it is actually using getopt_long or
getopt_long_only. If it is only using getopt, it should include unistd.h
and be patched accordingly. If it does not, the conditional version it
ok.

Joerg

NetBSD source update | 3 Jan 08:33 2007
Picon

daily pkgsrc CVS update output


Updating pkgsrc tree:
? pkgsrc/INDEX
? pkgsrc/README-IPv6.html
? pkgsrc/README-all.html
? pkgsrc/net/wget/work
P pkgsrc/audio/csound5/distinfo
P pkgsrc/audio/csound5/patches/patch-ac
P pkgsrc/audio/xmcd/Makefile
P pkgsrc/benchmarks/httperf/Makefile
P pkgsrc/benchmarks/httperf/distinfo
U pkgsrc/benchmarks/httperf/patches/patch-ad
P pkgsrc/cad/qcad-partlibrary/Makefile
P pkgsrc/converters/skf/Makefile
P pkgsrc/converters/skf/distinfo
P pkgsrc/databases/nss_ldap/Makefile
P pkgsrc/databases/openldap/Makefile
P pkgsrc/databases/openldap/Makefile.common
P pkgsrc/databases/openldap/distinfo
cvs update: pkgsrc/databases/openldap/patches/patch-ap is no longer in the repository
P pkgsrc/databases/openldap-client/Makefile
P pkgsrc/databases/openldap-doc/Makefile
P pkgsrc/databases/openldap-server/Makefile
P pkgsrc/databases/p5-DBD-Oracle/Makefile
P pkgsrc/databases/pgadmin3/Makefile
P pkgsrc/devel/libextractor/Makefile
P pkgsrc/devel/libextractor/PLIST
P pkgsrc/devel/libextractor/distinfo
P pkgsrc/devel/libextractor/patches/patch-ab
P pkgsrc/devel/libextractor/patches/patch-ac
(Continue reading)

Jukka Salmi | 3 Jan 18:00 2007
Picon

Re: CVS commit: pkgsrc/net/gethost

Hi,

Todd Vierling --> pkgsrc-changes (2007-01-03 15:51:24 +0000):
> 
> Module Name:	pkgsrc
> Committed By:	tv
> Date:		Wed Jan  3 15:51:24 UTC 2007
> 
> Modified Files:
> 	pkgsrc/net/gethost: Makefile
> 
> Log Message:
> Require "inet6" option to be present.
> 
> 
> To generate a diff of this commit:
> cvs rdiff -r1.6 -r1.7 pkgsrc/net/gethost/Makefile
> 
> Please note that diffs are not public domain; they are subject to the
> copyright notices on the relevant files.

hmm, if group `inet6' is required and consists only of option `inet6',
why is `inet6' an option at all? (same for net/dnstracer, net/fping6,
net/netcat6, net/scamper, net/sipcalc, net/totd, www/www6to4, et al.)

Regards, Jukka

--

-- 
bashian roulette:
$ ((RANDOM%6)) || rm -rf ~
(Continue reading)


Gmane