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

add arm64 and armv7l arches to linux CI #2187

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

theofficialgman
Copy link

@theofficialgman theofficialgman commented Jan 15, 2024

Known issues:
3rd party library is 4+ years unmaintained and does not have ARM64 or ARMhf scrot native binaries on linux https://www.npmjs.com/package/electron-color-picker

electron-color-picker should be moved away from anyway as it is not wayland compatible which is the current default window system on linux desktops in 2024

you will likely want to move to gcolor3 as it seems to work well for me on wayland and x11 due to using the xdg desktop portals for obtaining color information https://www.hjdskes.nl/projects/gcolor3/

I am not actually sure what this library is being used for in blockbench as the "pick screen color" option in the paint section does work on ARM64 xorg linux in testing but not in wayland on any architecture.

theofficialgman added a commit to Botspot/pi-apps that referenced this pull request Jan 15, 2024
based on upstream PR JannisX11/blockbench#2187

disable updater until upstream PR has merged and downloads are switched to their releases
@theofficialgman
Copy link
Author

@JannisX11 thoughts? I would have expected some response by now.

@txtsd
Copy link

txtsd commented May 10, 2024

@JannisX11 Getting rid of electron-color-picker is an excellent idea. Please let us know what kind of feedback you need. There are a lot of Linux users here.

@txtsd
Copy link

txtsd commented Nov 2, 2024

Gentle nudge~

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.

2 participants