Re: wip/kegs

Thomas Klausner <[email protected]>
Newsgroups gmane.os.netbsd.devel.pkgsrc.wip.review
Message-ID <[email protected]>
On Thu, Aug 06, 2009 at 05:33:32PM +0200, Thomas Klausner wrote:
> ===> Building for kegs-0.91
> if [ -L /scratch/wip/kegs/work/kegs.0.91/src/vars ]; then /bin/rm /scratch/wip/kegs/work/kegs.0.91/src/vars; fi
> Building for NetBSD on x86_64...
> x86_64 is not supported, but might work...
> cc -O2 -Wall -fomit-frame-pointer -march=pentium -I/usr/pkg/include -c -DKEGS_LITTLE_ENDIAN -I. adb.c
> ...
> cc -O2 -Wall -fomit-frame-pointer -march=pentium -I/usr/pkg/include -c -DKEGS_LI
> TTLE_ENDIAN -I. scc_socket_driver.c
> clock.c:1: error: CPU you selected does not support x86-64 instruction set
> clock.c:1: error: CPU you selected does not support x86-64 instruction set
> config.c:1: error: CPU you selected does not support x86-64 instruction set
> config.c:1: error: CPU you selected does not support x86-64 instruction set
> ...

Havard gave me the hint that it might be "-march=pentium"'s fault, and
it was, so I removed it.

Usually, packages shouldn't set optimization flags themselves, but use
the passed-in ones (via CFLAGS).

> pkglint complains:
> WARN: Makefile:14: Unknown hardware architecture: ppc
> WARN: Makefile:16: Unknown hardware architecture: ppc
> WARN: Makefile:26--38: The shell command "case" should not be hidden.
> WARN: Makefile:41: Using "${WRKSRC}/.." is conceptually wrong. Please use a combination of WRKSRC, CONFIGURE_DIRS and BUILD_DIRS instead.
> WARN: Makefile:42: Using "${WRKSRC}/.." is conceptually wrong. Please use a combination of WRKSRC, CONFIGURE_DIRS and BUILD_DIRS instead.
> WARN: Makefile:43: Using "${WRKSRC}/.." is conceptually wrong. Please use a combination of WRKSRC, CONFIGURE_DIRS and BUILD_DIRS instead.

Additionally:
=> Checking for missing run-time search paths in kegs-0.91
ERROR: bin/xkegs: missing library: libXext.so.6
*** Error code 1

Looks like the rpath isn't set (correctly?).
 Thomas

------------------------------------------------------------------------------
Let Crystal Reports handle the reporting - Free Crystal Reports 2008 30-Day 
trial. Simplify your report design, integration and deployment - and focus on 
what you do best, core application coding. Discover what's new with 
Crystal Reports now.  http://p.sf.net/sfu/bobj-july
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.