>Number:         36259
>Category:       pkg
>Synopsis:       sysutils/xentools30 has test == portability problems
>Confidential:   no
>Severity:       serious
>Priority:       medium
>Responsible:    pkg-manager
>State:          open
>Class:          sw-bug
>Submitter-Id:   net
>Arrival-Date:   Wed May 02 07:35:00 +0000 2007
>Originator:     Robert Elz
>Release:        NetBSD 3.99.15   (pkgsrc current within past 12 hours)
>Organization:
	Prince of Songkla University
>Environment:
System: NetBSD jade.coe.psu.ac.th 3.99.15 NetBSD 3.99.15 (GENERIC-1.696-20060125) #8: Wed Jan 25 04:59:39 ICT 2006 kre@jade.coe.psu.ac.th:/usr/obj/current/kernels/JADE_ASUS i386
Architecture: i386
Machine: i386
>Description:
	With PKG_DEVELOPER set (as is the default when using pkg_comp,
	which is what I do), pkgsrc checks scripts in packages for
	use of (truly idiotic) "test x == y" type commands that are
	apaprently becoming common in some mutant universe.

	sysutils/xentools30 is (again) failing that test.

	"Again" as an earlier PR was files against an earlier version
	of this package.   The solution at the time was to simply
	turn off the tests (the scripts in question, then, not being
	used or installed on NetBSD anyway).

	That now looks to have been a mistake - much better to fix the
	scripts, and send the fixes back upstream, so the turkeys who
	perpetrate this nonsense can learn how test should be used,
	and this problem may decrease over time, instead of magnifying.

>How-To-Repeat:
	Attempt to build sysutils/xentools30 with PKG_DEVELOPER set
	(why don't all pkgsrc developers simply always have PKG_DEVELOPER
	set in their mk.conf files?)

	Expect to see ...
=> Returning to build of xentools30-3.0.4
===> Overriding tools for xentools30-3.0.4
===> Extracting for xentools30-3.0.4
===> Patching for xentools30-3.0.4
=> Applying pkgsrc patches for xentools30-3.0.4
===> Creating toolchain wrappers for xentools30-3.0.4
===> Configuring for xentools30-3.0.4
=> Fixing hardcoded paths.
=> Checking for portability problems in extracted files
ERROR: [check-portability.awk] => Found test ... == ...:
ERROR: [check-portability.awk] xm-test/ramdisk/make-release.sh: if [ "$1" == ""
]
ERROR: [check-portability.awk] xm-test/ramdisk/make-release.sh: elif [ "$1" == "
powerpc" ]

Explanation:
	[gratuituous noise omitted here...]

>Fix:
	I don't know if the script detected (this time) is an important
	one, and really does need to be fixed, or just more noise that
	pkgsrc could be told to not bother checking, but I'd really
	strongly suggest fixing the problem, and never ignoring it!