You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@sling.apache.org by pa...@apache.org on 2017/09/12 09:50:01 UTC
svn commit: r1808095 -
/sling/whiteboard/cziegeler/osgifeature-maven-plugin/src/main/java/org/apache/sling/feature/maven/Preprocessor.java
Author: pauls
Date: Tue Sep 12 09:50:01 2017
New Revision: 1808095
URL: http://svn.apache.org/viewvc?rev=1808095&view=rev
Log:
Wrap featureprovider in a builder context where applicable.
Modified:
sling/whiteboard/cziegeler/osgifeature-maven-plugin/src/main/java/org/apache/sling/feature/maven/Preprocessor.java
Modified: sling/whiteboard/cziegeler/osgifeature-maven-plugin/src/main/java/org/apache/sling/feature/maven/Preprocessor.java
URL: http://svn.apache.org/viewvc/sling/whiteboard/cziegeler/osgifeature-maven-plugin/src/main/java/org/apache/sling/feature/maven/Preprocessor.java?rev=1808095&r1=1808094&r2=1808095&view=diff
==============================================================================
--- sling/whiteboard/cziegeler/osgifeature-maven-plugin/src/main/java/org/apache/sling/feature/maven/Preprocessor.java (original)
+++ sling/whiteboard/cziegeler/osgifeature-maven-plugin/src/main/java/org/apache/sling/feature/maven/Preprocessor.java Tue Sep 12 09:50:01 2017
@@ -23,6 +23,7 @@ import org.apache.sling.feature.Artifact
import org.apache.sling.feature.Extension;
import org.apache.sling.feature.ExtensionType;
import org.apache.sling.feature.Feature;
+import org.apache.sling.feature.process.BuilderContext;
import org.apache.sling.feature.process.FeatureBuilder;
import org.apache.sling.feature.process.FeatureProvider;
import org.apache.sling.feature.support.FeatureUtil;
@@ -119,12 +120,11 @@ public class Preprocessor {
}
}
- final Feature assembledFeature = FeatureBuilder.assemble(feature, this.createFeatureProvider(env,
+ final Feature assembledFeature = FeatureBuilder.assemble(feature, new BuilderContext(this.createFeatureProvider(env,
info,
config.isTestConfig(),
config.isSkipAddDependencies(),
- config.getScope(),
- null));
+ config.getScope(), null)));
if ( config.isTestConfig() ) {
info.assembledTestFeature = assembledFeature;
} else {
@@ -210,12 +210,12 @@ public class Preprocessor {
}
final List<Feature> assembledFeatureList = new ArrayList<>();
for(final Feature feature : featureList) {
- final Feature assembledFeature = FeatureBuilder.assemble(feature, this.createFeatureProvider(env,
+ final Feature assembledFeature = FeatureBuilder.assemble(feature, new BuilderContext(this.createFeatureProvider(env,
info,
config.isTestConfig(),
config.isSkipAddDependencies(),
config.getScope(),
- featureList));
+ featureList)));
assembledFeatureList.add(assembledFeature);
}
if ( config.isTestConfig() ) {
@@ -258,12 +258,12 @@ public class Preprocessor {
}
final List<Feature> assembledFeatureRefList = new ArrayList<>();
for(final Feature feature : featureRefList) {
- final Feature assembledFeature = FeatureBuilder.assemble(feature, this.createFeatureProvider(env,
+ final Feature assembledFeature = FeatureBuilder.assemble(feature, new BuilderContext(this.createFeatureProvider(env,
info,
config.isTestConfig(),
config.isSkipAddDependencies(),
config.getScope(),
- featureList));
+ featureList)));
assembledFeatureRefList.add(assembledFeature);
}
if ( config.isTestConfig() ) {
@@ -287,7 +287,7 @@ public class Preprocessor {
* Add all dependencies from the feature
* @param env The environment
* @param info The project info
- * @param assembleFeature The assembled feature for finding the artifacts.
+ * @param assembledFeature The assembled feature for finding the artifacts.
* @param scope The scope which the new dependencies should have
*/
private void addDependenciesFromFeature(