[GitHub] [maven-archetype] eolivelli commented on issue #28: ARCHETYPE-567: switch to dom4j 2.1.1 (and Java 8)

eolivelli commented on issue #28: ARCHETYPE-567: switch to dom4j 2.1.1 (and Java 8) URL: https://github.com/apache/maven-archetype/pull/28#issuecomment-498736789 @tony-- Shall I add you to the contributors list in yhe pom of the plugin?

[GitHub] [maven-archetype] eolivelli commented on issue #28: ARCHETYPE-567: switch to dom4j 2.1.1 (and Java 8)

eolivelli commented on issue #28: ARCHETYPE-567: switch to dom4j 2.1.1 (and Java 8) URL: https://github.com/apache/maven-archetype/pull/28#issuecomment-498652638 thank you @tony-- This is an automated message from the

[GitHub] [maven-archetype] eolivelli commented on issue #28: ARCHETYPE-567: switch to dom4j 2.1.1 (and Java 8)

eolivelli commented on issue #28: ARCHETYPE-567: switch to dom4j 2.1.1 (and Java 8) URL: https://github.com/apache/maven-archetype/pull/28#issuecomment-498652589 committed as fc9a5ff0efe1b076313b51180ac53d841d430620 This is

[GitHub] [maven-archetype] eolivelli commented on issue #28: ARCHETYPE-567: switch to dom4j 2.1.1 (and Java 8)

eolivelli commented on issue #28: ARCHETYPE-567: switch to dom4j 2.1.1 (and Java 8) URL: https://github.com/apache/maven-archetype/pull/28#issuecomment-498038064 @Tibor17 thank you for the heads up. I will continue as soon as possible thank you @tony-- I will be back with news.

[GitHub] [maven-archetype] eolivelli commented on issue #28: ARCHETYPE-567: switch to dom4j 2.1.1 (and Java 8)

eolivelli commented on issue #28: ARCHETYPE-567: switch to dom4j 2.1.1 (and Java 8) URL: https://github.com/apache/maven-archetype/pull/28#issuecomment-498037456 I have added debug as suggested by @Tibor17 but the is no much to need, at least in the scope of this issue I am seeing

[GitHub] [maven-archetype] eolivelli commented on issue #28: ARCHETYPE-567: switch to dom4j 2.1.1 (and Java 8)

eolivelli commented on issue #28: ARCHETYPE-567: switch to dom4j 2.1.1 (and Java 8) URL: https://github.com/apache/maven-archetype/pull/28#issuecomment-498014132 @Tibor17 is there any way to get build files? In surefire we are creating a zip with all of the logs

[GitHub] [maven-archetype] eolivelli commented on issue #28: ARCHETYPE-567: switch to dom4j 2.1.1 (and Java 8)

eolivelli commented on issue #28: ARCHETYPE-567: switch to dom4j 2.1.1 (and Java 8) URL: https://github.com/apache/maven-archetype/pull/28#issuecomment-498014094 I have reverted last commit. I don't think that it was the cause. I am digging into logs. Apparently there is no

[GitHub] [maven-archetype] eolivelli commented on issue #28: ARCHETYPE-567: switch to dom4j 2.1.1 (and Java 8)

eolivelli commented on issue #28: ARCHETYPE-567: switch to dom4j 2.1.1 (and Java 8) URL: https://github.com/apache/maven-archetype/pull/28#issuecomment-497976308 I have pushed your change, jenkins is running This is an

[GitHub] [maven-archetype] eolivelli commented on issue #28: ARCHETYPE-567: switch to dom4j 2.1.1 (and Java 8)

eolivelli commented on issue #28: ARCHETYPE-567: switch to dom4j 2.1.1 (and Java 8) URL: https://github.com/apache/maven-archetype/pull/28#issuecomment-497968674 You are right @Tobor17. I was trying to say that in the scope of this change it is fine to drop only 7. We can do a

[GitHub] [maven-archetype] eolivelli commented on issue #28: ARCHETYPE-567: switch to dom4j 2.1.1 (and Java 8)

eolivelli commented on issue #28: ARCHETYPE-567: switch to dom4j 2.1.1 (and Java 8) URL: https://github.com/apache/maven-archetype/pull/28#issuecomment-497967888 @Tibor17 let's stick to dropping java 7 for this patch. We should remove java10 in a more maven-wide way (in the common lib)

[GitHub] [maven-archetype] eolivelli commented on issue #28: ARCHETYPE-567: switch to dom4j 2.1.1 (and Java 8)

eolivelli commented on issue #28: ARCHETYPE-567: switch to dom4j 2.1.1 (and Java 8) URL: https://github.com/apache/maven-archetype/pull/28#issuecomment-497964380 It is okay. I don't have my laptop my. I will push your commit as soon as I can.

[GitHub] [maven-archetype] eolivelli commented on issue #28: ARCHETYPE-567: switch to dom4j 2.1.1 (and Java 8)

eolivelli commented on issue #28: ARCHETYPE-567: switch to dom4j 2.1.1 (and Java 8) URL: https://github.com/apache/maven-archetype/pull/28#issuecomment-497937360 Please do the same as I did for checkstyle plugin here

[GitHub] [maven-archetype] eolivelli commented on issue #28: ARCHETYPE-567: switch to dom4j 2.1.1 (and Java 8)

eolivelli commented on issue #28: ARCHETYPE-567: switch to dom4j 2.1.1 (and Java 8) URL: https://github.com/apache/maven-archetype/pull/28#issuecomment-497855733 https://builds.apache.org/job/maven-box/job/maven-archetype/job/ARCHETYPE-567/