Re: [joerg%netbsd.org@localhost: CVS commit: pkgsrc/devel/nbpatch]

[Eric Gillespie <epg%pretzelnet.org@localhost>]

Joerg Sonnenberger <joerg%britannica.bec.de@localhost> writes:

> I'd appreciate testing on all platforms to sort out the remaining
> modernisms in this code.

On Linux:

backupfile.c:119: error: 'struct dirent' has no member named 'd_namlen'

I think you have to check for d_namlen in configure and
strlen(d_name) when missing.  Linux sucks, I know.

I hacked around that, and get:

patch.o: In function `get_some_switches':
patch.c:(.text+0x38): undefined reference to `optreset'
patch.c:(.text+0x240): undefined reference to `strlcpy'
patch.o: In function `main':
patch.c:(.text+0x1da3): undefined reference to `strlcpy'
patch.c:(.text+0x1dca): undefined reference to `strlcat'
pch.o: In function `pgets':
pch.c:(.text+0x214): undefined reference to `strlcpy'
pch.o: In function `another_hunk':
pch.c:(.text+0x223e): undefined reference to `strlcpy'
pch.c:(.text+0x23af): undefined reference to `strlcpy'
pch.c:(.text+0x30b6): undefined reference to `strlcpy'
util.o: In function `backup_file':
util.c:(.text+0x5f9): undefined reference to `strlcpy'
util.c:(.text+0x792): undefined reference to `strlcat'
util.c:(.text+0x7fa): undefined reference to `strlcpy'

optreset is in nbcompat's getopt_long; I guess it isn't being
used for some reason?  As for the others, jebus, this is the
latest release of Ubuntu and Linux folks still haven't picked up
strl* yet?  Oh, but they're in nbcompat, too; maybe that's not
being used at all.  Hmm, adding -lnbcompat to the link command
still leaves the optreset error.

-- 
Eric Gillespie <*> epg%NetBSD.org@localhost