Skip to content

resolve issue #1958 (jmf library coming with j-ogg-all is not required)#1959

Closed
Ali-RS wants to merge 1 commit intojMonkeyEngine:masterfrom
Ali-RS:fix-issue-1958
Closed

resolve issue #1958 (jmf library coming with j-ogg-all is not required)#1959
Ali-RS wants to merge 1 commit intojMonkeyEngine:masterfrom
Ali-RS:fix-issue-1958

Conversation

@Ali-RS
Copy link
Member

@Ali-RS Ali-RS commented Feb 20, 2023

This PR excludes javax.media:jmf library from jme3-jogg module as it is not required for decoding ogg files in JME.

Fix issue #1958

@Ali-RS Ali-RS added enhancement buildscript An issue with the buildscript labels Feb 20, 2023
@Ali-RS Ali-RS added this to the Future Release milestone Feb 20, 2023
@Ali-RS Ali-RS linked an issue Feb 20, 2023 that may be closed by this pull request
@stephengold
Copy link
Member

stephengold commented Feb 20, 2023

I plan to release a new library "j-ogg-vorbis" which will be like "j-ogg-all" but will exclude the flac and Theora decoders. It won't depend on JMF. That will make this PR redundant, so perhaps you should hold off on this...

@Ali-RS
Copy link
Member Author

Ali-RS commented Feb 20, 2023

Ok, no problem. I will discard this PR then and will submit a new one that makes use of j-ogg-vorbis.

Thanks

@Ali-RS
Copy link
Member Author

Ali-RS commented Feb 21, 2023

Closing this for #1962

@Ali-RS Ali-RS closed this Feb 21, 2023
@Ali-RS Ali-RS removed this from the Future Release milestone Feb 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

buildscript An issue with the buildscript enhancement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

It seems jmf library coming with j-ogg-all is not required?

2 participants