Add missing pyproject.toml data#2
Conversation
This should fix CI by adding missing required info
This reverts commit 78b5e31.
|
So I tried to get things running locally, and heat.exe appears to be crashing due to something related to the .NET framework, but I have no idea what wix needs (the completely unhelpful error message is taking me to https://learn.microsoft.com/en-us/dotnet/framework/install/application-not-started?version=(null)&processName=heat.exe&platform=0009&osver=6&isServer=0&shimver=4.0.30319.0 -- my error is the first dialog in that link). @freakboy3742 do you happen to know what version of the .NET framework I might need? The log itself is mostly useless (aside from laughing at how much of a thin veneer windows 11 really is): |
|
@drewbrew AFAIK, WiX has a .NET requirement of .Net framework 3.5. This is an optional component; but I thought it was provided as part of the Windows CI environment. It's especially weird that this is only affecting the PySide builds - the Toga and PPB builds both call HEAT as well, but they aren't complaining... |
|
This error is on my local box, not the GitHub runner, so needing 3.5 makes sense. |
|
Installing the 3.5 framework didn't help, so I'm even more confused. |
|
Want to see something really fun? but: So... the file exists... unless you look for just that file, in which case it doesn't. |
|
Thanks to Mongoose-Q and davisagli on twitter, it appears Windows has a 260 character limit on file paths. I'm seeing conflicting reports on whether this is or isn't disabled by default on Github Actions - but it's something to investigate. |
797ae04 to
a5478cd
Compare
39aedf5 to
0104444
Compare
freakboy3742
left a comment
There was a problem hiding this comment.
It turns out this is a known problem with WiX. It's apparently fixed in WiX 4... which has been in prerelease for as long as I've known about the existence of WiX.
So - I'm going to make an executive decision to disable the PySide2/6 tests for now, on the basis that there's no way to make them work right now.
|
Wow that is truly impressive. Thanks for digging into it! |
This should fix CI by adding missing required info
PR Checklist: