[ 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)