Update revapi configuration

Use commons.bc.version for artifact to compare.

Use distinct ids in each configuration section.

Use ignore explicitly in the build configuration but not in reporting
(to allow the change to be reported).

Add missing 1.7 root to reporting section.

Author: aherbert

pom.xml

@@ -287,8 +287,14 @@
           </dependency>
         </dependencies>
         <configuration>
+          <oldArtifacts>
+            <artifact>${project.groupId}:${project.artifactId}:${commons.bc.version}</artifact>
+          </oldArtifacts>
           <analysisConfiguration>
-            <revapi.differences id="intentional-api-changes">
+            <revapi.differences id="intentional-api-changes-1_5">
+              <ignore>true</ignore>
+            </revapi.differences>
+            <revapi.differences id="intentional-api-changes-1_7">
               <ignore>true</ignore>
             </revapi.differences>
           </analysisConfiguration>
@@ -637,13 +643,17 @@ This is avoided by creating an empty directory when svn is not available.
             <artifactId>revapi-maven-plugin</artifactId>
             <version>${rng.revapi.version}</version>
             <configuration>
+              <oldArtifacts>
+                <artifact>${project.groupId}:${project.artifactId}:${commons.bc.version}</artifact>
+              </oldArtifacts>
               <!-- For reporting do not ignore the "intentional-api-changes" so they are included -->
               <analysisConfigurationFiles>
                 <configurationFile>
                   <path>${rng.parent.dir}/src/conf/revapi/api-changes.json</path>
                   <roots>
                     <!-- Path(s) to configuration in the JSON config file -->
                     <root>1.5</root>
+                    <root>1.7</root>
                   </roots>
                 </configurationFile>
               </analysisConfigurationFiles>

src/conf/revapi/api-changes.json

@@ -2,7 +2,7 @@
   "1.5": [
     {
       "extension": "revapi.differences",
-      "id": "intentional-api-changes",
+      "id": "intentional-api-changes-1_5",
       "configuration": {
         "differences": [
           {
@@ -11,7 +11,6 @@
             "justification": "Abstract method added to enum; all implementations are within this class. This is an internal package with no compatibility enforcement."
           },
           {
-            "ignore": true,
             "code": "java.class.externalClassExposedInAPI",
             "new": "interface org.apache.commons.rng.SplittableUniformRandomProvider",
             "justification": "Split support was added to the client API and can be used by other modules."
@@ -23,11 +22,10 @@
   "1.7": [
     {
       "extension": "revapi.differences",
-      "id": "intentional-api-changes",
+      "id": "intentional-api-changes-1_7",
       "configuration": {
         "differences": [
           {
-            "ignore": true,
             "code": "java.class.externalClassExposedInAPI",
             "new": "interface org.apache.commons.rng.ArbitrarilyJumpableUniformRandomProvider",
             "justification": "Arbitrary jump support was added to the client API and can be used by other modules."