DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND
INSERTED IN THE BUG DATABASE.
Patch, Avalon Logging to Commons Logging Conversion.
Summary: Patch, Avalon Logging to Commons Logging Conversion.
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
Attaching the patch that will convert FOP to using Commons Logging .
I've tested several sample files with it, it appears to run fine throughout,
but as the changes are extensive, I suspect there will be a few places where a
logger was not properly set, etc. Let me know what you see, but I suspect many
small bugs we can fix after the patch is applied.
Only issue found was in setting a logger for an Avalon "ContainerUtil", which
currently won't work with Commons Logging and needs its own Avalon logger. I
just commented out the line (in two places) where that ContainerUtil logging
would otherwise have been enabled (I don't think we're currently using this
functionality anyway.) In the future, we can look at other options here--such
as just feeding it an Avalon console logger, to providing a file conversion
between the two logging choices, to just not logging here, to using an
alternative to ContainerUtil.
I'll hold off on this for three days to allow for team comments. (Please
minimize your changes to the HEAD code base in the interim, as updating this
patch would be very complex.)