YunoHost-Apps / nextcloud_ynh

Nextcloud package for YunoHost
https://nextcloud.com/
GNU Affero General Public License v3.0
147 stars 63 forks source link

move the nextcloud log file to /var/log #669

Closed OniriCorpe closed 7 months ago

OniriCorpe commented 7 months ago

Problem

Solution

PR Status

Automatic tests

Automatic tests can be triggered on https://ci-apps-dev.yunohost.org/ after creating the PR, by commenting "!testme", "!gogogadgetoci" or "By the power of systemd, I invoke The Great App CI to test this Pull Request!". (N.B. : for this to work you need to be a member of the Yunohost-Apps organization)

yunohost-bot commented 7 months ago

:sunflower: Test Badge

ericgaspar commented 7 months ago

shall we also remove logs ynh_secure_remove --file="/var/log/$app" in remove script?

OniriCorpe commented 7 months ago

shall we also remove logs ynh_secure_remove --file="/var/log/$app" in remove script?

yeah why not

yunohost-bot commented 7 months ago

:rocket: Test Badge

niko2 commented 7 months ago

This will also help for logrotate that needs to be updated when data_dir is changed

OniriCorpe commented 7 months ago

that's the whole point 😅😅😅😅😅