>Number:         7651
>Category:       pkg
>Synopsis:       broken NOT_FOR_PLATFORM in net/ntop Makefile
>Confidential:   no
>Severity:       non-critical
>Priority:       low
>Responsible:    pkg-manager (NetBSD software packages system bug manager)
>State:          open
>Class:          sw-bug
>Submitter-Id:   net
>Arrival-Date:   Wed May 26 15:05:00 1999
>Last-Modified:
>Originator:     Eric Mumpower
>Organization:
Arepa
	
>Release:        19960526 <NetBSD-current source date>
>Environment:
	
System: NetBSD dr-teeth 1.4 NetBSD 1.4 (TEETH) #3: Mon May 17 13:15:15 PDT 1999 root@dr-teeth:/usr/src/sys/arch/i386/compile/TEETH i386


>Description:

ntop cannot be installed on i386, apparently due to a error in the
composition of a glob in its Makefile.

Perhaps more critically, ntop cannot be *deinstalled* on an i386. One
might think that alleged platform compatibility should have nothing to
do with one's ability to deinstall a platform. :-)

	
>How-To-Repeat:

I built and installed the 'ntop' package on an i386 system using the
pkgsrc.tgz snapshot provided with the NetBSD 1.4 distribution. In
particular, its pkgsrc/net/ntop/Makefile contained this line:

> NOT_FOR_ARCHS= alpha mips # elf problems WRT libcurses

This, clearly, should have worked, and did. Later, however, after
sup'ing my pkgsrc installation to -current (as of 19960526), I found I
could not deinstall the package:

> dr-teeth# make deinstall
> ===>  ntop-1.0 does not work on NetBSD-1.4-i386.

apparently because the aforementioned line had changed to this:

> NOT_FOR_PLATFORM=       *-*-{alpha,mips}        # elf problems WRT libcurses

which, apparently, somehow matches my system configuration. My reading
of bsd.pkg.mk and the make(1) manpage suggests that this syntax is not
valid.

	
>Fix:

Assuming that I correctly infer the proper syntax for
NOT_FOR_PLATFORM, apply this patch to pkgsrc/net/ntop/Makefile:

--- Makefile.orig       Wed May 26 17:40:44 1999
+++ Makefile    Wed May 26 17:49:15 1999
@@ -12,7 +12,7 @@
 
 HOMEPAGE=              http://www-serra.unipi.it/~ntop/
 
-NOT_FOR_PLATFORM=      *-*-{alpha,mips}        # elf problems WRT libcurses
+NOT_FOR_PLATFORM=      *-*-alpha *-*-mips      # elf problems WRT libcurses
 
 HAS_CONFIGURE=         yes
 GNU_CONFIGURE=         yes
>Audit-Trail:
>Unformatted: