Skip to content
This repository has been archived by the owner on Jul 5, 2024. It is now read-only.

Upgrading OneDrive Client (config wise) #36

Closed
modelmat opened this issue Dec 5, 2018 · 4 comments
Closed

Upgrading OneDrive Client (config wise) #36

modelmat opened this issue Dec 5, 2018 · 4 comments

Comments

@modelmat
Copy link
Collaborator

modelmat commented Dec 5, 2018

As we saw in the recent issues caused by renaming the project, I propose that there should be same way of upgrading versions. Here are some solutions which may or may not be interchangeable:

  1. Store the latest version or commit hash in the config files. (this should be done anyhow) - though maybe a config specific version would be better so it could be known when it changes.

  2. Add to the upgrading steps to README (this akes it slightly harder if updates become automated e g through an AUR package)

  3. If the config specific version changes:
    a. Every run, until something is done (in which case the version indicator is changed), ask/recommend that the user delete and recreate the config - this runs into problems because of Unable to compare existing files downloaded via other methods. As result, onedriveClient is downloading/uploading everything again #22 and Systemd Service #20 (the latter as the message would not be seen)
    b. Use input() inside of setup.py (is this bad practice?)

  4. Create some sort of automated tool that can migrate between versions (harder but solves all the problems)

@derrix060
Copy link
Owner

There is already a version set in setup.py, that can be changed in the code. Everytime that someone calls onedrive-client --version will get it.

I'm a little bit sceptic of having an automated tool to do the migration. I like the idea, but I think there will be a lot of development involved, and also we would have to support different versions and so on. I don't think these "break changes" are so frequent, so I would rather use pip install --upgrade onedrive-client instead...

@modelmat
Copy link
Collaborator Author

modelmat commented Dec 6, 2018

I do agree with the problems with merging.

I am meaning to store the version in the config - separately to the actual runner version. Maybe a better solution is to add an issue template and when adding issues require the config to be removed and recreated as a way of debugging (and hence the version would allow us to tell if any major changes).

@derrix060
Copy link
Owner

I think I'm not following... What config is this? Why the config and the runner can have a different version? And can you say an example for a issue that requires removing and recreating the config?

@modelmat
Copy link
Collaborator Author

modelmat commented Jan 9, 2019

Sorry for the late reply. What I'm trying to say is in the config (~/.config/onedrive_client) we store the application version at the time the config was created and request that when someone files an issue; as well as the onedrive-client --version (of which doesn' work for me).

Thinking about it, this seems to have little point. Closing anyway.

@modelmat modelmat closed this as completed Jan 9, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants