Skip to content

Keep the libcurl header out of CcdbApi.h - #15745

Open
sawenzel wants to merge 2 commits into
AliceO2Group:devfrom
sawenzel:ccdb-curl-header
Open

Keep the libcurl header out of CcdbApi.h#15745
sawenzel wants to merge 2 commits into
AliceO2Group:devfrom
sawenzel:ccdb-curl-header

Conversation

@sawenzel

@sawenzel sawenzel commented Sep 1, 2026

Copy link
Copy Markdown
Collaborator

Moves the libcurl dependency out of the public CcdbApi.h header and keeps it in the implementation where it belongs. This makes the header self-contained and allows ROOT to interpret it without requiring the curl headers.

Moves the libcurl dependency out of the public `CcdbApi.h` header and keeps it
in the implementation where it belongs. This makes the header self-contained and
allows ROOT to interpret it without requiring the curl headers.

@Barthelemy Barthelemy left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this is great, thanks !

@alibuild

alibuild commented Sep 1, 2026

Copy link
Copy Markdown
Collaborator

Error while checking build/O2/fullCI_slc9 for 10df9ff at 2026-09-02 19:28:

## sw/BUILD/QualityControl-latest/log
/sw/SOURCES/QualityControl/v1.195.4/v1.195.4/Framework/test/testCcdbDatabase.cxx:310:15: error: 'thread' is not a member of 'std'
/sw/SOURCES/QualityControl/v1.195.4/v1.195.4/Framework/test/testCcdbDatabase.cxx:310:21: error: template argument 1 is invalid
/sw/SOURCES/QualityControl/v1.195.4/v1.195.4/Framework/test/testCcdbDatabase.cxx:310:21: error: template argument 2 is invalid
/sw/SOURCES/QualityControl/v1.195.4/v1.195.4/Framework/test/testCcdbDatabase.cxx:314:13: error: request for member 'emplace_back' in 'threads', which is of non-class type 'int'
/sw/SOURCES/QualityControl/v1.195.4/v1.195.4/Framework/test/testCcdbDatabase.cxx:317:13: error: 'thread' is not a member of 'std'
/sw/SOURCES/QualityControl/v1.195.4/v1.195.4/Framework/test/testCcdbDatabase.cxx:317:23: error: found ':' in nested-name-specifier, expected '::'
/sw/SOURCES/QualityControl/v1.195.4/v1.195.4/Framework/test/testCcdbDatabase.cxx:317:21: error: 't' has not been declared
/sw/SOURCES/QualityControl/v1.195.4/v1.195.4/Framework/test/testCcdbDatabase.cxx:320:11: error: request for member 'clear' in 'threads', which is of non-class type 'int'
/sw/SOURCES/QualityControl/v1.195.4/v1.195.4/Framework/test/testCcdbDatabase.cxx:321:1: error: expected primary-expression before '}' token
/sw/SOURCES/QualityControl/v1.195.4/v1.195.4/Framework/test/testCcdbDatabase.cxx:320:19: error: expected ')' before '}' token
/sw/SOURCES/QualityControl/v1.195.4/v1.195.4/Framework/test/testCcdbDatabase.cxx:321:1: error: expected primary-expression before '}' token
/sw/SOURCES/QualityControl/v1.195.4/v1.195.4/Framework/test/testCcdbDatabase.cxx:345:15: error: 'thread' is not a member of 'std'
/sw/SOURCES/QualityControl/v1.195.4/v1.195.4/Framework/test/testCcdbDatabase.cxx:345:21: error: template argument 1 is invalid
/sw/SOURCES/QualityControl/v1.195.4/v1.195.4/Framework/test/testCcdbDatabase.cxx:345:21: error: template argument 2 is invalid
/sw/SOURCES/QualityControl/v1.195.4/v1.195.4/Framework/test/testCcdbDatabase.cxx:349:13: error: request for member 'emplace_back' in 'threads', which is of non-class type 'int'
/sw/SOURCES/QualityControl/v1.195.4/v1.195.4/Framework/test/testCcdbDatabase.cxx:350:10: error: 'std::this_thread' has not been declared
/sw/SOURCES/QualityControl/v1.195.4/v1.195.4/Framework/test/testCcdbDatabase.cxx:353:13: error: 'thread' is not a member of 'std'
/sw/SOURCES/QualityControl/v1.195.4/v1.195.4/Framework/test/testCcdbDatabase.cxx:353:23: error: found ':' in nested-name-specifier, expected '::'
/sw/SOURCES/QualityControl/v1.195.4/v1.195.4/Framework/test/testCcdbDatabase.cxx:353:21: error: 't' has not been declared
/sw/SOURCES/QualityControl/v1.195.4/v1.195.4/Framework/test/testCcdbDatabase.cxx:356:11: error: request for member 'clear' in 'threads', which is of non-class type 'int'
/sw/SOURCES/QualityControl/v1.195.4/v1.195.4/Framework/test/testCcdbDatabase.cxx:357:1: error: expected primary-expression before '}' token
/sw/SOURCES/QualityControl/v1.195.4/v1.195.4/Framework/test/testCcdbDatabase.cxx:356:19: error: expected ')' before '}' token
/sw/SOURCES/QualityControl/v1.195.4/v1.195.4/Framework/test/testCcdbDatabase.cxx:357:1: error: expected primary-expression before '}' token
ninja: build stopped: cannot make progress due to previous errors.


## sw/BUILD/o2checkcode-latest/log
--
========== List of errors found ==========
++ GRERR=0
++ grep -v clang-diagnostic-error error-log.txt
++ grep ' error:'
++ GRERR=1
++ [[ 1 == 0 ]]
++ mkdir -p /sw/INSTALLROOT/2fe1fa4aceb2a78deed5ceed8421dac6de68cd70/slc9_x86-64/o2checkcode/1.0-local502/etc/modulefiles
++ alibuild-generate-module
+ HAS_BIN=
+ HAS_LIB=
+ HAS_CMAKE=
+ HAS_ROOT=
+ '[' 0 '!=' 0 ']'
+ cat
+ printf 'if ![ is-loaded '\''BASE/1.0'\'' ] {\n module load BASE/1.0\n}'
+ echo 'BUILD_REQUIRES=CMake alibuild-recipe-tools defaults-release'
BUILD_REQUIRES=CMake alibuild-recipe-tools defaults-release
+ FULL_BUILD_REQUIRES='CMake O2-customization FairCMakeModules pytorch_cpuinfo alibuild-recipe-tools safe_int json-c ninja defaults-release Alice-GRID-Utils MPFR googlebenchmark date double-conversion UUID Python-modules-list ninja-fortran bz2'
+ echo 'FULL_BUILD_REQUIRES=CMake O2-customization FairCMakeModules pytorch_cpuinfo alibuild-recipe-tools safe_int json-c ninja defaults-release Alice-GRID-Utils MPFR googlebenchmark date double-conversion UUID Python-modules-list ninja-fortran bz2'
FULL_BUILD_REQUIRES=CMake O2-customization FairCMakeModules pytorch_cpuinfo alibuild-recipe-tools safe_int json-c ninja defaults-release Alice-GRID-Utils MPFR googlebenchmark date double-conversion UUID Python-modules-list ninja-fortran bz2
[0 more errors; see full log]

Full log here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

5 participants