maven-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Michael Osipov (JIRA)" <j...@codehaus.org>
Subject [jira] (MPMD-170) Have targetJdk default to maven.compiler.target
Date Sun, 02 Nov 2014 14:44:10 GMT

    [ https://jira.codehaus.org/browse/MPMD-170?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=355429#comment-355429
] 

Michael Osipov edited comment on MPMD-170 at 11/2/14 8:42 AM:
--------------------------------------------------------------

I'd like to fix this finally by apply to proposed fix. As you have already [layed out|http://jira.codehaus.org/browse/MJAVADOC-310?focusedCommentId=354200&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-354200]
Robert, plugins should not know of each other. Reading compiler plugin's config is the wrong
way to go. If no one opposes, I will by default provide {{maven.compiler.target}}.


was (Author: michael-o):
I'd like to fix this finally by apply to proposed fix. As you have already layed out Robert,
plugins should not know of each other. Reading compiler plugin's config is the wrong way to
go. If no one opposes, I will by default provide {{maven.compiler.target}}.

> Have targetJdk default to maven.compiler.target
> -----------------------------------------------
>
>                 Key: MPMD-170
>                 URL: https://jira.codehaus.org/browse/MPMD-170
>             Project: Maven PMD Plugin
>          Issue Type: Improvement
>          Components: PMD
>    Affects Versions: 3.0.1
>            Reporter: Stevo Slavic
>            Priority: Minor
>         Attachments: MPMD-170.patch
>
>
> Currently {{targetJdk}} has to be defined separately from {{maven.compiler.target}} property
that most plugins already default to, so target JDK information is unnecessarily repeated,
making POMs with maven-pmd-plugin less DRY, harder to maintain, prone to mistakes and unwanted
behavior.
> So, please make {{targetJdk}} default to {{maven.compiler.target}} property.
> Attached is [^MPMD-170.patch] which adds this behavior.



--
This message was sent by Atlassian JIRA
(v6.1.6#6162)

Mime
View raw message