On Mon, Jul 23, 2018 at 10:59 AM Matt Sicker <boa...@gmail.com> wrote:
> Can't we just set up our rat rules globally rather than via a profile? It'd > be nice if CI could catch missing license headers before working on making > an RC. > +1. I do not see why we need a profile. Gary > > On Mon, 23 Jul 2018 at 11:30, Ralph Goers <ralph.go...@dslextreme.com> > wrote: > > > When you do not use -P rat you are not using our configuration so you are > > including files we specifically exclude. It is bound to fail if you don’t > > use the profile. > > > > Ralph > > > > > > > On Jul 23, 2018, at 8:15 AM, Gary Gregory <garydgreg...@gmail.com> > > wrote: > > > > > > On Mon, Jul 23, 2018 at 9:09 AM Ralph Goers < > ralph.go...@dslextreme.com> > > > wrote: > > > > > >> Interesting. As part of the release I always run > > >> > > >> mvn -P rat -DskipTests clean verify > > >> > > >> And it failed to complain about that. > > >> > > > > > > I first ran 'mvn clean install' which I should not have. But since I > did > > > that first and THEN ran the RAT check, more files must have been > slogged > > > around and hence the failure. What is odd is that I just ran 'mvn > clean' > > > and then 'mvn clean apache-rat:check' and it fails as I noted so there > > > might be something off in the POM config between running 'mvn clean > > > apache-rat:check' and using the 'rat' profile. > > > > > > Gary > > > > > > > > > > > >> > > >> As far as the MD5’s I seem to recall there was a recommendation not to > > use > > >> them but it wasn’t a mandate. I’m not sure where that email is. > > >> > > >> Ralph > > >> > > >> > > >> > > >>> On Jul 23, 2018, at 8:03 AM, Gary Gregory <garydgreg...@gmail.com> > > >> wrote: > > >>> > > >>> +1 > > >>> > > >>> - From src zip: ASC OK, SHA1 OK. > > >>> > > >>> - Maven RAT check FAILs using 'mvn apache-rat:check': > > >>> > > >>> Files with unapproved licenses: > > >>> > > >>> C:/temp/rc/log4j/apache-log4j-2.11.1-src/log4j-cassandra/.toDelete > > >>> > > >>> Not a blocker since this is not a "source" file but should be fixed > > ASAP > > >>> IMO to make RC reviews simpler. > > >>> > > >>> Subsequent modules pass RAT check using 'mvn apache-rat:check -rf > > >>> log4j-iostreams' > > >>> > > >>> - Clirr check using 'mvn clirr:check -pl log4j-api' FAILS with: > > >>> > > >>> [INFO] --- clirr-maven-plugin:2.8:check (default-cli) @ log4j-api --- > > >>> [INFO] Comparing to version: 2.11.0 > > >>> [ERROR] Unable to find information in class > > >>> org.apache.logging.log4j.util.StackLocator referring back to nested > > class > > >>> org.apache.logging.log4j.util.StackLocator$PrivateSecurityManager > > >>> > > >>> Probably OK since this is in the 'util' package. Probably OK here > > though > > >> we > > >>> should try to use the japicmp Maven plugin instead of clirr. > > >>> > > >>> - Clirr check OK for 'mvn clirr:check -pl log4j-1.2-api' > > >>> > > >>> - Building 'mvn clean install' OK using: > > >>> > > >>> Apache Maven 3.5.4 (1edded0938998edf8bf061f1ceb3cfdeccf443fe; > > >>> 2018-06-17T12:33:14-06:00) > > >>> Maven home: C:\Java\apache-maven-3.5.4\bin\.. > > >>> Java version: 1.8.0_172, vendor: Oracle Corporation, runtime: > > C:\Program > > >>> Files\Java\jdk1.8.0_172\jre > > >>> Default locale: en_US, platform encoding: Cp1252 > > >>> OS name: "windows 10", version: "10.0", arch: "amd64", family: > > "windows" > > >>> > > >>> and tool chain: > > >>> > > >>> <?xml version="1.0" encoding="UTF8"?> > > >>> <toolchains> > > >>> <!-- JDK toolchains --> > > >>> <toolchain> > > >>> <type>jdk</type> > > >>> <provides> > > >>> <version>1.6</version> > > >>> <vendor>sun</vendor> > > >>> </provides> > > >>> <configuration> > > >>> <jdkHome>C:\Program Files\Java\jdk1.6.0_45</jdkHome> > > >>> </configuration> > > >>> </toolchain> > > >>> <toolchain> > > >>> <type>jdk</type> > > >>> <provides> > > >>> <version>1.7</version> > > >>> <vendor>sun</vendor> > > >>> </provides> > > >>> <configuration> > > >>> <jdkHome>C:\Program Files\Java\jdk1.7.0_80</jdkHome> > > >>> </configuration> > > >>> </toolchain> > > >>> <toolchain> > > >>> <type>jdk</type> > > >>> <provides> > > >>> <version>1.8</version> > > >>> <vendor>sun</vendor> > > >>> </provides> > > >>> <configuration> > > >>> <jdkHome>C:\Program Files\Java\jdk1.8.0_172</jdkHome> > > >>> </configuration> > > >>> </toolchain> > > >>> <toolchain> > > >>> <type>jdk</type> > > >>> <provides> > > >>> <version>9</version> > > >>> <vendor>sun</vendor> > > >>> </provides> > > >>> <configuration> > > >>> <jdkHome>C:\Program Files\Java\jdk-9.0.4</jdkHome> > > >>> </configuration> > > >>> </toolchain> > > >>> <toolchain> > > >>> <type>jdk</type> > > >>> <provides> > > >>> <version>10</version> > > >>> <vendor>sun</vendor> > > >>> </provides> > > >>> <configuration> > > >>> <jdkHome>C:\Program Files\Java\jdk-10.0.1</jdkHome> > > >>> </configuration> > > >>> </toolchain> > > >>> <toolchain> > > >>> <type>jdk</type> > > >>> <provides> > > >>> <version>11</version> > > >>> <vendor>sun</vendor> > > >>> </provides> > > >>> <configuration> > > >>> <jdkHome>C:\Program Files\Java\jdk-11</jdkHome> > > >>> </configuration> > > >>> </toolchain> > > >>> </toolchains> > > >>> > > >>> Gary > > >>> > > >>> > > >>> On Sun, Jul 22, 2018 at 10:44 PM Ralph Goers < > > ralph.go...@dslextreme.com > > >>> > > >>> wrote: > > >>> > > >>>> This is a vote to release Log4j 2.11.1, the next version of the > Log4j > > 2 > > >>>> project. > > >>>> > > >>>> Please download, test, and cast your votes on the log4j developers > > list. > > >>>> [] +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 and we encourage everyone to test the release, but only > > >> Logging > > >>>> PMC votes are “officially” counted. As always, at least 3 +1 votes > and > > >> more > > >>>> positive than negative votes are required. > > >>>> > > >>>> Changes in this release include: > > >>>> > > >>>> Fixed Bugs > > >>>> > > >>>> • LOG4J2-2389: ThrowableProxy was saving and retrieving cache > > >>>> entries using different keys. Thanks to Liu Wen. > > >>>> • LOG4J2-2316: If root LoggerConfig does not have a Level > return > > >>>> ERROR. > > >>>> • LOG4J2-2390: Fix broken links in log4j web documentation. > > >> Thanks > > >>>> to anton-balaniuc. > > >>>> • LOG4J2-2343: The OSGi Activator specified an incorrect > > version. > > >>>> Thanks to Raymond Augé. > > >>>> • LOG4J2-2305: Make java.util.ServiceLoader properly work in > > OSGi > > >>>> by using the Service Loader Mediator Specification. Thanks to Björn > > >> Kautler. > > >>>> • LOG4J2-2305: Split the SLF4J binding into 2 implementations > - > > >>>> one for SLF4J 1.7.x and one for SLF4J 1.8+. > > >>>> • LOG4J2-2268: Improve plugin error message when elements are > > >>>> missing. Thanks to Tilman Hausherr. > > >>>> • LOG4J2-2283: ParserConfigurationException when using Log4j > > with > > >>>> oracle.xml.jaxp.JXDocumentBuilderFactory. Thanks to Vishnu Priya > > Matha. > > >>>> • LOG4J2-2300: PoolingDriverConnectionSource does not take > into > > >>>> account properties, user name, and password. > > >>>> • LOG4J2-2307: MutableLogEvent and RingBufferLogEvent message > > >>>> mementos retain the original format string. > > >>>> • LOG4J2-2032: Curly braces in parameters are not treated as > > >>>> placeholders. Thanks to Kostiantyn Shchepanovskyi. > > >>>> • LOG4J2-2317: MutableLogEvent.getNonNullImmutableMessage and > > >>>> Log4jLogEvent.makeMessageImmutable retain format and parameters. > > >>>> • LOG4J2-2318: Messages are no longer mutated when the > > >>>> asynchronous queue is full. A warning is logged to the status logger > > >>>> instead. > > >>>> • LOG4J2-2320: Fix NPE in AbstractLogger when another > exception > > >> is > > >>>> thrown, masking the root cause. > > >>>> • LOG4J2-2321: AsyncLogger uses the correct level when > > >>>> unspecified. This provides parity between AsyncLogger and Logger. > > >>>> • LOG4J2-2322: Custom ContextSelector implementations which > > >> select > > >>>> an AsyncLoggerContext disable LoggerConfig.includeLocation by > default > > >> for > > >>>> parity with AsyncLoggerContextSelector. > > >>>> • LOG4J2-2269: MutableLogEvent references to other objects are > > >>>> cleared after each use. Fix a memory leak causing references to > > >> parameters > > >>>> to be held after synchronous logging with thread locals enabled. > > >>>> • LOG4J2-2301: Mixed async loggers no longer forget parameter > > >>>> values, providing some appenders with an array of nulls. > > >>>> • LOG4J2-2331: RollingFileManager debug logging avoids string > > >>>> concatenation and errant braces in favor of parameterized logging. > > >> Thanks > > >>>> to Mike Baranski. > > >>>> • LOG4J2-2333: Handle errors thrown in default disruptor > > >>>> ExceptionHandler implementations to avoid killing background > threads. > > >>>> • LOG4J2-2334: Add API > > >>>> org.apache.logging.log4j.core.appender.AsyncAppender.getQueueSize(). > > >>>> • LOG4J2-2336: Remove duplicate hyphen from the > > AsyncLoggerConfig > > >>>> background thread name. > > >>>> • LOG4J2-2347: Update Apache Commons Compress from 1.16.1 to > > >> 1.17. > > >>>> • LOG4J2-2352: RingBufferLogEvent memento messages provide the > > >>>> expected format string, and no longer attempt to substitute > parameters > > >> into > > >>>> curly braces in parameter toString values. Both RingBufferLogEvent > and > > >>>> MutableLogEvent memento implementations memoize results to avoid > > >> rebuilding > > >>>> formatted string values. > > >>>> • LOG4J2-2355: PropertiesUtil ignores non-string system > > >>>> properties. Fixes a NoClassDefFoundError initializing StatusLogger > > >> caused > > >>>> by an NPE while initializing the static PropertiesUtil field. Thanks > > to > > >>>> Henrik Brautaset Aronsen. > > >>>> • LOG4J2-2362: Fixed a memory leak in which > > ReusableObjectMessage > > >>>> would hold a reference to the most recently logged object. > > >>>> • LOG4J2-2312: Jackson layouts used with > > >>>> AsyncLoggerContextSelector output the expected format rather than > > only a > > >>>> JSON string of the message text. > > >>>> • LOG4J2-2364: Fixed a memory leak in which > > >>>> ReusableParameterizedMessage would hold a reference to the most > > recently > > >>>> logged throwable and provided varargs array. > > >>>> • LOG4J2-2368: Nested logging doesn't clobber > > >> AbstractStringLayout > > >>>> cached StringBuidlers > > >>>> • LOG4J2-2373: StringBuilders.escapeJson implementation runs > in > > >>>> linear time. Escaping large JSON strings in EncodingPatternConverter > > and > > >>>> MapMessage will perform significantly better. Thanks to Kevin > Meurer. > > >>>> • LOG4J2-2376: StringBuilders.escapeXml implementation runs in > > >>>> linear time. Escaping large XML strings in EncodingPatternConverter > > and > > >>>> MapMessage will perform significantly better. Thanks to Kevin > Meurer. > > >>>> • LOG4J2-2377: NullPointerException in > > >>>> org.apache.logging.log4j.util.LoaderUtil.getClassLoaders() when > using > > >>>> Bootstrap class loader. Thanks to Mirko Rzehak, Gary Gregory. > > >>>> • LOG4J2-2382: Update Mongodb 3 driver from 3.6.3 to 3.8.0. > > >>>> • LOG4J2-2388: Thread indefinitely blocked when logging a > > message > > >>>> in an interrupted thread. Thanks to Failled. > > >>>> > > >>>> Changes > > >>>> > > >>>> • LOG4J2-1721: Allow composite configuration for context > > >>>> parameter. Thanks to Phokham Nonava. > > >>>> • LOG4J2-2302: Status logger should show the Log4j name and > > >>>> version when initializing itself. > > >>>> • LOG4J2-2304: Log4j2 2.8.2 JMX unregister > NullPointerException. > > >>>> Thanks to wumengsheng. > > >>>> • LOG4J2-2311: Update Jackson from 2.9.4 to 2.9.5. > > >>>> • LOG4J2-2313: Update LMAX Disruptor from 3.3.7 to 3.4.2. > > >>>> • LOG4J2-548: Log4j 2.0 ERROR "Could not search jar" with > JBoss > > >>>> EAP 6.2. Thanks to Shehata, Paresh Varke, Eric Victorson, Martin > > >> Laforet. > > >>>> • LOG4J2-2328: Update JAnsi from 1.17 to 1.17.1. > > >>>> • LOG4J2-2351: Added AbstractLogEvent.getMutableInstant to > allow > > >>>> the MutableInstant instance to be modified by classes extending > > >>>> AbstractLogEvent. > > >>>> • LOG4J2-2357: Update Jackson from 2.9.5 to 2.9.6. > > >>>> • LOG4J2-2358: Update Kafka client from 1.0.0 to 1.1.0. > > >>>> • LOG4J2-2384: Update Kafka client from 1.1.0 to 1.1.1. > > >>>> • LOG4J2-2385: Update Groovy from 2.4.13 to 2.5.1. > > >>>> • LOG4J2-2386: Update optional Apache Commons DBCP from 2.2.0 > to > > >>>> 2.4.0. > > >>>> > > >>>> Tag: > > >>>> a) for a new copy do "git clone > > >>>> https://git-wip-us.apache.org/repos/asf/logging-log4j2.git" and > then > > >> "git > > >>>> checkout tags/log4j-2.11.1-rc1” > > >>>> b) for an existing working copy to “git pull” and then “git checkout > > >>>> tags/log4j-2.11.1-rc1” > > >>>> > > >>>> Web Site: http://rgoers.github.io/log4j2-site/index.html. > > >>>> > > >>>> Maven Artifacts: > > >>>> > > >> > > https://repository.apache.org/content/repositories/orgapachelogging-1037 > > >>>> > > >>>> Distribution archives: > > >>>> https://dist.apache.org/repos/dist/dev/logging/log4j/ > > >>>> > > >>>> You may download all the Maven artifacts by executing: > > >>>> wget -e robots=off --cut-dirs=7 -nH -r -p -np --no-check-certificate > > >>>> > > >> > > > https://repository.apache.org/content/repositories/orgapachelogging-1037/org/apache/logging/log4j/ > > >>>> > > >>>> Ralph > > >>>> > > >> > > >> > > > > > > > > -- > Matt Sicker <boa...@gmail.com> >