search

YunoHost-Apps / shaarli_ynh

Shaarli package for YunoHost
GNU General Public License v3.0
20 stars 13 forks source link

Backup/restore scripts #23

Closed rokaz closed 7 years ago

rokaz commented 7 years ago

Hi,

I wrote these basic backup/restore scripts. I tested them and they seem to be working as expected.

lapineige commented 7 years ago

Sorry for the delay. I'm testing those scripts right now :)

lapineige commented 7 years ago

Maybe it's because of my installation, but when making a backup, removing shaarli, and restoring later, it raises and error like: An error happened during LDAP operation

I'm on Yunohost 2.5.6.

rokaz commented 7 years ago

I have the same error when backing up and restoring both hooks and other official apps (tested with wallabag and dokuwiki). Backup/restore without the hooks (only the app) works on my test system (the ynh_dev script, on stable), as for the other apps and the hooks alone. I didn't take the time to talk with the ynh devs of this issue, which doesn't seem to be triggered by this only app.

lapineige commented 7 years ago

I did it just for shaarli app, but I don't remember if I added the hooks too. I'll test again.

If it works, and as it seems to be a known bug, I'll merge your PR and add a warning in the README :)

lapineige commented 7 years ago

Ok, if I had the --ignore-hooks option it works. But I have a warning for Insufficient permissions in "cache" , "data", "pagecache" and"tmp/" directories. Looking at the script, it setup the owner as root (same for the group). Shouldn't it be www-data ? (as during install)

It works like that on your config ?

rokaz commented 7 years ago

No, it is a mistake from myself. I pushed the correction.

lapineige commented 7 years ago

Everything works right now :) Thanks a lot for your work !