Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
I forgot to test the pyinstaller binary when I upgraded all the other dependencies, and there's a breaking change in 6.0.0: https://pyinstaller.org/en/latest/CHANGES.html#incompatible-changes > All of onedir build's contents except for the executable are now moved > into a sub-directory (called _internal by default). sys._MEIPASS is > adjusted to point to this _internal directory. The breaking > implications for this are: > > * Assumptions that os.path.dirname(sys.executable) == sys._MEIPASS > will break. Code locating application resources using > os.path.dirname(sys.executable) should be adjusted to use __file__ > or sys._MEIPASS and any code locating the original executable using > sys._MEIPASS should use sys.executable directly. > * Any custom post processing steps (either in the .spec file or > externally) which modify the bundle will likely need adjusting to > accommodate the new directory. This is actually great because it declutters the top level directory to just `dcs_liberation.exe` and a directory named `_internal` that has all the guts, but the CWD is no longer the directory that has `resources/` in it, so we can't find any of our resources. There are a few options for fixing that (cd into that directory probably being the easiest, or we could stop relying on CWD relative paths), but for now just downgrade to unbreak the build.
- Loading branch information