Rich Burridge wrote:
> http://jurassic.us.oracle.com/~richb/7183526-v1/
Makefile:
- line 45-46: How about something like:
# If SHELLOPTS is exported (as it is by the userland makefiles),
# then all shell options get exported to child invocations of bash,
# which results in test failures due to nounset and xtrace being
# set unexpectedly, and errors such as "$1: unbound variable" and
# diffs failing due to script tracing in output files.
- line 49: I think it's the case that the things being tested are
actually pulled from the build area, but the scripts run in the course
of the tests may rely on things that aren't in /usr/bin. That made
more sense for coreutils, where it relied on GNU coreutils instead of
the Solaris commands; I don't really understand what it needs from the
build area for gzip. Or does it really only test the component once
it's installed?
- line 50: could you pull from the proto area's usr/bin directory
instead? Every component will have its executables in different
locations in the build area, but the executables in the proto area
should generally be in consistent locations, and it would be nice to
have this line be consistent across components.
> /net/stard.us.oracle.com/tank/ws/UL/7183526/components/gzip/test-trans.txt
- I see that the zgrep-f test is skipped because we're using Solaris
grep, which doesn't grok the -f option. It looks like we're patching
the test to force the use of /usr/gnu/bin/grep, but it's not working,
probably because the test framework checks for a -f-supporting grep
before running the test (see init.cfg). You could set PATH to include
/usr/gnu/bin or /usr/xpg4/bin before /usr/bin for the test run, or
patch zgrep to make sure that a POSIX-compliant grep is used. The
latter might be nicer, since it would then not require gnu coreutils
to be installed -- it could check for grep from /usr/gnu/bin, then
/usr/xpg4, and finally /usr/bin.
- Did the zdiff test fail the first time through (test-trans.txt.prev)
because SHELLOPTS was still defined?
Thanks,
Danek
_______________________________________________
userland-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/userland-discuss
