Fix GPSim_INCLUDE_DIR to not include gpsim/ prefix
ClosedPublic

Authored by kossebau on Mar 4 2020, 2:33 PM.

Details

Summary

All include directives for gpsim headers are done using the gpsim/ prefix,
e.g. #include "gpsim/stimuli.h"
So the include directories passed to the compiler need to point to the
path before the gpsim subdir.
This has been broken since the first FindGPSim.cmake version, the existing
include directives only ever worked by chance if the installation prefix
of gpsim was the system one or one of the other dependencies whose include
dirs matched the one where the gpsim subdir was present.

Diff Detail

Repository
R496 KTechlab
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
kossebau requested review of this revision.Mar 4 2020, 2:33 PM
kossebau created this revision.
padrah accepted this revision.Mar 5 2020, 12:31 PM

Looks good. Most probably the gpsim path has been in system path, so it has been actually working.

This revision is now accepted and ready to land.Mar 5 2020, 12:31 PM
This revision was automatically updated to reflect the committed changes.