Apache OpenOffice (AOO) Bugzilla – Issue 123595
Cleanup logging of make_installer.pl and its modules
Last modified: 2017-05-20 10:44:54 UTC
The make_installer.pl in solenv/bin and its modules in solenv/bin/modules/installer(/windows) use very heterogeneous methods to write log information to console or log files. These methods include - simple print() calls - adding lines to global arrays @logfileinfo and @globallogfileinfo - calling functions in installer::logger There are three different log channels: the console, language dependent log information, language independent log information. Output to the console is not included in any log file. The goal of this issue is to make logging a little bit more understandable and logical.
This is part of preliminary work for adding support for the creation of Windows patches. See bug 123531 for more details.
"af" committed SVN revision 1537433 into trunk: 123595: Cleanup logging in make_installer.pl and its modules.
installer::logger is now partly a class. Its three default objects are installer::logger::Info installer::logger::Lang installer::logger::Global Info is for writing messages to the console. They are copied to Lang. Lang is for more detailed messages that are language dependent. They are only written to a log file. Global is for more detailed messages that are language independent. They are copied to the start of every language dependent log file (this is the original behavior). Log messages are written to the console as is. Messages are written to log files with additional information: the relative time since logging started and the original channel (abbreviated to info, lang, and glob). Log messages are written to the channels via print and printf methods. More information is available in installer/logger.pm.
Fixed a build breaker: missed a call to print_message() which now dies with a deprecation message. Added some diagnostics to printf (output of stack trace) when the message contains a % after sprintf formatting. Apparently there are some printf calls that use a broken format string. A % in the formatted message is a valid case, therefore this change is intended to be temporary.
"af" committed SVN revision 1537878 into trunk: 123595: logger::printf now writes stack trace when formatted message still co...
"af" committed SVN revision 1537879 into trunk: 123595: Use print instead of printf when text is to be printed verbatim.
"af" committed SVN revision 1538529 into trunk: 123595: Add temporary debug code to find who prints '%{'.
"af" committed SVN revision 1539620 into trunk: 123595: Added missing .idl extension to exclude from error messages.
"af" committed SVN revision 1539661 into trunk: 123595: RPM output lines need no printf
"af" committed SVN revision 1546570 into trunk: 123595: Logging of make_installer.pl now supports indentation.
Created attachment 82021 [details] Fix build breaker triggered in packagepool.pm There is a build breaker that is triggered in some cases when a Linux build is made. It is caused by a log message printed in solenv/bin/modules/installer/packagepool.pm.
"af" committed SVN revision 1547024 into trunk: 123595: Don't print newline as first character in line.
@Andre: Is this issue solved?
Reset the assignee to the default "issues@openoffice.apache.org".