kwin edited a comment on pull request #74: URL: https://github.com/apache/sling-org-apache-sling-feature-cpconverter/pull/74#issuecomment-826806670
> do we need to be more fine-grained when it comes to what to be stripping out of the bundle? I.e., maybe there are cases where somebody expects the content to be inside the bundle (hence, might only be fine with stripping the header) or where they want to deploy the bundle unchanged - just have the content pre-extracted... I have never seen cases like this. We could add a flag but I would vote for making stripping it out of the bundle by default. Otherwise you will run into the warning which we wanted to prevent in the first place. > should we write them back to a different id if we stripped something from them? Not sure if that would cause more confusion or not - but arguably, they are not the same bundle anymore... Right, I can change the artifact id of the bundle by adding a classifier. Do you think we should also modify the BSN in the manifest? -- 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]
