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

CURA 11132 fix missing icon crash #17080

Merged
merged 2 commits into from
Oct 23, 2023
Merged

CURA 11132 fix missing icon crash #17080

merged 2 commits into from
Oct 23, 2023

Conversation

jellespijker
Copy link
Member

Description

Make sure that all relevant icons and pix-buf libs are packaged. Tested in on Manjaro, Mint 20.4, Debian and Ubuntu

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Printer definition file(s)
  • Translations

How Has This Been Tested?

By running this build on various systems: UltiMaker-Cura-5.6.0-alpha+cura_11132_dd2a80-linux-X64-AppImage

Test Configuration:

  • Operating System: Linux see above

Checklist:

This update removes various icon-theme exclusions from the AppImage build configuration and includes specific hicolor
icon files. It also adds new environment variables and path mappings for handling user data more efficiently within the
AppImage. This aims to optimize the packing process and improve the resultant AppImage's compatibility and performance.

Contributes to CURA-11132
The changes include adding several packages to the list of dependencies for the AppImage-builder, such as libgtk,
librsvg2, imagemagick and icon themes. Furthermore, adjustments were made to the runtime environment variables
(GDK_PIXBUF_MODULEDIR and GDK_PIXBUF_MODULE_FILE). Excluded file paths were also updated to exclude unnecessary
documentation files.

Contributes to CURA-11132
@casperlamboo casperlamboo merged commit 996bea1 into 5.5 Oct 23, 2023
9 checks passed
@jellespijker jellespijker deleted the CURA-11132 branch October 24, 2023 05:51
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