Re: [1/2] git commit: CSS tweaks.

2014-10-04 Thread Gary Gregory
Maybe 152B55 or 061739 See http://paletton.com/#uid=13J0u0kaFw0g0qFqFg0w0aF Gary On Sat, Oct 4, 2014 at 11:37 PM, Matt Sicker wrote: > You got a hex code for that color? :) > > On 4 October 2014 22:10, Gary Gregory wrote: > >> >> -- Forwarded message -- >> From: >> Date:

Re: [1/2] git commit: CSS tweaks.

2014-10-04 Thread Matt Sicker
You got a hex code for that color? :) On 4 October 2014 22:10, Gary Gregory wrote: > > -- Forwarded message -- > From: > Date: Sat, Oct 4, 2014 at 8:56 PM > Subject: [1/2] git commit: CSS tweaks. > To: [email protected] > > > Repository: logging-log4j2 > Updated Branche

Fwd: [1/2] git commit: CSS tweaks.

2014-10-04 Thread Gary Gregory
-- Forwarded message -- From: Date: Sat, Oct 4, 2014 at 8:56 PM Subject: [1/2] git commit: CSS tweaks. To: [email protected] Repository: logging-log4j2 Updated Branches: refs/heads/master f36b214ab -> 394475673 CSS tweaks. Project: http://git-wip-us.apache.org/repo

Re: [1/3] git commit: Use of constants in JConsole plugin.

2014-10-04 Thread Remko Popma
Thanks! On Sun, Oct 5, 2014 at 10:58 AM, Remko Popma wrote: > Thanks! > > On Sun, Oct 5, 2014 at 8:55 AM, wrote: > >> Repository: logging-log4j2 >> Updated Branches: >> refs/heads/master 5ae0e6c00 -> 85f586c99 >> >> >> Use of constants in JConsole plugin. >> >> >> Project: http://git-wip-us.a

Re: [VOTE] Log4j 2.1 candidate 3

2014-10-04 Thread Remko Popma
Otsukaresama! On Sun, Oct 5, 2014 at 10:00 AM, Matt Sicker wrote: > I'm going to start the release process for candidate 4 on Sunday. > > On 4 October 2014 17:43, Matt Sicker wrote: > >> Alright, I've updated all the poms to use essentially the same reporting >> section (except for additional c

Re: [VOTE] Log4j 2.1 candidate 3

2014-10-04 Thread Matt Sicker
I'm going to start the release process for candidate 4 on Sunday. On 4 October 2014 17:43, Matt Sicker wrote: > Alright, I've updated all the poms to use essentially the same reporting > section (except for additional customizations in a couple modules). Added > ProviderUtil.loadProviders() back

Re: [3/3] git commit: Add more type safety.

2014-10-04 Thread Gary Gregory
YW! G On Sat, Oct 4, 2014 at 8:23 PM, Matt Sicker wrote: > My IDE settings must be messed up. Let me look into this. Thanks for > noticing, though! > > On 4 October 2014 19:06, Gary Gregory wrote: > >> -1. This is backwards: >> >> -import java.awt.BorderLayout; >> -import java.awt.Color; >> -i

Re: [3/3] git commit: Add more type safety.

2014-10-04 Thread Matt Sicker
My IDE settings must be messed up. Let me look into this. Thanks for noticing, though! On 4 October 2014 19:06, Gary Gregory wrote: > -1. This is backwards: > > -import java.awt.BorderLayout; > -import java.awt.Color; > -import java.awt.Component; > -import java.awt.Font; > +import java.awt.*; >

Fwd: [3/3] git commit: Add more type safety.

2014-10-04 Thread Gary Gregory
-1. This is backwards: -import java.awt.BorderLayout; -import java.awt.Color; -import java.awt.Component; -import java.awt.Font; +import java.awt.*; We do _not_ want * imports for normal imports, only for special cases like static imports of JUnit Assert methods. Gary -- Forwarded messa

Re: git commit: Add method removed from API.

2014-10-04 Thread Matt Sicker
Got it. On 4 October 2014 18:06, Gary Gregory wrote: > Can you add a Javadoc with an @deprecated tag that says something to the > effect of "do this instead" or "there is no replacement" and "might be > removed in 3.0". > > Thank you! > Gary > > -- Forwarded message -- > From: >

Fwd: git commit: Add method removed from API.

2014-10-04 Thread Gary Gregory
Can you add a Javadoc with an @deprecated tag that says something to the effect of "do this instead" or "there is no replacement" and "might be removed in 3.0". Thank you! Gary -- Forwarded message -- From: Date: Sat, Oct 4, 2014 at 6:41 PM Subject: git commit: Add method removed

Fwd: [1/4] git commit: Normalize reporting sections in poms.

2014-10-04 Thread Gary Gregory
Music to my ears :-) Gary -- Forwarded message -- From: Date: Sat, Oct 4, 2014 at 6:38 PM Subject: [1/4] git commit: Normalize reporting sections in poms. To: [email protected] Repository: logging-log4j2 Updated Branches: refs/heads/master ef23c80d7 -> 1c488bf42 No

Re: [VOTE] Log4j 2.1 candidate 3

2014-10-04 Thread Matt Sicker
Alright, I've updated all the poms to use essentially the same reporting section (except for additional customizations in a couple modules). Added ProviderUtil.loadProviders() back. Added clirr to the root pom and log4j-api. Also updated the other files that are missing headers. On 4 October 2014

Re: [2/2] git commit: Add license headers.

2014-10-04 Thread Gary Gregory
Just run the plugin as part of the build. Gary On Sat, Oct 4, 2014 at 6:21 PM, Matt Sicker wrote: > Agreed. Just gotta configure rat I guess? > > On 4 October 2014 17:13, Gary Gregory wrote: > >> Would it be better to make the build fail if the license header is >> missing? This would catch th

Re: [2/2] git commit: Add license headers.

2014-10-04 Thread Matt Sicker
Agreed. Just gotta configure rat I guess? On 4 October 2014 17:13, Gary Gregory wrote: > Would it be better to make the build fail if the license header is > missing? This would catch these errors earlier. > > Gary > > -- Forwarded message -- > From: > Date: Sat, Oct 4, 2014 at

Fwd: [2/2] git commit: Add license headers.

2014-10-04 Thread Gary Gregory
Would it be better to make the build fail if the license header is missing? This would catch these errors earlier. Gary -- Forwarded message -- From: Date: Sat, Oct 4, 2014 at 6:06 PM Subject: [2/2] git commit: Add license headers. To: [email protected] Add license hea

Re: [VOTE] Log4j 2.1 candidate 3

2014-10-04 Thread Gary Gregory
On Sat, Oct 4, 2014 at 5:40 PM, Matt Sicker wrote: > log4j-core: > MainInputArgumentsLookup doesn't have a license header along with a couple > tests. > > http://people.apache.org/~mattsicker/log4j/2.1/log4j-jul/index.html > The javadoc links to custom levels are wrong. > > > http://people.apache

Re: [VOTE] Log4j 2.1 candidate 3

2014-10-04 Thread Gary Gregory
FYI: I am going to be AFK soon but this is a good process. Hopefully the usual suspects will chime in this weekend. Thank you again, Matt, for doing the RM work. Gary On Sat, Oct 4, 2014 at 5:43 PM, Matt Sicker wrote: > javadoc links in log4j-web are not being generated for javaee6, nor to any

Re: [VOTE] Log4j 2.1 candidate 3

2014-10-04 Thread Gary Gregory
Here is how I set up Clirr: diff --git a/log4j-api/pom.xml b/log4j-api/pom.xml index a81117d..a9effe2 100644 --- a/log4j-api/pom.xml +++ b/log4j-api/pom.xml @@ -120,6 +120,11 @@ +org.codehaus.mojo +clirr-maven-plugin +${clirr.plugin.version} + +

Re: [VOTE] Log4j 2.1 candidate 3

2014-10-04 Thread Matt Sicker
javadoc links in log4j-web are not being generated for javaee6, nor to any other log4j modules (including some that are in log4j-web for some reason!) Actually, there are linking problems in all the javadocs. On 4 October 2014 16:40, Matt Sicker wrote: > log4j-core: > MainInputArgumentsLookup d

Re: [VOTE] Log4j 2.1 candidate 3

2014-10-04 Thread Gary Gregory
Oh, obviously the simple solution is to add the API back and deprecate it. Gary On Sat, Oct 4, 2014 at 5:42 PM, Gary Gregory wrote: > In master, Clirr says: > > [ERROR] 7002: org.apache.logging.log4j.util.ProviderUtil: Method > 'protected void loadProviders(java.util.Enumeration, > java.lang.Cl

Re: [VOTE] Log4j 2.1 candidate 3

2014-10-04 Thread Gary Gregory
In master, Clirr says: [ERROR] 7002: org.apache.logging.log4j.util.ProviderUtil: Method 'protected void loadProviders(java.util.Enumeration, java.lang.ClassLoader)' has been removed So we need to decide if the util package is off limits to BC or not. If so, we might need to rename it to internal

Re: [VOTE] Log4j 2.1 candidate 3

2014-10-04 Thread Matt Sicker
log4j-core: MainInputArgumentsLookup doesn't have a license header along with a couple tests. http://people.apache.org/~mattsicker/log4j/2.1/log4j-jul/index.html The javadoc links to custom levels are wrong. http://people.apache.org/~mattsicker/log4j/2.1/log4j-jul/project-reports.html Is this mis

Re: [VOTE] Log4j 2.1 candidate 3

2014-10-04 Thread Gary Gregory
I see you're using Java 8 to build which gives us the more modern looking Javadocs, nice! :-) Gary On Sat, Oct 4, 2014 at 5:30 PM, Matt Sicker wrote: > Not sure how to enable Clirr or anything. I'm uploading the site right > now, so you can give it another look in a few minutes. > > On 4 Octob

Re: [VOTE] Log4j 2.1 candidate 3

2014-10-04 Thread Matt Sicker
Not sure how to enable Clirr or anything. I'm uploading the site right now, so you can give it another look in a few minutes. On 4 October 2014 16:23, Gary Gregory wrote: > The site: That's usually the first thing I look at to find obvious > blockers like a dirty RAT or Clirr report. > > Speakin

Re: [VOTE] Log4j 2.1 candidate 3

2014-10-04 Thread Gary Gregory
The site: That's usually the first thing I look at to find obvious blockers like a dirty RAT or Clirr report. Speaking of Clirr reports... where are they? We MUST have Clirr reports at least for the API module to make sure we are not breaking binary compatibility. For non-API modules, we are bre

Re: [VOTE] Log4j 2.1 candidate 3

2014-10-04 Thread Matt Sicker
Oh no, I haven't regenerated the site yet. I'll update that in a bit. On 4 October 2014 16:13, Gary Gregory wrote: > Did you update the site on your people page? The RAT report is still dirty. > > Gary > > On Sat, Oct 4, 2014 at 4:48 PM, Matt Sicker wrote: > >> This is a vote to release Log4j 2

Re: [VOTE] Log4j 2.1 candidate 3

2014-10-04 Thread Gary Gregory
Did you update the site on your people page? The RAT report is still dirty. Gary On Sat, Oct 4, 2014 at 4:48 PM, Matt Sicker wrote: > This is a vote to release Log4j 2.1, the next major version of the Log4j 2 > project. > > Please download, test, and cast your votes. > [] +1, release the artifa

Re: [VOTE] Log4j 2.1 candidate 3

2014-10-04 Thread Gary Gregory
The source zip contains all of the .gitignore files. There should not be in there. The assembly can exclude them. I am not sure this is a blocker but no other Apache and FOSS projects I've seen allows it. Gary On Sat, Oct 4, 2014 at 4:48 PM, Matt Sicker wrote: > This is a vote to release Log4j

[VOTE] Log4j 2.1 candidate 3

2014-10-04 Thread Matt Sicker
This is a vote to release Log4j 2.1, the next major version of the Log4j 2 project. Please download, test, and cast your votes. [] +1, release the artifacts [] -1, don't release because... The vote will remain open for 72 hours (or more if required). All votes are welcome, but only Logging PMC vo

Re: [VOTE] Log4j 2.1 candidate 1

2014-10-04 Thread Matt Sicker
Alright, I'll update the pom in a bit. Trying rc3 right now (rc2 turned out to be a "tag" of 2.2-SNAPSHOT for some reason). On 4 October 2014 15:30, Gary Gregory wrote: > Well, you can exclude this one generated file from RAT processing. But in > general we want a clean RAT report, the POM can d

Re: [VOTE] Log4j 2.1 candidate 1

2014-10-04 Thread Gary Gregory
Well, you can exclude this one generated file from RAT processing. But in general we want a clean RAT report, the POM can document why it is OK to exclude this and that file from RAT. Gary On Sat, Oct 4, 2014 at 4:26 PM, Matt Sicker wrote: > How is a generated failsafe-summary.xml a licensed fi

Re: [VOTE] Log4j 2.1 candidate 1

2014-10-04 Thread Matt Sicker
How is a generated failsafe-summary.xml a licensed file? Also, took care of the other one. On 4 October 2014 15:20, Gary Gregory wrote: > Wait up! > > You should fix: > > Unapproved licenses: > > atlassian-ide-plugin.xml > log4j-distribution/target/failsafe-reports/failsafe-summary.xml > > G

Re: [VOTE] Log4j 2.1 candidate 1

2014-10-04 Thread Gary Gregory
Not a blocker: I really dislike the use of red as a font color for the code style. It's alarming. Red still means stop in my book. Gary On Sat, Oct 4, 2014 at 3:50 PM, Matt Sicker wrote: > This is a vote to release Log4j 2.1, the next major version of the Log4j 2 > project. > > Please download,

Re: [VOTE] Log4j 2.1 candidate 1

2014-10-04 Thread Gary Gregory
On the site, the link at the bottom left (in the nav menu) for surefire is broken: https://people.apache.org/~mattsicker/log4j/2.1/surefire-report.html Gary On Sat, Oct 4, 2014 at 3:50 PM, Matt Sicker wrote: > This is a vote to release Log4j 2.1, the next major version of the Log4j 2 > project

Re: [VOTE] Log4j 2.1 candidate 1

2014-10-04 Thread Gary Gregory
Wait up! You should fix: Unapproved licenses: atlassian-ide-plugin.xml log4j-distribution/target/failsafe-reports/failsafe-summary.xml Gary On Sat, Oct 4, 2014 at 4:14 PM, Matt Sicker wrote: > Whoops, I mixed up the order there. Trying again with candidate 2. > > On 4 October 2014 15:1

Re: [VOTE] Log4j 2.1 candidate 1

2014-10-04 Thread Matt Sicker
Whoops, I mixed up the order there. Trying again with candidate 2. On 4 October 2014 15:12, Ralph Goers wrote: > > Yes. The release plugin should have deployed those to Nexus. You delete > them from Nexus after the vote when you publish them to the distribution > archives and before the are mov

Re: [VOTE] Log4j 2.1 candidate 1

2014-10-04 Thread Ralph Goers
Yes. The release plugin should have deployed those to Nexus. You delete them from Nexus after the vote when you publish them to the distribution archives and before the are moved from Staging to Release. Ralph On Oct 4, 2014, at 1:07 PM, Matt Sicker wrote: > In the Nexus? > > On 4 October

Re: [VOTE] Log4j 2.1 candidate 1

2014-10-04 Thread Matt Sicker
In the Nexus? On 4 October 2014 15:06, Gary Gregory wrote: > Something is missing: there is supposed to be a set of zip/tar balls that > are a binary and source distributions. > > Gary > > On Sat, Oct 4, 2014 at 3:50 PM, Matt Sicker wrote: > >> This is a vote to release Log4j 2.1, the next majo

Re: [VOTE] Log4j 2.1 candidate 1

2014-10-04 Thread Gary Gregory
Something is missing: there is supposed to be a set of zip/tar balls that are a binary and source distributions. Gary On Sat, Oct 4, 2014 at 3:50 PM, Matt Sicker wrote: > This is a vote to release Log4j 2.1, the next major version of the Log4j 2 > project. > > Please download, test, and cast yo

Re: [VOTE] Log4j 2.1 candidate 1

2014-10-04 Thread Matt Sicker
Apologies, the correct wget command to download everything is: wget -e robots=off --cut-dirs=3 -r -p -np --no-check-certificate https://repository.apache.org/content/repositories/orgapachelogging-1008/org/apache/logging/log4j/ On 4 October 2014 14:50, Matt Sicker wrote: > This is a vote to rele

[VOTE] Log4j 2.1 candidate 1

2014-10-04 Thread Matt Sicker
This is a vote to release Log4j 2.1, the next major version of the Log4j 2 project. Please download, test, and cast your votes. [] +1, release the artifacts [] -1, don't release because... The vote will remain open for 72 hours (or more if required). All votes are welcome, but only Logging PMC vo

Re: logback in POMs

2014-10-04 Thread Matt Sicker
Remko, I fixed that in my local branch. I'll push that today before I officially start the release process. On 4 October 2014 07:47, Remko Popma wrote: > The perf module doesn't have unit tests, all benchmarks are in main. Some > benchmarks test logback, so that's ok to remain as a main dependen

Re: logback in POMs

2014-10-04 Thread Remko Popma
The perf module doesn't have unit tests, all benchmarks are in main. Some benchmarks test logback, so that's ok to remain as a main dependency. Sent from my iPhone > On 2014/10/04, at 21:18, Gary Gregory wrote: > > Done. Just one change. The perf module is a "main" dep. > > Gary > >> On Sa

Re: logback in POMs

2014-10-04 Thread Gary Gregory
Done. Just one change. The perf module is a "main" dep. Gary On Sat, Oct 4, 2014 at 1:11 AM, Matt Sicker wrote: > That would make sense indeed. > > On 3 October 2014 21:16, Gary Gregory wrote: > >> Hi All: >> >> Shouldn't all the logback references in POMs be in the test scope? >> >> Gary >> >

test failure on Linux only

2014-10-04 Thread Gary Gregory
Odd: I see a test failure only on Linux with mvn test in org.apache.log4j.LoggingTest.testParent() The Maven build is fine on Windows. Anyone else? On Linux in am in VirtualBox, Maven 3.0.5, Oracle Java 1.7.0_65, Linux 3.13.0-36-generic. Gary -- E-Mail: [email protected] | ggreg...@apac