Skip to content

Conversation

@xarvex
Copy link
Member

@xarvex xarvex commented Apr 22, 2025

Summary

Temporary solution for #915, long term should involve a proper upgrade on PySide6. This only applies the override if a later version of PySide6 is found. Again, this should only be temporary, my poor machine took a beating compiling this; does give some motivation for exploring Cachix further, however.

As far as I have been able to tell, Qt does not need any override treatment.

Tasks Completed

  • Platforms Tested:
    • Windows x86
    • Windows ARM
    • macOS x86
    • macOS ARM
    • Linux x86
    • Linux ARM
  • Tested For:
    • Basic functionality
    • PyInstaller executable

@CyanVoxel CyanVoxel added Type: Installation Installing, building, and/or launching the program Priority: High An important issue requiring attention System: Linux For Linux/BSD distributions Status: Mergeable The code is ready to be merged labels Apr 24, 2025
@CyanVoxel CyanVoxel moved this to 🍃 Pending Merge in TagStudio Development Apr 24, 2025
@CyanVoxel CyanVoxel modified the milestone: Alpha v9.5.3 Apr 24, 2025
@CyanVoxel CyanVoxel changed the base branch from main to Alpha-v9.5.3 May 5, 2025 19:48
@xarvex xarvex force-pushed the nix-pyside-override branch from 9d6ab08 to 3220526 Compare May 6, 2025 05:22
@xarvex xarvex force-pushed the nix-pyside-override branch from 3220526 to 08dad92 Compare May 6, 2025 05:26
@xarvex xarvex merged commit 1e783a5 into Alpha-v9.5.3 May 6, 2025
5 checks passed
@github-project-automation github-project-automation bot moved this from 🍃 Pending Merge to ✅ Done in TagStudio Development May 6, 2025
@xarvex xarvex deleted the nix-pyside-override branch May 6, 2025 23:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Priority: High An important issue requiring attention Status: Mergeable The code is ready to be merged System: Linux For Linux/BSD distributions Type: Installation Installing, building, and/or launching the program

Projects

Status: ✅ Done

Development

Successfully merging this pull request may close these issues.

3 participants