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

🆕 [ARM/ARM64 users] Feedback on new packages #903

Closed
shiftkey opened this issue Jul 11, 2023 · 18 comments
Closed

🆕 [ARM/ARM64 users] Feedback on new packages #903

shiftkey opened this issue Jul 11, 2023 · 18 comments
Labels

Comments

@shiftkey
Copy link
Owner

shiftkey commented Jul 11, 2023

Please submit any feedback you have here about the new ARM and ARM64 installers.

Latest release: 3.2.7-linux2

@shiftkey shiftkey changed the title [ARM/ARM64 users] Feedback on test packages 🆕 [ARM/ARM64 users] Feedback on test packages Jul 11, 2023
@shiftkey shiftkey pinned this issue Jul 11, 2023
@isuro
Copy link

isuro commented Jul 13, 2023

I'm using it via X11 & Docker, so some limitations apply from not having any real desktop environment or browser, but generally working great.

(Ubuntu 20.04.5 LTS, installed via .deb, current XQuartz & Docker, Macbook Pro M1)

@md-shamim-ahmad

This comment was marked as off-topic.

@shiftkey shiftkey changed the title 🆕 [ARM/ARM64 users] Feedback on test packages 🆕 [ARM/ARM64 users] Feedback on new packages Jul 29, 2023
@Lunarequest
Copy link

the flatpak now builds for aarch64 as well(flathub doesn't support arm so we haven't build it for it)

@isuro
Copy link

isuro commented Aug 8, 2023

Launching from cli no longer works ☹️.
As of 3.2.7-linux2, if I comment out the redirect to /dev/null in /usr/lib/github-desktop/resources/app/static/github, I get:

$ github
ERR! Desktop command line interface not currently supported on platform linux
Error: Desktop command line interface not currently supported on platform linux
    at t (/usr/lib/github-desktop/resources/app/cli.js:1:26473)
    at Object.handler (/usr/lib/github-desktop/resources/app/cli.js:1:26160)
    at h (/usr/lib/github-desktop/resources/app/cli.js:1:28549)
    at e (/usr/lib/github-desktop/resources/app/cli.js:1:28693)
    at /usr/lib/github-desktop/resources/app/cli.js:1:28764
    at /usr/lib/github-desktop/resources/app/cli.js:1:28783
    at Object.<anonymous> (/usr/lib/github-desktop/resources/app/cli.js:1:28808)
    at Module._compile (node:internal/modules/cjs/loader:1241:14)
    at Module._extensions..js (node:internal/modules/cjs/loader:1296:10)
    at Module.load (node:internal/modules/cjs/loader:1096:32)

(Ubuntu 20.04.5 LTS, installed via .deb, current XQuartz & Docker, Macbook Pro M1)

@shiftkey
Copy link
Owner Author

shiftkey commented Aug 8, 2023

@isuro which architecture are you installing? That appears to be this error:

throw new Error(
`Desktop command line interface not currently supported on platform ${process.platform}`
)

But I'm puzzled about how it's reaching there because the __LINUX__ global that we set when building also uses process.platform:

__LINUX__: process.platform === 'linux',

@isuro
Copy link

isuro commented Aug 8, 2023

Installed via GitHubDesktop-linux-arm64-3.2.7-linux2.deb

@isuro
Copy link

isuro commented Aug 8, 2023

And indeed:

$ npx ts-node
> process.platform
'linux'

@isuro
Copy link

isuro commented Aug 8, 2023

According to the source map, the case wasn't built in somehow:
Screenshot 2023-08-08 at 14 16 05

@shiftkey
Copy link
Owner Author

shiftkey commented Aug 8, 2023

Cool, that gives me some context to dig into @isuro - thanks for poking at the package contents...

@isuro
Copy link

isuro commented Aug 8, 2023

Ping me if you have a new .deb you'd like me to try out!

@Bezierr
Copy link

Bezierr commented Aug 16, 2023

This is not about the ARM installer but about AMD64.deb .
Issue: Geany is not supported as external editor anymore. This worked up to and including 3.2.5.

(System: Mint 21.1 Mate)

@shiftkey
Copy link
Owner Author

@Bezierr

Issue: Geany is not supported as external editor anymore. This worked up to and including 3.2.5.

This was removed in #894 because it doesn't support opening a directory, and I was receiving bug reports about it #871

@Bezierr
Copy link

Bezierr commented Aug 18, 2023

@shiftkey
Ok. I never tried opening a directory but only individual files, which works (worked) as it's supposed to. So I guess I'll have to stick with 3.2.5 .

Anyway, thanks a lot for providing this for Linux -- really much appreciated!

@shiftkey
Copy link
Owner Author

@isuro just letting you know that the newest release (3.2.8-linux1) has fixed this issue with the github command line helper.

@sabrehagen
Copy link

My environment uses OpenGL 2.1. The UI renders without some elements. Screenshot for reference:

image

Terminal log:

[1200283:1014/012916.918090:ERROR:shared_context_state.cc(77)] Skia shader compilation error
------------------------
#version 100

#extension GL_EXT_blend_func_extended : require
precision mediump float;
precision mediump sampler2D;
uniform sampler2D uTextureSampler_0_S0;
varying highp vec2 vTextureCoords_S0;
varying highp float vTexIndex_S0;
varying mediump vec4 vinColor_S0;
void main() {
    mediump vec4 outputColor_S0;
    outputColor_S0 = vinColor_S0;
    mediump vec4 texColor;
    {
        texColor = texture2D(uTextureSampler_0_S0, vTextureCoords_S0, -0.5);
    }
    mediump vec4 outputCoverage_S0 = texColor;
    {
        outputCoverage_S0.w = max(max(outputCoverage_S0.x, outputCoverage_S0.y), outputCoverage_S0.z);
        gl_SecondaryFragColorEXT = outputColor_S0.w * outputCoverage_S0;
        gl_FragColor = outputColor_S0 * outputCoverage_S0;
    }
}

Errors:
0:1(12): warning: extension `GL_ARB_gpu_shader5' unsupported in fragment shader
0:2(12): warning: extension `GL_EXT_gpu_shader5' unsupported in fragment shader
0:3(1): error: `out' qualifier in declaration of `webgl_SecondaryFragColor' only valid for function parameters in GLSL 1.10
0:0(0): error: fragment shader writes to both `gl_FragColor' and `webgl_SecondaryFragColor'

@Lunarequest
Copy link

I've had similar output when my nvidia driver updated. Maybe its the same for you? You could try disabling gpu bug workarounds?

@sabrehagen
Copy link

I've had similar output when my nvidia driver updated. Maybe its the same for you? You could try disabling gpu bug workarounds?

Yes, using GITHUB_DESKTOP_DISABLE_HARDWARE_ACCELERATION=1 github-desktop worked. Thank you!

@shiftkey shiftkey added the meta label Nov 27, 2023
@shiftkey
Copy link
Owner Author

Closing this out as things appear to be stable, and I suspect folks are fine to open fresh issues for whatever support questions they have

@shiftkey shiftkey unpinned this issue Jan 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

7 participants
@shiftkey @isuro @sabrehagen @Lunarequest @Bezierr @md-shamim-ahmad and others