search

ppy / osu-wine

Wine container for running osu! on OS X. http://osx.ppy.sh
139 stars 20 forks source link

Fix curl command again #2

Closed Artoria2e5 closed 8 years ago

Artoria2e5 commented 8 years ago

Human generally prefer escaping over quoting.

magicnat commented 8 years ago

You know, some osu! players aren't human. ( Especially those play osu! on OSX. lol

Artoria2e5 commented 8 years ago

In order to be further non-human, my suggestion is to use printf '%(%s)T' instead of date +%s... XD

ghost commented 8 years ago

Why are you making the curl command so complicated?

I'd just use something like: curl -SL# 'https://m1.ppy.sh/r/osu!.exe' -o osu.exe

peppy commented 8 years ago

The request string was added because the executable can get cached otherwise, but I agree that focus is being placed on the wrong thing here. This was made open source to fix the actual bundle, not the script which won't even be included in the final distribution.