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

Fix appimage build on Linux #742

Merged
merged 1 commit into from
Oct 10, 2023
Merged

Fix appimage build on Linux #742

merged 1 commit into from
Oct 10, 2023

Conversation

tk-nguyen
Copy link
Collaborator

@tk-nguyen tk-nguyen commented Oct 3, 2023

Need some more testing with different distros: Fedora, Debian, Ubuntu, Centos, Arch, ...

Closes: #674

@tk-nguyen tk-nguyen temporarily deployed to PR-742 October 3, 2023 10:54 — with GitHub Actions Inactive
@tk-nguyen tk-nguyen temporarily deployed to PR-742 October 3, 2023 10:57 — with GitHub Actions Inactive
@github-actions
Copy link

github-actions bot commented Oct 3, 2023

This PR has been deployed to https://linagora.github.io/twake-on-matrix/742

@tk-nguyen tk-nguyen force-pushed the fix-appimage-builds branch from f96df78 to 3be0cca Compare October 4, 2023 09:40
@tk-nguyen tk-nguyen changed the title WIP: Fix appimage builds Fix appimage build on Linux Oct 4, 2023
@tk-nguyen tk-nguyen marked this pull request as ready for review October 4, 2023 09:58
Fixed library name and removed unneeded packages

Use azure mirror, skip tests

Because GH Actions docker is flaky

Use png file for icon

Added additional libraries

Correct the distro codename
@hoangdat
Copy link
Member

hoangdat commented Oct 9, 2023

is it ok for merge? @tk-nguyen

@tk-nguyen
Copy link
Collaborator Author

is it ok for merge? @tk-nguyen

Should be good.

@hoangdat hoangdat merged commit 72129f2 into main Oct 10, 2023
3 checks passed
@hoangdat hoangdat deleted the fix-appimage-builds branch October 10, 2023 09:14
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