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

Use jetty-alpn-agent to simplify pom.xml #78

Merged
merged 1 commit into from
Jan 5, 2016

Conversation

trustin
Copy link
Member

@trustin trustin commented Jan 4, 2016

Motivation:

We had to add a new profile for each OpenJDK/OracleJDK release to make
Maven choose the correct alpn-boot.jar and npn-boot.jar. As a result,
our pom.xml has a large number of <profile/> sections.

Modifications:

Result:

Cleaner pom.xml

@trustin trustin added the cleanup label Jan 4, 2016
@trustin trustin added this to the 0.7.0.Final milestone Jan 4, 2016
@trustin
Copy link
Member Author

trustin commented Jan 4, 2016

@inch772 @anuraaga PTAL

@anuraaga
Copy link
Collaborator

anuraaga commented Jan 5, 2016

LGreatTM

Motivation:

We had to add a new profile for each OpenJDK/OracleJDK release to make
Maven choose the correct alpn-boot.jar and npn-boot.jar. As a result,
our pom.xml has a large number of `<profile/>` sections.

Modifications:

- Use jetty-alpn-agent, which chooses the correct alpn-boot.jar and
  npn-boot.jar automatically to remove all the nasty profile sections
  from pom.xml
  - Visit https://github.com/trustin/jetty-alpn-agent for more info

Result:

Cleaner pom.xml
@inch772
Copy link
Contributor

inch772 commented Jan 5, 2016

Cool! LGTM!! 👍

anuraaga added a commit that referenced this pull request Jan 5, 2016
Use jetty-alpn-agent to simplify pom.xml
@anuraaga anuraaga merged commit 2cf7e01 into line:master Jan 5, 2016
@trustin trustin deleted the use_jetty-alpn-agent branch January 6, 2016 05:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants