YunoHost-Apps / freshrss_ynh

FreshRSS package for YunoHost
https://freshrss.org
GNU Affero General Public License v3.0
28 stars 20 forks source link

set app as owner of log folder in install/restore/update script #150

Closed xoddark closed 1 year ago

xoddark commented 1 year ago

This fix problem of writing log from php actualize_script runed by cron. Issue #148

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)

plopoyop commented 1 year ago

!testme

yunohost-bot commented 1 year ago

:v: Test Badge

plopoyop commented 1 year ago

Install fail :


153734 ERROR Unable to install freshrss: An error occurred inside the app installation script ```

You must remove spaces when affecting defining variable in bash.
You should try to manually install the package to tests you changes
plopoyop commented 1 year ago

I updated my comments (bad copy paste)

plopoyop commented 1 year ago

!testme

yunohost-bot commented 1 year ago

Meow :cat2: Test Badge

xoddark commented 1 year ago

@plopoyop : thank you for your feedback. I do this PR rapidly, as a start for a fix ;)

plopoyop commented 1 year ago

All ok. Thanks for your contribution :)