karlpauls commented on pull request #70: URL: https://github.com/apache/sling-org-apache-sling-feature-cpconverter/pull/70#issuecomment-818702640
> That is where we don't agree. I don't see a good use case for that (even if it the current status quo). Can you give one? I'm not sure how to make this any clearer: I do not _disagree_. > For me this is not going "further", but probably related to the first point. It is clearly something new. Right now, the converter doesn't output "content" content packages - with your change it will. > Again, I don't see any use case for the current (errorneous) way of working. I agree. Nevertheless, it is how it is currently used. > No, just outputting a non-referenced artifact does not do any harm. It all depends on whether you use it afterwards or not. It is putting (potentially lots and big) content packages on disc where they are not needed. In summary, you want to change the current behavior - whatever usage there is currently, has to adapt. I don't understand why you want to insist that everybody that wants the old way has to now wrap the output in something that deletes the content package manually that are not used in the first place. I understand you think it is wrong to not use them - that I can agree with but why can't we just be BC in the default case and make it an option to include the content packages? -- 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: us...@infra.apache.org
