{"action":"created","issue":{"url":"https://api.github.com/repos/eclipse-ee4j/jaxrs-api/issues/686","repository_url":"https://api.github.com/repos/eclipse-ee4j/jaxrs-api","labels_url":"https://api.github.com/repos/eclipse-ee4j/jaxrs-api/issues/686/labels{/name}","comments_url":"https://api.github.com/repos/eclipse-ee4j/jaxrs-api/issues/686/comments","events_url":"https://api.github.com/repos/eclipse-ee4j/jaxrs-api/issues/686/events","html_url":"https://github.com/eclipse-ee4j/jaxrs-api/pull/686","id":374608717,"node_id":"MDExOlB1bGxSZXF1ZXN0MjI2MjcxMDAz","number":686,"title":"Fixed spec-version properties and Travis config for 2.1 releases","user":{"login":"chkal","id":159245,"node_id":"MDQ6VXNlcjE1OTI0NQ==","avatar_url":"https://avatars2.githubusercontent.com/u/159245?v=4","gravatar_id":"","url":"https://api.github.com/users/chkal","html_url":"https://github.com/chkal","followers_url":"https://api.github.com/users/chkal/followers","following_url":"https://api.github.com/users/chkal/following{/other_user}","gists_url":"https://api.github.com/users/chkal/gists{/gist_id}","starred_url":"https://api.github.com/users/chkal/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/chkal/subscriptions","organizations_url":"https://api.github.com/users/chkal/orgs","repos_url":"https://api.github.com/users/chkal/repos","events_url":"https://api.github.com/users/chkal/events{/privacy}","received_events_url":"https://api.github.com/users/chkal/received_events","type":"User","site_admin":false},"labels":[{"id":840843097,"node_id":"MDU6TGFiZWw4NDA4NDMwOTc=","url":"https://api.github.com/repos/eclipse-ee4j/jaxrs-api/labels/project","name":"project","color":"0b258e","default":false}],"state":"closed","locked":false,"assignee":{"login":"chkal","id":159245,"node_id":"MDQ6VXNlcjE1OTI0NQ==","avatar_url":"https://avatars2.githubusercontent.com/u/159245?v=4","gravatar_id":"","url":"https://api.github.com/users/chkal","html_url":"https://github.com/chkal","followers_url":"https://api.github.com/users/chkal/followers","following_url":"https://api.github.com/users/chkal/following{/other_user}","gists_url":"https://api.github.com/users/chkal/gists{/gist_id}","starred_url":"https://api.github.com/users/chkal/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/chkal/subscriptions","organizations_url":"https://api.github.com/users/chkal/orgs","repos_url":"https://api.github.com/users/chkal/repos","events_url":"https://api.github.com/users/chkal/events{/privacy}","received_events_url":"https://api.github.com/users/chkal/received_events","type":"User","site_admin":false},"assignees":[{"login":"chkal","id":159245,"node_id":"MDQ6VXNlcjE1OTI0NQ==","avatar_url":"https://avatars2.githubusercontent.com/u/159245?v=4","gravatar_id":"","url":"https://api.github.com/users/chkal","html_url":"https://github.com/chkal","followers_url":"https://api.github.com/users/chkal/followers","following_url":"https://api.github.com/users/chkal/following{/other_user}","gists_url":"https://api.github.com/users/chkal/gists{/gist_id}","starred_url":"https://api.github.com/users/chkal/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/chkal/subscriptions","organizations_url":"https://api.github.com/users/chkal/orgs","repos_url":"https://api.github.com/users/chkal/repos","events_url":"https://api.github.com/users/chkal/events{/privacy}","received_events_url":"https://api.github.com/users/chkal/received_events","type":"User","site_admin":false}],"milestone":{"url":"https://api.github.com/repos/eclipse-ee4j/jaxrs-api/milestones/9","html_url":"https://github.com/eclipse-ee4j/jaxrs-api/milestone/9","labels_url":"https://api.github.com/repos/eclipse-ee4j/jaxrs-api/milestones/9/labels","id":3743847,"node_id":"MDk6TWlsZXN0b25lMzc0Mzg0Nw==","number":9,"title":"2.1.3","description":"JAX RS 2.1.2 + bug fixes + Dependency Maven coordinate changes\r\n\r\n**This release was requested by the PMC to ensure all dependencies will be taken from the `jakara` namespace.**","creator":{"login":"mkarg","id":1701815,"node_id":"MDQ6VXNlcjE3MDE4MTU=","avatar_url":"https://avatars2.githubusercontent.com/u/1701815?v=4","gravatar_id":"","url":"https://api.github.com/users/mkarg","html_url":"https://github.com/mkarg","followers_url":"https://api.github.com/users/mkarg/followers","following_url":"https://api.github.com/users/mkarg/following{/other_user}","gists_url":"https://api.github.com/users/mkarg/gists{/gist_id}","starred_url":"https://api.github.com/users/mkarg/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mkarg/subscriptions","organizations_url":"https://api.github.com/users/mkarg/orgs","repos_url":"https://api.github.com/users/mkarg/repos","events_url":"https://api.github.com/users/mkarg/events{/privacy}","received_events_url":"https://api.github.com/users/mkarg/received_events","type":"User","site_admin":false},"open_issues":3,"closed_issues":2,"state":"open","created_at":"2018-10-16T17:31:08Z","updated_at":"2018-11-06T16:17:41Z","due_on":"2018-11-05T08:00:00Z","closed_at":null},"comments":18,"created_at":"2018-10-27T05:27:34Z","updated_at":"2018-11-06T17:52:14Z","closed_at":"2018-11-06T16:17:27Z","author_association":"CONTRIBUTOR","pull_request":{"url":"https://api.github.com/repos/eclipse-ee4j/jaxrs-api/pulls/686","html_url":"https://github.com/eclipse-ee4j/jaxrs-api/pull/686","diff_url":"https://github.com/eclipse-ee4j/jaxrs-api/pull/686.diff","patch_url":"https://github.com/eclipse-ee4j/jaxrs-api/pull/686.patch"},"body":"I had a deeper look at the OSGi manifest issues described in #681. It looks like the wrong version numbers were created because the `final` profile wasn't active during the release. Unfortunately activating this profile currently fails the build with:\r\n\r\n```\r\n[INFO] --- spec-version-maven-plugin:1.5:check-module (default) @ jakarta.ws.rs-api ---\r\n\r\n[ jakarta.ws.rs:jakarta.ws.rs-api:2.1.2 ]\r\n{ groupIdPrefix=jakarta. spec-version=2.2 apiPackage=javax.ws.rs API final spec-impl-version=2.1.2 }\r\n- WARNING: spec-impl-version (2.1.2) must start with JCP spec-version number (2.2)\r\n```\r\n\r\nThe root cause for this seems to be that the Maven properties `last.final.spec.version` and `next.final.spec.version` aren't set correctly. As the `EE4J_8` branch is a maintenance branch for JAX-RS 2.1, the properties need to point to different versions.\r\n\r\nAfter applying this change, running the Maven build with `-Pfinal` seems to create the correct version numbers in the OSGi manifest:\r\n\r\n```\r\nBundle-Version: 2.1.2\r\nImplementation-Version: 2.1.2\r\nSpecification-Version: 2.1\r\n```\r\n\r\nAs this PR doesn't change the JAX-RS API in any way and just affects the build, I would like to apply the rule for a shorter review period of only one week. "},"comment":{"url":"https://api.github.com/repos/eclipse-ee4j/jaxrs-api/issues/comments/436345851","html_url":"https://github.com/eclipse-ee4j/jaxrs-api/pull/686#issuecomment-436345851","issue_url":"https://api.github.com/repos/eclipse-ee4j/jaxrs-api/issues/686","id":436345851,"node_id":"MDEyOklzc3VlQ29tbWVudDQzNjM0NTg1MQ==","user":{"login":"mkarg","id":1701815,"node_id":"MDQ6VXNlcjE3MDE4MTU=","avatar_url":"https://avatars2.githubusercontent.com/u/1701815?v=4","gravatar_id":"","url":"https://api.github.com/users/mkarg","html_url":"https://github.com/mkarg","followers_url":"https://api.github.com/users/mkarg/followers","following_url":"https://api.github.com/users/mkarg/following{/other_user}","gists_url":"https://api.github.com/users/mkarg/gists{/gist_id}","starred_url":"https://api.github.com/users/mkarg/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mkarg/subscriptions","organizations_url":"https://api.github.com/users/mkarg/orgs","repos_url":"https://api.github.com/users/mkarg/repos","events_url":"https://api.github.com/users/mkarg/events{/privacy}","received_events_url":"https://api.github.com/users/mkarg/received_events","type":"User","site_admin":false},"created_at":"2018-11-06T17:52:14Z","updated_at":"2018-11-06T17:52:14Z","author_association":"MEMBER","body":"@chkal Darn! You're right, I actually missed the last line of the original description. Sorry, I'm getting old. Maybe we should add the rule to have that line written in 72 point fonts always. ;-) Anyways, as I wrote, I'm actually +1 for fast-tracking this."}}
{"actor":{"display_login":"mkarg"}}