[cfe-commits] [PATCH] Warn if -isysroot is given an invalid path
matthieu.monrocq at gmail.com
Fri Dec 14 11:20:34 PST 2012
On Fri, Dec 14, 2012 at 7:41 PM, Eric Christopher <echristo at gmail.com>wrote:
> Sure, but I just don't see that providing much value. The point of adding
>> a warning was to help catch things like typos in the sysroot path. What
>> are the chances that someone is going to mis-type the sysroot and also
>> specify -Wmissing-sysroot? I'm assuming that typos are most likely to come
>> up when someone is running the compiler by hand, rather than from a
>> makefile. I suppose that it could still help when a makefile specifies an
>> SDK that is not currently installed. I'm just not convinced that's worth
>> the effort of adding and maintaining a new warning, but I'm open to being
> My only concerns are false positives in fairly standard use cases. I think
> the warning could be useful when debugging problems and if you know you
> aren't in the use cases that might give a false positive you can turn it on
> by default (perhaps with a new umbrella?) but false positives are bad for
> everyone :)
> cfe-commits mailing list
> cfe-commits at cs.uiuc.edu
Actually maybe the best way is in putting this warning in -Wmissing-sysroot
AND make -Wmissing-sysroot part of -Wunused-command-line-argument.
=> if you turn on -Wunused-command-line-argument you get warned if the flag
is ignored because of an invalid path.
=> if you are warned and you do not care, you can always use
In this scenario, -Wmissing-sysroot is not of much use by itself, but its
negative form is.
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the cfe-commits