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

Re-enable the Windows installer for GTK4 #622

Merged
merged 10 commits into from
Dec 19, 2023

Conversation

cameronwhite
Copy link
Member

  • Generate output files under a build folder, rather than just bin. This allows copying the GTK library dependencies to sibling share and lib folders.
  • Copy the GTK library dependencies to the build folder. This allows running Pinta for dev workflows without MSYS in the path. Previously for GTK3, this implicitly used the GTK libraries bundled with GtkSharp
  • Fix a regression in the GTK4 port which caused the non-native file picker to be used on Windows
  • Update the CI builds and installer for GTK4 and the output path changes

For Windows the plan is to copy the MSYS GTK libs to the build folder so Pinta can be easily launched without manipulating the path, so this is most easily done by adding sibling `lib` and `share` folders.

This doesn't affect the output of 'dotnet publish', so the existing Linux and macOS install scripts should still work.
This allows Pinta to successfully run without MSYS in the path.
- The CI machines seem to have MinGW installed under a non-standard prefix, so this also must be overridden
- Avoid warning from using the -o parameter when publishing
- Merge the locale folders, in addition to icons
- Revert changes to the icon lookup path, which weren't actually necessary since the bundled GTK libraries include `../share` in their search path automatically
- Fix missing libraries from the latest MSYS packages
Similar to the macOS installer, we need to update the index.theme to include the expected folders.
This avoids having an existing `icon-theme.cache` for the `hicolor` theme which is out of date vs the updated index.theme we install
@cameronwhite cameronwhite merged commit 715db6e into master Dec 19, 2023
9 checks passed
@cameronwhite cameronwhite deleted the feature/windows-installer branch December 19, 2023 18:02
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.

1 participant