findunref build shouldn't fail if SCM can't be determined
usr/src/tools/findunref/Makefile deals with the possibility SCM_TYPE is "none", but not the possibility that it's "unknown", which in some cases it may be (when using bldenv, for instance).
- It should deal with either possibility in the same way
- bldenv should use "none" the same as nightly does
Updated by Rich Lowe over 8 years ago
- Status changed from New to In Progress
Actually, this whole thing is bogus. There's utterly no reason to go through any of this pain, and the very idea of Makefile concerning itself with the SCM bothers me greatly in the first place.
The only somewhat valid reason this is the way it is is that it is possible, in theory, is that it may be necessary to pacify checkpaths. It turns out that's never true: any valid entry in the SCM-specific lists is also an entry with a leading wildcard, which checkpaths explicitly ignores. I plan to delete all the SCM specific files and the build junk, and add those entries which actually need to exist to exception_list.common, fixing the subversion entry in the process (I'm not certain why the file existed, but it was clearly never tested).
In doing this, bldenv doesn't even need to know anything about the SCM (as it should be!) and will cease to know about it, and nightly will cease exporting the SCM type to deter anyone else from depending on it.