[00:46:09]
<Yunohost Git/Infra notifications> [mastodon_ynh] Thatoo opened [issue #485](https://github.com/YunoHost-Apps/mastodon_ynh/issues/485): upgrade to 4.3.3 fails : Gem::Net::OpenTimeout
[07:17:22]
<Yunohost Git/Infra notifications> [piped_ynh] loijay0808 [commented](https://github.com/YunoHost-Apps/piped_ynh/issues/173#issuecomment-2629277110) on [issue #173](https://github.com/YunoHost-Apps/piped_ynh/issues/173) IOS player response is not valid: Hi,
I tried installing the testing branch, yet Im still getting the same errors:
org.schabi.newpipe.extractor.excepti...
[09:37:09]
<Yunohost Git/Infra notifications> [apps] ericgaspar [commented](https://github.com/YunoHost/apps/pull/2792#issuecomment-2629318325) on [issue #2792](https://github.com/YunoHost/apps/pull/2792) Add conduwuit to wishlist: already present https://github.com/YunoHost-Apps/conduwuit_ynh
[09:37:09]
<Yunohost Git/Infra notifications> [apps] ericgaspar closed [pull request #2792](https://github.com/YunoHost/apps/pull/2792): Add conduwuit to wishlist
[11:19:48]
<Yunohost Git/Infra notifications> [apps] yunohost-bot labeled Wishlist on [pull request #2793](https://github.com/YunoHost/apps/pull/2793): Add PaperlessAI to wishlist
[11:19:48]
<Yunohost Git/Infra notifications> [apps] yunohost-bot opened [pull request #2793](https://github.com/YunoHost/apps/pull/2793): Add PaperlessAI to wishlist
[12:25:23]
<Yunohost Git/Infra notifications> [freshrss_ynh] dalgwen [commented](https://github.com/YunoHost-Apps/freshrss_ynh/issues/199#issuecomment-2629374993) on [issue #199](https://github.com/YunoHost-Apps/freshrss_ynh/issues/199) Hooks are called with php7 but FreshRSS 1.25 required PHP8: > It should not because of this https://github.com/YunoHost-Apps/freshrss_ynh/blob/master/manifest.toml#L78
I think thi...
[12:28:35]
<Yunohost Git/Infra notifications> [freshrss_ynh] lapineige [commented](https://github.com/YunoHost-Apps/freshrss_ynh/issues/199#issuecomment-2629376120) on [issue #199](https://github.com/YunoHost-Apps/freshrss_ynh/issues/199) Hooks are called with php7 but FreshRSS 1.25 required PHP8: > I think this installs a specific php version, but thats all. It doesnt specify which version to use, if several are ...
[12:29:40]
<lapineige> Any idea how to fix this PHP version issue in hooks ? 😅 https://github.com/YunoHost-Apps/freshrss_ynh/issues/199#issuecomment-2629000107
[12:31:40]
<Aleks (he/him/il/lui)> zmlerp
[12:32:00]
<Aleks (he/him/il/lui)> what about calling `phpX.Y script` instead of just `script` here ? https://github.com/YunoHost-Apps/freshrss_ynh/blob/master/hooks/post_user_create#L11
[12:32:22]
<Aleks (he/him/il/lui)> i.e. something like `sudo -u $app php8.2 $install_dir/cli/create-user.php`
[12:41:40]
<lapineige> Yeah I thought about this. So this is a way forward, nice.
Do you know why the issue can happen ? I thought in packaging v2 you specify PHP version in dependencies and never touch it again ?
[12:45:50]
<Aleks (he/him/il/lui)> because in the context of the script, it will use the default php version on the system, there's no magic way around this because there's nothing in the script explicitly stating the version to be used
[12:46:23]
<Aleks (he/him/il/lui)> (in fact also in the context of an install script, you need to explicitly call `phpX.Y somescript`, or possibly replacing X.Y with `$php_version`
[12:46:50]
<Aleks (he/him/il/lui)> the "magic" bit is that the `$php_version` setting is automatically defined according to the apt dependency
[12:47:14]
<Aleks (he/him/il/lui)> and then it gets replaced in the nginx conf and ynh_add_php_config will create the conf in the appropriate directory etc
[12:47:17]
<Aleks (he/him/il/lui)> but that's pretty much it
[12:48:08]
<Aleks (he/him/il/lui)> for example the "exec_occ" helper in nextcloud has the explicit version (well via the php_version var) : https://github.com/YunoHost-Apps/nextcloud_ynh/blob/master/scripts/_common.sh#L10
[13:25:56]
<Yunohost Git/Infra notifications> [freshrss_ynh] dalgwen [commented](https://github.com/YunoHost-Apps/freshrss_ynh/issues/199#issuecomment-2629396043) on [issue #199](https://github.com/YunoHost-Apps/freshrss_ynh/issues/199) Hooks are called with php7 but FreshRSS 1.25 required PHP8: Interesting
So, does it means that you do not have the issue ? If so, do you also have another (older) PHP version in...
[13:26:11]
<Yunohost Git/Infra notifications> [freshrss_ynh] dalgwen [commented](https://github.com/YunoHost-Apps/freshrss_ynh/issues/199#issuecomment-2629396043) on [issue #199](https://github.com/YunoHost-Apps/freshrss_ynh/issues/199) Hooks are called with php7 but FreshRSS 1.25 required PHP8: Interesting
So, does it means that you do not have the issue ? If so, do you also have another (older) PHP version in...
[13:26:20]
<Yunohost Git/Infra notifications> yalh76 created repository private-gpt_ynh https://github.com/YunoHost-Apps/private-gpt_ynh
[13:26:21]
<Yunohost Git/Infra notifications> yalh76 edited repository private-gpt_ynh https://github.com/YunoHost-Apps/private-gpt_ynh
[13:46:41]
<orhtej2> While this is true wasn't env for hooks supposed to have alias set so that `php` is aliased to the right version?
[13:48:18]
<Aleks (he/him/il/lui)> hmmmm
[13:48:36]
<Aleks (he/him/il/lui)> uuuuuh idk x_X
[13:49:04]
<orhtej2> It's set the same way ynh app shell is, right?
[13:49:39]
<Aleks (he/him/il/lui)> ah indeed the app shell has this trick : https://github.com/YunoHost/yunohost/blob/dev/helpers/helpers.v2.1.d/utils#L429
[13:49:46]
<Aleks (he/him/il/lui)> but that's specific to app shell
[13:50:12]
<Aleks (he/him/il/lui)> and wouldn't work in the case of fresh_rss either i think, because the custom bash function wouldnt be exported to subshells
[13:52:19]
<orhtej2> Sure, we can call right version and call it a day
[15:07:50]
<Yunohost Git/Infra notifications> [jellyfin_ynh] tituspijean [commented](https://github.com/YunoHost-Apps/jellyfin_ynh/pull/184#issuecomment-2629433591) on [issue #184](https://github.com/YunoHost-Apps/jellyfin_ynh/pull/184) upgrade to 10.10.5: Works for me, thanks :)
[15:08:18]
<Yunohost Git/Infra notifications> [jellyfin_ynh] tituspijean pushed 1 commit to testing: Upgrade to 10.10.5 (#184) ([70805533](https://github.com/YunoHost-Apps/jellyfin_ynh/commit/7080553301afd0b878cd07f09bc007ff615d918c))
[15:08:18]
<Yunohost Git/Infra notifications> [jellyfin_ynh] tituspijean merged [pull request #184](https://github.com/YunoHost-Apps/jellyfin_ynh/pull/184): Upgrade to 10.10.5
[15:08:19]
<Yunohost Git/Infra notifications> [jellyfin_ynh] tituspijean edited [pull request #184](https://github.com/YunoHost-Apps/jellyfin_ynh/pull/184): Upgrade to 10.10.5
[15:08:42]
<Yunohost Git/Infra notifications> [jellyfin_ynh] tituspijean opened [pull request #186](https://github.com/YunoHost-Apps/jellyfin_ynh/pull/186): Upgrade to 10.10.5
[15:08:53]
<Yunohost Git/Infra notifications> [jellyfin_ynh] tituspijean merged [pull request #186](https://github.com/YunoHost-Apps/jellyfin_ynh/pull/186): Upgrade to 10.10.5
[15:20:26]
<Yunohost Git/Infra notifications> [apps] tituspijean deleted branch add-to-wishlist-conduwuit
[17:17:04]
<Yunohost Git/Infra notifications> [apps] tituspijean merged [pull request #2790](https://github.com/YunoHost/apps/pull/2790): Add Argos to catalog
[17:17:04]
<Yunohost Git/Infra notifications> [apps] tituspijean deleted branch argos
[17:17:04]
<Yunohost Git/Infra notifications> [apps] tituspijean pushed 1 commit to master: Add Argos to catalog (#2790) ([3a3b2072](https://github.com/YunoHost/apps/commit/3a3b207250cc3ae746873b3f0a9afdce8f6c24e7))
[17:35:47]
<Yunohost Git/Infra notifications> [borg_ynh] olberger [commented](https://github.com/YunoHost-Apps/borg_ynh/issues/120#issuecomment-2629484447) on [issue #120](https://github.com/YunoHost-Apps/borg_ynh/issues/120) Possibility to add --remote-path parameter: > ForceCommand
> For security reasons you should use ForceCommand on your synology. https://github.com/YunoHost-Apps/...
[17:44:24]
<Yunohost Git/Infra notifications> [borg_ynh] zamentur [commented](https://github.com/YunoHost-Apps/borg_ynh/issues/120#issuecomment-2629487308) on [issue #120](https://github.com/YunoHost-Apps/borg_ynh/issues/120) Possibility to add --remote-path parameter: You should use ForceCommand SSH feature to define a command like bellow in the file /home/USER/.ssh//authorized_keys...
[17:44:55]
<Yunohost Git/Infra notifications> [borg_ynh] zamentur [commented](https://github.com/YunoHost-Apps/borg_ynh/issues/120#issuecomment-2629487308) on [issue #120](https://github.com/YunoHost-Apps/borg_ynh/issues/120) Possibility to add --remote-path parameter: @olberger You should use ForceCommand SSH feature to define a command like bellow in the file /home/USER/.ssh/authori...
[17:48:39]
<Yunohost Git/Infra notifications> [borg_ynh] olberger [commented](https://github.com/YunoHost-Apps/borg_ynh/issues/120#issuecomment-2629489583) on [issue #120](https://github.com/YunoHost-Apps/borg_ynh/issues/120) Possibility to add --remote-path parameter: > [@olberger](https://github.com/olberger) You should use ForceCommand SSH feature to define a command like bellow in ...
[17:51:35]
<Yunohost Git/Infra notifications> [borg_ynh] zamentur [commented](https://github.com/YunoHost-Apps/borg_ynh/issues/120#issuecomment-2629490548) on [issue #120](https://github.com/YunoHost-Apps/borg_ynh/issues/120) Possibility to add --remote-path parameter: I guess it could, feel free to make some test by changing your /etc/yunohost/apps/borg/settings.yml to add the settings ...
[17:51:39]
<Yunohost Git/Infra notifications> [borg_ynh] zamentur reopened [issue #120](https://github.com/YunoHost-Apps/borg_ynh/issues/120): Possibility to add --remote-path parameter
[18:54:35]
<Yunohost Git/Infra notifications> [borg_ynh] olberger [commented](https://github.com/YunoHost-Apps/borg_ynh/issues/120#issuecomment-2629512581) on [issue #120](https://github.com/YunoHost-Apps/borg_ynh/issues/120) Possibility to add --remote-path parameter: Heres a preliminary attempt : https://github.com/olberger/borg_ynh/tree/remote-path
Havent tested it much, and not fu...
[19:02:04]
<Yunohost Git/Infra notifications> [borg_ynh] olberger [commented](https://github.com/YunoHost-Apps/borg_ynh/issues/120#issuecomment-2629515078) on [issue #120](https://github.com/YunoHost-Apps/borg_ynh/issues/120) Possibility to add --remote-path parameter: I havent yet thought about adding the configurable setting in the Web interface, but only support it if set through set...
[19:07:19]
<Yunohost Git/Infra notifications> [synapse_ynh] Josue-T merged [pull request #512](https://github.com/YunoHost-Apps/synapse_ynh/pull/512): Testing
[20:26:53]
<m606> Hello, I wonder - where would be the good place to add notes on specific tricks for a specific package for future maintenance (be it done by me who forget stuffs everytime, or potential other maintainer)?
[20:29:12]
<m606> like a README to the attention of maintainners... One place could be in the yunobot auto-PR's body, but I don't think packagers can't customizes its content, can they ?
[20:38:38]
<orhtej2> reading something from `doc/` folder [here](https://github.com/YunoHost/apps_tools/blob/0b1977c764f693522aabc12b7d1e5126d5ce4218/autoupdate_app_sources/autoupdate_app_sources.py#L220) seems doable so not a bad idea.
Or make the bot fill under the pr template from `.github/PULL_REQUEST_TEMPLATE.md`
[20:44:59]
<Yunohost Git/Infra notifications> [penpot_ynh] orhtej2 created new branch orhtej2-patch-1
[20:45:00]
<Yunohost Git/Infra notifications> [penpot_ynh] orhtej2 pushed 1 commit to orhtej2-patch-1: Update _common.sh ([6b7e85fc](https://github.com/YunoHost-Apps/penpot_ynh/commit/6b7e85fc5f3dc8318efce71d6681415b897072f7))
[20:45:05]
<Yunohost Git/Infra notifications> [penpot_ynh] orhtej2 opened [pull request #88](https://github.com/YunoHost-Apps/penpot_ynh/pull/88): Update _common.sh
[20:45:14]
<Yunohost Git/Infra notifications> [penpot_ynh] orhtej2 merged [pull request #88](https://github.com/YunoHost-Apps/penpot_ynh/pull/88): Update _common.sh
[20:45:22]
<Yunohost Git/Infra notifications> [penpot_ynh] orhtej2 opened [pull request #89](https://github.com/YunoHost-Apps/penpot_ynh/pull/89): v2.4.3
[20:46:05]
<Yunohost Git/Infra notifications> [penpot_ynh] orhtej2 pushed 2 commits to testing ([202bb64e5b0c...3940fea38b59](https://github.com/YunoHost-Apps/penpot_ynh/compare/202bb64e5b0c...3940fea38b59))
[20:46:06]
<Yunohost Git/Infra notifications> [penpot_ynh/testing] Merge branch master into testing - orhtej2
[20:46:06]
<Yunohost Git/Infra notifications> [penpot_ynh/testing] Auto-update READMEs - yunohost-bot
[21:24:47]
<lapineige> > <@m606:matrix.org> like a README to the attention of maintainners... One place could be in the yunobot auto-PR's body, but I don't think packagers can't customizes its content, can they ?
In the package base readme ? Sure it's not the best place for general users, but it's visible.
[21:38:07]
<m606> > reading something from `doc/` folder [here](https://github.com/YunoHost/apps_tools/blob/0b1977c764f693522aabc12b7d1e5126d5ce4218/autoupdate_app_sources/autoupdate_app_sources.py#L220) seems doable so not a bad idea.
> Or make the bot fill under the pr template from `.github/PULL_REQUEST_TEMPLATE.md`
Interesting thanks. the issue with `PULL_REQUEST_TEMPLATE.md` is that it would apply to all PR, which in the case of that app I think of would be strange as instructions I'd like to note down are related to the "upgrade sources" case.
[21:38:44]
<m606> > In the package base readme ? Sure it's not the best place for general users, but it's visible.
Indeed, I feel that would too be long and meaningless (not to say frightening) for general users.
[21:39:17]
<lapineige> Even if it's in the end of the readme, with a tittle like "for package contributors" ?
[21:41:00]
<Yunohost Git/Infra notifications> [roundcube_ynh] texttheater opened [pull request #228](https://github.com/YunoHost-Apps/roundcube_ynh/pull/228): Update ADMIN.md
[21:41:49]
<Yunohost Git/Infra notifications> [roundcube_ynh] texttheater opened [pull request #229](https://github.com/YunoHost-Apps/roundcube_ynh/pull/229): Update ADMIN_fr.md
[21:42:51]
<m606> Yes I mean, that's an option worth to be considered of course. But I may be more comfortable with a `/doc/CONTRIBUTORS.md` (or so) that would ideally be reflected in yunobot's autoupdate PR
[21:45:39]
<m606> At the same time, I believe non-potential-maintainers are not that likely to read the package base README (they would discover app via the catalog I guess)
[21:57:08]
<Yunohost Git/Infra notifications> [apps] yunohost-bot opened [pull request #2794](https://github.com/YunoHost/apps/pull/2794): Add DD Builder to wishlist
[21:57:09]
<Yunohost Git/Infra notifications> [apps] yunohost-bot labeled Wishlist on [pull request #2794](https://github.com/YunoHost/apps/pull/2794): Add DD Builder to wishlist
[23:19:47]
<Yunohost Git/Infra notifications> [piped_ynh] yunohost-bot opened [pull request #178](https://github.com/YunoHost-Apps/piped_ynh/pull/178): Upgrade to v2025.02.01
[23:20:44]
<Yunohost Git/Infra notifications> [searxng_ynh] yunohost-bot opened [pull request #366](https://github.com/YunoHost-Apps/searxng_ynh/pull/366): Upgrade to v2025.02.02
[23:25:41]
<Yunohost Git/Infra notifications> Autoupdater just ran, here are the results:
- 16 pending update PRs
- 22 new apps PRs
- 9 failed apps updates: appflowy, autobd, benevalibre, elabftw, gitlab-runner, lemmy, owncast, stremio, zoraxy
See the full log here: https://paste.yunohost.org/raw/hozoqutono
[23:28:45]
<miro5001> > <@m606:matrix.org> Indeed, I feel that would too be long and meaningless (not to say frightening) for general users.
General users may become app maintainers. It's not frightening, it will lead curious people to understand how it works and read more about packaging. Non interested people will stop reading where they want
[23:41:07]
<Yunohost Git/Infra notifications> [penpot_ynh] orhtej2 merged [pull request #89](https://github.com/YunoHost-Apps/penpot_ynh/pull/89): v2.4.3
[23:51:52]
<Yunohost Git/Infra notifications> [apps] yunohost-bot opened [pull request #2795](https://github.com/YunoHost/apps/pull/2795): Add Cytube to wishlist
[23:51:53]
<Yunohost Git/Infra notifications> [apps] yunohost-bot labeled Wishlist on [pull request #2795](https://github.com/YunoHost/apps/pull/2795): Add Cytube to wishlist
[23:58:52]
<Yunohost Git/Infra notifications> yalh76 deleted repository private-gpt_ynh https://github.com/YunoHost-Apps/private-gpt_ynh