Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Issue compiling larwirecell: SPDLOG_ACTIVE_LEVEL in Logging.h #288

Open
HaiwangYu opened this issue Mar 23, 2024 · 2 comments
Open

Issue compiling larwirecell: SPDLOG_ACTIVE_LEVEL in Logging.h #288

HaiwangYu opened this issue Mar 23, 2024 · 2 comments

Comments

@HaiwangYu
Copy link
Member

Encountered this when trying to compile larwirecell against wirecell
Screenshot 2024-03-23 at 3 02 42 PM

@brettviren
Copy link
Member

Hi @HaiwangYu

What version of WCT is this?

Here is my guess, let me know if I get close.

I suspect you are using a common source area to install multiple versions and an incorrect build/WireCellUtil/BuildConfig.h is getting installed due to switching to a new source branch. I believe is possible (wcb bug) even if a ./wcb configure is done after the switch. I think there is some bug that keeps the BuildConfig.h from being updated if it already exists.

If this is still on track, the BuildConfig.h that you have installed is probably from a branch that adds support for ./wcb --with-spdlog-active-level and in that branch you should have a version of Logging.h not like what you show but like:

a5ef5d3fd (Brett Viren 2023-12-01 16:44:57 -0500 16) // #define SPDLOG_ACTIVE_LEVEL SPDLOG_LEVEL_DEBUG
a5ef5d3fd (Brett Viren 2023-12-01 16:44:57 -0500 17) #include "WireCellUtil/BuildConfig.h"

This is git blame from my apply-pointcloud branch. So, there, everything is consistent. But, then switching a branch changes Logging.h but you get stuck with the previous BuildConfig.h.

I believe the fix after you switch to a source branch that lacks .wcb --with-spdlog-active-level is to manually make sure BuildConfig.h is gone

rm build/WireCellUtil/BuildConfig.h
./wcb configure ...
./wcb
./wcb install

@HaiwangYu
Copy link
Member Author

Thanks! It seems to be the case. I have two local repos. I will check your solution.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants