Skip to content

fix(main/openjdk): Build jsound #25152

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

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from
Draft

Conversation

alexytomi
Copy link

WIP because git is refusing to apply the patches with git am and thus I cannot regenerate them to make this a proper PR

Help appreciated.

Copy link
Contributor

@robertkirkman robertkirkman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@alexytomi fantastic job!!! I couldn't really figure out how to do this, but you did.

This build actually works to solve the problem as-is.

So that others understand how to test this PR: these are the commands that don't work currently in Termux, but will work and actually play audible sounds, if this PR is merged:

pkg upgrade
pkg install x11-repo
pkg install termux-x11-nightly xfce openjdk-21-x
export DISPLAY=:0 TERMUX_X11_XSTARTUP=xfce4-session
LIBGL_ALWAYS_SOFTWARE=1 termux-x11 &
wget https://www.oracle.com/technetwork/java/javasounddemo-150249.zip
unzip javasounddemo-150249.zip
cd JavaSoundDemo/
java -jar JavaSoundDemo.jar

@TomJo2000
Copy link
Member

I'll see if I can "properly" do a v2 of the patches modified in this PR, since we have a proper git format-patch generated patch series for the OpenJDK packages.

So having the proper headers and metadata on the patches would be good to maintain.

Since alsa-lib and alsa-plugin were introduced, building jsound has
since been possible, but no one did it for some reason.
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

Successfully merging this pull request may close these issues.

3 participants