-
Notifications
You must be signed in to change notification settings - Fork 374
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
Update requirements.txt #513
Conversation
Changed the version to 6.7.0 because it is marked as a fixed version for the but where it says: ```No QtMultimedia backends found. Only QMediaDevices, QAudioDevice, QSoundEffect, QAudioSink, and QAudioSource are available. Failed to create QVideoSink "Not available" Failed to initialize QMediaPlayer "Not available" ``` Bug Report is attached below and the change log from 6.7.0 -> 6.7.1 is listed. two CVE's were listed but they are not directly used by QT and only used if the developer is using them. I did not check to see if the code contains, but since you are trying to use sqlite DB and are not networking this should be in the clear. https://bugreports.qt.io/browse/PYSIDE-2656 https://code.qt.io/cgit/qt/qtreleasenotes.git/about/qt/6.7.1/release-note.md https://forum.qt.io/topic/157026/no-qtmultimedia-backends-found-on-macos-with-pyside-6-7-1
Added Quotes around more of the directory fields. Found this error because I have a space in the name of my external drive where I was trying to build this.
Quality of life change, added new line to the End of Transmission
6.7.0 has historically had issues for us including a bug involving a TypeError in the main window class, plus an issue where it doesn't play nicely with MyPy causing the workflow to always fail.
However I am currently unable to reproduce the same TypeError issue with the current codebase, so maybe that particular bug isn't an issue anymore. Either way we may just need to try/wait for a future PySide version instead of downgrading to versions with preexisting bugs. |
No problem, was not sure if any of the other tweaks I made were worth it. I am on a Mac Studio (Last intel model) so the error I was getting on running the script could just be a me thing is someone else is running it on an M1. |
I think there's been some reports on Linux about a similar issue, if not then I'm thinking about other Mac reports. Either way I'll keep an eye on future PySide versions to fix this. According to one of the forum posts you linked it looks like 6.7.3 might include a fix, which looks like it's scheduled for release... wait, tomorrow? Okay I'll really be keeping an eye out for that then... |
Not sure if you are able to commit just the Build_MacOS_app.sh file but I downloaded the 9.4 branch again and tried to compile but was getting an error because the directories don't have quotes around them. |
Updated Pyinstaller to 6.11.0
Updated PySide6 (Qt) to 6.8.0.1 When compiled FFmpeg Checker worked for Mac in both versions. FFmpegChecker] FFmpeg found: True, FFprobe found: True
Creating a new pull request. Looks like the updated Qt fixes the ffmpeg not found problem |
Changed the version to 6.7.0 because it is marked as a fixed version for the but where it says:
Bug Report is attached below and the change log from 6.7.0 -> 6.7.1 is listed. two CVE's were listed but they are not directly used by QT and only used if the developer is using them. I did not check to see if the code contains, but since you are trying to use sqlite DB and are not networking this should be in the clear.
https://bugreports.qt.io/browse/PYSIDE-2656
https://code.qt.io/cgit/qt/qtreleasenotes.git/about/qt/6.7.1/release-note.md
https://forum.qt.io/topic/157026/no-qtmultimedia-backends-found-on-macos-with-pyside-6-7-1