I see that you check coverage via jacoco-maven-plugin <https://github.com/edwgiz/maven-shaded-log4j-transformer/blob/master/pom.xml#L211>. Repeating this practice for this module (only?) is fine by me.
On Tue, Oct 19, 2021 at 9:52 AM Eduard Gizatullin <[email protected]> wrote: > Done, https://github.com/apache/logging-log4j-tools/pull/2 > > The new artifact can be built, > I care about one minor thing: whether it is possible to enable code > coverage check during a test stage? Original project was configured for 96% > threshold for LOC > > -- > Best regards, > Ed > > On Tue, Oct 19, 2021 at 9:26 AM Volkan Yazıcı <[email protected]> wrote: > > > Makes sense Eduardo, go ahead with your proposal for the first draft. > > > > On Mon, Oct 18, 2021 at 7:22 PM Eduard Gizatullin <[email protected]> > > wrote: > > > > > Hello Volkan, > > > > > > Thanks for your explanation, I really appreciate it and I think I can > > make > > > a pull request tomorrow. > > > > > > With hindsight, couldn't you please consider below points: > > > > > > 1. maven-shaded-log4j-transformer is not a plugin, it's an extension > for > > > the existing shaded plugin. Also the dependency must be explicitly > > > declared. > > > <groupId>org.apache.maven.plugins</groupId> > > > <artifactId>maven-shade-plugin</artifactId> > > > <version>3.2.4</version> > > > <executions> > > > <execution> > > > <phase>package</phase> > > > <goals> > > > <goal>shade</goal> > > > </goals> > > > <configuration> > > > <transformers> > > > <transformer > > > > > > > > > > > > implementation="org.apache.logging.maven.plugins.shade.Log4j2PluginCacheFileTransformer"> > > > </transformer> > > > </transformers> > > > ... > > > </configuration> > > > </execution> > > > </executions> > > > <dependencies> > > > <dependency> > > > <groupId>org.apache.logging.maven</groupId> > > > <artifactId>log4j-maven-plugin</artifactId> > > > <version>2.15.0</version> > > > </dependency> > > > In this case a more specific name like shade-plugin-log4j-transfomer > > > reflects exactly the essence of the extension. > > > > > > > > > 2. Putting all eggs in one basket can lead to misunderstanding when one > > > artifact should be declared as dependency for different purposes. Also > > > let's take into account the popularity of log4j and imagine new code of > > > log4j-maven-plugin might require heavy dependencies, which can slow > > down > > > those maven builds that using only the transformer or other > light-weight > > > functionality. > > > > > > Therefore let's consider a risk that a single artifact might become a > > > burden. > > > > > > > > > Looking forward for your answer > > > > > > > > > -- > > > Best regards, > > > Ed > > > > > > > > > > > > > > > On Sat, Oct 16, 2021 at 1:04 AM Gary Gregory <[email protected]> > > > wrote: > > > > > > > Here it is: https://github.com/apache/logging-log4j-tools > > > > > > > > Gary > > > > > > > > > > > > On Fri, Oct 15, 2021, 14:12 Ralph Goers <[email protected]> > > > > wrote: > > > > > > > > > Yeah, that makes sense. It is less specific than just Maven > plugins. > > > > > > > > > > Ralph > > > > > > > > > > > On Oct 15, 2021, at 9:51 AM, Gary Gregory < > [email protected]> > > > > > wrote: > > > > > > > > > > > > I think Ralph started a tools repo which we should reuse for this > > > > > component > > > > > > IMO. > > > > > > > > > > > > Gary > > > > > > > > > > > > On Fri, Oct 15, 2021, 11:22 Matt Sicker <[email protected]> > wrote: > > > > > > > > > > > >> I’m in favor of making this its own repo. Making the build, > site, > > > and > > > > > >> release process as simple as possible would be great. I can help > > > with > > > > > the > > > > > >> release process at least. > > > > > >> > > > > > >> Matt Sicker > > > > > >> > > > > > >>> On Oct 15, 2021, at 09:56, Volkan Yazıcı <[email protected]> > wrote: > > > > > >>> > > > > > >>> License headers, `changes.xml` integration, JPMS shizzle(?), > > etc. > > > I > > > > > >> guess > > > > > >>> we will need to copy quite some plumbing code from Log4j 2. > > > > > Nevertheless, > > > > > >>> they are all doable. > > > > > >>> I am also in favor of using GitHub all the way down: GitHub > > Issues > > > > for > > > > > >>> issue tracking, GitHub Actions for CI/CD (yes, even CD!), > GitHub > > > > pages > > > > > >> for > > > > > >>> publishing the Maven-generated site, etc. > > > > > >>> > > > > > >>> > > > > > >>>> On Fri, Oct 15, 2021 at 4:48 PM Gary Gregory < > > > > [email protected]> > > > > > >> wrote: > > > > > >>>> > > > > > >>>> Hi Ed, > > > > > >>>> > > > > > >>>> Your files must have the Apache License header comment, > > otherwise > > > > > >> running > > > > > >>>> 'mvn apache-rat:check' will fail. > > > > > >>>> > > > > > >>>> Gary > > > > > >>>> > > > > > >>>> > > > > > >>>>> On Fri, Oct 15, 2021, 09:49 Eduard Gizatullin < > > [email protected]> > > > > > >> wrote: > > > > > >>>>> > > > > > >>>>> Hello dear log4j team > > > > > >>>>> > > > > > >>>>> Volkan Yazıcı asked me to make > maven-shaded-log4j-transformer > > > > > >>>>> <https://github.com/edwgiz/maven-shaded-log4j-transformer> a > > > part > > > > of > > > > > >>>>> log4j2 and I tend to accomplish the proposal. > > > > > >>>>> > > > > > >>>>> Can you please confirm that > > > > > >>>>> new sub-module name log4j-maven-shade-plugin is ok > > > > > >>>>> > > > > > >>>>> Any preliminary advice will be appreciated > > > > > >>>>> > > > > > >>>>> > > > > > >>>>> -- > > > > > >>>>> Best regards, > > > > > >>>>> Ed > > > > > >>>>> > > > > > >>>>> > > > > > >>>>>> On Fri, Oct 15, 2021 at 1:39 PM Volkan Yazıcı < > [email protected] > > > > > > > > wrote: > > > > > >>>>> > > > > > >>>>>> Thanks for the prompt (and positive!) reply Eduard! > > > > > >>>>>> I think it is best to first lay out the details of the plan > > in a > > > > > post > > > > > >>>>> to the > > > > > >>>>>> dev mailing list < > > > > > >> https://logging.apache.org/log4j/2.x/mail-lists.html > > > > > >>>>>> . > > > > > >>>>>> For instance, the module name, transformer name, > documentation > > > > > >> changes, > > > > > >>>>>> etc. > > > > > >>>>>> This will give others an opportunity to share their feedback > > and > > > > > >>>>> remarks. > > > > > >>>>>> Then simply create a JIRA < > > > > > >>>>> https://issues.apache.org/jira/projects/LOG4J2> > > > > > >>>>>> ticket and submit a GitHub < > > > > > https://github.com/apache/logging-log4j2> > > > > > >>>>> PR. > > > > > >>>>>> > > > > > >>>>>> `master` branch targets Log4j 3, which is not released yet. > > > > > >>>>>> It uses a different plugin loading mechanism than the one > used > > > in > > > > > >> Log4j > > > > > >>>>> 2. > > > > > >>>>>> Log4j 3 doesn't suffer from this "override of plugins after > > > > shading" > > > > > >>>>>> problem. > > > > > >>>>>> Hence, the PR needs to target the `release-2.x` branch. > > > > > >>>>>> > > > > > >>>>>> Also note that since this is a non-trivial contribution, you > > > need > > > > to > > > > > >>>>> sign the > > > > > >>>>>> ICLA document <https://www.apache.org/licenses/icla.pdf> > and > > > > email > > > > > it > > > > > >>>>> to > > > > > >>>>>> the ASF <[email protected]>. > > > > > >>>>>> Once you have done this, it is good to mention this in the > dev > > > > > mailing > > > > > >>>>>> list. > > > > > >>>>>> > > > > > >>>>>> On Fri, Oct 15, 2021 at 12:26 PM Eduard Gizatullin < > > > > > [email protected]> > > > > > >>>>>> wrote: > > > > > >>>>>> > > > > > >>>>>>> Hello Volkan, > > > > > >>>>>>> > > > > > >>>>>>> Thank you for letting me know, I'm all for it. > > > > > >>>>>>> > > > > > >>>>>>> Couldn't you please confirm that target branch is master > > > > > >>>>>>> and log4j-maven-plugins is ok as new name of submodule > > > > > >>>>>>> > > > > > >>>>>>> Any other advices will be appreciated, > > > > > >>>>>>> > > > > > >>>>>>> -- > > > > > >>>>>>> Best regards, > > > > > >>>>>>> Ed > > > > > >>>>>>> > > > > > >>>>>>> > > > > > >>>>>>> On Fri, Oct 15, 2021 at 10:10 AM Volkan Yazıcı < > > [email protected] > > > > > > > > > >> wrote: > > > > > >>>>>>> > > > > > >>>>>>>> Hello, > > > > > >>>>>>>> > > > > > >>>>>>>> My name is Volkan Yazici and I am a PMC committee member > of > > > the > > > > > ASF > > > > > >>>>>>>> Logging Services, which develops Log4j too. > > > > > >>>>>>>> maven-shaded-log4j-transformer > > > > > >>>>>>>> <https://github.com/edwgiz/maven-shaded-log4j-transformer > > > > > > plugin > > > > > >>>>>>>> addresses an important shortcoming of the Log4j 2.x plugin > > > > design > > > > > >>>>> surfacing > > > > > >>>>>>>> when users want to shade it. We have recently had a chat > > about > > > > it > > > > > in > > > > > >>>>>>>> the mailing list > > > > > >>>>>>>> < > > > > > >>>>> > > > > > >> > > > > > > > > > > > > > > > https://lists.apache.org/thread.html/rcfa4fc8678642a51e3a69dd2b14848fe4e1e5b71de7c99a7b55ff182%40%3Cdev.logging.apache.org%3E > > > > > >>>>>> , > > > > > >>>>>>>> and the maintainers (incl. me) are inclined to ship it as > a > > > part > > > > > of > > > > > >>>>> the > > > > > >>>>>>>> Log4j project. Would you like to contribute it yourself in > > the > > > > > form > > > > > >>>>> of a > > > > > >>>>>>>> GitHub PR? Note that this route is subject to > > > update-push-review > > > > > >>>>> cycles, > > > > > >>>>>>>> yet they are pretty rewarding for both parties, IMHO. What > > do > > > > you > > > > > >>>>> think? > > > > > >>>>>>>> > > > > > >>>>>>>> Kind regards. > > > > > >>>>>>>> > > > > > >>>>>>> > > > > > >>>>> > > > > > >>>> > > > > > >> > > > > > > > > > > > > > > > > > > > > > > > > >
