YunoHost-Apps / freshrss_ynh

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

Testing : v1.20 + Bulleyes fixes #146

Closed lapineige closed 1 year ago

lapineige commented 1 year 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)

lapineige commented 1 year ago

!testme

yunohost-bot commented 1 year ago

Alrighty! Test Badge

Alkarex commented 1 year ago

FYI, by the end of the month, there will be a minor release 1.20.1 fixing a few bugs https://github.com/FreshRSS/FreshRSS/pulls?q=milestone%3A1.20.1+is%3Aclosed+is%3Apr

lapineige commented 1 year ago

Ok then I'm ok to wait for that bug fix release. Meanwhile users can still use this version with the testing branch.

lapineige commented 1 year ago

This causes trouble https://forum.yunohost.org/t/probleme-avec-la-mise-a-jour-de-freshrss/21875 ?

““Cron freshrss@p…56 /usr/bin/php7.4 /var/www/freshrss/app/actualize_script.php >/var/log/freshrss/freshrss.log 2>&1"" “”/bin/sh: 1: cannot create /var/log/freshrss/freshrss.log: Permission denied””

@yalh76 I believe it's because of the change you made to the log file location (/var/www -> /var/log) and permission issues: https://github.com/YunoHost-Apps/freshrss_ynh/pull/146/files#diff-b6fe977119ccbe78ecd82239e9027e7ad39da209af7969156687d037bce754b0

yalh76 commented 1 year ago

This causes trouble https://forum.yunohost.org/t/probleme-avec-la-mise-a-jour-de-freshrss/21875 ?

““Cron freshrss@p…56 /usr/bin/php7.4 /var/www/freshrss/app/actualize_script.php >/var/log/freshrss/freshrss.log 2>&1"" “”/bin/sh: 1: cannot create /var/log/freshrss/freshrss.log: Permission denied””

@yalh76 I believe it's because of the change you made to the log file location (/var/www -> /var/log) and permission issues: https://github.com/YunoHost-Apps/freshrss_ynh/pull/146/files#diff-b6fe977119ccbe78ecd82239e9027e7ad39da209af7969156687d037bce754b0

Do you plan to rollback the change I made on freshrss.cron

lapineige commented 1 year ago

150 fixed it :)

6-ril commented 1 year ago

Hello, I'm using FressRss version1.20.0~ynh1. Does the issue on the log ( Permission denied ) is supposed to be fixed ?

6-ril