Skip to content

Fix/review f39 root library load status #37

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.

Sign up for GitHub

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

Merged
GernotMaier merged 3 commits into from
Oct 4, 2026
Merged

Fix/review f39 root library load status #37

Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
2eb294a
Reject interpreter errors when loading the analysis ROOT library
GernotMaier Oct 4, 2026
467a7b7
no tests needed
GernotMaier Oct 4, 2026
63c3115
Change error type to TInterpreter::EErrorCode
GernotMaier Oct 4, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
11 changes: 7 additions & 4 deletions release_tests/utilities/parameters.C
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,7 @@
#include <sstream>
#include <string>
#include <vector>
#include "TInterpreter.h"

/*
* Helper for ROOT macros: load libVAnaSum from environment.
Expand Down Expand Up @@ -56,17 +57,19 @@ bool loadVAnaSumLibrary()
{
// Use gROOT->ProcessLine to properly load the library and its dictionaries
string loadCmd = "R__LOAD_LIBRARY(" + iLibPath + ");";
Longptr_t result = gROOT->ProcessLine( loadCmd.c_str() );
if( result >= 0 )
TInterpreter::EErrorCode error = TInterpreter::kNoError;
Longptr_t result = gROOT->ProcessLine( loadCmd.c_str(), &error );
if( error == TInterpreter::kNoError && result >= 0 )
{
iLibraryLoaded = true;
return true;
}
}

// Try with just the library name
Longptr_t result = gROOT->ProcessLine( "R__LOAD_LIBRARY(libVAnaSum.so);" );
if( result >= 0 )
Int_t error = TInterpreter::kNoError;
Longptr_t result = gROOT->ProcessLine( "R__LOAD_LIBRARY(libVAnaSum.so);", &error );
if( error == TInterpreter::kNoError && result >= 0 )
{
iLibraryLoaded = true;
return true;
Expand Down
Loading