-
Notifications
You must be signed in to change notification settings - Fork 144
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
Check for latest version before execution #142
Comments
Sure. I suppose it would first require us to actually set up version numbers for migrate2rocky, though. Originally posted by @pajamian in #141 (comment) |
How about checking for differences between running Originally posted by @aminvakil in #141 (comment) |
The only real way to do that would be some sort of checksum, and that would be spoiled by any kind of local modification to the script. Originally posted by @pajamian in #141 (comment) |
Let's continue here :) Well it's a warning, right? They can skip the warning. Also I'm not sure how versioning can help in this situtation. |
Well, versioning is on the todo list anyways (just sort of way down on the list). So it would make sense to me for this feature to use it. |
So do you think this should wait out until versioning has been done? |
I'm more than happy if you want to add versioning as well. It could be as simple as just setting a variable at the top of the script and displaying it in the usage function. |
I'll set up versioning, and once I've done that you can look into this commit. I'll bump it up on my radar. |
There could be a soft warning which checks the latest
migrate2rocky.sh
from github and warns if it is not the latest one, this could help for other circumstances too IMHO.This should definitely not be a barrier and force users to update as probably there are someones which execute this on local networks without internet access.
Not related to this issue but I could help creating a PR for this if you think it's useful.
Originally posted by @aminvakil in #141 (comment)
The text was updated successfully, but these errors were encountered: