Paul Wernau wrote: > > Vladimir Kotal wrote: >> Paul Wernau wrote: >>>> Here you go (portranges only, new fixed bits): >>>> http://cr.opensolaris.org/~vkotal/portranges.journal.debug >>>> http://cr.opensolaris.org/~vkotal/portranges.journal.non-debug >>>> >>> OK, 1 small thing. >>> >>> This is pre-existing, but should be fixed: >>> >>> Every tests has what looks like an error about the pid file not being >>> readable, which will cause alarm for people unfamiliar with the suite. >>> >>> 520|0 4 18172 1 11|killing nc with pidfile '/var/tmp/nc.pid' >>> 520|0 4 18172 1 12|pid file '/var/tmp/nc.pid' is not readable >> This is only printed in the debug journal but it's confusing, I agree. >> Removed, incremental webrev is here: >> >> http://cr.opensolaris.org/~vkotal/nc_tet-ext_portlist-6754842.stcnv.Paul-2/ >> >> The debug journal has been refreshed: >> http://cr.opensolaris.org/~vkotal/portranges.journal.debug >> > > OK, all LGTM now. Thanks for the quick turnaround on the comments.
Cool. Thanks for the quick review. v.
