Skip to content
This repository has been archived by the owner on Jan 19, 2023. It is now read-only.

Automatic updating #22

Closed
dirkgroenen opened this issue Jan 8, 2015 · 7 comments
Closed

Automatic updating #22

dirkgroenen opened this issue Jan 8, 2015 · 7 comments

Comments

@dirkgroenen
Copy link
Owner

Let Mopify automatically update itself. Should be possible with the tornado.web request handler and pip update.

Worth trying..

@dirkgroenen dirkgroenen added the bug label Jan 8, 2015
@dirkgroenen dirkgroenen added this to the 1.2.0 milestone Jan 18, 2015
@dirkgroenen dirkgroenen modified the milestone: 1.2.0 Jan 28, 2015
@dirkgroenen dirkgroenen modified the milestone: 1.3.3 Mar 25, 2015
dirkgroenen added a commit that referenced this issue Apr 4, 2015
@dirkgroenen
Copy link
Owner Author

Added in next release (1.4.0)

@kingosticks
Copy link

Cool! I wanted something like this in mopidy-websettings.

@dirkgroenen
Copy link
Owner Author

Must say that it still isn't perfect since I haven't found a way to restart Mopidy after updating, but for now it's a nice gimmick.

@kingosticks
Copy link

There's some hack in websettings for restarting it when you're just running
it in the terminal but it doesn't work when it's a service. I played around
it before but lost interest. Will post back if we do find a nice solution.
On 5 Apr 2015 15:56, "Dirk Groenen" notifications@github.com wrote:

Must say that it still isn't perfect since I haven't found a way to
restart Mopidy after updating, but for now it's a nice gimmick.


Reply to this email directly or view it on GitHub
#22 (comment)
.

@dirkgroenen
Copy link
Owner Author

Yes I've seen that hack, but I don't want to restart the whole system after updating. At the moment a reboot isn't always necessary. It depends on the changes made in Mopify. When changes have been made in the python core files a reboot is needed, but when only html, js and css files have been updates it isn't required.

@kingosticks
Copy link

There should be another (!) hack, maybe commented out, that just reloads
the python process rather than the whole system. And Yeh, a reboot is
complete overkill most of the time but I think getting something working
was the idea at the time.
On 5 Apr 2015 16:00, "Dirk Groenen" notifications@github.com wrote:

Yes I've seen that hack, but I don't want to restart the whole system
after updating. At the moment a reboot isn't always necessary. It depends
on the changes made in Mopify. When changes have been made in the python
core files a reboot is needed, but when only html, js and css files have
been updates it isn't required.


Reply to this email directly or view it on GitHub
#22 (comment)
.

@dirkgroenen
Copy link
Owner Author

I have been experimenting with restarting the process, but I haven't been able to do this successfully. The biggest problem I encountered was the process being killed (duh) which disables the ability to start a new process.

In the meantime I've created a new issue #87 where we can discuss further improvements to this feature.

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