Skip to content

Commit 8e8b6bb

Browse files
committed
Remove stray linebreak when running root -q with no input files
There is an additional line break when running `root -q` with no input files: ```txt ------------------------------------------------------------------ | Welcome to ROOT 6.36.02 https://root.cern | | (c) 1995-2025, The ROOT Team; conception: R. Brun, F. Rademakers | | Built for linuxx8664gcc on Jul 08 2025, 21:22:35 | | From tags/6-36-02@6-36-02 | | With g++ (GCC) 14.3.0 | | Try '.help'/'.?', '.demo', '.license', '.credits', '.quit'/'.q' | ------------------------------------------------------------------ ``` I don't think the second newline at the end should be there. The reason why I suggest to remove it now is because it's still there when you disable the banner, as running `root -b -q -l` gives: ``` ``` That is a bit awkward for something that is supposed to do absolutely nothing, and pollutes the build log with blank likes when it calls `root -b -q -l` to implicitly generate the modules.
1 parent fde0c6d commit 8e8b6bb

File tree

2 files changed

+9
-4
lines changed

2 files changed

+9
-4
lines changed

README/ReleaseNotes/v638/index.md

Lines changed: 9 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -75,7 +75,7 @@ RooAbsData *dataForChan = found != splits.end() ? found->get() : nullptr;
7575
are filled at very high rates. Use lower number in this case.
7676
- The Snapshot method has been refactored so that it does not need anymore compile-time information (i.e. either template arguments or JIT-ting) to know the input column types. This means that any Snapshot call that specifies the template arguments, e.g. `Snapshot<int, float>(..., {"intCol", "floatCol"})` is now redundant and the template arguments can safely be removed from the call. At the same time, Snapshot does not need to JIT compile the column types, practically giving huge speedups depending on the number of columns that need to be written to disk. In certain cases (e.g. when writing O(10000) columns) the speedup can be larger than an order of magnitude. The Snapshot template is now deprecated and it will issue a compile-time warning when called. The function overload is scheduled for removal in ROOT 6.40.
7777

78-
## PyROOT
78+
## Python Interface
7979

8080
### Deprecate the attribute pythonization of `TDirectory` in favor of item-getting syntax
8181

@@ -89,8 +89,14 @@ The deprecated pythonization with the `__getattr__` syntax is now removed.
8989
It was originally schedeuled for removal in 6.34 according to the 6.32 release notes, but since it was still used quite a bit,
9090
the deprecation period was extended.
9191

92-
## Command line utils
93-
- `rootls` has a new native implementation and can now be run without Python. The options and the output of the new rootls are identical to the previous implementation but it should run faster (typically about 3 to 4x faster).
92+
## ROOT executable
93+
94+
- Removed stray linebreak when running `root -q` with no input files.
95+
This ensures that there is no superfluous output when running `root` without the banner and without input files (`root -q -l`).
96+
97+
## Command-line utilities
98+
- The `rootls` utility has a new native implementation and can now be run without Python.
99+
The options and the output of the new rootls are identical to the previous implementation but it should run faster (typically about 3 to 4x faster).
94100

95101
## JavaScript ROOT
96102
- A new configuration option `Jupyter.JSRoot` was added in .rootrc to set the default mode for JSROOT in Jupyter notebooks (on or off).

core/rint/src/TRint.cxx

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -498,7 +498,6 @@ void TRint::Run(Bool_t retrn)
498498
}
499499

500500
if (QuitOpt()) {
501-
printf("\n");
502501
if (retrn) return;
503502
Terminate(fCaughtSignal != -1 ? fCaughtSignal + 128 : 0);
504503
}

0 commit comments

Comments
 (0)