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

[ELY-1398] Add support for public API compatibility check using japicmp #1092

Merged
merged 2 commits into from Feb 13, 2018

Conversation

darranl
Copy link
Contributor

@darranl darranl commented Feb 12, 2018

This contains Martin's PR but I have added a commit to split the packages into their own line so we can see future modifications more easily.

Using japicmp tool [1] profile was created to compare two jars of library:

last 1.1.x released jar (currently 1.1.7.Final)
currently SNAPSHOT version (e.g. 1.2.0.Beta12-SNAPSHOT)
Tool comes with lots of options to specify what to check [2]
In this initial version breakBuildBasedOnSemanticVersioning is turned
on. And set of public packages is specified.

This is alternative to #1066

Usage:
mvn verify -DskipTests -PcompatibilityCheck

[1] https://github.com/siom79/japicmp
[2] https://siom79.github.io/japicmp/MavenPlugin.html

mchoma and others added 2 commits February 12, 2018 14:59
tool

Using japicmp tool [1] 	profile was created to compare two jars
 - last 1.1.x released jar (currently 1.1.7.Final)
 - currently SNAPSHOT version (e.g. 1.2.0.Beta12-SNAPSHOT)

Tool comes with lots of options to specify what to check [2]
In this initial version breakBuildBasedOnSemanticVersioning is turned
on. And set of public packages is specified.

[1] https://github.com/siom79/japicmp
[2] https://siom79.github.io/japicmp/MavenPlugin.html
… the source history we can see when items were added more easily.
@wildfly-ci
Copy link

Linux - JDK10 EA Build 44 outcome was FAILURE using a merge of e3e6d5b
Summary: Tests passed: 931, ignored: 9; exit code 1 (new) Build time: 00:04:07

@wildfly-ci
Copy link

Linux - JDK9 Build 208 outcome was FAILURE using a merge of e3e6d5b
Summary: Tests passed: 931, ignored: 9; exit code 1 (new) Build time: 00:04:18

@hkalina hkalina added the +1 HK label Feb 12, 2018
@fjuma fjuma added the +1 FJ label Feb 13, 2018
@fjuma fjuma merged commit 1b6f690 into wildfly-security:master Feb 13, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
5 participants