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

Java 9 dies not provide sun.security.action.GetPropertyAction #26

Open
jpsacha opened this issue Jun 14, 2016 · 7 comments
Open

Java 9 dies not provide sun.security.action.GetPropertyAction #26

jpsacha opened this issue Jun 14, 2016 · 7 comments

Comments

@jpsacha
Copy link

jpsacha commented Jun 14, 2016

Running on Java 9 fails due to use of sun.security.action.GetPropertyAction that is not available in Java 9. The problem is with PNMImageReader line 93 and PNMImageWriter line 102 initializing lineSeparator.

@rleigh-codelibre
Copy link
Contributor

rleigh-codelibre commented Jun 19, 2017

@jpsacha I've opened a PR to fix this here: #41

If you'd like to give this a try, that would be much appreciated.

@rleigh-codelibre
Copy link
Contributor

The second PR is here: jai-imageio/jai-imageio-jpeg2000#21

@tomhood
Copy link

tomhood commented Aug 24, 2017

Any plan to fix this soon? For the suggested replacement, see https://bugs.openjdk.java.net/browse/JDK-8035808

@rleigh-codelibre
Copy link
Contributor

@tomhood Mainly the two open PRs need some review. Are any changes required in addition to what's in these PRs?

@tomhood
Copy link

tomhood commented Aug 24, 2017

@rleigh-codelibre Lol...sorry, had PR stuck in my mind as Personal Record (my running group mentions PR all the time), not Pull Request. :-) That makes more sense. I downloaded the master branch and it still had the GetPropertyAction issue. I guess I need to apply the individual commits that haven't made it into master yet or something like that. Haven't used git much at all. I see now that I can view the individual commits you made. I may just manually apply them to the master zip I downloaded. I'll try out the changes and let you know if I have any problems with them. Thanks.

@gexclaude
Copy link

Any plans to get this fixed?

@stain
Copy link
Member

stain commented Apr 11, 2018

Thanks to @rleigh-codelibre for the fix, his #41 is now merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants