search

YunoHost-Apps / pluxml_ynh

PluXml package for YunoHost
http://pluxml.org
GNU General Public License v3.0
9 stars 4 forks source link

Multi-instance #11

Closed scith closed 9 years ago

scith commented 9 years ago

I think this is all that is needed :) Your scripts are very clean so YunoHost takes care of the rest (adds 1, 2, etc... behind $app names).

matlink commented 9 years ago

Well, I trust you on this pull request, since you've tested it ! Thanks !

scith commented 9 years ago

Anyway, worst case scenario is that only 1 pluxml could be installed, which is the current situation anyway. But yes it worked in my VM, I'll install it soon in prod Cheers

matlink commented 9 years ago

One case that could be tricky is If there is one pluxml installed without multi_instance (before the merge) that tries to update or install a new instance of it. I don't know how Yunohost reacts to it.

Le 20/08/2015 10:07, Jeff a écrit :

Anyway, worst case scenario is that only 1 pluxml could be installed, which is the current situation anyway. But yes it worked in my VM, I'll install it soon in prod Cheers

— Reply to this email directly or view it on GitHub https://github.com/matlink/pluxml_ynh/pull/11#issuecomment-132926503.