[VOTE] Release Apache Maven Checkstyle Plugin version 3.1.0

classic Classic list List threaded Threaded
7 messages Options
Reply | Threaded
Open this post in threaded view
|

[VOTE] Release Apache Maven Checkstyle Plugin version 3.1.0

Enrico Olivelli
Hi,

We solved 13 issues:
https://issues.apache.org/jira/secure/ReleaseNote.jspa?version=12342397&styleName=Text&projectId=12317223

Please note that this version now only supports CheckStyle 8+ and Java 8
(as required by latest Checkstyle)

There are still a couple of issues left in JIRA:
https://issues.apache.org/jira/issues/?jql=project%20%3D%20XXXXXXXXXX%20AND%20status%20%3D%20Open%20ORDER%20BY%20key%20DESC%2C%20priority%20DESC

Staging repo:
https://repository.apache.org/content/repositories/maven-1503/
https://repository.apache.org/content/repositories/maven-1503/org/apache/maven/plugins/maven-checkstyle-plugin/3.1.0/maven-checkstyle-plugin-3.1.0-source-release.zip

Source release checksum(s):
maven-checkstyle-plugin-3.1.0-source-release.zip sha512:
eca46edb4d2f6cf2e250169ce5d5e510781b4bba116cc7b5655797cdb109ccdb0da883a1761347c6e2c77fe58395aff17e45c46d85a8a823340ae2b11c92852e

Staging site:
https://maven.apache.org/plugins-archives/maven-checkstyle-plugin-LATEST/

Guide to testing staged releases:
https://maven.apache.org/guides/development/guide-testing-releases.html

Vote open for at least 72 hours.

[ ] +1
[ ] +0
[ ] -1


Enrico Olivelli
Reply | Threaded
Open this post in threaded view
|

Re: [VOTE] Release Apache Maven Checkstyle Plugin version 3.1.0

Karl Heinz Marbaise-3
Hi,

I have checked the SHA512 Ok...and unpacked the release and tested via:

mvn -Prun-its clean verify

Get a complete fine build without any issue on MacOS JDK11... Maven 3.6.1

so +1 from me...

Kind regards
Karl Heinz Marbaise

On 13.05.19 11:52, Enrico Olivelli wrote:

> Hi,
>
> We solved 13 issues:
> https://issues.apache.org/jira/secure/ReleaseNote.jspa?version=12342397&styleName=Text&projectId=12317223
>
> Please note that this version now only supports CheckStyle 8+ and Java 8
> (as required by latest Checkstyle)
>
> There are still a couple of issues left in JIRA:
> https://issues.apache.org/jira/issues/?jql=project%20%3D%20XXXXXXXXXX%20AND%20status%20%3D%20Open%20ORDER%20BY%20key%20DESC%2C%20priority%20DESC
>
> Staging repo:
> https://repository.apache.org/content/repositories/maven-1503/
> https://repository.apache.org/content/repositories/maven-1503/org/apache/maven/plugins/maven-checkstyle-plugin/3.1.0/maven-checkstyle-plugin-3.1.0-source-release.zip
>
> Source release checksum(s):
> maven-checkstyle-plugin-3.1.0-source-release.zip sha512:
> eca46edb4d2f6cf2e250169ce5d5e510781b4bba116cc7b5655797cdb109ccdb0da883a1761347c6e2c77fe58395aff17e45c46d85a8a823340ae2b11c92852e
>
> Staging site:
> https://maven.apache.org/plugins-archives/maven-checkstyle-plugin-LATEST/
>
> Guide to testing staged releases:
> https://maven.apache.org/guides/development/guide-testing-releases.html
>
> Vote open for at least 72 hours.
>
> [ ] +1
> [ ] +0
> [ ] -1
>
>
> Enrico Olivelli
>

---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|

Re: [VOTE] Release Apache Maven Checkstyle Plugin version 3.1.0

Enrico Olivelli
In reply to this post by Enrico Olivelli
Il lun 13 mag 2019, 23:48 Tibor Digana <[hidden email]> ha scritto:

> Robert, I did *not* use the source zip.
>

IMHO we should vote on the staged zip


> git clone https://gitbox.apache.org/repos/asf/maven-checkstyle-plugin
> Oracle jdk 1.8.0u212, Maven 3.3.9, Windows 10
>

This is what CI does and tests are passing.

Do you have some global git configuration?

Enrico

>
> I see there is a new line, but the checkstyle does not care if you put one
> or two lines.
> No idea why.
>
> Even if you go to the target and run it from the folder
> c:\vcs\github\maven-checkstyle-plugin\target\it\MCHECKSTYLE-54\
> it's the same as if you run the project root - mvn verify -P
> run-its,quality-checks
>
>
> *mvn -nsu checkstyle:check*
>
> [INFO] --- maven-checkstyle-plugin:3.1.1-SNAPSHOT:check (default-cli) @
> mcheckstyle-54 ---
> [INFO] There are 2 errors reported by Checkstyle 8.19 with sun_checks.xml
> ruleset.
> [ERROR]
>
> src\main\java\org\apache\maven\plugins\checkstyle\mcheckstyle54\Mcheckstyle54.java:[1]
> (misc) NewlineAtEndOfFile: File does not end with a newline.
> [ERROR]
>
> src\main\java\org\apache\maven\plugins\checkstyle\mcheckstyle54\package-info.java:[1]
> (misc) NewlineAtEndOfFile: File does not end with a newline.
> [INFO]
> ------------------------------------------------------------------------
> [INFO] BUILD FAILURE
> [INFO]
> ------------------------------------------------------------------------
> [INFO] Total time: 3.450 s
> [INFO] Finished at: 2019-05-13T23:42:55+02:00
> [INFO] Final Memory: 12M/193M
> [INFO]
> ------------------------------------------------------------------------
> [ERROR] Failed to execute goal
> org.apache.maven.plugins:maven-checkstyle-plugin:3.1.1-SNAPSHOT:check
> (default-cli) on project mcheckstyle-54: You have 2 Checkstyle violations.
> -> [Help 1]
> [ERROR]
> [ERROR] To see the full stack trace of the errors, re-run Maven with the -e
> switch.
> [ERROR] Re-run Maven using the -X switch to enable full debug logging.
> [ERROR]
> [ERROR] For more information about the errors and possible solutions,
> please read the following articles:
> [ERROR] [Help 1]
> http://cwiki.apache.org/confluence/display/MAVEN/MojoFailureException
>
>
> On Mon, May 13, 2019 at 10:29 PM Robert Scholte <[hidden email]>
> wrote:
>
> > This can happen when source-release.zip was created on a different OS
> > compared to the verifying OS.
> >
> > With Git it will switch to the line endings of the operating system, but
> > with a zip that's not possible.
> >
> > I'd say not blocking, but the IT needs to be fixed to handle these
> > situations.
> >
> > Robert
> >
> >
> > On Mon, 13 May 2019 21:55:27 +0200, Enrico Olivelli <[hidden email]
> >
> >
> > wrote:
> >
> > > Tibor
> > > It is strage all its are passing on CI.
> > > Are you sure you have correcly unpacked the package?
> > >
> > >
> > > Il lun 13 mag 2019, 21:23 Tibor Digana <[hidden email]> ha
> > > scritto:
> > >
> > >> checked the sha512 of src zip, ok
> > >> checked the build, failed (mvn verify -P run-its,quality-checks)
> > >>
> > >
> > > What is 'quality-checks' profile? I have never heard about it
> > >
> > > Enrico
> > >
> > >
> > >> [INFO] Building: MCHECKSTYLE-54\pom.xml
> > >> [INFO]           MCHECKSTYLE-54\pom.xml ...........................
> > >> FAILED
> > >> (5.3 s)
> > >>
> > >> *Mcheckstyle54.java:[1] (misc) NewlineAtEndOfFile: File does not end
> > >> with a
> > >> newline.*
> > >>
> > >> but I checkted this file and it ends with a new line. Is it bug in the
> > >> Checkstyle dependency?
> > >>
> > >> [INFO] BUILD FAILURE
> > >> [ERROR] Failed to execute goal
> > >> org.apache.maven.plugins:maven-pmd-plugin:3.8:cpd-check (cpd-check) on
> > >> project maven-checkstyle-plugin: You have 1 CPD duplication.
> > >>
> > >> There are exactly the same methods. That's why CPD fails, see
> > >> CheckstyleViolationCheckMojo L813
> > >> AbstractCheckstyleReport        L581
> > >>
> > >>
> > >>
> > >>
> > >>
> > >>
> > >>
> > >>
> > >>
> > >> On Mon, May 13, 2019 at 11:52 AM Enrico Olivelli <[hidden email]
> >
> > >> wrote:
> > >>
> > >> > Hi,
> > >> >
> > >> > We solved 13 issues:
> > >> >
> > >> >
> > >>
> >
> https://issues.apache.org/jira/secure/ReleaseNote.jspa?version=12342397&styleName=Text&projectId=12317223
> > >> >
> > >> > Please note that this version now only supports CheckStyle 8+ and
> > >> Java 8
> > >> > (as required by latest Checkstyle)
> > >> >
> > >> > There are still a couple of issues left in JIRA:
> > >> >
> > >> >
> > >>
> >
> https://issues.apache.org/jira/issues/?jql=project%20%3D%20XXXXXXXXXX%20AND%20status%20%3D%20Open%20ORDER%20BY%20key%20DESC%2C%20priority%20DESC
> > >> >
> > >> > Staging repo:
> > >> > https://repository.apache.org/content/repositories/maven-1503/
> > >> >
> > >> >
> > >>
> >
> https://repository.apache.org/content/repositories/maven-1503/org/apache/maven/plugins/maven-checkstyle-plugin/3.1.0/maven-checkstyle-plugin-3.1.0-source-release.zip
> > >> >
> > >> > Source release checksum(s):
> > >> > maven-checkstyle-plugin-3.1.0-source-release.zip sha512:
> > >> >
> > >> >
> > >>
> >
> eca46edb4d2f6cf2e250169ce5d5e510781b4bba116cc7b5655797cdb109ccdb0da883a1761347c6e2c77fe58395aff17e45c46d85a8a823340ae2b11c92852e
> > >> >
> > >> > Staging site:
> > >> >
> > >>
> >
> https://maven.apache.org/plugins-archives/maven-checkstyle-plugin-LATEST/
> > >> >
> > >> > Guide to testing staged releases:
> > >> >
> > >>
> https://maven.apache.org/guides/development/guide-testing-releases.html
> > >> >
> > >> > Vote open for at least 72 hours.
> > >> >
> > >> > [ ] +1
> > >> > [ ] +0
> > >> > [ ] -1
> > >> >
> > >> >
> > >> > Enrico Olivelli
> > >> >
> >
> > ---------------------------------------------------------------------
> > To unsubscribe, e-mail: [hidden email]
> > For additional commands, e-mail: [hidden email]
> >
> >
>
Reply | Threaded
Open this post in threaded view
|

Problems with checkstyle tests on Windows (Was [VOTE] Release Apache Maven Checkstyle Plugin version 3.1.0)

Enrico Olivelli
I will get a windows box and try to reproduce.
It is weird that on ASF Jenkins the build is passing even on windows

Enrico

---------- Forwarded message ---------
Da: Enrico Olivelli <[hidden email]>
Date: mar 14 mag 2019, 13:58
Subject: Re: [VOTE] Release Apache Maven Checkstyle Plugin version 3.1.0
To: Maven Developers List <[hidden email]>


Eric and Tibor,
Thank you so much for your effort in testing Maven Checkstyle Plugin.

This is the "official" VOTE thread, here we have to decide if the staged
artifacts are good to be released or not.

Feel free to cast a -1 if you think that the staged artifacts are not
"stable" or there is any showstopper problem for the release.

Let's move this discussion to a separate thread, something like "Validation
failures in Windows over current checkstyle plugin master branch")

Enrico




Il giorno mar 14 mag 2019 alle ore 13:51 Eric Lilja <[hidden email]>
ha scritto:

> Tried overriding line.separator when running using -Dline.separator="\n",
> but then the builds fails (early) in maven-plugin-plugin:
> [ERROR] Failed to execute goal
> org.apache.maven.plugins:maven-plugin-plugin:3.5:descriptor
> (default-descriptor) on project maven-checkstyle-plugin: Execution
> default-descriptor of goal
> org.apache.maven.plugins:maven-plugin-plugin:3.5:descriptor failed:
> Requested line separator is invalid. -> [Help 1]
> [ERROR]
> [ERROR] To see the full stack trace of the errors, re-run Maven with the -e
> switch.
> [ERROR] Re-run Maven using the -X switch to enable full debug logging.
> [ERROR]
> [ERROR] For more information about the errors and possible solutions,
> please read the following articles:
> [ERROR] [Help 1]
> http://cwiki.apache.org/confluence/display/MAVEN/PluginExecutionException
>
> Tried upgrading maven-plugin-plugin to 3.6.0 (lots of stuff
> upgradeable...), but same error
>
> I also happened to notice this (probably unrelated, but wanted to bring it
> to attention anyway so it can be fixed) warning:
> [INFO] java-annotations mojo extractor found 4 mojo descriptors.
> [WARNING]
>
> Unexpected situation: destinationDirectory not defined in
> maven-plugin-help.properties during help mojo source generation but
> expected during XML descriptor generation.
> [WARNING] Please check helpmojo goal version used in previous build phase.
> [WARNING] If you just upgraded to plugin-tools >= 3.2 you must run a clean
> build at least once.
> [WARNING] Trying default location: target\generated-sources\plugin
>
> - Eric L
>
> On Tue, May 14, 2019 at 11:04 AM Eric Lilja <[hidden email]> wrote:
>
> > I tried bumping checkstyle to 8.20, plus a few of the plexus
> dependencies,
> > but that just brought an additional failure... (to
> > MCHECKSTYLE-70-multi-sourcefolder\pom.xml)  :-)
> >
> > I suppose the problem might be that the files has linux-style line breaks
> > (this is desired for me, I don't want to convert to windows-style line
> > breaks locally), but the test think I should have windows-style line
> > separators. It seems these files are generated by the tests because I
> tried
> > changing them to Windows style line breaks for re-running just to see if
> > that would work, but those changes were overwritten)
> >
> > - Eric L
> >
> > On Tue, May 14, 2019 at 10:38 AM Eric Lilja <[hidden email]>
> wrote:
> >
> >> I also see a failure for MCHECKSTYLE-54 on Windows. (Sorry, I didn't try
> >> the source zip, just cloned master)
> >>
> >> I tested on one of our corporate laptops:
> >> Windows 10
> >> Cygwin 64-bit (I use it to clone the repo and use Maven)
> >> Maven 3.6.0
> >> Java 8 update 202
> >>
> >> The build log says:
> >> [INFO] There are 2 errors reported by Checkstyle 8.19 with
> sun_checks.xml
> >> ruleset.
> >> [ERROR]
> >>
> src\main\java\org\apache\maven\plugins\checkstyle\mcheckstyle54\Mcheckstyle54.java:[1]
> >> (misc) NewlineAtEndOfFile: File does not end with a newline.
> >> [ERROR]
> >>
> src\main\java\org\apache\maven\plugins\checkstyle\mcheckstyle54\package-info.java:[1]
> >> (misc) NewlineAtEndOfFile: File does not end with a newline.
> >>
> >> These two files end with unix-style line breaks (as expected with my
> >> setup).
> >>
> >> - Eric L
> >>
> >>
> >> On Tue, May 14, 2019 at 8:10 AM Enrico Olivelli <[hidden email]>
> >> wrote:
> >>
> >>> Il lun 13 mag 2019, 23:48 Tibor Digana <[hidden email]> ha
> >>> scritto:
> >>>
> >>> > Robert, I did *not* use the source zip.
> >>> >
> >>>
> >>> IMHO we should vote on the staged zip
> >>>
> >>>
> >>> > git clone
> https://gitbox.apache.org/repos/asf/maven-checkstyle-plugin
> >>> > Oracle jdk 1.8.0u212, Maven 3.3.9, Windows 10
> >>> >
> >>>
> >>> This is what CI does and tests are passing.
> >>>
> >>> Do you have some global git configuration?
> >>>
> >>> Enrico
> >>>
> >>> >
> >>> > I see there is a new line, but the checkstyle does not care if you
> put
> >>> one
> >>> > or two lines.
> >>> > No idea why.
> >>> >
> >>> > Even if you go to the target and run it from the folder
> >>> > c:\vcs\github\maven-checkstyle-plugin\target\it\MCHECKSTYLE-54\
> >>> > it's the same as if you run the project root - mvn verify -P
> >>> > run-its,quality-checks
> >>> >
> >>> >
> >>> > *mvn -nsu checkstyle:check*
> >>> >
> >>> > [INFO] --- maven-checkstyle-plugin:3.1.1-SNAPSHOT:check
> (default-cli) @
> >>> > mcheckstyle-54 ---
> >>> > [INFO] There are 2 errors reported by Checkstyle 8.19 with
> >>> sun_checks.xml
> >>> > ruleset.
> >>> > [ERROR]
> >>> >
> >>> >
> >>>
> src\main\java\org\apache\maven\plugins\checkstyle\mcheckstyle54\Mcheckstyle54.java:[1]
> >>> > (misc) NewlineAtEndOfFile: File does not end with a newline.
> >>> > [ERROR]
> >>> >
> >>> >
> >>>
> src\main\java\org\apache\maven\plugins\checkstyle\mcheckstyle54\package-info.java:[1]
> >>> > (misc) NewlineAtEndOfFile: File does not end with a newline.
> >>> > [INFO]
> >>> >
> >>>
> ------------------------------------------------------------------------
> >>> > [INFO] BUILD FAILURE
> >>> > [INFO]
> >>> >
> >>>
> ------------------------------------------------------------------------
> >>> > [INFO] Total time: 3.450 s
> >>> > [INFO] Finished at: 2019-05-13T23:42:55+02:00
> >>> > [INFO] Final Memory: 12M/193M
> >>> > [INFO]
> >>> >
> >>>
> ------------------------------------------------------------------------
> >>> > [ERROR] Failed to execute goal
> >>> > org.apache.maven.plugins:maven-checkstyle-plugin:3.1.1-SNAPSHOT:check
> >>> > (default-cli) on project mcheckstyle-54: You have 2 Checkstyle
> >>> violations.
> >>> > -> [Help 1]
> >>> > [ERROR]
> >>> > [ERROR] To see the full stack trace of the errors, re-run Maven with
> >>> the -e
> >>> > switch.
> >>> > [ERROR] Re-run Maven using the -X switch to enable full debug
> logging.
> >>> > [ERROR]
> >>> > [ERROR] For more information about the errors and possible solutions,
> >>> > please read the following articles:
> >>> > [ERROR] [Help 1]
> >>> >
> http://cwiki.apache.org/confluence/display/MAVEN/MojoFailureException
> >>> >
> >>> >
> >>> > On Mon, May 13, 2019 at 10:29 PM Robert Scholte <
> [hidden email]>
> >>> > wrote:
> >>> >
> >>> > > This can happen when source-release.zip was created on a different
> OS
> >>> > > compared to the verifying OS.
> >>> > >
> >>> > > With Git it will switch to the line endings of the operating
> system,
> >>> but
> >>> > > with a zip that's not possible.
> >>> > >
> >>> > > I'd say not blocking, but the IT needs to be fixed to handle these
> >>> > > situations.
> >>> > >
> >>> > > Robert
> >>> > >
> >>> > >
> >>> > > On Mon, 13 May 2019 21:55:27 +0200, Enrico Olivelli <
> >>> [hidden email]
> >>> > >
> >>> > >
> >>> > > wrote:
> >>> > >
> >>> > > > Tibor
> >>> > > > It is strage all its are passing on CI.
> >>> > > > Are you sure you have correcly unpacked the package?
> >>> > > >
> >>> > > >
> >>> > > > Il lun 13 mag 2019, 21:23 Tibor Digana <[hidden email]>
> ha
> >>> > > > scritto:
> >>> > > >
> >>> > > >> checked the sha512 of src zip, ok
> >>> > > >> checked the build, failed (mvn verify -P run-its,quality-checks)
> >>> > > >>
> >>> > > >
> >>> > > > What is 'quality-checks' profile? I have never heard about it
> >>> > > >
> >>> > > > Enrico
> >>> > > >
> >>> > > >
> >>> > > >> [INFO] Building: MCHECKSTYLE-54\pom.xml
> >>> > > >> [INFO]           MCHECKSTYLE-54\pom.xml
> >>> ...........................
> >>> > > >> FAILED
> >>> > > >> (5.3 s)
> >>> > > >>
> >>> > > >> *Mcheckstyle54.java:[1] (misc) NewlineAtEndOfFile: File does not
> >>> end
> >>> > > >> with a
> >>> > > >> newline.*
> >>> > > >>
> >>> > > >> but I checkted this file and it ends with a new line. Is it bug
> >>> in the
> >>> > > >> Checkstyle dependency?
> >>> > > >>
> >>> > > >> [INFO] BUILD FAILURE
> >>> > > >> [ERROR] Failed to execute goal
> >>> > > >> org.apache.maven.plugins:maven-pmd-plugin:3.8:cpd-check
> >>> (cpd-check) on
> >>> > > >> project maven-checkstyle-plugin: You have 1 CPD duplication.
> >>> > > >>
> >>> > > >> There are exactly the same methods. That's why CPD fails, see
> >>> > > >> CheckstyleViolationCheckMojo L813
> >>> > > >> AbstractCheckstyleReport        L581
> >>> > > >>
> >>> > > >>
> >>> > > >>
> >>> > > >>
> >>> > > >>
> >>> > > >>
> >>> > > >>
> >>> > > >>
> >>> > > >>
> >>> > > >> On Mon, May 13, 2019 at 11:52 AM Enrico Olivelli <
> >>> [hidden email]
> >>> > >
> >>> > > >> wrote:
> >>> > > >>
> >>> > > >> > Hi,
> >>> > > >> >
> >>> > > >> > We solved 13 issues:
> >>> > > >> >
> >>> > > >> >
> >>> > > >>
> >>> > >
> >>> >
> >>>
> https://issues.apache.org/jira/secure/ReleaseNote.jspa?version=12342397&styleName=Text&projectId=12317223
> >>> > > >> >
> >>> > > >> > Please note that this version now only supports CheckStyle 8+
> >>> and
> >>> > > >> Java 8
> >>> > > >> > (as required by latest Checkstyle)
> >>> > > >> >
> >>> > > >> > There are still a couple of issues left in JIRA:
> >>> > > >> >
> >>> > > >> >
> >>> > > >>
> >>> > >
> >>> >
> >>>
> https://issues.apache.org/jira/issues/?jql=project%20%3D%20XXXXXXXXXX%20AND%20status%20%3D%20Open%20ORDER%20BY%20key%20DESC%2C%20priority%20DESC
> >>> > > >> >
> >>> > > >> > Staging repo:
> >>> > > >> >
> https://repository.apache.org/content/repositories/maven-1503/
> >>> > > >> >
> >>> > > >> >
> >>> > > >>
> >>> > >
> >>> >
> >>>
> https://repository.apache.org/content/repositories/maven-1503/org/apache/maven/plugins/maven-checkstyle-plugin/3.1.0/maven-checkstyle-plugin-3.1.0-source-release.zip
> >>> > > >> >
> >>> > > >> > Source release checksum(s):
> >>> > > >> > maven-checkstyle-plugin-3.1.0-source-release.zip sha512:
> >>> > > >> >
> >>> > > >> >
> >>> > > >>
> >>> > >
> >>> >
> >>>
> eca46edb4d2f6cf2e250169ce5d5e510781b4bba116cc7b5655797cdb109ccdb0da883a1761347c6e2c77fe58395aff17e45c46d85a8a823340ae2b11c92852e
> >>> > > >> >
> >>> > > >> > Staging site:
> >>> > > >> >
> >>> > > >>
> >>> > >
> >>> >
> >>>
> https://maven.apache.org/plugins-archives/maven-checkstyle-plugin-LATEST/
> >>> > > >> >
> >>> > > >> > Guide to testing staged releases:
> >>> > > >> >
> >>> > > >>
> >>> >
> >>>
> https://maven.apache.org/guides/development/guide-testing-releases.html
> >>> > > >> >
> >>> > > >> > Vote open for at least 72 hours.
> >>> > > >> >
> >>> > > >> > [ ] +1
> >>> > > >> > [ ] +0
> >>> > > >> > [ ] -1
> >>> > > >> >
> >>> > > >> >
> >>> > > >> > Enrico Olivelli
> >>> > > >> >
> >>> > >
> >>> > >
> ---------------------------------------------------------------------
> >>> > > To unsubscribe, e-mail: [hidden email]
> >>> > > For additional commands, e-mail: [hidden email]
> >>> > >
> >>> > >
> >>> >
> >>>
> >>
>
Reply | Threaded
Open this post in threaded view
|

Re: Problems with checkstyle tests on Windows (Was [VOTE] Release Apache Maven Checkstyle Plugin version 3.1.0)

Enrico Olivelli
Il mer 15 mag 2019, 17:44 Tibor Digana <[hidden email]> ha scritto:

> @Enrico did you announce the Checkstyle team about our issue [1]?
> What is going on with that?
> [1]: https://issues.apache.org/jira/browse/MCHECKSTYLE-376
> Thx
> Tibor17
>


Not yet sorry. I have been busy today. Will do this evening


Enrico



>
> On Tue, May 14, 2019 at 2:11 PM Enrico Olivelli <[hidden email]>
> wrote:
>
> > I will get a windows box and try to reproduce.
> > It is weird that on ASF Jenkins the build is passing even on windows
> >
> > Enrico
> >
> > ---------- Forwarded message ---------
> > Da: Enrico Olivelli <[hidden email]>
> > Date: mar 14 mag 2019, 13:58
> > Subject: Re: [VOTE] Release Apache Maven Checkstyle Plugin version 3.1.0
> > To: Maven Developers List <[hidden email]>
> >
> >
> > Eric and Tibor,
> > Thank you so much for your effort in testing Maven Checkstyle Plugin.
> >
> > This is the "official" VOTE thread, here we have to decide if the staged
> > artifacts are good to be released or not.
> >
> > Feel free to cast a -1 if you think that the staged artifacts are not
> > "stable" or there is any showstopper problem for the release.
> >
> > Let's move this discussion to a separate thread, something like
> "Validation
> > failures in Windows over current checkstyle plugin master branch")
> >
> > Enrico
> >
> >
> >
> >
> > Il giorno mar 14 mag 2019 alle ore 13:51 Eric Lilja <
> [hidden email]>
> > ha scritto:
> >
> > > Tried overriding line.separator when running using
> -Dline.separator="\n",
> > > but then the builds fails (early) in maven-plugin-plugin:
> > > [ERROR] Failed to execute goal
> > > org.apache.maven.plugins:maven-plugin-plugin:3.5:descriptor
> > > (default-descriptor) on project maven-checkstyle-plugin: Execution
> > > default-descriptor of goal
> > > org.apache.maven.plugins:maven-plugin-plugin:3.5:descriptor failed:
> > > Requested line separator is invalid. -> [Help 1]
> > > [ERROR]
> > > [ERROR] To see the full stack trace of the errors, re-run Maven with
> the
> > -e
> > > switch.
> > > [ERROR] Re-run Maven using the -X switch to enable full debug logging.
> > > [ERROR]
> > > [ERROR] For more information about the errors and possible solutions,
> > > please read the following articles:
> > > [ERROR] [Help 1]
> > >
> >
> http://cwiki.apache.org/confluence/display/MAVEN/PluginExecutionException
> > >
> > > Tried upgrading maven-plugin-plugin to 3.6.0 (lots of stuff
> > > upgradeable...), but same error
> > >
> > > I also happened to notice this (probably unrelated, but wanted to bring
> > it
> > > to attention anyway so it can be fixed) warning:
> > > [INFO] java-annotations mojo extractor found 4 mojo descriptors.
> > > [WARNING]
> > >
> > > Unexpected situation: destinationDirectory not defined in
> > > maven-plugin-help.properties during help mojo source generation but
> > > expected during XML descriptor generation.
> > > [WARNING] Please check helpmojo goal version used in previous build
> > phase.
> > > [WARNING] If you just upgraded to plugin-tools >= 3.2 you must run a
> > clean
> > > build at least once.
> > > [WARNING] Trying default location: target\generated-sources\plugin
> > >
> > > - Eric L
> > >
> > > On Tue, May 14, 2019 at 11:04 AM Eric Lilja <[hidden email]>
> > wrote:
> > >
> > > > I tried bumping checkstyle to 8.20, plus a few of the plexus
> > > dependencies,
> > > > but that just brought an additional failure... (to
> > > > MCHECKSTYLE-70-multi-sourcefolder\pom.xml)  :-)
> > > >
> > > > I suppose the problem might be that the files has linux-style line
> > breaks
> > > > (this is desired for me, I don't want to convert to windows-style
> line
> > > > breaks locally), but the test think I should have windows-style line
> > > > separators. It seems these files are generated by the tests because I
> > > tried
> > > > changing them to Windows style line breaks for re-running just to see
> > if
> > > > that would work, but those changes were overwritten)
> > > >
> > > > - Eric L
> > > >
> > > > On Tue, May 14, 2019 at 10:38 AM Eric Lilja <[hidden email]>
> > > wrote:
> > > >
> > > >> I also see a failure for MCHECKSTYLE-54 on Windows. (Sorry, I didn't
> > try
> > > >> the source zip, just cloned master)
> > > >>
> > > >> I tested on one of our corporate laptops:
> > > >> Windows 10
> > > >> Cygwin 64-bit (I use it to clone the repo and use Maven)
> > > >> Maven 3.6.0
> > > >> Java 8 update 202
> > > >>
> > > >> The build log says:
> > > >> [INFO] There are 2 errors reported by Checkstyle 8.19 with
> > > sun_checks.xml
> > > >> ruleset.
> > > >> [ERROR]
> > > >>
> > >
> >
> src\main\java\org\apache\maven\plugins\checkstyle\mcheckstyle54\Mcheckstyle54.java:[1]
> > > >> (misc) NewlineAtEndOfFile: File does not end with a newline.
> > > >> [ERROR]
> > > >>
> > >
> >
> src\main\java\org\apache\maven\plugins\checkstyle\mcheckstyle54\package-info.java:[1]
> > > >> (misc) NewlineAtEndOfFile: File does not end with a newline.
> > > >>
> > > >> These two files end with unix-style line breaks (as expected with my
> > > >> setup).
> > > >>
> > > >> - Eric L
> > > >>
> > > >>
> > > >> On Tue, May 14, 2019 at 8:10 AM Enrico Olivelli <
> [hidden email]>
> > > >> wrote:
> > > >>
> > > >>> Il lun 13 mag 2019, 23:48 Tibor Digana <[hidden email]> ha
> > > >>> scritto:
> > > >>>
> > > >>> > Robert, I did *not* use the source zip.
> > > >>> >
> > > >>>
> > > >>> IMHO we should vote on the staged zip
> > > >>>
> > > >>>
> > > >>> > git clone
> > > https://gitbox.apache.org/repos/asf/maven-checkstyle-plugin
> > > >>> > Oracle jdk 1.8.0u212, Maven 3.3.9, Windows 10
> > > >>> >
> > > >>>
> > > >>> This is what CI does and tests are passing.
> > > >>>
> > > >>> Do you have some global git configuration?
> > > >>>
> > > >>> Enrico
> > > >>>
> > > >>> >
> > > >>> > I see there is a new line, but the checkstyle does not care if
> you
> > > put
> > > >>> one
> > > >>> > or two lines.
> > > >>> > No idea why.
> > > >>> >
> > > >>> > Even if you go to the target and run it from the folder
> > > >>> > c:\vcs\github\maven-checkstyle-plugin\target\it\MCHECKSTYLE-54\
> > > >>> > it's the same as if you run the project root - mvn verify -P
> > > >>> > run-its,quality-checks
> > > >>> >
> > > >>> >
> > > >>> > *mvn -nsu checkstyle:check*
> > > >>> >
> > > >>> > [INFO] --- maven-checkstyle-plugin:3.1.1-SNAPSHOT:check
> > > (default-cli) @
> > > >>> > mcheckstyle-54 ---
> > > >>> > [INFO] There are 2 errors reported by Checkstyle 8.19 with
> > > >>> sun_checks.xml
> > > >>> > ruleset.
> > > >>> > [ERROR]
> > > >>> >
> > > >>> >
> > > >>>
> > >
> >
> src\main\java\org\apache\maven\plugins\checkstyle\mcheckstyle54\Mcheckstyle54.java:[1]
> > > >>> > (misc) NewlineAtEndOfFile: File does not end with a newline.
> > > >>> > [ERROR]
> > > >>> >
> > > >>> >
> > > >>>
> > >
> >
> src\main\java\org\apache\maven\plugins\checkstyle\mcheckstyle54\package-info.java:[1]
> > > >>> > (misc) NewlineAtEndOfFile: File does not end with a newline.
> > > >>> > [INFO]
> > > >>> >
> > > >>>
> > >
> ------------------------------------------------------------------------
> > > >>> > [INFO] BUILD FAILURE
> > > >>> > [INFO]
> > > >>> >
> > > >>>
> > >
> ------------------------------------------------------------------------
> > > >>> > [INFO] Total time: 3.450 s
> > > >>> > [INFO] Finished at: 2019-05-13T23:42:55+02:00
> > > >>> > [INFO] Final Memory: 12M/193M
> > > >>> > [INFO]
> > > >>> >
> > > >>>
> > >
> ------------------------------------------------------------------------
> > > >>> > [ERROR] Failed to execute goal
> > > >>> >
> > org.apache.maven.plugins:maven-checkstyle-plugin:3.1.1-SNAPSHOT:check
> > > >>> > (default-cli) on project mcheckstyle-54: You have 2 Checkstyle
> > > >>> violations.
> > > >>> > -> [Help 1]
> > > >>> > [ERROR]
> > > >>> > [ERROR] To see the full stack trace of the errors, re-run Maven
> > with
> > > >>> the -e
> > > >>> > switch.
> > > >>> > [ERROR] Re-run Maven using the -X switch to enable full debug
> > > logging.
> > > >>> > [ERROR]
> > > >>> > [ERROR] For more information about the errors and possible
> > solutions,
> > > >>> > please read the following articles:
> > > >>> > [ERROR] [Help 1]
> > > >>> >
> > > http://cwiki.apache.org/confluence/display/MAVEN/MojoFailureException
> > > >>> >
> > > >>> >
> > > >>> > On Mon, May 13, 2019 at 10:29 PM Robert Scholte <
> > > [hidden email]>
> > > >>> > wrote:
> > > >>> >
> > > >>> > > This can happen when source-release.zip was created on a
> > different
> > > OS
> > > >>> > > compared to the verifying OS.
> > > >>> > >
> > > >>> > > With Git it will switch to the line endings of the operating
> > > system,
> > > >>> but
> > > >>> > > with a zip that's not possible.
> > > >>> > >
> > > >>> > > I'd say not blocking, but the IT needs to be fixed to handle
> > these
> > > >>> > > situations.
> > > >>> > >
> > > >>> > > Robert
> > > >>> > >
> > > >>> > >
> > > >>> > > On Mon, 13 May 2019 21:55:27 +0200, Enrico Olivelli <
> > > >>> [hidden email]
> > > >>> > >
> > > >>> > >
> > > >>> > > wrote:
> > > >>> > >
> > > >>> > > > Tibor
> > > >>> > > > It is strage all its are passing on CI.
> > > >>> > > > Are you sure you have correcly unpacked the package?
> > > >>> > > >
> > > >>> > > >
> > > >>> > > > Il lun 13 mag 2019, 21:23 Tibor Digana <
> [hidden email]
> > >
> > > ha
> > > >>> > > > scritto:
> > > >>> > > >
> > > >>> > > >> checked the sha512 of src zip, ok
> > > >>> > > >> checked the build, failed (mvn verify -P
> > run-its,quality-checks)
> > > >>> > > >>
> > > >>> > > >
> > > >>> > > > What is 'quality-checks' profile? I have never heard about it
> > > >>> > > >
> > > >>> > > > Enrico
> > > >>> > > >
> > > >>> > > >
> > > >>> > > >> [INFO] Building: MCHECKSTYLE-54\pom.xml
> > > >>> > > >> [INFO]           MCHECKSTYLE-54\pom.xml
> > > >>> ...........................
> > > >>> > > >> FAILED
> > > >>> > > >> (5.3 s)
> > > >>> > > >>
> > > >>> > > >> *Mcheckstyle54.java:[1] (misc) NewlineAtEndOfFile: File does
> > not
> > > >>> end
> > > >>> > > >> with a
> > > >>> > > >> newline.*
> > > >>> > > >>
> > > >>> > > >> but I checkted this file and it ends with a new line. Is it
> > bug
> > > >>> in the
> > > >>> > > >> Checkstyle dependency?
> > > >>> > > >>
> > > >>> > > >> [INFO] BUILD FAILURE
> > > >>> > > >> [ERROR] Failed to execute goal
> > > >>> > > >> org.apache.maven.plugins:maven-pmd-plugin:3.8:cpd-check
> > > >>> (cpd-check) on
> > > >>> > > >> project maven-checkstyle-plugin: You have 1 CPD duplication.
> > > >>> > > >>
> > > >>> > > >> There are exactly the same methods. That's why CPD fails,
> see
> > > >>> > > >> CheckstyleViolationCheckMojo L813
> > > >>> > > >> AbstractCheckstyleReport        L581
> > > >>> > > >>
> > > >>> > > >>
> > > >>> > > >>
> > > >>> > > >>
> > > >>> > > >>
> > > >>> > > >>
> > > >>> > > >>
> > > >>> > > >>
> > > >>> > > >>
> > > >>> > > >> On Mon, May 13, 2019 at 11:52 AM Enrico Olivelli <
> > > >>> [hidden email]
> > > >>> > >
> > > >>> > > >> wrote:
> > > >>> > > >>
> > > >>> > > >> > Hi,
> > > >>> > > >> >
> > > >>> > > >> > We solved 13 issues:
> > > >>> > > >> >
> > > >>> > > >> >
> > > >>> > > >>
> > > >>> > >
> > > >>> >
> > > >>>
> > >
> >
> https://issues.apache.org/jira/secure/ReleaseNote.jspa?version=12342397&styleName=Text&projectId=12317223
> > > >>> > > >> >
> > > >>> > > >> > Please note that this version now only supports CheckStyle
> > 8+
> > > >>> and
> > > >>> > > >> Java 8
> > > >>> > > >> > (as required by latest Checkstyle)
> > > >>> > > >> >
> > > >>> > > >> > There are still a couple of issues left in JIRA:
> > > >>> > > >> >
> > > >>> > > >> >
> > > >>> > > >>
> > > >>> > >
> > > >>> >
> > > >>>
> > >
> >
> https://issues.apache.org/jira/issues/?jql=project%20%3D%20XXXXXXXXXX%20AND%20status%20%3D%20Open%20ORDER%20BY%20key%20DESC%2C%20priority%20DESC
> > > >>> > > >> >
> > > >>> > > >> > Staging repo:
> > > >>> > > >> >
> > > https://repository.apache.org/content/repositories/maven-1503/
> > > >>> > > >> >
> > > >>> > > >> >
> > > >>> > > >>
> > > >>> > >
> > > >>> >
> > > >>>
> > >
> >
> https://repository.apache.org/content/repositories/maven-1503/org/apache/maven/plugins/maven-checkstyle-plugin/3.1.0/maven-checkstyle-plugin-3.1.0-source-release.zip
> > > >>> > > >> >
> > > >>> > > >> > Source release checksum(s):
> > > >>> > > >> > maven-checkstyle-plugin-3.1.0-source-release.zip sha512:
> > > >>> > > >> >
> > > >>> > > >> >
> > > >>> > > >>
> > > >>> > >
> > > >>> >
> > > >>>
> > >
> >
> eca46edb4d2f6cf2e250169ce5d5e510781b4bba116cc7b5655797cdb109ccdb0da883a1761347c6e2c77fe58395aff17e45c46d85a8a823340ae2b11c92852e
> > > >>> > > >> >
> > > >>> > > >> > Staging site:
> > > >>> > > >> >
> > > >>> > > >>
> > > >>> > >
> > > >>> >
> > > >>>
> > >
> >
> https://maven.apache.org/plugins-archives/maven-checkstyle-plugin-LATEST/
> > > >>> > > >> >
> > > >>> > > >> > Guide to testing staged releases:
> > > >>> > > >> >
> > > >>> > > >>
> > > >>> >
> > > >>>
> > >
> https://maven.apache.org/guides/development/guide-testing-releases.html
> > > >>> > > >> >
> > > >>> > > >> > Vote open for at least 72 hours.
> > > >>> > > >> >
> > > >>> > > >> > [ ] +1
> > > >>> > > >> > [ ] +0
> > > >>> > > >> > [ ] -1
> > > >>> > > >> >
> > > >>> > > >> >
> > > >>> > > >> > Enrico Olivelli
> > > >>> > > >> >
> > > >>> > >
> > > >>> > >
> > > ---------------------------------------------------------------------
> > > >>> > > To unsubscribe, e-mail: [hidden email]
> > > >>> > > For additional commands, e-mail: [hidden email]
> > > >>> > >
> > > >>> > >
> > > >>> >
> > > >>>
> > > >>
> > >
> >
>
Reply | Threaded
Open this post in threaded view
|

Re: Problems with checkstyle tests on Windows (Was [VOTE] Release Apache Maven Checkstyle Plugin version 3.1.0)

Tibor Digana
In reply to this post by Enrico Olivelli
Enrico, I checked the Checkstyle rules again (default value = system) and
Robert is right, please adjust the IT sources on the fly.
Not sure why INFRA or Jenkins sets the Windows EOL to LF.
Therefore we could not find this issue in version 3.0.0 (Jan 04) because
the IT was created on Jan 17 and we run it on local Windows the first time.

The library is fine. I used several old versions of the plugin having
identical results.
Few links:

http://checkstyle.sourceforge.net/config_misc.html#NewlineAtEndOfFile
<module name="NewlineAtEndOfFile"/>
http://checkstyle.sourceforge.net/google_style.html
https://github.com/checkstyle/checkstyle/blob/master/src/main/resources/sun_checks.xml
https://github.com/checkstyle/checkstyle/blob/master/src/main/resources/google_checks.xml

I expected platform independence because my company used sun_checks.xml on
Windows with Unix EOL in Git/IDE without this problem.
The company improved sun_checks.xml long time ago, so I realized that later
that it is slightly different XML and different experience. Sorry.
Cheers
Tibor17


On Tue, May 14, 2019 at 2:11 PM Enrico Olivelli <[hidden email]> wrote:

> I will get a windows box and try to reproduce.
> It is weird that on ASF Jenkins the build is passing even on windows
>
> Enrico
>
> ---------- Forwarded message ---------
> Da: Enrico Olivelli <[hidden email]>
> Date: mar 14 mag 2019, 13:58
> Subject: Re: [VOTE] Release Apache Maven Checkstyle Plugin version 3.1.0
> To: Maven Developers List <[hidden email]>
>
>
> Eric and Tibor,
> Thank you so much for your effort in testing Maven Checkstyle Plugin.
>
> This is the "official" VOTE thread, here we have to decide if the staged
> artifacts are good to be released or not.
>
> Feel free to cast a -1 if you think that the staged artifacts are not
> "stable" or there is any showstopper problem for the release.
>
> Let's move this discussion to a separate thread, something like "Validation
> failures in Windows over current checkstyle plugin master branch")
>
> Enrico
>
>
>
>
> Il giorno mar 14 mag 2019 alle ore 13:51 Eric Lilja <[hidden email]>
> ha scritto:
>
> > Tried overriding line.separator when running using -Dline.separator="\n",
> > but then the builds fails (early) in maven-plugin-plugin:
> > [ERROR] Failed to execute goal
> > org.apache.maven.plugins:maven-plugin-plugin:3.5:descriptor
> > (default-descriptor) on project maven-checkstyle-plugin: Execution
> > default-descriptor of goal
> > org.apache.maven.plugins:maven-plugin-plugin:3.5:descriptor failed:
> > Requested line separator is invalid. -> [Help 1]
> > [ERROR]
> > [ERROR] To see the full stack trace of the errors, re-run Maven with the
> -e
> > switch.
> > [ERROR] Re-run Maven using the -X switch to enable full debug logging.
> > [ERROR]
> > [ERROR] For more information about the errors and possible solutions,
> > please read the following articles:
> > [ERROR] [Help 1]
> >
> http://cwiki.apache.org/confluence/display/MAVEN/PluginExecutionException
> >
> > Tried upgrading maven-plugin-plugin to 3.6.0 (lots of stuff
> > upgradeable...), but same error
> >
> > I also happened to notice this (probably unrelated, but wanted to bring
> it
> > to attention anyway so it can be fixed) warning:
> > [INFO] java-annotations mojo extractor found 4 mojo descriptors.
> > [WARNING]
> >
> > Unexpected situation: destinationDirectory not defined in
> > maven-plugin-help.properties during help mojo source generation but
> > expected during XML descriptor generation.
> > [WARNING] Please check helpmojo goal version used in previous build
> phase.
> > [WARNING] If you just upgraded to plugin-tools >= 3.2 you must run a
> clean
> > build at least once.
> > [WARNING] Trying default location: target\generated-sources\plugin
> >
> > - Eric L
> >
> > On Tue, May 14, 2019 at 11:04 AM Eric Lilja <[hidden email]>
> wrote:
> >
> > > I tried bumping checkstyle to 8.20, plus a few of the plexus
> > dependencies,
> > > but that just brought an additional failure... (to
> > > MCHECKSTYLE-70-multi-sourcefolder\pom.xml)  :-)
> > >
> > > I suppose the problem might be that the files has linux-style line
> breaks
> > > (this is desired for me, I don't want to convert to windows-style line
> > > breaks locally), but the test think I should have windows-style line
> > > separators. It seems these files are generated by the tests because I
> > tried
> > > changing them to Windows style line breaks for re-running just to see
> if
> > > that would work, but those changes were overwritten)
> > >
> > > - Eric L
> > >
> > > On Tue, May 14, 2019 at 10:38 AM Eric Lilja <[hidden email]>
> > wrote:
> > >
> > >> I also see a failure for MCHECKSTYLE-54 on Windows. (Sorry, I didn't
> try
> > >> the source zip, just cloned master)
> > >>
> > >> I tested on one of our corporate laptops:
> > >> Windows 10
> > >> Cygwin 64-bit (I use it to clone the repo and use Maven)
> > >> Maven 3.6.0
> > >> Java 8 update 202
> > >>
> > >> The build log says:
> > >> [INFO] There are 2 errors reported by Checkstyle 8.19 with
> > sun_checks.xml
> > >> ruleset.
> > >> [ERROR]
> > >>
> >
> src\main\java\org\apache\maven\plugins\checkstyle\mcheckstyle54\Mcheckstyle54.java:[1]
> > >> (misc) NewlineAtEndOfFile: File does not end with a newline.
> > >> [ERROR]
> > >>
> >
> src\main\java\org\apache\maven\plugins\checkstyle\mcheckstyle54\package-info.java:[1]
> > >> (misc) NewlineAtEndOfFile: File does not end with a newline.
> > >>
> > >> These two files end with unix-style line breaks (as expected with my
> > >> setup).
> > >>
> > >> - Eric L
> > >>
> > >>
> > >> On Tue, May 14, 2019 at 8:10 AM Enrico Olivelli <[hidden email]>
> > >> wrote:
> > >>
> > >>> Il lun 13 mag 2019, 23:48 Tibor Digana <[hidden email]> ha
> > >>> scritto:
> > >>>
> > >>> > Robert, I did *not* use the source zip.
> > >>> >
> > >>>
> > >>> IMHO we should vote on the staged zip
> > >>>
> > >>>
> > >>> > git clone
> > https://gitbox.apache.org/repos/asf/maven-checkstyle-plugin
> > >>> > Oracle jdk 1.8.0u212, Maven 3.3.9, Windows 10
> > >>> >
> > >>>
> > >>> This is what CI does and tests are passing.
> > >>>
> > >>> Do you have some global git configuration?
> > >>>
> > >>> Enrico
> > >>>
> > >>> >
> > >>> > I see there is a new line, but the checkstyle does not care if you
> > put
> > >>> one
> > >>> > or two lines.
> > >>> > No idea why.
> > >>> >
> > >>> > Even if you go to the target and run it from the folder
> > >>> > c:\vcs\github\maven-checkstyle-plugin\target\it\MCHECKSTYLE-54\
> > >>> > it's the same as if you run the project root - mvn verify -P
> > >>> > run-its,quality-checks
> > >>> >
> > >>> >
> > >>> > *mvn -nsu checkstyle:check*
> > >>> >
> > >>> > [INFO] --- maven-checkstyle-plugin:3.1.1-SNAPSHOT:check
> > (default-cli) @
> > >>> > mcheckstyle-54 ---
> > >>> > [INFO] There are 2 errors reported by Checkstyle 8.19 with
> > >>> sun_checks.xml
> > >>> > ruleset.
> > >>> > [ERROR]
> > >>> >
> > >>> >
> > >>>
> >
> src\main\java\org\apache\maven\plugins\checkstyle\mcheckstyle54\Mcheckstyle54.java:[1]
> > >>> > (misc) NewlineAtEndOfFile: File does not end with a newline.
> > >>> > [ERROR]
> > >>> >
> > >>> >
> > >>>
> >
> src\main\java\org\apache\maven\plugins\checkstyle\mcheckstyle54\package-info.java:[1]
> > >>> > (misc) NewlineAtEndOfFile: File does not end with a newline.
> > >>> > [INFO]
> > >>> >
> > >>>
> > ------------------------------------------------------------------------
> > >>> > [INFO] BUILD FAILURE
> > >>> > [INFO]
> > >>> >
> > >>>
> > ------------------------------------------------------------------------
> > >>> > [INFO] Total time: 3.450 s
> > >>> > [INFO] Finished at: 2019-05-13T23:42:55+02:00
> > >>> > [INFO] Final Memory: 12M/193M
> > >>> > [INFO]
> > >>> >
> > >>>
> > ------------------------------------------------------------------------
> > >>> > [ERROR] Failed to execute goal
> > >>> >
> org.apache.maven.plugins:maven-checkstyle-plugin:3.1.1-SNAPSHOT:check
> > >>> > (default-cli) on project mcheckstyle-54: You have 2 Checkstyle
> > >>> violations.
> > >>> > -> [Help 1]
> > >>> > [ERROR]
> > >>> > [ERROR] To see the full stack trace of the errors, re-run Maven
> with
> > >>> the -e
> > >>> > switch.
> > >>> > [ERROR] Re-run Maven using the -X switch to enable full debug
> > logging.
> > >>> > [ERROR]
> > >>> > [ERROR] For more information about the errors and possible
> solutions,
> > >>> > please read the following articles:
> > >>> > [ERROR] [Help 1]
> > >>> >
> > http://cwiki.apache.org/confluence/display/MAVEN/MojoFailureException
> > >>> >
> > >>> >
> > >>> > On Mon, May 13, 2019 at 10:29 PM Robert Scholte <
> > [hidden email]>
> > >>> > wrote:
> > >>> >
> > >>> > > This can happen when source-release.zip was created on a
> different
> > OS
> > >>> > > compared to the verifying OS.
> > >>> > >
> > >>> > > With Git it will switch to the line endings of the operating
> > system,
> > >>> but
> > >>> > > with a zip that's not possible.
> > >>> > >
> > >>> > > I'd say not blocking, but the IT needs to be fixed to handle
> these
> > >>> > > situations.
> > >>> > >
> > >>> > > Robert
> > >>> > >
> > >>> > >
> > >>> > > On Mon, 13 May 2019 21:55:27 +0200, Enrico Olivelli <
> > >>> [hidden email]
> > >>> > >
> > >>> > >
> > >>> > > wrote:
> > >>> > >
> > >>> > > > Tibor
> > >>> > > > It is strage all its are passing on CI.
> > >>> > > > Are you sure you have correcly unpacked the package?
> > >>> > > >
> > >>> > > >
> > >>> > > > Il lun 13 mag 2019, 21:23 Tibor Digana <[hidden email]
> >
> > ha
> > >>> > > > scritto:
> > >>> > > >
> > >>> > > >> checked the sha512 of src zip, ok
> > >>> > > >> checked the build, failed (mvn verify -P
> run-its,quality-checks)
> > >>> > > >>
> > >>> > > >
> > >>> > > > What is 'quality-checks' profile? I have never heard about it
> > >>> > > >
> > >>> > > > Enrico
> > >>> > > >
> > >>> > > >
> > >>> > > >> [INFO] Building: MCHECKSTYLE-54\pom.xml
> > >>> > > >> [INFO]           MCHECKSTYLE-54\pom.xml
> > >>> ...........................
> > >>> > > >> FAILED
> > >>> > > >> (5.3 s)
> > >>> > > >>
> > >>> > > >> *Mcheckstyle54.java:[1] (misc) NewlineAtEndOfFile: File does
> not
> > >>> end
> > >>> > > >> with a
> > >>> > > >> newline.*
> > >>> > > >>
> > >>> > > >> but I checkted this file and it ends with a new line. Is it
> bug
> > >>> in the
> > >>> > > >> Checkstyle dependency?
> > >>> > > >>
> > >>> > > >> [INFO] BUILD FAILURE
> > >>> > > >> [ERROR] Failed to execute goal
> > >>> > > >> org.apache.maven.plugins:maven-pmd-plugin:3.8:cpd-check
> > >>> (cpd-check) on
> > >>> > > >> project maven-checkstyle-plugin: You have 1 CPD duplication.
> > >>> > > >>
> > >>> > > >> There are exactly the same methods. That's why CPD fails, see
> > >>> > > >> CheckstyleViolationCheckMojo L813
> > >>> > > >> AbstractCheckstyleReport        L581
> > >>> > > >>
> > >>> > > >>
> > >>> > > >>
> > >>> > > >>
> > >>> > > >>
> > >>> > > >>
> > >>> > > >>
> > >>> > > >>
> > >>> > > >>
> > >>> > > >> On Mon, May 13, 2019 at 11:52 AM Enrico Olivelli <
> > >>> [hidden email]
> > >>> > >
> > >>> > > >> wrote:
> > >>> > > >>
> > >>> > > >> > Hi,
> > >>> > > >> >
> > >>> > > >> > We solved 13 issues:
> > >>> > > >> >
> > >>> > > >> >
> > >>> > > >>
> > >>> > >
> > >>> >
> > >>>
> >
> https://issues.apache.org/jira/secure/ReleaseNote.jspa?version=12342397&styleName=Text&projectId=12317223
> > >>> > > >> >
> > >>> > > >> > Please note that this version now only supports CheckStyle
> 8+
> > >>> and
> > >>> > > >> Java 8
> > >>> > > >> > (as required by latest Checkstyle)
> > >>> > > >> >
> > >>> > > >> > There are still a couple of issues left in JIRA:
> > >>> > > >> >
> > >>> > > >> >
> > >>> > > >>
> > >>> > >
> > >>> >
> > >>>
> >
> https://issues.apache.org/jira/issues/?jql=project%20%3D%20XXXXXXXXXX%20AND%20status%20%3D%20Open%20ORDER%20BY%20key%20DESC%2C%20priority%20DESC
> > >>> > > >> >
> > >>> > > >> > Staging repo:
> > >>> > > >> >
> > https://repository.apache.org/content/repositories/maven-1503/
> > >>> > > >> >
> > >>> > > >> >
> > >>> > > >>
> > >>> > >
> > >>> >
> > >>>
> >
> https://repository.apache.org/content/repositories/maven-1503/org/apache/maven/plugins/maven-checkstyle-plugin/3.1.0/maven-checkstyle-plugin-3.1.0-source-release.zip
> > >>> > > >> >
> > >>> > > >> > Source release checksum(s):
> > >>> > > >> > maven-checkstyle-plugin-3.1.0-source-release.zip sha512:
> > >>> > > >> >
> > >>> > > >> >
> > >>> > > >>
> > >>> > >
> > >>> >
> > >>>
> >
> eca46edb4d2f6cf2e250169ce5d5e510781b4bba116cc7b5655797cdb109ccdb0da883a1761347c6e2c77fe58395aff17e45c46d85a8a823340ae2b11c92852e
> > >>> > > >> >
> > >>> > > >> > Staging site:
> > >>> > > >> >
> > >>> > > >>
> > >>> > >
> > >>> >
> > >>>
> >
> https://maven.apache.org/plugins-archives/maven-checkstyle-plugin-LATEST/
> > >>> > > >> >
> > >>> > > >> > Guide to testing staged releases:
> > >>> > > >> >
> > >>> > > >>
> > >>> >
> > >>>
> > https://maven.apache.org/guides/development/guide-testing-releases.html
> > >>> > > >> >
> > >>> > > >> > Vote open for at least 72 hours.
> > >>> > > >> >
> > >>> > > >> > [ ] +1
> > >>> > > >> > [ ] +0
> > >>> > > >> > [ ] -1
> > >>> > > >> >
> > >>> > > >> >
> > >>> > > >> > Enrico Olivelli
> > >>> > > >> >
> > >>> > >
> > >>> > >
> > ---------------------------------------------------------------------
> > >>> > > To unsubscribe, e-mail: [hidden email]
> > >>> > > For additional commands, e-mail: [hidden email]
> > >>> > >
> > >>> > >
> > >>> >
> > >>>
> > >>
> >
>
Reply | Threaded
Open this post in threaded view
|

[RESULT] [VOTE] Release Apache Maven Checkstyle Plugin version 3.1.0

Enrico Olivelli
In reply to this post by Enrico Olivelli
- Hi,
-
- The vote has passed with the following result:
-
- +1 :  Tibor Digana, Sylwester Lachiewicz, Hervé Boutemy, Karl Heinz Marbaise
-
- PMC quorum: reached
-
- I will promote the artifacts to the central repo.


Enrico

Il giorno dom 19 mag 2019 alle ore 11:44 Sylwester Lachiewicz <
[hidden email]> ha scritto:

> +1
> Sylwester
>
> W dniu niedz., 19.05.2019 o 11:09 Hervé BOUTEMY <[hidden email]>
> napisał(a):
>
> > +1
> >
> > Regards,
> >
> > Hervé
> >
> > Le lundi 13 mai 2019, 11:52:30 CEST Enrico Olivelli a écrit :
> > > Hi,
> > >
> > > We solved 13 issues:
> > >
> >
> https://issues.apache.org/jira/secure/ReleaseNote.jspa?version=12342397&styl
> > > eName=Text&projectId=12317223
> > >
> > > Please note that this version now only supports CheckStyle 8+ and Java
> 8
> > > (as required by latest Checkstyle)
> > >
> > > There are still a couple of issues left in JIRA:
> > >
> >
> <a href="https://issues.apache.org/jira/issues/?jql=project%20%3D%20XXXXXXXXXX%20AND%">https://issues.apache.org/jira/issues/?jql=project%20%3D%20XXXXXXXXXX%20AND%
> > > 20status%20%3D%20Open%20ORDER%20BY%20key%20DESC%2C%20priority%20DESC
> > >
> > > Staging repo:
> > > https://repository.apache.org/content/repositories/maven-1503/
> > >
> >
> https://repository.apache.org/content/repositories/maven-1503/org/apache/mav
> > >
> >
> en/plugins/maven-checkstyle-plugin/3.1.0/maven-checkstyle-plugin-3.1.0-sourc
> > > e-release.zip
> > >
> > > Source release checksum(s):
> > > maven-checkstyle-plugin-3.1.0-source-release.zip sha512:
> > >
> >
> eca46edb4d2f6cf2e250169ce5d5e510781b4bba116cc7b5655797cdb109ccdb0da883a17613
> > > 47c6e2c77fe58395aff17e45c46d85a8a823340ae2b11c92852e
> > >
> > > Staging site:
> > >
> >
> https://maven.apache.org/plugins-archives/maven-checkstyle-plugin-LATEST/
> > >
> > > Guide to testing staged releases:
> > >
> https://maven.apache.org/guides/development/guide-testing-releases.html
> > >
> > > Vote open for at least 72 hours.
> > >
> > > [ ] +1
> > > [ ] +0
> > > [ ] -1
> > >
> > >
> > > Enrico Olivelli
> >
> >
> >
> >
> >
> > ---------------------------------------------------------------------
> > To unsubscribe, e-mail: [hidden email]
> > For additional commands, e-mail: [hidden email]
> >
> >
>