mvn qulice:check -Pqulice fails to read license #400

asaen · 2015-02-09T22:38:00Z

Running locally under windows 7 mvn qulice:check -Pqulice -e on a project with LICENSE.txt inside the project folder leads to the following error:

[INFO] --- qulice-maven-plugin:0.12:check (default-cli) @ aProject ---
[INFO] ------------------------------------------------------------------------
[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 6.774 s
[INFO] Finished at: 2015-02-10T01:30:57+03:00
[INFO] Final Memory: 15M/308M
[INFO] ------------------------------------------------------------------------
[ERROR] Failed to execute goal com.qulice:qulice-maven-plugin:0.12:check (default-cli) on project aProject: Execution default-cli of goal com.qulice:qulice-maven-plugin:0.12:check failed: Failed to read license: Unknown host D -> [Help 1]
org.apache.maven.lifecycle.LifecycleExecutionException: Failed to execute goal com.qulice:qulice-maven-plugin:0.12:check (default-cli) on project aProject: Execution default-cli of goal com.qulice:qulice-maven-plugin:0.12:check failed: Failed to read license
        at org.apache.maven.lifecycle.internal.MojoExecutor.execute(MojoExecutor.java:224)
        at org.apache.maven.lifecycle.internal.MojoExecutor.execute(MojoExecutor.java:153)
        at org.apache.maven.lifecycle.internal.MojoExecutor.execute(MojoExecutor.java:145)
        at org.apache.maven.lifecycle.internal.LifecycleModuleBuilder.buildProject(LifecycleModuleBuilder.java:116)
        at org.apache.maven.lifecycle.internal.LifecycleModuleBuilder.buildProject(LifecycleModuleBuilder.java:80)
        at org.apache.maven.lifecycle.internal.builder.singlethreaded.SingleThreadedBuilder.build(SingleThreadedBuilder.java:51)
        at org.apache.maven.lifecycle.internal.LifecycleStarter.execute(LifecycleStarter.java:120)
        at org.apache.maven.DefaultMaven.doExecute(DefaultMaven.java:355)
        at org.apache.maven.DefaultMaven.execute(DefaultMaven.java:155)
        at org.apache.maven.cli.MavenCli.execute(MavenCli.java:584)
        at org.apache.maven.cli.MavenCli.doMain(MavenCli.java:216)
        at org.apache.maven.cli.MavenCli.main(MavenCli.java:160)
        at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
        at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:57)
        at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
        at java.lang.reflect.Method.invoke(Method.java:606)
        at org.codehaus.plexus.classworlds.launcher.Launcher.launchEnhanced(Launcher.java:289)
        at org.codehaus.plexus.classworlds.launcher.Launcher.launch(Launcher.java:229)
        at org.codehaus.plexus.classworlds.launcher.Launcher.mainWithExitCode(Launcher.java:415)
        at org.codehaus.plexus.classworlds.launcher.Launcher.main(Launcher.java:356)
Caused by: org.apache.maven.plugin.PluginExecutionException: Execution default-cli of goal com.qulice:qulice-maven-plugin:0.12:check failed: Failed to read license
        at org.apache.maven.plugin.DefaultBuildPluginManager.executeMojo(DefaultBuildPluginManager.java:143)
        at org.apache.maven.lifecycle.internal.MojoExecutor.execute(MojoExecutor.java:208)
        ... 19 more
Caused by: java.lang.IllegalStateException: Failed to read license
        at com.qulice.checkstyle.CheckstyleValidator.header(CheckstyleValidator.java:151)
        at com.qulice.checkstyle.CheckstyleValidator.configuration(CheckstyleValidator.java:120)
        at com.qulice.checkstyle.CheckstyleValidator.validate(CheckstyleValidator.java:86)
        at com.qulice.maven.CheckMojo.run(CheckMojo.java:83)
        at com.qulice.maven.CheckMojo.doExecute(CheckMojo.java:59)
        at com.qulice.maven.AbstractQuliceMojo.execute(AbstractQuliceMojo.java:175)
        at org.apache.maven.plugin.DefaultBuildPluginManager.executeMojo(DefaultBuildPluginManager.java:132)
        ... 20 more
Caused by: java.net.UnknownHostException: D
        at java.net.AbstractPlainSocketImpl.connect(AbstractPlainSocketImpl.java:178)
        at java.net.PlainSocketImpl.connect(PlainSocketImpl.java:172)
        at java.net.Socket.connect(Socket.java:579)
        at java.net.Socket.connect(Socket.java:528)
        at sun.net.ftp.impl.FtpClient.doConnect(FtpClient.java:958)
        at sun.net.ftp.impl.FtpClient.tryConnect(FtpClient.java:918)
        at sun.net.ftp.impl.FtpClient.connect(FtpClient.java:1013)
        at sun.net.ftp.impl.FtpClient.connect(FtpClient.java:999)
        at sun.net.www.protocol.ftp.FtpURLConnection.connect(FtpURLConnection.java:294)
        at sun.net.www.protocol.ftp.FtpURLConnection.getInputStream(FtpURLConnection.java:393)
        at java.net.URL.openStream(URL.java:1037)
        at com.qulice.checkstyle.CheckstyleValidator.header(CheckstyleValidator.java:148)
        ... 26 more
[ERROR]
[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

It seems that the method com.qulice.checkstyle.CheckstyleValidator.header(Environment) builds a wrong URL.

The text was updated successfully, but these errors were encountered:

yegor256 · 2015-02-09T22:42:21Z

isn't it a duplicate of #397?

asaen · 2015-02-09T22:52:19Z

@yegor256 they are probably different sides of the same coin. Let's wait till #397 is resolved and check this one again.

asaen · 2015-08-26T13:39:56Z

@yegor256 it is still reproducible with qulice version 0.12.1 even when #397 is closed. I would say they are not duplicates, aren't they?

yegor256 · 2015-08-26T14:51:45Z

@darkled ok, let's try to fix this separately

davvd · 2015-08-27T07:42:54Z

@darkled this task will get someone's attention soon

davvd · 2015-08-27T07:45:16Z

@krzyk this task is yours, go ahead

davvd · 2015-08-27T08:57:20Z

@darkled thanks for tis bug, I topped your account for 15 mins, transaction 63938954

krzyk · 2015-08-28T17:51:13Z

@darkled could you share pom.xml and the full path name you use for the project?
I tried it in few projects that use qulice 0.12, and all succeed under windows.

asaen · 2015-08-30T08:41:03Z

@krzyk I think they succeed because your pom.xml's has configured license:

<license>file:${basedir}/LICENSE.txt</license>

In my case qulice has the default configuration for the license and its configuration looks like:

<profile>
    <id>qulice</id>
    <build>
        <pluginManagement>
            <plugins>
                <plugin>
                    <groupId>com.jcabi</groupId>
                    <artifactId>jcabi-maven-plugin</artifactId>
                    <dependencies>
                        <dependency>
                            <groupId>org.hibernate</groupId>
                            <artifactId>hibernate-validator</artifactId>
                            <version>${hibernate.validator}</version>
                            <scope>runtime</scope>
                        </dependency>
                    </dependencies>
                </plugin>
                <plugin>
                    <groupId>com.qulice</groupId>
                    <artifactId>qulice-maven-plugin</artifactId>
                    <version>0.12.1</version>
                    <configuration>
                        <license>file://${basedir}/LICENSE.txt</license>
                        <excludes combine.children="append">
                            <exclude>findbugs:.*</exclude>
                            <exclude>checkstyle:/src/main/resources/com/sigzig/.*</exclude>
                        </excludes>
                    </configuration>
                    <dependencies>
                        <dependency>
                            <groupId>org.hibernate</groupId>
                            <artifactId>hibernate-validator</artifactId>
                            <version>${hibernate.validator}</version>
                            <scope>runtime</scope>
                        </dependency>
                    </dependencies>
                </plugin>
            </plugins>
        </pluginManagement>
    </build>
</profile>

krzyk · 2015-08-30T11:19:35Z

@darkled In your example profile, license is defined as <license>file://${basedir}/LICENSE.txt</license>, does it work when you remove the //?

krzyk · 2015-08-30T11:51:22Z

@darkled also, why override license configuration, and add // after file:? Is this just a mistake, or is it done on purpose?

krzyk · 2015-09-01T09:36:17Z

@davvd please give me more time

davvd · 2015-09-01T11:12:46Z

@davvd please give me more time

@krzyk agreed, take your time

asaen · 2015-09-01T11:19:53Z

@krzyk you're right, when if I remove // it works fine. I am not sure about the reason of overriding default configuration, this pom.xml is taken from another teamed's project. Could you please put some short note about the correct way of configuration of license as well as it's default value into readme.md ?

krzyk · 2015-09-05T17:28:55Z

@darkled actually there is already info about correct configuration on www.qulice.com, let me know if you think we should add more info there

krzyk · 2015-09-05T17:29:51Z

@darkled also the same configuration is in the README.md

asaen · 2015-09-05T17:38:11Z

@krzyk it looks very confusing, I think we need to say it explicitly in one or two sentences.

asaen · 2015-09-05T17:41:39Z

@krzyk e.g. The path to license has to be declared in the following format: "file:${basedir}/LICENSE.txt", it's the default value, one can use any full path instead of ${basedir}.

krzyk · 2015-09-05T18:23:24Z

@davvd please see PR #458

davvd · 2015-09-07T07:19:22Z

@davvd please see PR #458

@krzyk thank you, I will check it

krzyk · 2015-09-07T07:40:40Z

@darkled I've added explanation to the README, please close this issue

asaen · 2015-09-07T07:57:12Z

@krzyk thanks!

davvd · 2015-09-08T10:34:56Z

@krzyk 30 mins was added to your account, many thanks for your contribution (payment 64780896)! The task took 265 hours and 50 mins.. added +30 to your rating, now it is equal to +3944

yegor256 added bug @krzyk labels Aug 26, 2015

davvd added the DEV label Aug 27, 2015

davvd assigned krzyk Aug 27, 2015

davvd added the 30 mins label Aug 27, 2015

krzyk pushed a commit to krzyk/qulice that referenced this issue Sep 5, 2015

yegor256#400 added explanation for license

d9dd0ec

krzyk mentioned this issue Sep 5, 2015

#400 added explanation for license #458

Merged

asaen closed this as completed Sep 7, 2015

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

mvn qulice:check -Pqulice fails to read license #400

mvn qulice:check -Pqulice fails to read license #400

asaen commented Feb 9, 2015

yegor256 commented Feb 9, 2015

asaen commented Feb 9, 2015

asaen commented Aug 26, 2015

yegor256 commented Aug 26, 2015

davvd commented Aug 27, 2015

davvd commented Aug 27, 2015

davvd commented Aug 27, 2015

krzyk commented Aug 28, 2015

asaen commented Aug 30, 2015

krzyk commented Aug 30, 2015

krzyk commented Aug 30, 2015

krzyk commented Sep 1, 2015

davvd commented Sep 1, 2015

asaen commented Sep 1, 2015

krzyk commented Sep 5, 2015

krzyk commented Sep 5, 2015

asaen commented Sep 5, 2015

asaen commented Sep 5, 2015

krzyk commented Sep 5, 2015

davvd commented Sep 7, 2015

krzyk commented Sep 7, 2015

asaen commented Sep 7, 2015

davvd commented Sep 8, 2015

mvn qulice:check -Pqulice fails to read license #400

mvn qulice:check -Pqulice fails to read license #400

Comments

asaen commented Feb 9, 2015

yegor256 commented Feb 9, 2015

asaen commented Feb 9, 2015

asaen commented Aug 26, 2015

yegor256 commented Aug 26, 2015

davvd commented Aug 27, 2015

davvd commented Aug 27, 2015

davvd commented Aug 27, 2015

krzyk commented Aug 28, 2015

asaen commented Aug 30, 2015

krzyk commented Aug 30, 2015

krzyk commented Aug 30, 2015

krzyk commented Sep 1, 2015

davvd commented Sep 1, 2015

asaen commented Sep 1, 2015

krzyk commented Sep 5, 2015

krzyk commented Sep 5, 2015

asaen commented Sep 5, 2015

asaen commented Sep 5, 2015

krzyk commented Sep 5, 2015

davvd commented Sep 7, 2015

krzyk commented Sep 7, 2015

asaen commented Sep 7, 2015

davvd commented Sep 8, 2015