> Right. Seems I missed this (which was no wonder at that point after to
 > many hours without sleep (near the point where you start to see the
 > little pink flying elephants... :-) ) ... that's why I asked "... please
 > verify..." to avoid that we're checking-in something dumb...)
 > 
 > Here comes the fixed list:
 > -- snip --
 > usr/src/cmd/ast/msgcc/Mamfile
 > usr/src/lib/libast/common/Mamfile
 > usr/src/lib/libast/common/Makefile
 > usr/src/lib/libast/common/astsa/align.h
 > usr/src/lib/libast/common/astsa/astwinsize.c
 > usr/src/lib/libast/common/astsa/ccode.h
 > usr/src/lib/libast/common/astsa/strmatch.c
 > usr/src/lib/libast/common/astsa/times.h
 > usr/src/lib/libast/common/astsa/sig.h
 > usr/src/lib/libast/common/astsa/lclib.h
 > usr/src/lib/libast/common/astsa/README
 > usr/src/lib/libast/common/astsa/ast.h
 > usr/src/lib/libast/common/dir/dirstd.h
 > usr/src/lib/libast/common/misc/magic.tab
 > usr/src/lib/libast/common/port/atmain.C
 > usr/src/lib/libast/common/uwin/mini.sym
 > usr/src/lib/libcmd/common/Mamfile
 > usr/src/lib/libcmd/common/Makefile
 > usr/src/lib/libdll/common/Mamfile
 > usr/src/lib/libdll/common/Makefile
 > usr/src/lib/libpp/common/Mamfile
 > usr/src/lib/libpp/common/Makefile
 > usr/src/lib/libpp/common/probe.win32
 > usr/src/lib/libshell/common/Mamfile
 > usr/src/lib/libshell/common/Makefile
 > usr/src/lib/libshell/common/mamexec
 > usr/src/lib/libshell/common/mamstate.c
 > -- snip --
 > 
 > Better ?

Looks good.  You or April can verify that it's correct by updating
findunref's exception_list with the patterns I listed earlier, doing a
build with "ft" included in NIGHTLY_OPTIONS, and verifying that the only
remaining files are those listed above (or are only used on another ISA).

 > >  > >  > >    ./lib/libpp/sparc/probe
 > >  > >  > >    ./lib/libpp/sparc/probe.sh
 > 
 > I am not sure. I don't see anything special for these files and AFAIK
 > none of the Makefiles in usr/src/lib/libpp/${TRANSMACH)/ uses something
 > like /usr/bin/file which may touch the files for reading. Short: I have
 > zero clues why they are not listed...

Certainly not a high-priority concern, but it would be nice to do some
build experiments to tease out the answer.

-- 
meem

Reply via email to