#253 using gpsim library: breakpoints.h references nonexistent ../config.h

[Zoltan P]

Hi,

as a developer of KTechLab [1], I've run into the issue that when using gpsim as a library, the compilation of code using the non-modified headers of gpsim fails.
The header breakpoints.h includes the header ../config.h, which is an internal header of gpsim, I assume [2]. And this ../config.h header is not shipped with gpsim, so the compilation fails.
Thus, it would be nice to fix the public headers to be compilable witout patching. I propose to move the inclusion of ../config.h into the relevant cpp files.

This issue has been reported to Debian [3] and to KTechLab [4].

[1]
https://userbase.kde.org/KTechlab

[2]
https://sourceforge.net/p/gpsim/code/HEAD/tree/trunk/src/breakpoints.h

[3]
https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=939338

[4]
https://github.com/ktechlab/ktechlab/issues/55

Best regards,

Zoltan

[Roy Rankin]

../config.h is generated by the linux configure system. ../config.h contains the build version and refines required by the Dprintf function as well a system information. It will not be removed. You might try making an appropriate ../config.h to suit your application

[Zoltan P]

I definitely do not propose to remove the config.h file; I'm trying to figure out how to setup a correct build environment for using gpsim as a library.

If I understand correctly, either

(a) gpsim's ../config.h file should exist in the application's build environment. I think this should be fine, but as I observe the Linux distributions shipping gpsim-0.31.0 do not include the config.h file in the gpsim-dev (or equivalent) package [1]. Debian even patches the headers from gpsim-dev and removes the line #include "../config.h" [2]. This situation means that the distributions are not packaging the latest gpsim correctly?

(b) an application using gpsim as library should contain a config.h for gpsim. This should work, as long as it doesn't require some very specific or esoteric information in the file. Although the original information from ../config.h would be useful (version information...).
Actually I ended up implementing this approach, based on a bugreport.

[1]
https://pkgs.org/download/gpsim
[2]
https://packages.debian.org/sid/gpsim-dev

Can you confirm that the original ../config.h file should be included in the gpsim-dev
packages of Linux distributions?