YunoHost-Apps / kimai2_ynh

Kimai v2 package for YunoHost
https://www.kimai.org/
MIT License
13 stars 8 forks source link

Fix Buster #24

Closed yalh76 closed 4 years ago

yalh76 commented 4 years ago

Package_check results


Build Status
Revision: 93968e7326d7aa1d6361437ed061a5b4a0ce9829

Stretch:

image

Buster:

image

kevinpapst commented 4 years ago

Any chance you can use the GIT installation? If it works: great! But I don't support the FTP installation and for long term support git should be preferred.

yalh76 commented 4 years ago

Any chance you can use the GIT installation? If it works: great! But I don't support the FTP installation and for long term support git should be preferred.

I switched to the git version

yalh76 commented 4 years ago

Seems OK, if someone want to test it.

kevinpapst commented 4 years ago

By "installing the git version" I actually meant using git clone and not the git release archive. I never used YunoHost, so I don't know if that is feasible.

Thanks for all your work you put into this installation and for supporting Kimai 👍 Very much appreciated!!!!!

yalh76 commented 4 years ago

By "installing the git version" I actually meant using git clone and not the git release archive. I never used YunoHost, so I don't know if that is feasible.

Thanks for all your work you put into this installation and for supporting Kimai 👍 Very much appreciated!!!!!

We can use git clone but YunoHost packages usually use .tar.gz archive:

kevinpapst commented 4 years ago

I am just afraid, that I might move files around in Kimai and these old files won't get deleted during an updated. Thats why I recommend git, because git would take care of deleting these old files.

But if you delete the complete application before the update, this problem should not occur.

yalh76 commented 4 years ago

I am just afraid, that I might move files around in Kimai and these old files won't get deleted during an updated. Thats why I recommend git, because git would take care of deleting these old files.

But if you delete the complete application before the update, this problem should not occur.

Thanks, I didn't reread that part of the code, now after saving in a tempdir some files:

https://github.com/YunoHost-Apps/kimai2_ynh/blob/d44cfc271a7f8975ec87cc6e005eccb2a09e2a28/scripts/upgrade#L96-L129

kevinpapst commented 4 years ago

This is much better!

Just found one more thing. Can you add -n to php7.3 bin/console kimai:update so it becomes:

php7.3 bin/console kimai:update -n

Just in case there will ever be interactive questions added to the update mechanism, this flag will prevent the update from being interrupted, instead the default answer will be used.

yalh76 commented 4 years ago
php7.3 bin/console kimai:update -n

Updated ;)

kevinpapst commented 4 years ago

Great work! I am installing a Virtualbox image for YunoHost, lets see if I can test this... I might come back with a couple of questions as I never used it before.

SiM commented 4 years ago

Hello,

Any chance this PR will be merged soon ?

yalh76 commented 4 years ago

Hello,

Any chance this PR will be merged soon ?

not that soon, I just worked to make it YunoHost 3.8 ready and have to wait for a package_check