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

fixed plugin can not find assembly descriptor #89

Closed
wants to merge 1 commit into from

Conversation

kuhnroyal
Copy link
Contributor

The plugin could not find the assembly.xml file in the default source directory in multi-module maven projects.
It works if docker:build is executed from the sub-module but not if execute from the parent module.
When building the absolute path to the assembly.xml or Dockerfile the project baseDir is now respected.

@rhuss
Copy link
Collaborator

rhuss commented Feb 6, 2015

Awesome. Thanks a lot for the fix. Could you please do me a favor and sign-off your commits as explained here ? Please also for you other PRs which I'm going to integrate today.

You simply need to do a

git commit -s --amend

on this bug fix branch (works for the last commit, if you need to sign-off more commits in retrospective, you need an interactive rebase (git -i rebase))

Thanks a lot !

@jgangemi
Copy link
Collaborator

jgangemi commented Feb 6, 2015

do you want to do a quick bugfix release w/ these and maybe slate new functionality for 0.12.0?

@rhuss
Copy link
Collaborator

rhuss commented Feb 6, 2015

Yes, think this is a good idea. I will do a a 0.11.2 this weekend.

@jgangemi
Copy link
Collaborator

jgangemi commented Feb 6, 2015

was there a 0.11.1 that i missed?

@rhuss
Copy link
Collaborator

rhuss commented Feb 6, 2015

Ehm, no. I was wrong, no 0.11.1 yet, so that's the next one.

… started from top module

Signed-off-by: Peter Leibiger <[email protected]>
rhuss added a commit that referenced this pull request Feb 10, 2015
@rhuss
Copy link
Collaborator

rhuss commented Feb 10, 2015

Applied and available on branch integration

@rhuss rhuss closed this Feb 10, 2015
@kuhnroyal kuhnroyal deleted the fixedBaseDir branch February 12, 2015 10:01
leusonmario pushed a commit to leusonmario/docker-maven-plugin that referenced this pull request Aug 18, 2018
leusonmario pushed a commit to leusonmario/docker-maven-plugin that referenced this pull request Aug 18, 2018
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.

3 participants