search

cDima / Aerial

Aerial Apple TV screen saver for Windows
1.97k stars 181 forks source link

Rename v3 to pretty name #91

Closed cDima closed 7 years ago

cDima commented 7 years ago

Caused confution after installation, should correlate with v5.1 or have no version in name at all.

alistairmcmillan commented 7 years ago

I vote for no version in name at all.

alistairmcmillan commented 7 years ago

Each new version should just replace the previous one. You don't want people hanging onto AerialScreenSaverV3.scr, AerialScreenSaverV4.scr, AerialScreenSaverV5.scr as you fix bugs. Just asking for more confusion. :)

cDima commented 7 years ago

@alistairmcmillan you're right. But for traceability, should we have a version number visible in the options dialog or in a log file? If someone finds an error, they can see that they aren't on the latest build instead of guessing. Perhaps the app can reach out to github and see the latest version. :)

cDima commented 7 years ago

aka https://api.github.com/repos/cdima/aerial/releases - but the current apps don't carry version info, and the dates don't seem to correlate. :\

alistairmcmillan commented 7 years ago

@cDima Oh yes, absolutely have the version number in the Settings dialog.

And yeah. If only Windows had something like Sparkle on the Mac.

alistairmcmillan commented 7 years ago

Well what do you know, there is a Windows version of Sparkle. :) https://github.com/vslavik/winsparkle/wiki/Basic-Setup

cDima commented 7 years ago

The windows way would be a Click Once application, which is possible as well from GitHub. http://stackoverflow.com/questions/13512029/is-it-possible-to-offer-a-clickonce-installer-on-github

On Wed, Dec 28, 2016, 5:40 PM Alistair McMillan notifications@github.com wrote:

Well what do you know, there is a Windows version of Sparkle. :) https://github.com/vslavik/winsparkle/wiki/Basic-Setup

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/cDima/Aerial/issues/91#issuecomment-269552806, or mute the thread https://github.com/notifications/unsubscribe-auth/AADl3X57-dmI1qpsjUx8bEVhflhME_u5ks5rMuVagaJpZM4LXOlL .

alistairmcmillan commented 7 years ago

But as mentioned in another issue you don't really need an installer.

Maybe just a little thing in the Settings dialog that flags up for the user when a new release is available.

2016-12-29 12_06_58-screensaver - sharpdevelop

With "Click Here!" just pointing to https://github.com/cDima/Aerial/releases.

cDima commented 7 years ago

Adding a version tag, but closing for now as the name was changed.