Skip to content

Restore Pyside6 tests, enable long filename support#68

Merged
mhsmith merged 3 commits intomainfrom
long-path-support
Jul 11, 2025
Merged

Restore Pyside6 tests, enable long filename support#68
mhsmith merged 3 commits intomainfrom
long-path-support

Conversation

@freakboy3742
Copy link
Member

@freakboy3742 freakboy3742 commented Jul 10, 2025

beeware/briefcase#2355 upgrades the WiX version to WiX 5, which should no longer have the the path length restriction - so we should be able to restore pyside6 tests to the Windows tests.

The app template isn't affected by the long path problem in CI, but the pyside6 tests were disabled. This restores the pyside6 tests.

Includes the \\?\ long path name support fix identified by beeware/briefcase-windows-VisualStudio-template#62

Refs beeware/briefcase#948.

PR Checklist:

  • All new features have been tested
  • All new features have been documented
  • I have read the CONTRIBUTING.md file
  • I will abide by the code of conduct

@freakboy3742 freakboy3742 requested a review from mhsmith July 10, 2025 00:38
@freakboy3742
Copy link
Member Author

Although this has passed CI, beeware/briefcase-windows-VisualStudio-template#62 indicates the underlying problem is still present (or, at least, requires some sort of additional handling in the WiX configuration).

@freakboy3742 freakboy3742 changed the title Restore Pyside6 tests, as WiX 5 now supports long paths. Restore Pyside6 tests Jul 10, 2025
@mhsmith mhsmith merged commit ac5bf44 into main Jul 11, 2025
21 checks passed
@mhsmith mhsmith deleted the long-path-support branch July 11, 2025 13:41
@mhsmith mhsmith changed the title Restore Pyside6 tests Restore Pyside6 tests, enable long filename support Oct 9, 2025
@mhsmith
Copy link
Member

mhsmith commented Oct 9, 2025

Even Microsoft documentation about this syntax is often incomplete or incorrect. This is the best explanation I could find.

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