-
-
Notifications
You must be signed in to change notification settings - Fork 301
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
[Bug]: Red and blue color channels swapped on build #523 on Exynos #495
Comments
Does it work fine with termux's own xfce4? |
No, it also has the same issue (just tested). Here's a screenshot of what the behavior is (Build #523): And here's a screenshot of what the behavior should be (Build #522): |
Post logs. |
From both builds or just the one that doesn't work? |
The latest one. |
Here:
|
Yep ... I'm also having this issue . Also on an S22 |
Confirmed on S22 Ultra (Exynos) |
I've been playing with the source, bumped the SDK, changed some code and somehow got the color swap issue fixed. The problem is, I'm not sure what made the difference. I'm also a total greenhorn in this domain. Will report back if I have more details. |
hello. Even on S24+ if |
@hansm629 I wish I could just send a pr with the changes, but a quick diff didn't give me anything to work with. |
thank you! :) Thanks for solving it! ah! Exynos2400 Xclipse 940 Both show the worst performance and compatibility.
For Exynos2200 Xclipse 920 |
Glad it works for you too! |
I'm using S24, have the same issue. https://drive.google.com/file/d/1WOb2T3PUpCzMvwsIq9EOP8D6kMQOS0S1/view?usp=drivesdk didn't seem to resolve it. I'm using native XFCE4. For comparison, top half shows a photo in GIMP using tigervnc server & MultiVNC client (expected colours), bottom half shows the same using termux-X11 (unexpected colours). |
This is PRoot-Distro Debian Sid installed on Galaxy S24+ (SM-S926N). I have XFCE4 session colors displayed correctly in the APK shared above. |
@arenddeboer @hansm629 Using https://github.com/termux/termux-x11/releases/download/nightly/app-universal-debug.apk for native, proot-distro debian & proot-distro ubuntu: Using https://drive.google.com/file/d/1WOb2T3PUpCzMvwsIq9EOP8D6kMQOS0S1/view?usp=drivesdk for native, proot-distro debian & proot-distro ubuntu: |
After 6fcf1b7 you can use |
@twaik thanks for the fix. Just a minor comment that the README.md line 59 mentioned the wrong flag. It should be For some reason some devices show screen with swapped colours, in this case you should pass |
Are you looking at readme of master branch or the readme of commit itself? |
Apologies I looked at the commit at 6fcf1b7. Noted the main page at https://github.com/termux/termux-x11/ is showing correct option. |
@lengors @sfantu @arenddeboer @hansm629 @whjou can you please confirm problem existing on the latest build? With |
@twaik When I install the latest build from https://github.com/termux/termux-x11/releases, -force-bgra only doesn't work for me, but -legacy-drawing shows the colors correctly. Without needing -force-bgra. Or am I doing something wrong, I was expecting to need the option -force-bgra for the colors to show up correctly. |
@arenddeboer does it work without flags |
@twaik |
Ok, so that means texture format is fine, but drawing it on actual surface with shaders is problematic. I'll check what can be done. |
|
Working here ... with -legacy-drawing ... haven't test without yet. |
I fixed one little thing so termux may work on Exynos without |
Tested, it works on my Samsung Galaxy S24 without the
|
yep working here without the -legacy-drawing flag. |
Problem description
I own a Galaxy S22+ with an Exynos CPU and with build #523 from 20231006 (last successful build from master branch) referent to commit 84901c2, the red and blue color channels seem to be swapped (red colors become blue and vice-versa, other tones of colors like dark blue-purple ish becomes orange, etc). With the build previous to that one though, there are no issues (#522 from 20231002 referent to commit 4acf4b2).
Some notes:
-legacy-drawing
flag as without it I get a black screen (this is true independently of which build I choose)What steps will reproduce the bug?
x11-repo
and thetermux-x11
deb package referent to the giventermux-x11
app (from termux-companion packages)termux.properties
proot-distro
xfce4
desktop environment inside proottermux-x11 :0 -legacy-drawing
env DISPLAY=:0 dbus-launch --exit-with-session startxfce4
What is the expected behavior?
Red and blue color channels should be swapped as to give the correct color on, well, anything really.
The text was updated successfully, but these errors were encountered: