You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@sling.apache.org by "Carsten Ziegeler (JIRA)" <ji...@apache.org> on 2009/03/28 15:27:50 UTC

[jira] Resolved: (SLING-876) maven-sling-plugin ignores targetPath on resource path checking

     [ https://issues.apache.org/jira/browse/SLING-876?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Carsten Ziegeler resolved SLING-876.
------------------------------------

    Resolution: Fixed

Hi Stefan,
in revision 759467 I've committed a slightly modified patch which checks for a trailing "/" after the target path. Can you please cross check and then close this bug?
Thanks

> maven-sling-plugin ignores targetPath on resource path checking
> ---------------------------------------------------------------
>
>                 Key: SLING-876
>                 URL: https://issues.apache.org/jira/browse/SLING-876
>             Project: Sling
>          Issue Type: Bug
>          Components: Maven Plugins
>    Affects Versions: Maven Sling Plugin 2.0.2, Maven Sling Plugin 2.0.4
>            Reporter: Stefan Seifert
>             Fix For: Maven Sling Plugin 2.0.4
>
>         Attachments: targetpath-patch.patch
>
>
> the sling plugins configures and checks initial content path mappings in method AbstractBundleInstallMojo.configure.
> this does work fine with maven default mapping.
> but if the project pom defines a special resource mapping with target path like this:
> 	  <resources>
> 	    <!-- copy webapp resources to SLING-INF/content/apps/${project.name} -->
> 	    <resource>
> 	      <directory>src/main/webapp</directory>
> 	      <targetPath>SLING-INF/content/apps/${project.name}</targetPath>
> 	      <filtering>false</filtering>
> 	    </resource>
> 	  </resources>
> this does not work any longer, because the target path is ignored and the wrong path is checked.
> the patch attached solves the problem.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.