-
Notifications
You must be signed in to change notification settings - Fork 208
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
add Cube 2 Sauerbraten #1875
base: master
Are you sure you want to change the base?
add Cube 2 Sauerbraten #1875
Conversation
@theofficialgman successful launch Debian Buster 32. |
these look to me just like sourceforge download errors... I got some as well, it reconnected and still finished thought without error. I don't think there is anything that can be done about that... sourceforge is just kinda crap edit: actually no, those aren't errors at all. thats just the redirect reponding with a new URL (a mirror) to download from |
@theofficialgman are you able to see my suggestions? I think it would be a better idea to use the direct link to the tar.bz2 file (from |
I didn't see your suggestion, where did you make it? I'm not sure if directly linking that sourceforge mirror is a good idea. I think pointing to sourceforge will automatically redirect to the best mirror available, right? |
My suggestions have a Pending tag. Maybe it’s because I made them while the PR was a draft. I am pretty sure it downloads from the closest mirror, yes. |
yeah I can't see them at all. you must have made them right before I swapped over to ready for merge and maybe that cleared them? |
@ryanfortner pending comments means you haven't submitted them yet |
Oops. You should be able to see them now. Sorry |
so what is this url? https://versaweb.dl.sourceforge.net is this just one of the mirrors or does sourceforge say to download from this? |
That’s where the original link redirected to. I think it is a mirror though. |
@theofficialgman I’ve changed the link to SourceForge’s master mirror. |
Why exactly do you want to make this change? the resulting redirect ends up in the same place as what it is currently |
I will test this once @ryanfortner's suggestions are either added or dismissed. |
according to sourceforge docs, downloading the way I have shown is the "most correct" and supported way https://sourceforge.net/p/forge/documentation/Downloading%20files%20via%20the%20command%20line/ going to dismiss @ryanfortner suggestions for now. if there are failures in the future we can look for other options @Botspot you can test now |
@Botspot I dismissed ryanfortners suggestions in the the above comment |
Are these errors of any concern?
It did eventually appear to begin successfully downloading... until I accidentally pressed Ctrl+C when trying to copy output. 🤦♂️ |
These errors could be due to sourceforge bandwidth limits in combination with aria2c's 16 threads. |
yes thats redirection. they aren't errors are are misclassified as such by aria2c |
I've seen redirection before and it doesn't look like that. In any case, if the app fails to install for any other reason, |
for sourceforge, this is what redirection looks like. if you download with wget it is more clear and correctly classifies this as redirection. |
I just tested this and the performance when fullscreen is not quite playable. I tried to reduce the resolution, and it looked like the game tried to do so, but then it reverted back to fullscreen in a split second. |
I can take a screen recording if that would be helpful. |
this is intentional. fullscreen and resolution are adjusted in the in game settings under the display and res tabs |
exit 1 is enough for install_packages
9b411bf
to
0b424fa
Compare
Bump |
@Botspot I think we are just stuck because of this ^. not really anything we can do about that besides report it to the devs |
Just do Sudo apt update |
1d137bb
to
6119ae6
Compare
closes #1475
I have edited the scripts to fix issues and tested on switch. please someone test on 32bit buster/bullseye
(click for video)