bugzilla | 1 Jul 2005 01:13
Picon
Favicon

DO NOT REPLY [Bug 35568] New: - log4j 1.2 unit tests have stray imports and test order change

DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://issues.apache.org/bugzilla/show_bug.cgi?id=35568>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=35568

           Summary: log4j 1.2 unit tests have stray imports and test order
                    change
           Product: Log4j
           Version: 1.2
          Platform: Other
        OS/Version: other
            Status: NEW
          Severity: normal
          Priority: P2
         Component: Other
        AssignedTo: log4j-dev <at> logging.apache.org
        ReportedBy: carnold <at> apache.org

When attempting to use the log4j 1.2 unit tests against the CVS HEAD (the future 1.3), several tests fail to 
compile due to unnecessary imports of org.apache.log4j.performance.NullAppender and 
org.apache.log4j.LocationInfo.

Three classes will fail to compile against log4j 1.3, MyPatternLayout, MyPatternParser and
DRFATestCase.  
DRFATestCase is a test of the internal implementation of the inner class RollingCalendar which is only 
visible within the org.apache.log4j package which does not make a test of the external log4j API.  
MyPatternLayout and MyPatternParser do test the publically exposed extension points for PatternLayout 
(Continue reading)

carnold | 1 Jul 2005 01:16
Picon
Favicon

cvs commit: logging-log4j/tests/src/java/org/apache/log4j/defaultInit TestCase1.java TestCase2.java TestCase3.java TestCase4.java

carnold     2005/06/30 16:16:59

  Modified:    tests    Tag: v1_2-branch build.xml
               tests/src/java/org/apache/log4j Tag: v1_2-branch
                        AsyncAppenderTestCase.java VectorAppender.java
               tests/src/java/org/apache/log4j/defaultInit Tag: v1_2-branch
                        TestCase1.java TestCase2.java TestCase3.java
                        TestCase4.java
  Log:
  Bug 35568: Remove stray imports in log4j 1.2 unit tests

  Revision  Changes    Path
  No                   revision
  No                   revision
  1.16.2.7  +7 -4      logging-log4j/tests/build.xml

  Index: build.xml
  ===================================================================
  RCS file: /home/cvs/logging-log4j/tests/build.xml,v
  retrieving revision 1.16.2.6
  retrieving revision 1.16.2.7
  diff -u -r1.16.2.6 -r1.16.2.7
  --- build.xml	3 Jun 2005 05:29:54 -0000	1.16.2.6
  +++ build.xml	30 Jun 2005 23:16:58 -0000	1.16.2.7
   <at>  <at>  -22,6 +22,7  <at>  <at> 
     <property name="project.classes.home" value="../dist/classes/"/>

     <property name="tests.source.home" value="./src/java/"/>
  +  <property name="excludes" value=""/>

(Continue reading)

bugzilla | 1 Jul 2005 01:16
Picon
Favicon

DO NOT REPLY [Bug 35568] - log4j 1.2 unit tests have stray imports and test order change

DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://issues.apache.org/bugzilla/show_bug.cgi?id=35568>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=35568

------- Additional Comments From carnold <at> apache.org  2005-07-01 01:16 -------
Created an attachment (id=15569)
 --> (http://issues.apache.org/bugzilla/attachment.cgi?id=15569&action=view)
Removal of stray imports, resequence tests in tests/build to put 1.3 failing
test at end

--

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
bugzilla | 1 Jul 2005 01:17
Picon
Favicon

DO NOT REPLY [Bug 35568] - log4j 1.2 unit tests have stray imports and test order change

DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://issues.apache.org/bugzilla/show_bug.cgi?id=35568>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=35568

carnold <at> apache.org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |FIXED

--

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
bugzilla | 1 Jul 2005 06:37
Picon
Favicon

DO NOT REPLY [Bug 35572] New: - Test coverage analysis for log4j 1.2

DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://issues.apache.org/bugzilla/show_bug.cgi?id=35572>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=35572

           Summary: Test coverage analysis for log4j 1.2
           Product: Log4j
           Version: 1.2
          Platform: Other
        OS/Version: other
            Status: NEW
          Severity: normal
          Priority: P2
         Component: Other
        AssignedTo: log4j-dev <at> logging.apache.org
        ReportedBy: carnold <at> apache.org

Bug 32722 added test coverage analysis using Clover for the CVS HEAD.  This bug adds it to the 
v1_2branch.

Test coverage analysis on the v1_2 branch should allow developers to identify compatibility issues that
are 
not tested in the current test suite so that tests can be added to check to insure compatibility between v 
1.2 and 1.3.  Instructions for use should be identical to CVS HEAD.

Modify tests/build.properties or specify -Dclover.jar to point to clover.jar.  Run tests/build.xml
with target 
(Continue reading)

carnold | 1 Jul 2005 06:38
Picon
Favicon

cvs commit: logging-log4j/tests .cvsignore build.properties.sample build.xml

carnold     2005/06/30 21:38:34

  Modified:    tests    Tag: v1_2-branch .cvsignore build.properties.sample
                        build.xml
  Log:
  Bug 35572: Add test coverage analysis to v1_2branch

  Revision  Changes    Path
  No                   revision
  No                   revision
  1.3.2.2   +2 -0      logging-log4j/tests/.cvsignore

  Index: .cvsignore
  ===================================================================
  RCS file: /home/cvs/logging-log4j/tests/.cvsignore,v
  retrieving revision 1.3.2.1
  retrieving revision 1.3.2.2
  diff -u -r1.3.2.1 -r1.3.2.2
  --- .cvsignore	27 Jun 2005 19:40:18 -0000	1.3.2.1
  +++ .cvsignore	1 Jul 2005 04:38:34 -0000	1.3.2.2
   <at>  <at>  -1,3 +1,5  <at>  <at> 
   output
   classes
   build.properties
  +coverage
  +log4j.db*

  

  1.3.2.2   +7 -9      logging-log4j/tests/build.properties.sample
(Continue reading)

bugzilla | 1 Jul 2005 06:39
Picon
Favicon

DO NOT REPLY [Bug 35572] - Test coverage analysis for log4j 1.2

DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://issues.apache.org/bugzilla/show_bug.cgi?id=35572>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=35572

carnold <at> apache.org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |FIXED

--

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
Mark Womack | 1 Jul 2005 18:46
Picon
Favicon

RE: [logging] Log4J forward compatibility with version 1.3

[getting ready to duck rotten eggs and tomatoes...]

What about just doing what the deprecation warning has been saying for all
of the 1.2 release and getting rid of Category and Priority altogether in
the 1.3 release?  I think everyone has been fairly warned about it.

Simon, is there a pending release of JCL that is waiting for an answer to
this issue?  It is on my list of items to look at in more detail this
weekend.  Don't know if I will have an answer to suggest, but I hope to be
more familiar with it.

-Mark

> -----Original Message-----
> From: Simon Kitching [mailto:skitching <at> apache.org]
> Sent: Sunday, June 19, 2005 11:12 PM
> To: Log4J Developers List
> Subject: Re: [logging] Log4J forward compatibility with version 1.3
> 
> On Sat, 2005-06-18 at 16:23 +0200, Ceki Gülcü wrote:
> > I see your point. Assuming everyone involved in log4j is cognizant of
> > this problem, I think you can expect current arrangement of Logger,
> > Category, Level etc. to persist through until 1.3 final.
> 
> In the 1.3alpha6 release has swapped the hierarchy positions for Level
> and Priority.
> 
> But it hasn't swapped Category and Logger. Is this still in the plan for
> 1.3? If so, when is this work likely to be done?
> 
(Continue reading)

Yoav Shapira | 1 Jul 2005 18:52
Picon
Favicon

RE: [logging] Log4J forward compatibility with version 1.3

Hi,

> What about just doing what the deprecation warning has been saying for all
> of the 1.2 release and getting rid of Category and Priority altogether in
> the 1.3 release?  I think everyone has been fairly warned about it.

I've been itching to do just that.  +1.

Yoav Shapira
System Design and Management Fellow
MIT Sloan School of Management / School of Engineering
Cambridge, MA USA
yoavsh <at> sloan.mit.edu / yoavs <at> computer.org
bugzilla | 1 Jul 2005 18:57
Picon
Favicon

DO NOT REPLY [Bug 35052] - Problem with String.intern() in CategoryKey

DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://issues.apache.org/bugzilla/show_bug.cgi?id=35052>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=35052

------- Additional Comments From mwomack <at> apache.org  2005-07-01 18:56 -------
1.2.12 candidate

I don't know if this is fixable, but I think it should be looked at to 
understand why we are using String.intern().

--

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.

Gmane