-
Notifications
You must be signed in to change notification settings - Fork 404
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
RPM nightly release fails to install #5542
Comments
Yeah the files section is what you “did” install and we don’t install usr/bin. Or usr/lib/vst3 for that matter. I’ll clean it up in a sweep before the weekend and tag this issue when I do thanks! |
1. RPM file has correct %files% section. Addresses surge-synthesizer#5542 2. XDG_DOCUMENTS_DIR used to resolve documents dir if set
So I think I have this resolved but it is super hard for me to test. I'll merge 5543 but leave this issue open until we have a confirm. Will need the nightlies to run also. |
1. RPM file has correct %files% section. Addresses #5542 2. XDG_DOCUMENTS_DIR used to resolve documents dir if set
Sigh my fix blew up the nightly so no test image yet. Working on it |
1; Have the correct %files% 2. Dont crash on build Addresses surge-synthesizer#5542
1; Have the correct %files% 2. Dont crash on build Addresses #5542
This is fixed in the latest nightly. Thank you! |
Great thanks! |
Bug Description:
Reopening in this repo as requested:
I'm using Fedora 35. I tried to install the RPM release and got the following (the last two lines are the error):
Surge Version
Reproduction Steps:
Steps to reproduce the behavior:
sudo dnf install ./surge-xt-x86_64-NIGHTLY-2021-12-01-bc53d94.rpm
Expected Behavior:
Installation succeeds.
Computer Information (please complete the following information):
Additional Information:
As a temporary workaround, I was able to use rpmrebuild on the file and remove the line specifying /usr/bin. The resulting RPM was installable, but did not contain a standalone binary, which I now suspect is what the /usr/bin line should have been specifying.
The issue appears to be with this line, according to @baconpaul:
surge/scripts/installer_linux/make_rpm.sh
Line 130 in 2f2509e
The text was updated successfully, but these errors were encountered: