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.

Sign up for GitHub

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

Jump to bottom

Java 9 Requirement in GSON Library #1608

Closed
amitjoy opened this issue Nov 6, 2019 · 5 comments
Closed

Java 9 Requirement in GSON Library #1608

amitjoy opened this issue Nov 6, 2019 · 5 comments

Comments

@amitjoy
Copy link

amitjoy commented Nov 6, 2019 •
edited

The newest version 2.8.6 comprises the following in its MANIFEST.MF

Require-Capability: osgi.ee;filter:="(&(osgi.ee=JavaSE)(version=9.0))"

This prevents the bundle to be installed in Java 8 execution environment. Is there really any Java 9 dependency in this version at all? I hope not since the patch version is only bumped up and the 2.8.5 version is compatible with Java 8.
@inder123
Copy link
Collaborator

inder123 commented Nov 6, 2019

Yes, this is a bug that we will fix soon. Can you test the latest snapshot to see if the issue is gone?

@elharo
Copy link
Contributor

elharo commented Nov 7, 2019

This is a dupe of #1601. It's fixed at head. When is the next release with this fix scheduled?

@amitjoy
Copy link
Author

amitjoy commented Nov 7, 2019

@inder123 I just noticed. It has already been fixed.

@chanseokoh
Copy link
Member

#1601 (and this one as a dup) fixed by #1603. Please close this.

@inder123 inder123 closed this as completed Nov 8, 2019
odl-github pushed a commit to opendaylight/odlparent that referenced this issue Dec 10, 2019
@rovarga
Revert "Update GSON to 2.8.6"
da949f3 
This reverts commit edbeb3e, as
GSON 2.8.6 seems to have quite a bit of trouble:

google/gson#1601
google/gson#1602
google/gson#1608
google/gson#1630

Let's wait for the dust to settle first.

Change-Id: I7d0c7b671a31bf9bf3a1861cc56b009bd0f724b6
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
@markwoon
Copy link

Can we please get a new release to fix this? Pretty please? Before Christmas?

@jsimomaa jsimomaa mentioned this issue Oct 27, 2020
Merged
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@markwoon @elharo @inder123 @chanseokoh @amitjoy