Remove -l argument for when using root with input files or commands

The logo is now now printed anymore anyway if we use `root` with inputs,
so we can drop some redundant `-l` flags to set the right precedent for
your users and don't confuse them with ineffective flags.

Author: guitargeek

CMakeLists.txt

@@ -590,7 +590,7 @@ add_custom_command(OUTPUT tutorials/hsimple.root
                      ROOT_INCLUDE_PATH=${DEFAULT_ROOT_INCLUDE_PATH}
                      ROOTIGNOREPREFIX=1
                      ROOT_HIST=0
-                     $<TARGET_FILE:root.exe> -l -q -b -n
+                     $<TARGET_FILE:root.exe> -q -b -n
                        -e ".L ${CMAKE_SOURCE_DIR}/tutorials/hsimple.C"
                        -e "hsimple();"
                        -e return

bindings/jupyroot/python/JupyROOT/helpers/utils.py

@@ -253,7 +253,7 @@ def _invokeAclicMac(fileName):
     to exclude these libraries, so we launch a second root session to compile
     the library, which we then load.
     """
-    command = 'root -l -q -b -e gSystem->CompileMacro("%s","k")*0' % fileName
+    command = 'root -q -b -e gSystem->CompileMacro("%s","k")*0' % fileName
     out = _checkOutput(command, "Error ivoking ACLiC")  # noqa: F841
     libNameBase = fileName.replace(".C", "_C")
     ROOT.gSystem.Load(libNameBase)

bindings/pyroot/cppyy/cppyy/test/CMakeLists.txt

@@ -10,7 +10,7 @@ if(NOT MSVC)
   list(APPEND common_args ENVIRONMENT ${ld_library_path}=${CMAKE_CURRENT_BINARY_DIR})
 endif()
 
-set(root_cmd $<TARGET_FILE:root.exe> -b -q -l)
+set(root_cmd $<TARGET_FILE:root.exe> -b -q)
 
 function(ADD_PYUNITTEST_ACLIC TESTNAME)
   set(options)

cmake/modules/RootMacros.cmake

@@ -2502,7 +2502,7 @@ macro(ROOTTEST_COMPILE_MACRO filename)
       ${ROOT_root_CMD}
       -e "gSystem->SetBuildDir(\"${CMAKE_CURRENT_BINARY_DIR}\", true)"
       ${RootMacroDirDefines}
-      -q -l -b
+      -q -b
   )
 
   get_filename_component(realfp ${filename} ABSOLUTE)
@@ -2916,9 +2916,9 @@ macro(ROOTTEST_SETUP_MACROTEST)
                -e "gInterpreter->AddIncludePath(\"-I${CMAKE_CURRENT_BINARY_DIR}\")"
                -e "gSystem->AddIncludePath(\"-I${CMAKE_CURRENT_BINARY_DIR}\")"
                ${ARG_ROOTEXE_OPTS}
-               -q -l -b)
+               -q -b)
 
-  set(root_buildcmd ${ROOT_root_CMD} ${RootExeDefines} -q -l -b)
+  set(root_buildcmd ${ROOT_root_CMD} ${RootExeDefines} -q -b)
 
   # Compile macro, then add to CTest.
   if(ARG_MACRO MATCHES "[.]C\\+" OR ARG_MACRO MATCHES "[.]cxx\\+" OR ARG_MACRO MATCHES "[.]cpp\\+" OR ARG_MACRO MATCHES "[.]cc\\+")

documentation/doxygen/Makefile

@@ -17,7 +17,7 @@ export PYSPARK_PYTHON := $(Python3_EXECUTABLE)
 DOXYGEN_OUTPUT_DIRECTORY ?= $(HOME)/rootdoc
 export DOXYGEN_OUTPUT_DIRECTORY
 export DOXYGEN_SOURCE_DIRECTORY := ../..
-export DOXYGEN_ROOT_VERSION := $(shell root -l -b -q -e 'printf("%s\n",gROOT->GetGitBranch());')
+export DOXYGEN_ROOT_VERSION := $(shell root -b -q -e 'printf("%s\n",gROOT->GetGitBranch());')
 export DOXYGEN_ROOT_VERSION := $(subst -00-patches,,$(DOXYGEN_ROOT_VERSION))
 export DOXYGEN_ROOT_VERSION := $(subst heads/,,$(DOXYGEN_ROOT_VERSION))
 export DOXYGEN_EXAMPLE_PATH := $(DOXYGEN_OUTPUT_DIRECTORY)/macros

documentation/doxygen/filter.cxx

@@ -242,7 +242,7 @@ void FilterClass()
                fclose(m);
                m = 0;
                ExecuteCommand(
-                  StringFormat("root -l -b -q \"makeimage.C+O(\\\"%s\\\",\\\"%s\\\",\\\"%s\\\",true,false)\"",
+                  StringFormat("root -b -q \"makeimage.C+O(\\\"%s\\\",\\\"%s\\\",\\\"%s\\\",true,false)\"",
                                StringFormat("%s_%3.3d.C", gClassName.c_str(), gMacroID).c_str(),
                                StringFormat("%s_%3.3d.%s", gClassName.c_str(), gImageID, gImageType.c_str()).c_str(),
                                gOutDir.c_str()));
@@ -360,11 +360,11 @@ void CreateTutorialImage(bool nobatch) {
    } else {
       if (nobatch) {
          ExecuteCommand(
-            StringFormat("root -l -q \"makeimage.C+O(\\\"%s\\\",\\\"%s\\\",\\\"%s\\\",false,false)\"",
+            StringFormat("root -q \"makeimage.C+O(\\\"%s\\\",\\\"%s\\\",\\\"%s\\\",false,false)\"",
                          gFileName.c_str(), gImageName.c_str(), gOutDir.c_str()));
       } else {
          ExecuteCommand(
-            StringFormat("root -l -b -q \"makeimage.C+O(\\\"%s\\\",\\\"%s\\\",\\\"%s\\\",false,false)\"",
+            StringFormat("root -b -q \"makeimage.C+O(\\\"%s\\\",\\\"%s\\\",\\\"%s\\\",false,false)\"",
                          gFileName.c_str(), gImageName.c_str(), gOutDir.c_str()));
       }
    }
@@ -449,7 +449,7 @@ void FilterTutorial()
             if (gPython) {
                ExecuteCommand(StringFormat("%s %s", gPythonExec.c_str(), gFileName.c_str()));
             } else {
-               ExecuteCommand(StringFormat("root -l -b -q %s", gFileName.c_str()));
+               ExecuteCommand(StringFormat("root -b -q %s", gFileName.c_str()));
             }
             ExecuteCommand(StringFormat("mv %s %s/html", image_name.c_str(), gOutDir.c_str()));
             ReplaceAll(gLineString, "macro_image (", "image html ");
@@ -459,7 +459,7 @@ void FilterTutorial()
             IN = gImageName;
             int i = IN.find(".");
             IN.erase(i,IN.length());
-            ExecuteCommand(StringFormat("root -l -b -q \"MakeTCanvasJS.C(\\\"%s\\\",\\\"%s\\\",\\\"%s\\\",false,%d,%d)\"",
+            ExecuteCommand(StringFormat("root -b -q \"MakeTCanvasJS.C(\\\"%s\\\",\\\"%s\\\",\\\"%s\\\",false,%d,%d)\"",
                                          gFileName.c_str(), IN.c_str(), gOutDir.c_str(), gPython, tcanvas_aclic));
             ReplaceAll(gLineString, "macro_image", StringFormat("htmlinclude %s.html",IN.c_str()));
          } else if (rcanvas_js) {
@@ -468,7 +468,7 @@ void FilterTutorial()
             int i = IN.find(".");
             IN.erase(i,IN.length());
             ExecuteCommand(StringFormat(
-               "root -l -b -q --web=batch \"MakeRCanvasJS.C+O(\\\"%s\\\",\\\"%s\\\",\\\"%s\\\",false,%d)\"",
+               "root -b -q --web=batch \"MakeRCanvasJS.C+O(\\\"%s\\\",\\\"%s\\\",\\\"%s\\\",false,%d)\"",
                gFileName.c_str(), IN.c_str(), gOutDir.c_str(), gPython));
             ReplaceAll(gLineString, "macro_image", StringFormat("htmlinclude %s.html",IN.c_str()));
          } else {
@@ -512,7 +512,7 @@ void FilterTutorial()
       // \macro_output found
       if (gLineString.find("\\macro_output") != string::npos) {
          remove(gOutputName.c_str());
-         if (!gPython) ExecuteCommand(StringFormat("root -l -b -q %s", gFileName.c_str()).c_str());
+         if (!gPython) ExecuteCommand(StringFormat("root -b -q %s", gFileName.c_str()).c_str());
          else          ExecuteCommand(StringFormat("%s %s", gPythonExec.c_str(), gFileName.c_str()).c_str());
          ExecuteCommand(StringFormat("sed -i '/Processing/d' %s", gOutputName.c_str()).c_str());
          rename(gOutputName.c_str(), StringFormat("%s/macros/%s",gOutDir.c_str(), gOutputName.c_str()).c_str());
@@ -588,7 +588,7 @@ void ExecuteMacro()
    gMacroName = gLineString.substr(i1,i2-i1+1);
 
    // Build the ROOT command to be executed.
-   gLineString.insert(0, StringFormat("root -l -b -q \"makeimage.C+O(\\\""));
+   gLineString.insert(0, StringFormat("root -b -q \"makeimage.C+O(\\\""));
    size_t l = gLineString.length();
    gLineString.replace(l-1,1,StringFormat("\\\",\\\"%s\\\",\\\"%s\\\",true,false)\"", gImageName.c_str(), gOutDir.c_str()));
 

documentation/doxygen/modifyClassWebpage.sh

@@ -11,7 +11,7 @@ if [ ! -d "$HTMLPATH" -o ! -f "$WORKFILE" ]; then
 fi
 
 # Find the libraries for the class $1
-libname=$(root -l -b -q "libs.C+g(\"$1\")" | grep 'mainlib=')
+libname=$(root -b -q "libs.C+g(\"$1\")" | grep 'mainlib=')
 
 # The class was not found. Remove the collaboration graph
 if [ -z "${libname}" ]; then

documentation/doxygen/modifyClassWebpages.sh

@@ -25,5 +25,5 @@ if [ ! -s "${listOfClasses}" ]; then
    exit 0
 fi
 
-root -l -b -q -e ".L libs.C++g"
+root -b -q -e ".L libs.C++g"
 xargs -L 1 -P ${NJOB:-1} ./modifyClassWebpage.sh < ${listOfClasses}

documentation/primer/macros/read_ntuple_with_chain.C

@@ -4,7 +4,7 @@
 // you can easily create some files with the following statement:
 //
 // for i in 0 1 2 3 4 5; \\
-// do root -l -x -b -q \\
+// do root -x -b -q \\
 // "write_ntuple_to_file.cxx \\
 // (\"conductivity_experiment_${i}.root\", 100)"; \\
 //  done

documentation/primer/macros/runall.py

@@ -32,7 +32,7 @@
 import sys
 
 for mName in macros:
-    command = "root -b -l -q %s" %mName
+    command = "root -b -q %s" %mName
     if mName == "slits.C": command = 'echo "2 4" | %s' %command
     print "\n ******* Running %s" %mName
     if 0 !=os.system(command):