Config.h Name Collision

[alex-shaw]

Hi,

Had an issue where i had 2 libraries installed (FirebaseClient and HX711_ADC) - and both use a configuration file names "Config.h. I had issues with the default defines werent visible to the FirebaseClient library (and i got a bunch of compile errors).

Is there some misconfiguration on my end? Or should the Config.h in firebaseclient library be renamed to something more unique (e.g. firebaseclient_config.h)?

Thanks!
Alex

[mobizt]

It is not because of the same header and source files of different libraries as all files from different paths that included are compiled together but the same class name with different parameter of member functions, generic macro name can conflict.

You should debug your code first to find the root cause or post the compilation error (first error and few lines).

[mobizt]

The hx library is simple with only two main files .h and .cpp whic the config.h and Config.h are different in naming in Windows unless you use Linux or Mac.

It is easy to test by renaming the config.h of that library to other name and change the #include line of that library header too.

[alex-shaw]

Thank you. Yes - that's how i resolved the issue for now (renaming the HX library config file as it was simple). This resolved my compile errors - it looked like the additional file was effectively hiding the the #define values in config.h (e.g. ENABLE_DATABASE wasnt defined which led to compilation errors).

I could recreate the problem with new clean project.
Add FirebaseClient library -> Compile ok
Add HX711_ADC library -> Compile Errors

Sample compile error : ".pio/libdeps/alktester_wrover/FirebaseClient/src/./core/AsyncResult/AsyncResult.h:55:1: error: expected class-name before '{' token" - (it is because in ResultBase.h there is #if defined(ENABLE_DATABASE)).

edit : this is on a windows 11 machine, VSCode, PlatformIO
Thanks
Alex

[mobizt]

You can update to v1.5.5 to fix this issue.

[alex-shaw]

Brilliant thank you.