karlpauls commented on a change in pull request #18:
URL:
https://github.com/apache/sling-org-apache-sling-feature-launcher/pull/18#discussion_r593811652
##########
File path: src/main/java/org/apache/sling/feature/launcher/impl/Main.java
##########
@@ -66,9 +67,11 @@
public static final String OPT_REPOSITORY_URLS = "u";
- public static final String OPT_CONFIG_CLASH = "CC";
+ public static final String OPT_CONFIG_CLASH = "C";
- public static final String OPT_ARTICACT_CLASH = "C";
Review comment:
This seems wrong - I don't think we want to change any of the switches
at this point. We could consider adding new ones that are essentially aliases
for the old switches but we have to stay backwards compatible at this point.
Furthermore, I think you don't reflect this change in the help nor in the
env var mappings.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
