Maven 2 & 3

generated source repo page is misleading when added to the "general site".

Details

  • Complexity:
    Intermediate
  • Number of attachments :
    1

Description

it says:
$ svn checkout http://svn.apache.org/repos/asf/maven/components/trunk/ maven-site

We really should set the SCM URL correctly in the site pom again (to point to the site SCM), and the generated report should be excluded from the navigation. It could still be generated and linked from another doc that describes helping with the docs.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Brian K. Wallace added a comment -

Attached is a change to the POM to fix the prefix for the source repository. However it does not address two issues:

1. It is still misleading to have SCM access for "Maven" only point to the site's source
2. maven-project-info-reports-plugin is coded to add the artifactId (maven-site in this case) to the end of all SCM URLs. A pre-existing comment in the POM states that the SCM section was repeated to prevent maven-site from being added, however the plugin will still add it regardless. I created MPIR-33 to address generated SCM access without the addition of the artifactId.

Show
Brian K. Wallace added a comment - Attached is a change to the POM to fix the prefix for the source repository. However it does not address two issues: 1. It is still misleading to have SCM access for "Maven" only point to the site's source 2. maven-project-info-reports-plugin is coded to add the artifactId (maven-site in this case) to the end of all SCM URLs. A pre-existing comment in the POM states that the SCM section was repeated to prevent maven-site from being added, however the plugin will still add it regardless. I created MPIR-33 to address generated SCM access without the addition of the artifactId.
Hide
Permalink
Brett Porter added a comment -

I think this patch should be applied (as it is correct), and then the source repository report actually disabled. I'll think some more about it.

Show
Brett Porter added a comment - I think this patch should be applied (as it is correct), and then the source repository report actually disabled. I'll think some more about it.
Hide
Permalink
Vincent Siveton added a comment -

Applied in svn with brett suggestions
Thanks.

Show
Vincent Siveton added a comment - Applied in svn with brett suggestions Thanks.

People

Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved:
Atlassian JIRA (v4.4.3#663-r165197) | Report a problem
Powered by a free Atlassian JIRA open source license for Codehaus. Try JIRA - bug tracking software for your team.