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

Update README.md docker build instructions in case of missing JDK from container image #507

Open
wants to merge 1,362 commits into
base: main
Choose a base branch
from

Conversation

andreaippo
Copy link

@andreaippo andreaippo commented Feb 4, 2025

This PR aims at addressing two issues I faced when following the docker-based build instructions:

  1. the configure script fails because the recommended container image does not come with a JDK (or the script can't find it? A whereis java didn't return anything, anyway). I solved this by installing a JDK via SDKMAN!, if that's fine by you. I'm an avid user of SDMAN! as it installs in user space and will work on any distro/system.
  2. permission issues when trying to run commands inside the container (couldn't even cd into the dir). This is because my host filesystem uses SELinux, therefore the -v mount is not enough to grant rights on the host filesystem to the container. AFAIK this can be solved by suffixing :Z. This switch will be ignored by non-SELinux systems, so AFAIK it doesn't harm to have it.

Error mentioned at 1. is:

checking for javac... [not found]
checking for java... [not found]
configure: Could not find a valid Boot JDK. OpenJDK distributions are available at http://jdk.java.net/.
configure: This might be fixed by explicitly setting --with-boot-jdk
configure: error: Cannot continue
configure exiting with result code 1

JB-Dmitry and others added 30 commits February 4, 2025 01:03
prevent using a pointer to destroyed surface

(cherry picked from commit ba1b5ce)
(cherry picked from commit 20b2eeb)
A partial solution. Cases not still covered:
* Alt+tab from the app and back should keep popup focused if it was focused initially
* Mouse clicks between popup and owner should transfer focus as expected

(cherry picked from commit 903231d)
Added --with-wayland-lib option to provide custom library path

(cherry picked from commit cdc2b1b)
1. Update dockerfile to checkout Vulkan headers
2. Fix --with-vulkan-include configure option

(cherry picked from commit 67c8c4d)
When popup's parent is also its top-level window, use that instead of
null

(cherry picked from commit 47d484b)
Also added a flush-to-server command following each Wayland request

(cherry picked from commit 58d2d42)
Also fixed the keyboard repeat manager so that it does not prevent
application from shutting down.
Also improved fullscreen support.

(cherry picked from commit a36c960)
Changed the logging level for such messages to FINE

(cherry picked from commit e946179)
fix activation not working on Ubuntu 23 (mutter 45.2), after a mouse button has been pressed in the originally active window

(cherry picked from commit 07fa181)
Refactored AWTAccessor code

(cherry picked from commit acbd18f)
-correctly positioning SplashScreen on multiple monitors;
-correctly correctly handles cases of SplashScreen with gif with transparent parts;
-refactoring;

(cherry picked from commit 7d750cf)
Improved rendering performance by
* reducing memory copy and making it more efficient,
* tying the next frame display to the frame event from Wayland,
  which dramatically reduces load for very quick Swing apps,
* limiting the number of buffers to 2.

(cherry picked from commit e625eec)
Cancel the frame callback when hiding a window.

(cherry picked from commit fd4ba3f)
Allow to associate the application's window with .desktop file with icon
and other info with -Dawt.app.id=... (DBus application name similar to
WM_CLASS in X

(cherry picked from commit 5901915)
@vprovodin vprovodin force-pushed the main branch 3 times, most recently from 71b988a to d20629a Compare February 20, 2025 00:03
@vprovodin vprovodin force-pushed the main branch 6 times, most recently from 4e2cce6 to c6bd451 Compare February 26, 2025 00:03
@vprovodin vprovodin force-pushed the main branch 4 times, most recently from d2c10f5 to 7ee2b57 Compare February 28, 2025 13:51
@vprovodin vprovodin force-pushed the main branch 2 times, most recently from 511323f to 428a663 Compare March 2, 2025 01:12
@vprovodin vprovodin force-pushed the main branch 2 times, most recently from c0c6e71 to ca26f47 Compare March 5, 2025 00:03
@vprovodin vprovodin force-pushed the main branch 4 times, most recently from 521fcb5 to cfa81d2 Compare March 10, 2025 00:03
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.