You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@struts.apache.org by "Wendy Smoak (JIRA)" <ji...@apache.org> on 2006/04/30 23:32:20 UTC
[jira] Created: (STR-2853) Confiure the CheckStyle plugin to use
the struts_checks.xml file
Confiure the CheckStyle plugin to use the struts_checks.xml file
----------------------------------------------------------------
Key: STR-2853
URL: http://issues.apache.org/struts/browse/STR-2853
Project: Struts Action 1
Type: Improvement
Components: Web Site
Versions: Nightly Build
Reporter: Wendy Smoak
Assigned to: Wendy Smoak
As shown at the bottom of [1], we need to put the struts_checks.xml and LICENSE.CheckStyle files in a jar and use it as an <extension> in the pom.
[1] http://maven.apache.org/plugins/maven-checkstyle-plugin/customize.html
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
http://issues.apache.org/struts/secure/Administrators.jspa
-
For more information on JIRA, see:
http://www.atlassian.com/software/jira
[jira] Commented: (STR-2853) Configure the CheckStyle plugin to use
the struts_checks.xml file
Posted by "Wendy Smoak (JIRA)" <ji...@apache.org>.
[ http://issues.apache.org/struts/browse/STR-2853?page=comments#action_37265 ]
Wendy Smoak commented on STR-2853:
----------------------------------
The struts/action/trunk/build directory has been moved to struts/maven/trunk/build, where it now creates a struts-build-1.0.0-SNAPSHOT.jar containing the config files. A snapshot has been published to cvs.apache.org, so the build should work from a clean checkout.
CheckStyle's regexpHeader property still needs to be set to the org/apache/struts/build/LICENSE.CheckStyle file in the struts-build jar.
> Configure the CheckStyle plugin to use the struts_checks.xml file
> -----------------------------------------------------------------
>
> Key: STR-2853
> URL: http://issues.apache.org/struts/browse/STR-2853
> Project: Struts Action 1
> Type: Improvement
> Components: Infrastructure
> Versions: Nightly Build
> Reporter: Wendy Smoak
> Assignee: Wendy Smoak
>
> As shown at the bottom of [1], we need to put the struts_checks.xml and LICENSE.CheckStyle files in a jar and use it as an <extension> in the pom.
> [1] http://maven.apache.org/plugins/maven-checkstyle-plugin/customize.html
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
http://issues.apache.org/struts/secure/Administrators.jspa
-
For more information on JIRA, see:
http://www.atlassian.com/software/jira
[jira] Commented: (STR-2853) Configure the CheckStyle plugin to use
the struts_checks.xml file
Posted by "James Mitchell (JIRA)" <ji...@apache.org>.
[ http://issues.apache.org/struts/browse/STR-2853?page=comments#action_37255 ]
James Mitchell commented on STR-2853:
-------------------------------------
It sounds like the first option would be the best for us. That would allow us to share anything (code, resources, *tests*, licenses, etc) across all of Struts projects (action1, action2, as well as Shale or any other projects in the sandbox or even external).
> Configure the CheckStyle plugin to use the struts_checks.xml file
> -----------------------------------------------------------------
>
> Key: STR-2853
> URL: http://issues.apache.org/struts/browse/STR-2853
> Project: Struts Action 1
> Type: Improvement
> Components: Infrastructure
> Versions: Nightly Build
> Reporter: Wendy Smoak
> Assignee: Wendy Smoak
>
> As shown at the bottom of [1], we need to put the struts_checks.xml and LICENSE.CheckStyle files in a jar and use it as an <extension> in the pom.
> [1] http://maven.apache.org/plugins/maven-checkstyle-plugin/customize.html
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
http://issues.apache.org/struts/secure/Administrators.jspa
-
For more information on JIRA, see:
http://www.atlassian.com/software/jira
[jira] Updated: (STR-2853) Configure the CheckStyle plugin to use
the struts_checks.xml file
Posted by "Wendy Smoak (JIRA)" <ji...@apache.org>.
[ http://issues.apache.org/struts/browse/STR-2853?page=all ]
Wendy Smoak updated STR-2853:
-----------------------------
Summary: Configure the CheckStyle plugin to use the struts_checks.xml file (was: Confiure the CheckStyle plugin to use the struts_checks.xml file)
Component: Infrastructure
(was: Web Site)
Description:
As shown at the bottom of [1], we need to put the struts_checks.xml and LICENSE.CheckStyle files in a jar and use it as an <extension> in the pom.
[1] http://maven.apache.org/plugins/maven-checkstyle-plugin/customize.html
was:
As shown at the bottom of [1], we need to put the struts_checks.xml and LICENSE.CheckStyle files in a jar and use it as an <extension> in the pom.
[1] http://maven.apache.org/plugins/maven-checkstyle-plugin/customize.html
> Configure the CheckStyle plugin to use the struts_checks.xml file
> -----------------------------------------------------------------
>
> Key: STR-2853
> URL: http://issues.apache.org/struts/browse/STR-2853
> Project: Struts Action 1
> Type: Improvement
> Components: Infrastructure
> Versions: Nightly Build
> Reporter: Wendy Smoak
> Assignee: Wendy Smoak
>
> As shown at the bottom of [1], we need to put the struts_checks.xml and LICENSE.CheckStyle files in a jar and use it as an <extension> in the pom.
> [1] http://maven.apache.org/plugins/maven-checkstyle-plugin/customize.html
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
http://issues.apache.org/struts/secure/Administrators.jspa
-
For more information on JIRA, see:
http://www.atlassian.com/software/jira
[jira] Commented: (STR-2853) Configure the CheckStyle plugin to use
the struts_checks.xml file
Posted by "Wendy Smoak (JIRA)" <ji...@apache.org>.
[ http://issues.apache.org/struts/browse/STR-2853?page=comments#action_37265 ]
Wendy Smoak commented on STR-2853:
----------------------------------
The struts/action/trunk/build directory has been moved to struts/maven/trunk/build, where it now creates a struts-build-1.0.0-SNAPSHOT.jar containing the config files. A snapshot has been published to cvs.apache.org, so the build should work from a clean checkout.
CheckStyle's regexpHeader property still needs to be set to the org/apache/struts/build/LICENSE.CheckStyle file in the struts-build jar.
> Configure the CheckStyle plugin to use the struts_checks.xml file
> -----------------------------------------------------------------
>
> Key: STR-2853
> URL: http://issues.apache.org/struts/browse/STR-2853
> Project: Struts Action 1
> Type: Improvement
> Components: Infrastructure
> Versions: Nightly Build
> Reporter: Wendy Smoak
> Assignee: Wendy Smoak
>
> As shown at the bottom of [1], we need to put the struts_checks.xml and LICENSE.CheckStyle files in a jar and use it as an <extension> in the pom.
> [1] http://maven.apache.org/plugins/maven-checkstyle-plugin/customize.html
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
http://issues.apache.org/struts/secure/Administrators.jspa
-
For more information on JIRA, see:
http://www.atlassian.com/software/jira
[jira] Commented: (STR-2853) Configure the CheckStyle plugin to use
the struts_checks.xml file
Posted by "Wendy Smoak (JIRA)" <ji...@apache.org>.
[ http://issues.apache.org/struts/browse/STR-2853?page=comments#action_37641 ]
Wendy Smoak commented on STR-2853:
----------------------------------
I've changed the struts-parent pom to use a URL instead of finding the Checkstyle config file in struts-build.jar.
IMO, while it's nice to make the build reproducible by having struts-build.jar in the repository, it isn't worth the additional effort required to coordinate a release when anything changes. No objection if someone else would like to release it and change the pom back, though.
> Configure the CheckStyle plugin to use the struts_checks.xml file
> -----------------------------------------------------------------
>
> Key: STR-2853
> URL: http://issues.apache.org/struts/browse/STR-2853
> Project: Struts 1
> Type: Improvement
> Components: Infrastructure
> Versions: Nightly Build
> Reporter: Wendy Smoak
> Assignee: Wendy Smoak
>
> As shown at the bottom of [1], we need to put the struts_checks.xml and LICENSE.CheckStyle files in a jar and use it as an <extension> in the pom.
> [1] http://maven.apache.org/plugins/maven-checkstyle-plugin/customize.html
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
http://issues.apache.org/struts/secure/Administrators.jspa
-
For more information on JIRA, see:
http://www.atlassian.com/software/jira
[jira] Commented: (STR-2853) Configure the CheckStyle plugin to use
the struts_checks.xml file
Posted by "James Mitchell (JIRA)" <ji...@apache.org>.
[ http://issues.apache.org/struts/browse/STR-2853?page=comments#action_37255 ]
James Mitchell commented on STR-2853:
-------------------------------------
It sounds like the first option would be the best for us. That would allow us to share anything (code, resources, *tests*, licenses, etc) across all of Struts projects (action1, action2, as well as Shale or any other projects in the sandbox or even external).
> Configure the CheckStyle plugin to use the struts_checks.xml file
> -----------------------------------------------------------------
>
> Key: STR-2853
> URL: http://issues.apache.org/struts/browse/STR-2853
> Project: Struts Action 1
> Type: Improvement
> Components: Infrastructure
> Versions: Nightly Build
> Reporter: Wendy Smoak
> Assignee: Wendy Smoak
>
> As shown at the bottom of [1], we need to put the struts_checks.xml and LICENSE.CheckStyle files in a jar and use it as an <extension> in the pom.
> [1] http://maven.apache.org/plugins/maven-checkstyle-plugin/customize.html
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
http://issues.apache.org/struts/secure/Administrators.jspa
-
For more information on JIRA, see:
http://www.atlassian.com/software/jira
[jira] Commented: (STR-2853) Configure the CheckStyle plugin to use
the struts_checks.xml file
Posted by "Wendy Smoak (JIRA)" <ji...@apache.org>.
[ http://issues.apache.org/struts/browse/STR-2853?page=comments#action_37254 ]
Wendy Smoak commented on STR-2853:
----------------------------------
One option is to have a top-level 'maven' project for the CheckStyle and other config files, the struts-parent pom, and any Maven plugins we end up writing. (Similar to MyFaces: http://svn.apache.org/repos/asf/myfaces )
Another is to create a jar from action/build, but then the plugin config has to be duplicated in all the modules to avoid a circular dependency.
Other ideas?
> Configure the CheckStyle plugin to use the struts_checks.xml file
> -----------------------------------------------------------------
>
> Key: STR-2853
> URL: http://issues.apache.org/struts/browse/STR-2853
> Project: Struts Action 1
> Type: Improvement
> Components: Infrastructure
> Versions: Nightly Build
> Reporter: Wendy Smoak
> Assignee: Wendy Smoak
>
> As shown at the bottom of [1], we need to put the struts_checks.xml and LICENSE.CheckStyle files in a jar and use it as an <extension> in the pom.
> [1] http://maven.apache.org/plugins/maven-checkstyle-plugin/customize.html
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
http://issues.apache.org/struts/secure/Administrators.jspa
-
For more information on JIRA, see:
http://www.atlassian.com/software/jira
[jira] Commented: (STR-2853) Configure the CheckStyle plugin to use
the struts_checks.xml file
Posted by "Wendy Smoak (JIRA)" <ji...@apache.org>.
[ http://issues.apache.org/struts/browse/STR-2853?page=comments#action_37259 ]
Wendy Smoak commented on STR-2853:
----------------------------------
Some discussion in the webwork chat:
http://forums.opensymphony.com/thread.jspa?threadID=28830
I'm not sure about sharing code and tests... what I'm thinking of here is maven-specific. So far my plan is to have
struts/maven/trunk
struts/maven/trunk/pom/pom.xml <--- currently in struts/pom, to be moved down.
struts/maven/trunk/build/ <--- Maven project for struts-build.jar, containing the CheckStyle config (and LICENSE.txt, Jalopy config...)
This is also where we'll put the (as yet unwritten) 'test build promotion' plugin.
These are things that must be released separately from, and in advance of, any Action 1 release. (Same for Action 2/Shale if those sub-projects elect to release with Maven 2 and need these or other build config files.)
Unless I hear objections I'll go ahead and try this out in a day or so. If it doesn't work out, we can always remove it.
> Configure the CheckStyle plugin to use the struts_checks.xml file
> -----------------------------------------------------------------
>
> Key: STR-2853
> URL: http://issues.apache.org/struts/browse/STR-2853
> Project: Struts Action 1
> Type: Improvement
> Components: Infrastructure
> Versions: Nightly Build
> Reporter: Wendy Smoak
> Assignee: Wendy Smoak
>
> As shown at the bottom of [1], we need to put the struts_checks.xml and LICENSE.CheckStyle files in a jar and use it as an <extension> in the pom.
> [1] http://maven.apache.org/plugins/maven-checkstyle-plugin/customize.html
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
http://issues.apache.org/struts/secure/Administrators.jspa
-
For more information on JIRA, see:
http://www.atlassian.com/software/jira
[jira] Commented: (STR-2853) Configure the CheckStyle plugin to use
the struts_checks.xml file
Posted by "Wendy Smoak (JIRA)" <ji...@apache.org>.
[ http://issues.apache.org/struts/browse/STR-2853?page=comments#action_37259 ]
Wendy Smoak commented on STR-2853:
----------------------------------
Some discussion in the webwork chat:
http://forums.opensymphony.com/thread.jspa?threadID=28830
I'm not sure about sharing code and tests... what I'm thinking of here is maven-specific. So far my plan is to have
struts/maven/trunk
struts/maven/trunk/pom/pom.xml <--- currently in struts/pom, to be moved down.
struts/maven/trunk/build/ <--- Maven project for struts-build.jar, containing the CheckStyle config (and LICENSE.txt, Jalopy config...)
This is also where we'll put the (as yet unwritten) 'test build promotion' plugin.
These are things that must be released separately from, and in advance of, any Action 1 release. (Same for Action 2/Shale if those sub-projects elect to release with Maven 2 and need these or other build config files.)
Unless I hear objections I'll go ahead and try this out in a day or so. If it doesn't work out, we can always remove it.
> Configure the CheckStyle plugin to use the struts_checks.xml file
> -----------------------------------------------------------------
>
> Key: STR-2853
> URL: http://issues.apache.org/struts/browse/STR-2853
> Project: Struts Action 1
> Type: Improvement
> Components: Infrastructure
> Versions: Nightly Build
> Reporter: Wendy Smoak
> Assignee: Wendy Smoak
>
> As shown at the bottom of [1], we need to put the struts_checks.xml and LICENSE.CheckStyle files in a jar and use it as an <extension> in the pom.
> [1] http://maven.apache.org/plugins/maven-checkstyle-plugin/customize.html
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
http://issues.apache.org/struts/secure/Administrators.jspa
-
For more information on JIRA, see:
http://www.atlassian.com/software/jira
[jira] Updated: (STR-2853) Configure the CheckStyle plugin to use
the struts_checks.xml file
Posted by "Wendy Smoak (JIRA)" <ji...@apache.org>.
[ http://issues.apache.org/struts/browse/STR-2853?page=all ]
Wendy Smoak updated STR-2853:
-----------------------------
Summary: Configure the CheckStyle plugin to use the struts_checks.xml file (was: Confiure the CheckStyle plugin to use the struts_checks.xml file)
Component: Infrastructure
(was: Web Site)
Description:
As shown at the bottom of [1], we need to put the struts_checks.xml and LICENSE.CheckStyle files in a jar and use it as an <extension> in the pom.
[1] http://maven.apache.org/plugins/maven-checkstyle-plugin/customize.html
was:
As shown at the bottom of [1], we need to put the struts_checks.xml and LICENSE.CheckStyle files in a jar and use it as an <extension> in the pom.
[1] http://maven.apache.org/plugins/maven-checkstyle-plugin/customize.html
> Configure the CheckStyle plugin to use the struts_checks.xml file
> -----------------------------------------------------------------
>
> Key: STR-2853
> URL: http://issues.apache.org/struts/browse/STR-2853
> Project: Struts Action 1
> Type: Improvement
> Components: Infrastructure
> Versions: Nightly Build
> Reporter: Wendy Smoak
> Assignee: Wendy Smoak
>
> As shown at the bottom of [1], we need to put the struts_checks.xml and LICENSE.CheckStyle files in a jar and use it as an <extension> in the pom.
> [1] http://maven.apache.org/plugins/maven-checkstyle-plugin/customize.html
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
http://issues.apache.org/struts/secure/Administrators.jspa
-
For more information on JIRA, see:
http://www.atlassian.com/software/jira
[jira] Resolved: (STR-2853) Configure the CheckStyle plugin to use
the struts_checks.xml file
Posted by "Wendy Smoak (JIRA)" <ji...@apache.org>.
[ http://issues.apache.org/struts/browse/STR-2853?page=all ]
Wendy Smoak resolved STR-2853:
------------------------------
Fix Version: 1.3.5
Resolution: Fixed
Fixed, with struts1/pom.xml pointing at struts_checks.xml in the svn repo, and the header added as a single line regexp since Checkstyle won't read a headerFile from a URL.
> Configure the CheckStyle plugin to use the struts_checks.xml file
> -----------------------------------------------------------------
>
> Key: STR-2853
> URL: http://issues.apache.org/struts/browse/STR-2853
> Project: Struts 1
> Type: Improvement
> Components: Infrastructure
> Versions: Nightly Build
> Reporter: Wendy Smoak
> Assignee: Wendy Smoak
> Fix For: 1.3.5
>
> As shown at the bottom of [1], we need to put the struts_checks.xml and LICENSE.CheckStyle files in a jar and use it as an <extension> in the pom.
> [1] http://maven.apache.org/plugins/maven-checkstyle-plugin/customize.html
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
http://issues.apache.org/struts/secure/Administrators.jspa
-
For more information on JIRA, see:
http://www.atlassian.com/software/jira
[jira] Commented: (STR-2853) Configure the CheckStyle plugin to use
the struts_checks.xml file
Posted by "Wendy Smoak (JIRA)" <ji...@apache.org>.
[ http://issues.apache.org/struts/browse/STR-2853?page=comments#action_37254 ]
Wendy Smoak commented on STR-2853:
----------------------------------
One option is to have a top-level 'maven' project for the CheckStyle and other config files, the struts-parent pom, and any Maven plugins we end up writing. (Similar to MyFaces: http://svn.apache.org/repos/asf/myfaces )
Another is to create a jar from action/build, but then the plugin config has to be duplicated in all the modules to avoid a circular dependency.
Other ideas?
> Configure the CheckStyle plugin to use the struts_checks.xml file
> -----------------------------------------------------------------
>
> Key: STR-2853
> URL: http://issues.apache.org/struts/browse/STR-2853
> Project: Struts Action 1
> Type: Improvement
> Components: Infrastructure
> Versions: Nightly Build
> Reporter: Wendy Smoak
> Assignee: Wendy Smoak
>
> As shown at the bottom of [1], we need to put the struts_checks.xml and LICENSE.CheckStyle files in a jar and use it as an <extension> in the pom.
> [1] http://maven.apache.org/plugins/maven-checkstyle-plugin/customize.html
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
http://issues.apache.org/struts/secure/Administrators.jspa
-
For more information on JIRA, see:
http://www.atlassian.com/software/jira