Improve readability of conddb console output #16187
Conversation
When a multi-line field is printed on the console, the output is severely compromised, to the point of being non-readable. The problem is due to the non correct handling of line-breaks. This PR cures the problem using native str methods. The special character '+' has been used to indicate that the original field was a multi-line field. The choice was completely arbitrary. Limited testing has been performed, but on the pathological case, the correct behaviour is restored. To test use, e.g.: * * conddb search SiStripDQMTrigger * before and after this patch.
|
A new Pull Request was created by @rovere (Marco Rovere) for CMSSW_8_1_X. It involves the following packages: CondCore/Utilities @ggovi, @cmsbuild, @davidlange6 can you please review it and eventually sign? Thanks. cms-bot commands are listed here #13028 |
|
please test |
|
The tests are being triggered in jenkins. |
|
Comparison is ready @slava77 comparisons for the following workflows were not done due to missing matrix map:
|
|
+1 |
|
This pull request is fully signed and it will be integrated in one of the next CMSSW_8_1_X IBs (tests are also fine). This pull request requires discussion in the ORP meeting before it's merged. @slava77, @davidlange6, @smuzaffar |
cmsbuild
added
db-approved
fully-signed
and removed
db-pending
pending-signatures
labels
|
+1 |
When a multi-line field is printed on the console, the output
is severely compromised, to the point of being non-readable.
The problem is due to the non correct handling of
line-breaks. This PR cures the problem using native str
methods. The special character '+' has been used to indicate
that the original field was a multi-line field. The choice
was completely arbitrary. Limited testing has been
performed, but on the pathological case, the correct
behaviour is restored. To test use, e.g.:
*
*
before and after this patch.