Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

WIP:[maven] use outputDir and waroutputDir correctly #5176

Closed
wants to merge 1 commit into from

Conversation

stbischof
Copy link
Contributor

@stbischof stbischof commented Mar 16, 2022

The new jarand testjargoals / BndMavenPackagingTestsPlugin and BndMavenPackagingPlugin could be configures according the documentation using the confguration-propertiers outputDir and waroutputDir. But this properties are not used to save the jars in this position.

TODO:

  • test that shows that a not default outputDir and waroutputDir work properly.
  • since waroutputDir and outputDir never worked we can decide to use only outputDirectory for both like in maven-jar-plugin and maven-war-plgin

Signed-off-by: Stefan Bischof <stbischof@bipolis.org>
@bjhargrave bjhargrave marked this pull request as draft March 21, 2022 20:24
@bjhargrave
Copy link
Member

It is unclear to me what problem you are solving. There is no issue describing the problem and there is no text in the pr or commit which describes the purpose of the change. The PR is also lacking tests for the change.

@bjhargrave
Copy link
Member

A war file never goes in the web app directory. This is only for the exploded wab. What does appear to be missing is the exploding of the wab into the web app directory directory when using a packaging goal. I will make a new PR for that.

@bjhargrave bjhargrave closed this Jul 6, 2022
@bjhargrave
Copy link
Member

See #5310.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants