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

Oliver Lietz commented on SLING-4147:
-------------------------------------

* {{artifactId}} should be changed from 
{{org.apache.sling.extensions.datasource}} to {{org.apache.sling.datasource}} 
accordingly
* configuration of {{jacoco-maven-plugin}} needs to be updated
* empty directory 
{{src/main/java/org/apache/sling/extensions/datasource/internal}} should be 
removed (Git used?)
* test classes should also be moved from 
{{src/test/java/org/apache/sling/extensions/datasource}} to 
{{src/test/java/org/apache/sling/datasource}}

> Remove extensions from package name 
> org.apache.sling.extensions.datasource.internal
> -----------------------------------------------------------------------------------
>
>                 Key: SLING-4147
>                 URL: https://issues.apache.org/jira/browse/SLING-4147
>             Project: Sling
>          Issue Type: Task
>          Components: Extensions
>            Reporter: Chetan Mehrotra
>            Assignee: Chetan Mehrotra
>            Priority: Minor
>             Fix For: DataSource Provider 1.0.0
>
>
> DataSource provider bundle currently has {{extensions}} in package name and 
> configuration pid. As {{extensions}} is mostly related to module location in 
> Sling svn it would be better to drop it from package name and component pid



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to