[
https://issues.apache.org/jira/browse/MNG-6207?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16046058#comment-16046058
]
ASF GitHub Bot commented on MNG-6207:
-------------------------------------
Github user dejan2609 commented on the issue:
https://github.com/apache/maven/pull/123
Just for the record: somehow it slipped of my mind that `package` phase
doesn't execute integration tests
(http://maven.apache.org/ref/3.5.0/maven-core/lifecycles.html#default_Lifecycle)
and hence this checkstyle error pass unnoticed on my side:
> [ERROR]
src/main/java/org/apache/maven/model/validation/DefaultModelValidator.java[471:104]
(blocks) LeftCurly: '{' should be on a new line.
Thanx @khmarbaise for this correction and for overall review.
> Create WARNINGs in case of using system scope
> ---------------------------------------------
>
> Key: MNG-6207
> URL: https://issues.apache.org/jira/browse/MNG-6207
> Project: Maven
> Issue Type: Improvement
> Affects Versions: 3.5.0
> Reporter: Karl Heinz Marbaise
> Assignee: Karl Heinz Marbaise
> Priority: Minor
> Fix For: 3.5.1
>
>
> Currently the documentation has already marked the usage of:
> {code:xml}
> <dependency>
> <groupId>javax.sql</groupId>
> <artifactId>jdbc-stdext</artifactId>
> <version>2.0</version>
> <scope>system</scope>
> <systemPath>${java.home}/lib/rt.jar</systemPath>
> </dependency>
> {code}
> as
> [deprecated|https://maven.apache.org/guides/introduction/introduction-to-dependency-mechanism.html#System_Dependencies].
> So we should start to produce WARNING during a build if someone uses it...
--
This message was sent by Atlassian JIRA
(v6.3.15#6346)
