You are viewing a plain text version of this content. The canonical link for it is here.
Posted to log4j-dev@logging.apache.org by bu...@apache.org on 2012/07/14 09:48:21 UTC
[Bug 53548] New: Log4j extras FormattingInfoTest failing
https://issues.apache.org/bugzilla/show_bug.cgi?id=53548
Priority: P2
Bug ID: 53548
CC: log4j-dev@logging.apache.org
Assignee: log4j-dev@logging.apache.org
Summary: Log4j extras FormattingInfoTest failing
Severity: normal
Classification: Unclassified
OS: Mac OS X 10.4
Reporter: sdeboy@iname.com
Hardware: PC
Status: NEW
Version: unspecified
Component: Companions
Product: Log4j
Gump failure:
test-compile:
[mkdir] Created dir:
/srv/gump/public/workspace/logging-log4j-extras/target/test-classes
[javac] Compiling 67 source files to
/srv/gump/public/workspace/logging-log4j-extras/target/test-classes
[javac]
/srv/gump/public/workspace/logging-log4j-extras/src/test/java/org/apache/log4j/pattern/FormattingInfoTest.java:56:
cannot find symbol
[javac] symbol : constructor FormattingInfo(boolean,boolean,int,int)
[javac] location: class org.apache.log4j.pattern.FormattingInfo
[javac] FormattingInfo field = new FormattingInfo(true, false, 3, 6);
--
You are receiving this mail because:
You are on the CC list for the bug.
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: log4j-dev-unsubscribe@logging.apache.org
For additional commands, e-mail: log4j-dev-help@logging.apache.org
[Bug 53548] Log4j extras FormattingInfoTest failing
Posted by bu...@apache.org.
https://issues.apache.org/bugzilla/show_bug.cgi?id=53548
Scott Deboy <sd...@iname.com> changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|NEW |RESOLVED
Resolution|--- |FIXED
--- Comment #1 from Scott Deboy <sd...@iname.com> ---
Fix committed in svn trunk revision 1361489.
Commit msg:
Fix FormattingInfoTest
(https://issues.apache.org/bugzilla/show_bug.cgi?id=53548)
- Updated test log files to include receivers package in class name
- Updated RewriteAppender appender definitions to correct package
(log4j.receivers.rewrite)
- Updated test-compile target in extras build.xml to put companion.jar before
log4j.jar (both contain FormattingInfo but they are different)
--
You are receiving this mail because:
You are on the CC list for the bug.
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: log4j-dev-unsubscribe@logging.apache.org
For additional commands, e-mail: log4j-dev-help@logging.apache.org
[Bug 53548] Log4j extras FormattingInfoTest failing
Posted by bu...@apache.org.
https://issues.apache.org/bugzilla/show_bug.cgi?id=53548
Scott Deboy <sd...@iname.com> changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|RESOLVED |REOPENED
Resolution|FIXED |---
--- Comment #2 from Scott Deboy <sd...@iname.com> ---
Still failing...issue is two forms of FormattingInfo exist, one in log4j core
and one in the extras companion, and the log4j core version is being picked up.
I thought I had resolve this by updating the classpath in the ant script, but
it didn't work when Gump ran. The tests fail for me at the XSLT test. I'm not
sure why the log4j snapshot is being picked up first when the tests run..any
ideas?
--
You are receiving this mail because:
You are on the CC list for the bug.
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: log4j-dev-unsubscribe@logging.apache.org
For additional commands, e-mail: log4j-dev-help@logging.apache.org
[Bug 53548] Log4j extras FormattingInfoTest failing
Posted by bu...@apache.org.
https://issues.apache.org/bugzilla/show_bug.cgi?id=53548
Scott Deboy <sd...@iname.com> changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|REOPENED |RESOLVED
Resolution|--- |FIXED
--- Comment #4 from Scott Deboy <sd...@iname.com> ---
Extras now passing in gump after a change to the gump config to put the extras
classes ahead of log4j in the test classpath.
--
You are receiving this mail because:
You are on the CC list for the bug.
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: log4j-dev-unsubscribe@logging.apache.org
For additional commands, e-mail: log4j-dev-help@logging.apache.org
[Bug 53548] Log4j extras FormattingInfoTest failing
Posted by bu...@apache.org.
https://issues.apache.org/bugzilla/show_bug.cgi?id=53548
--- Comment #3 from grobmeier <gr...@gmail.com> ---
I think the easiest way is to rename that class (the companion one).
Not sure if it is an general agreement but I really try to avoid classes with
same names/paths as it always leads to problems (at least in my projects).
As we have some non-bc changes we might easily rename this to for example
CompanionFormattingInfo or move it to another package
wdyt?
--
You are receiving this mail because:
You are on the CC list for the bug.
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: log4j-dev-unsubscribe@logging.apache.org
For additional commands, e-mail: log4j-dev-help@logging.apache.org