[ 
https://issues.apache.org/jira/browse/MSITE-738?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16785813#comment-16785813
 ] 

Steve Brown commented on MSITE-738:
-----------------------------------

This is broken in version 3.7.1 of the site plugin and an example of the impact 
is that the site for a child pom cannot be deployed to Artifactory because 
Artifactory does not allow relative paths (although they are addressing the 
issue for downloads from remote sites 
([RTFACT-16457|https://www.jfrog.com/jira/browse/RTFACT-16457]). Artifactory 
returns this error:

{{"status" : 500,"}}
 {{"Path element cannot end with a dot: sites/ ... redacted ... */../*"}}

> SiteDeployMojo#determineDeploySite code/javadoc inconsistent. Javadoc seems 
> more correct
> ----------------------------------------------------------------------------------------
>
>                 Key: MSITE-738
>                 URL: https://issues.apache.org/jira/browse/MSITE-738
>             Project: Maven Site Plugin
>          Issue Type: Bug
>          Components: site:deploy
>    Affects Versions: 3.4
>            Reporter: Grégory Joseph
>            Priority: Major
>
> The javadoc of this method seems to be the desired behavior:
> {code}
>     /**
>      * Deploy directly to the current project's distribution management site.
>      */
>     @Override
>     protected Site determineDeploySite()
>         throws MojoExecutionException
>     {
>         return getSite( getTopLevelProject( project ) );
>     }
> {code}
> However, the code indicates it goes all the way in the parent pom hierarchy ? 
> Why ?
> * The outcome is inconsistent with the effective-pom
> * I'd assume if my pom declares a {{distributionManagement/site}} section, it 
> should be used, rather than the site plugin trying to be smarter and use the 
> parent pom's info then somewhat relativize ? This leads to the same issues I 
> bumped into a couple years ago (when I didn't bother plugging my debugger in) 
> : 
> http://maven.40175.n5.nabble.com/Site-deployment-url-and-inheritance-td5712737.html
> This can cause at least two problems:
> * One can't deploy a site to a host that's different than that of the parent 
> pom
> * Permissions on the server-side might not be applied correctly (perhaps the 
> project's deployer doesn't have permissions to deploy into the path 
> configured in the parent, but does in the path of his project.. however we 
> try to deploy to {{...parent/../../project/...}}.
> Additionally, this just get confusing, because {{mvn help:effective-pom}} 
> gives me the {{distributionManagement/site}} section I expect, but the site 
> plugin ends up doing something else. 
> Reverting to {code}
>  protected Site determineDeploySite()
>         throws MojoExecutionException
>     {
>         return getSite( project );
>     }
>  {code}
> Fixes the problem as far as I can tell, since project is already 
> resolved/effective model, isn't it ? I'm not sure what sure what commit 
> #1480820 was fixing.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to