[jira] (MCHECKSTYLE-169) Suppressions property is incorrectly set if suppressions file is on classpath
[
https://jira.codehaus.org/browse/MCHECKSTYLE-169?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=362374#comment-362374
]
Dmitry Katsubo commented on MCHECKSTYLE-169:
Before v2.12 (that included this patch) it was possible to set
{{suppressionsLocation}} to some location, that does not necessarily exist in
particular project. This is handy e.g. during multi-module build, when some of
the projects have that file, and some ? don't. Now the location should exist.
How to make this setting optional (e.g. if suppressions file exists then it is
used, otherwise the build does not fail)?
Suppressions property is incorrectly set if suppressions file is on classpath
-
Key: MCHECKSTYLE-169
URL: https://jira.codehaus.org/browse/MCHECKSTYLE-169
Project: Maven Checkstyle Plugin
Issue Type: Bug
Affects Versions: 2.8
Reporter: Sergei Ivanov
Assignee: Dennis Lundberg
Fix For: 2.12
Attachments: MCHECKSTYLE-169-bugfix.patch, MCHECKSTYLE-169-it.patch
If checkstyle plugin loads suppressions file from its classpath, and the
checkstyle configuration contains suppression filter with a property
placeholder, that property is not expanded because an incorrect path is used.
--
This message was sent by Atlassian JIRA
(v6.1.6#6162)
[jira] (MCHECKSTYLE-169) Suppressions property is incorrectly set if suppressions file is on classpath
[ https://jira.codehaus.org/browse/MCHECKSTYLE-169?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Dennis Lundberg closed MCHECKSTYLE-169. --- Resolution: Fixed Fix Version/s: 2.12 Assignee: Dennis Lundberg Patch applied in [r1570730|http://svn.apache.org/viewvc?view=revisionrevision=1570730]. Many thanks! Suppressions property is incorrectly set if suppressions file is on classpath - Key: MCHECKSTYLE-169 URL: https://jira.codehaus.org/browse/MCHECKSTYLE-169 Project: Maven Checkstyle Plugin Issue Type: Bug Affects Versions: 2.8 Reporter: Sergei Ivanov Assignee: Dennis Lundberg Fix For: 2.12 Attachments: MCHECKSTYLE-169-bugfix.patch, MCHECKSTYLE-169-it.patch If checkstyle plugin loads suppressions file from its classpath, and the checkstyle configuration contains suppression filter with a property placeholder, that property is not expanded because an incorrect path is used. -- This message was sent by Atlassian JIRA (v6.1.6#6162)
[jira] (MCHECKSTYLE-169) Suppressions property is incorrectly set if suppressions file is on classpath
[ https://jira.codehaus.org/browse/MCHECKSTYLE-169?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=338108#comment-338108 ] Sergei Ivanov commented on MCHECKSTYLE-169: --- Can it be integrated into 2.12, please? It is frustrating that the issue is still unresolved after 2 years, despite a patch being available. We have effectively given up on Checkstyle integration into our Maven/Jenkins reports because of that. Suppressions property is incorrectly set if suppressions file is on classpath - Key: MCHECKSTYLE-169 URL: https://jira.codehaus.org/browse/MCHECKSTYLE-169 Project: Maven Checkstyle Plugin Issue Type: Bug Affects Versions: 2.8 Reporter: Sergei Ivanov Attachments: MCHECKSTYLE-169-bugfix.patch, MCHECKSTYLE-169-it.patch If checkstyle plugin loads suppressions file from its classpath, and the checkstyle configuration contains suppression filter with a property placeholder, that property is not expanded because an incorrect path is used. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] (MCHECKSTYLE-169) Suppressions property is incorrectly set if suppressions file is on classpath
[ https://jira.codehaus.org/browse/MCHECKSTYLE-169?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=327354#comment-327354 ] Edwin Shin commented on MCHECKSTYLE-169: This was driving me nuts. Is there any workaround in the meantime (e.g. is this a regression and is there an older version in which this does work)? Suppressions property is incorrectly set if suppressions file is on classpath - Key: MCHECKSTYLE-169 URL: https://jira.codehaus.org/browse/MCHECKSTYLE-169 Project: Maven 2.x Checkstyle Plugin Issue Type: Bug Affects Versions: 2.8 Reporter: Sergei Ivanov Attachments: MCHECKSTYLE-169-bugfix.patch, MCHECKSTYLE-169-it.patch If checkstyle plugin loads suppressions file from its classpath, and the checkstyle configuration contains suppression filter with a property placeholder, that property is not expanded because an incorrect path is used. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] (MCHECKSTYLE-169) Suppressions property is incorrectly set if suppressions file is on classpath
[ https://jira.codehaus.org/browse/MCHECKSTYLE-169?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=327354#comment-327354 ] Edwin Shin edited comment on MCHECKSTYLE-169 at 6/25/13 10:30 PM: -- This was driving me nuts (using version 2.10). Is there any workaround in the meantime (e.g. is this a regression and is there an older version in which this does work)? was (Author: eddies): This was driving me nuts. Is there any workaround in the meantime (e.g. is this a regression and is there an older version in which this does work)? Suppressions property is incorrectly set if suppressions file is on classpath - Key: MCHECKSTYLE-169 URL: https://jira.codehaus.org/browse/MCHECKSTYLE-169 Project: Maven 2.x Checkstyle Plugin Issue Type: Bug Affects Versions: 2.8 Reporter: Sergei Ivanov Attachments: MCHECKSTYLE-169-bugfix.patch, MCHECKSTYLE-169-it.patch If checkstyle plugin loads suppressions file from its classpath, and the checkstyle configuration contains suppression filter with a property placeholder, that property is not expanded because an incorrect path is used. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] (MCHECKSTYLE-169) Suppressions property is incorrectly set if suppressions file is on classpath
[ https://jira.codehaus.org/browse/MCHECKSTYLE-169?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=324310#comment-324310 ] Sergei Ivanov commented on MCHECKSTYLE-169: --- Can we have it integrated into 2.11, please? Suppressions property is incorrectly set if suppressions file is on classpath - Key: MCHECKSTYLE-169 URL: https://jira.codehaus.org/browse/MCHECKSTYLE-169 Project: Maven 2.x Checkstyle Plugin Issue Type: Bug Affects Versions: 2.8 Reporter: Sergei Ivanov Attachments: MCHECKSTYLE-169-bugfix.patch, MCHECKSTYLE-169-it.patch If checkstyle plugin loads suppressions file from its classpath, and the checkstyle configuration contains suppression filter with a property placeholder, that property is not expanded because an incorrect path is used. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] (MCHECKSTYLE-169) Suppressions property is incorrectly set if suppressions file is on classpath
[ https://jira.codehaus.org/browse/MCHECKSTYLE-169?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=307773#comment-307773 ] Sergei Ivanov commented on MCHECKSTYLE-169: --- Please, please! Suppressions property is incorrectly set if suppressions file is on classpath - Key: MCHECKSTYLE-169 URL: https://jira.codehaus.org/browse/MCHECKSTYLE-169 Project: Maven 2.x Checkstyle Plugin Issue Type: Bug Affects Versions: 2.8 Reporter: Sergei Ivanov Attachments: MCHECKSTYLE-169-bugfix.patch, MCHECKSTYLE-169-it.patch If checkstyle plugin loads suppressions file from its classpath, and the checkstyle configuration contains suppression filter with a property placeholder, that property is not expanded because an incorrect path is used. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators: https://jira.codehaus.org/secure/ContactAdministrators!default.jspa For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] (MCHECKSTYLE-169) Suppressions property is incorrectly set if suppressions file is on classpath
[ https://jira.codehaus.org/browse/MCHECKSTYLE-169?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=305468#comment-305468 ] Sergei Ivanov commented on MCHECKSTYLE-169: --- Any chance of the attached patch being integrated into 2.10? Suppressions property is incorrectly set if suppressions file is on classpath - Key: MCHECKSTYLE-169 URL: https://jira.codehaus.org/browse/MCHECKSTYLE-169 Project: Maven 2.x Checkstyle Plugin Issue Type: Bug Affects Versions: 2.8 Reporter: Sergei Ivanov Attachments: MCHECKSTYLE-169-bugfix.patch, MCHECKSTYLE-169-it.patch If checkstyle plugin loads suppressions file from its classpath, and the checkstyle configuration contains suppression filter with a property placeholder, that property is not expanded because an incorrect path is used. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators: https://jira.codehaus.org/secure/ContactAdministrators!default.jspa For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] (MCHECKSTYLE-169) Suppressions property is incorrectly set if suppressions file is on classpath
[ https://jira.codehaus.org/browse/MCHECKSTYLE-169?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=290591#comment-290591 ] Dennis Lundberg commented on MCHECKSTYLE-169: - Can you give a sample project that we can use to test this? Suppressions property is incorrectly set if suppressions file is on classpath - Key: MCHECKSTYLE-169 URL: https://jira.codehaus.org/browse/MCHECKSTYLE-169 Project: Maven 2.x Checkstyle Plugin Issue Type: Bug Affects Versions: 2.8 Reporter: Sergei Ivanov If checkstyle plugin loads suppressions file from its classpath, and the checkstyle configuration contains suppression filter with a property placeholder, that property is not expanded because an incorrect path is used. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators: https://jira.codehaus.org/secure/ContactAdministrators!default.jspa For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] (MCHECKSTYLE-169) Suppressions property is incorrectly set if suppressions file is on classpath
[ https://jira.codehaus.org/browse/MCHECKSTYLE-169?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Sergei Ivanov updated MCHECKSTYLE-169: -- Attachment: MCHECKSTYLE-169-bugfix.patch MCHECKSTYLE-169-it.patch Hello Dennis, Attached are an integration test for the described problem and a patch that fixes it. I wanted to add a couple of additional integration tests to check for regressions, but ran out of spare time. Please review and integrate into the next release if deemed suitable. The patch is against the SVN trunk for the plugin. Thank you. Suppressions property is incorrectly set if suppressions file is on classpath - Key: MCHECKSTYLE-169 URL: https://jira.codehaus.org/browse/MCHECKSTYLE-169 Project: Maven 2.x Checkstyle Plugin Issue Type: Bug Affects Versions: 2.8 Reporter: Sergei Ivanov Attachments: MCHECKSTYLE-169-bugfix.patch, MCHECKSTYLE-169-it.patch If checkstyle plugin loads suppressions file from its classpath, and the checkstyle configuration contains suppression filter with a property placeholder, that property is not expanded because an incorrect path is used. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators: https://jira.codehaus.org/secure/ContactAdministrators!default.jspa For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] (MCHECKSTYLE-169) Suppressions property is incorrectly set if suppressions file is on classpath
Sergei Ivanov created MCHECKSTYLE-169: - Summary: Suppressions property is incorrectly set if suppressions file is on classpath Key: MCHECKSTYLE-169 URL: https://jira.codehaus.org/browse/MCHECKSTYLE-169 Project: Maven 2.x Checkstyle Plugin Issue Type: Bug Affects Versions: 2.8 Reporter: Sergei Ivanov If checkstyle plugin loads suppressions file from its classpath, and the checkstyle configuration contains suppression filter with a property placeholder, that property is not expanded because an incorrect path is used. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators: https://jira.codehaus.org/secure/ContactAdministrators!default.jspa For more information on JIRA, see: http://www.atlassian.com/software/jira
