[02:00:38]
<Yunohost Git/Infra notifications> [package_linter] alexAubin pushed 1 commit to master: Complain about ynh_npm install --global é_è ([5d2c46a0](https://github.com/YunoHost/package_linter/commit/5d2c46a002e864250655deaeee5ac2f1a7096da3))
[02:01:06]
<Yunohost Git/Infra notifications> [package_linter] github-actions[bot] created new branch actions/black
[02:01:10]
<Yunohost Git/Infra notifications> [package_linter] github-actions[bot] pushed 1 commit to actions/black: :art: Format Python code with Black ([3ad32520](https://github.com/YunoHost/package_linter/commit/3ad32520ccae7a9d8d7752de4eda326388d279e7))
[02:01:19]
<Yunohost Git/Infra notifications> [package_linter] github-actions[bot] opened [pull request #153](https://github.com/YunoHost/package_linter/pull/153): Format Python code with Black
[02:19:32]
<Yunohost Git/Infra notifications> Autoupdater just ran, here are the results:
- 24 pending update PRs
- 14 new apps PRs
- 6 failed apps updates: calibreweb, focalboard, invoiceninja5, minetest, shiori, snserver
See the full log here: http://paste.yunohost.org/raw/pimemasesa
[09:48:00]
<Salamandar> > `100759 WARNING Packagers /!\ patch /var/cache/yunohost/app_tmp_work_dirs/app_4plo75ow/app_folder/sources/patches/main-02-oauth-workaround.patch failed to apply`
Shouldn’t this be an error instead of a warning ?
[09:57:35]
<Aleks (he/him/il/lui)> It was in the past and quickly turned into a dosaster
[11:49:53]
<Salamandar> ah…
[11:50:04]
<Salamandar> IMHO it should fail in CI
[11:50:09]
<Salamandar> at least
[11:51:10]
<Aleks (he/him/il/lui)> it's not that simple, for example some patches don't apply for old versions of Nextcloud, but ynh_setup_source will try to apply them when going through every major upgrade
hence you may end up with a failed "upgrade from old versions" because of this
[11:51:27]
<Aleks (he/him/il/lui)> though maybe that's only nextcloud and we could find a way to live with this
[11:51:51]
<Aleks (he/him/il/lui)> at least that's the reason the error was reverted to a warning
[11:53:12]
<Salamandar> > <@Alekswag:matrix.org> it's not that simple, for example some patches don't apply for old versions of Nextcloud, but ynh_setup_source will try to apply them when going through every major upgrade
>
> hence you may end up with a failed "upgrade from old versions" because of this
grmblbl
[11:53:16]
<Aleks (he/him/il/lui)> yeah...
[11:53:48]
<Salamandar> but i don't understand
[11:54:08]
<Salamandar> the patches used during test_upgrade_from are the patches from the old version
[11:54:13]
<Salamandar> so it, like, *should* apply
[11:55:42]
<Aleks (he/him/il/lui)> hmmm nah they are from the new version ?
[11:57:25]
<Salamandar> wut ?
[11:57:27]
<Salamandar> but no
[11:57:31]
<Salamandar> it can't
[11:57:44]
<Salamandar> oooooooh
[11:57:47]
<Salamandar> i misunderstood
[11:57:50]
<Salamandar> see that
[11:57:56]
<Salamandar> https://ci-apps.yunohost.org/ci/job/27360
[11:58:04]
<Salamandar> [Test 10/11] Upgrade from 27.1.4
[11:58:16]
<Salamandar> the preliminary install already has a patching issue
[11:58:59]
<Salamandar> But you were talking about this step during upgrade right ?
```
63221 INFO [#+++................] > Upgrading nextcloud... [.7 minutes]
64912 INFO Upgrading to Nextcloud 26
113986 INFO Upgrading to Nextcloud 27
151675 INFO Upgrading to Nextcloud 28.0.6
```
[11:59:53]
<Salamandar> but patches are named
[11:59:59]
<Salamandar> so there shouldn't be any conflict
[12:00:27]
<Salamandar> `ynh_setup_source --source_id=27` uses patches named `27-*`
`ynh_setup_source --source_id=latest` uses patches named `latest-*`
[12:00:47]
<Salamandar> so you can't have patches applied on the wrong assets
[12:02:01]
<Aleks (he/him/il/lui)> ah ?
[12:02:25]
<Salamandar> well i think yeah
[12:02:30]
<Aleks (he/him/il/lui)> hmm but maybe it was about patches that failed on the old apps and suddently they were raising an error so the old version couldnt be installed anymore
[12:02:39]
<Aleks (he/him/il/lui)> but maybe we could have an hack for this
[12:02:49]
<Salamandar> ah yeah ah ah
[12:02:59]
<Salamandar> maybe not fail during pre-install of `test_upgrade_from`
[12:03:08]
<Salamandar> idk
[12:03:17]
<Aleks (he/him/il/lui)> also it will break restoring the existing apps in the wild (eg if an upgrade fails and it wants to restore)
[12:03:35]
<Yunohost Git/Infra notifications> [my_webapp_ynh] bmp opened [issue #139](https://github.com/YunoHost-Apps/my_webapp_ynh/issues/139): Unable to use HTTP Basic Authentication with nginx
[12:04:09]
<Aleks (he/him/il/lui)> maybe we could have a "smooth" transition e.g. raising an error if app requires YunoHost >= 11.2.X or something
[12:04:17]
<Aleks (he/him/il/lui)> or a warning otherwise
[12:04:27]
<Aleks (he/him/il/lui)> i did this back in the days for 4.2 to fix a permission disaster
[13:55:58]
<rodinux> does someone have try if discourse issue https://github.com/YunoHost-Apps/discourse_ynh/issues/179 is resolved on testing branch ??
[13:57:48]
<rodinux> sorry it is the same
[14:30:29]
<rodinux> I was looking for a work around here https://github.com/YunoHost-Apps/discourse_ynh/issues/179#issuecomment-2029706595 but when I compare the last file install and the the one suggested, there is a mount of diffs, sooooo
[15:02:49]
<rodinux> Well I am scarred !! the fix is with old manifest.json file so no in v2
[15:07:20]
<rodinux> There is no way install a discourse ?
[15:16:21]
<rodinux> here it seems to be perhaps a solution ?? https://github.com/jonmbake/discourse-ldap-auth/issues/71#issuecomment-1374357112
[15:42:06]
<Guillaume Bouzige> yeah flarum is working better than discourse
[15:49:49]
<rodinux> I understand it's very new for v2 branch, only 3 weeks...
[15:58:44]
<rodinux> perhaps next version resolve the issue ??? https://github.com/YunoHost-Apps/discourse_ynh/pull/180
[16:22:35]
<rodinux> > <@rodinux:matrix.org> perhaps next version resolve the issue ??? https://github.com/YunoHost-Apps/discourse_ynh/pull/180
ce serait trop simple..;
[18:58:48]
<Yunohost Git/Infra notifications> App seafile rises from level 6 to 7 in job [#27386](https://ci-apps.yunohost.org/ci/job/27386) !
[20:39:00]
<Yunohost Git/Infra notifications> [apps] yunohost-bot opened [pull request #2405](https://github.com/YunoHost/apps/pull/2405): Translations update from Weblate for appstore
[23:12:43]
<Yunohost Git/Infra notifications> App ghost goes down from level 8 to 6 in job [#27390](https://ci-apps.yunohost.org/ci/job/27390)
[23:38:52]
<Yunohost Git/Infra notifications> App cheky goes down from level 8 to 7 in job [#27392](https://ci-apps.yunohost.org/ci/job/27392)
[23:50:55]
<Yunohost Git/Infra notifications> App friendica failed all tests in job [#27393](https://ci-apps.yunohost.org/ci/job/27393) :(