artemp/mapnik — Issue #250: Scons - must improve handling of "CPPPATH" "LIBPATH" environment variables
Author: artemp
Created: 2011-10-11 16:07:03+00:00
State: open
Labels: Scons_Build_Scripts

I'm convinced that we need to be much more careful in our adding and removing of path variables for libs and includes of mapnik dependencies.

So, instead of bulk unique adding and then removing a path for a when not found, we should:
1) backup the env
2) add paths for a given dependency during the checklibwithheader stage
3) restore the env if the test fails.

This will avoid indiscriminately removing paths when a test fails (particularly plugins) that might actually be needed for another dependency.