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

Ensure we also build dependent modules for deployments #10936

Merged
merged 1 commit into from
Jan 14, 2021
Merged

Conversation

normanmaurer
Copy link
Member

Motivation:

We need to ensure we also build the modules we depend on as otherwise we may not be able to resolve the dependencies correctly

Modifications:

  • Add -am when calling maven

Result:

Deployment works all the time

Motivation:

We need to ensure we also build the modules we depend on as otherwise we may not be able to resolve the dependencies correctly

Modifications:

- Add -am when calling maven

Result:

Deployment works all the time
@normanmaurer normanmaurer merged commit c509a47 into 4.1 Jan 14, 2021
@normanmaurer normanmaurer added this to the 4.1.59.Final milestone Jan 14, 2021
@normanmaurer normanmaurer deleted the deploy branch January 14, 2021 17:08
ZzxyNn pushed a commit to ZzxyNn/netty that referenced this pull request Jan 29, 2021
Motivation:

We need to ensure we also build the modules we depend on as otherwise we may not be able to resolve the dependencies correctly

Modifications:

- Add -am when calling maven

Result:

Deployment works all the time
normanmaurer added a commit that referenced this pull request Mar 15, 2021
Motivation:

We need to ensure we also build the modules we depend on as otherwise we may not be able to resolve the dependencies correctly

Modifications:

- Add -am when calling maven

Result:

Deployment works all the time
raidyue pushed a commit to raidyue/netty that referenced this pull request Jul 8, 2022
Motivation:

We need to ensure we also build the modules we depend on as otherwise we may not be able to resolve the dependencies correctly

Modifications:

- Add -am when calling maven

Result:

Deployment works all the time
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

1 participant