Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Log4j warning in execution logs #509

Open
mkordas opened this Issue Dec 10, 2015 · 6 comments

Comments

Projects
None yet
3 participants
@mkordas
Copy link
Contributor

mkordas commented Dec 10, 2015

Qulice execution logs contain log4j warnings:

[INFO] --- qulice-maven-plugin:0.14:check (default) @ template-project ---
[INFO] Starting Checkstyle validator
[INFO] LICENSE found: file:config/header.txt
log4j:WARN No appenders could be found for logger (com.puppycrawl.tools.checkstyle.ConfigurationLoader).
log4j:WARN Please initialize the log4j system properly.
log4j:WARN See http://logging.apache.org/log4j/1.2/faq.html#noconfig for more info.

Logs should not be polluted with such warnings.

To reproduce the issue you can use simplest possible Maven project with Qulice:

<?xml version="1.0" encoding="UTF-8" standalone="no"?>
<project xmlns="http://maven.apache.org/POM/4.0.0" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
         xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/maven-v4_0_0.xsd">
    <modelVersion>4.0.0</modelVersion>

    <groupId>any</groupId>
    <artifactId>any</artifactId>
    <version>any</version>
    <build>
        <plugins>
            <plugin>
                <groupId>com.qulice</groupId>
                <artifactId>qulice-maven-plugin</artifactId>
                <version>0.14</version>
                <configuration>
                    <license>file:${basedir}/header</license>
                </configuration>
                <executions>
                    <execution>
                        <phase>validate</phase>
                        <goals>
                            <goal>check</goal>
                        </goals>
                    </execution>
                </executions>
            </plugin>
        </plugins>
    </build>
</project>

and run mvn validate.

@krzyk

This comment has been minimized.

Copy link
Contributor

krzyk commented Dec 11, 2015

@davvd valid bug

@krzyk

This comment has been minimized.

Copy link
Contributor

krzyk commented Dec 11, 2015

@davvd this is postponed

@davvd davvd added the bug label Dec 11, 2015

@davvd

This comment has been minimized.

Copy link

davvd commented Dec 11, 2015

@davvd valid bug

@krzyk I just added bug tag here

@davvd davvd added the postponed label Dec 11, 2015

@davvd

This comment has been minimized.

Copy link

davvd commented Dec 11, 2015

@davvd this is postponed

@krzyk right, I added "postponed" label

@davvd

This comment has been minimized.

Copy link

davvd commented Dec 11, 2015

@davvd this is postponed

@krzyk I will try to find someone else

@davvd

This comment has been minimized.

Copy link

davvd commented Dec 11, 2015

@mkordas thanks for tis bug, I topped your account for 15 mins, transaction 566aa93cd2b2cc0064000036

@davvd davvd added DEV 30 mins labels Dec 11, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.