Monday, April 21, 2025
apps@conference.yunohost.org
April
Mon Tue Wed Thu Fri Sat Sun
  1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28 29 30        
             

[06:43:27] <Yunohost Git/Infra notifications> [searxng_ynh] e​willy merged [pull request #387](https://github.com/YunoHost-Apps/searxng_ynh/pull/387): Upgrade to v2025.04.18
[06:43:32] <Yunohost Git/Infra notifications> [searxng_ynh] e​willy deleted branch ci-auto-update-2025.04.18
[06:43:39] <Yunohost Git/Infra notifications> [searxng_ynh] g​ithub-actions[bot] opened [pull request #388](https://github.com/YunoHost-Apps/searxng_ynh/pull/388): Upgrade master from testing
[06:44:26] <Yunohost Git/Infra notifications> [searxng_ynh] e​willy merged [pull request #388](https://github.com/YunoHost-Apps/searxng_ynh/pull/388): Upgrade master from testing
[06:44:26] <Yunohost Git/Infra notifications> [searxng_ynh] e​willy deleted branch master-promotion
[09:35:14] <Yunohost Git/Infra notifications> [package_check] f​florent opened [issue #193](https://github.com/YunoHost/package_check/issues/193): How to run locally? (Python dependency issue)
[11:12:29] <tituspijean[m]> https://ci-apps-dev.yunohost.org/ci/job/9005
I see that the Diagnosis is being run in CI context. Is there a way to prevent that?
[11:13:07] <Aleks (he/him/il/lui)> sounds related to the preinstall steps o.O
[11:13:21] <tituspijean[m]> yup, I need to add a domain for Dex, so it triggers the diagnosis :/
[11:14:17] <Aleks (he/him/il/lui)> ah yeah
[11:14:38] <Aleks (he/him/il/lui)> meh
[11:14:57] <tituspijean[m]> (trolling: the other solution is to integrate OIDC in YunoHost 🙃 )
[11:15:04] <Aleks (he/him/il/lui)> lolze
[11:15:58] <Aleks (he/him/il/lui)> hmmyeah anyway i guess we could tweak the core code to skip the diagnosis if `YNH_CI` (or whatever the env var is) is set
[11:16:11] <Aleks (he/him/il/lui)> #goudefeurstichiou
[11:21:55] <tituspijean[m]> I'll write the ticket if you have not already started ;)
[12:36:58] <florent[m]> Question about the [_ynh_apply_default_permissions helpers function](https://github.com/YunoHost/yunohost/blob/f8f77ade6950590ef2771d4e0b550a8a591e5cd7/helpers/helpers.v2.1.d/0-utils#L237) in v2.1: it is typically used when we call `yny_setup_source` to change the permissions on the `$install_dir` folder to remove the rights for `others`, but if the app creates a new file during the runtime the permissions are not set accordingly.

I wonder if the umask should not be changed as well. Would it make sense?
[12:38:19] <Aleks (he/him/il/lui)> hmyeah idk if umask is the right way because the file could be created in various ways but yeah typically the perms have to be reapplied somehow after the entire "setup" is done i suppose
[12:39:08] <Aleks (he/him/il/lui)> removing permission for "others" is mainly important for the initial folder
[12:39:27] <Aleks (he/him/il/lui)> it doesn't matter that much for files themselves inside the folder since the access is already constrained by the parent
[12:41:13] <florent[m]> Right, I wondered what was the intent of changing the folders right recursively. Based on your answer, it makes sense to not worry about umask then. Thanks!
[13:37:03] <florent[m]> It is not official yet for legal reasons (the project should be white-labeled beforehand):
https://github.com/fflorent/visio_ynh
[13:37:05] <florent[m]> A videoconference solution powered by Livekit
[13:40:24] <florent[m]> I have just acquired the write permission for this Dex app, I took the opportunity to migrate the helpers to v2.1:
https://github.com/YunoHost-Apps/dex\_ynh/pull/75

Should I merge the PR myself or ask for a review? (maybe limezy ?)
[15:41:51] <florent[m]> I saw your approval, thanks limezy!
[15:52:25] <limezy> Thanks for all the work you do !
[15:53:15] <limezy> 5 kids to look after at home and a challenging daily job, hard to find enough time to spend on such updates !
[15:58:13] <eric_G> are the PHP settings set in install script applied on Nextcloud?
[15:58:37] <eric_G> https://github.com/YunoHost-Apps/nextcloud_ynh/blob/c048997b21867d635729c8b98b9097d485644257/scripts/install#L10-L11
[15:59:44] <eric_G> An extract from my `/etc+php/8.3/fpm/pool.d/nextcloud.conf` file:
```
pm.process_idle_timeout = 10s
php_admin_value[upload_max_filesize] = 50M
php_admin_value[post_max_size] = 50M
php_admin_value[memory_limit] = 128M
```

[16:27:06] <limezy> I tried to install the testing branch (with 2.1 helpers) on my server and it still fails. https://paste.yunohost.org/raw/axiyurekar
I think it's unrelated to your branch and work, I've had Dex installation failing on this server since a few weeks now... but nevertheless if you had a clue on what may be broken I'd be glad ! florent
[16:39:09] <m606> Hello @orhtej2:circledsquareroot.ovh, it doesn't seem enough unfortunately - all my recent PR on `testing` branch go without README.md auto-regeneration
[16:40:29] <m606> Hello @orhtej2:circledsquareroot.ovh, it doesn't seem enough unfortunately - all my recent PR on `testing` branches go without README.md auto-regeneration
[16:41:00] <m606> Hello @orhtej2:circledsquareroot.ovh, it doesn't seem enough unfortunately - all my recent PR on `testing` branches (in Yunohost-Apps) go without README.md auto-regeneration
[16:47:14] <florent[m]> Hmm, I do not understand this error. My hope is that I will finish the work on extracting the binary from the docker image very soon, so you won't be much bothered by this.
[17:36:40] <Yunohost Git/Infra notifications> [nextcloud_ynh] M​elolonn [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/801#issuecomment-2819090429) on [issue #801](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/801) Nextcloud gone after trying to upgrade: I now tried restoring with two different backup, and the result is different depending of using command line or admin UI...
[17:43:33] <Yunohost Git/Infra notifications> [nextcloud_ynh] M​elolonn [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/801#issuecomment-2819090429) on [issue #801](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/801) Nextcloud gone after trying to upgrade: I now tried restoring with two different backup, and the result is different depending of using command line or admin UI...
[17:56:45] <Yunohost Git/Infra notifications> S​alamandar edited repository apps: YunoHost apps catalog https://github.com/YunoHost/apps
[17:56:45] <Yunohost Git/Infra notifications> [apps] S​alamandar created new branch main
[17:56:46] <Yunohost Git/Infra notifications> [apps] S​alamandar deleted branch master
[17:56:47] <florent[m]> limezy: If you want to take a look, that should work:
https://github.com/YunoHost-Apps/dex_ynh/pull/77
[17:56:59] <Yunohost Git/Infra notifications> [apps] S​alamandar created new branch master
[17:58:44] <Yunohost Git/Infra notifications> [example_ynh] S​alamandar pushed 1 commit to main: Apps repo default branch is now main ([60c44773](https://github.com/YunoHost/example_ynh/commit/60c4477325a7c57a51ecd5606ebb223a0bbfe54c))
[17:59:50] <Yunohost Git/Infra notifications> [appgenerator] S​alamandar pushed 1 commit to main: Apps repo default branch is now main ([969402c1](https://github.com/YunoHost/appgenerator/commit/969402c14ee69febe6b403ab55449b1199262093))
[18:02:30] <Yunohost Git/Infra notifications> [apps] S​alamandar created new branch now_main
[18:02:31] <Yunohost Git/Infra notifications> [apps] S​alamandar pushed 1 commit to now_main: Apps repo default branch is now main ([af8f5003](https://github.com/YunoHost/apps/commit/af8f5003a94bb30cde57683ff975c09ff0dc6764))
[18:02:42] <Yunohost Git/Infra notifications> [apps] S​alamandar opened [pull request #2928](https://github.com/YunoHost/apps/pull/2928): Apps repo default branch is now main
[18:02:51] <Yunohost Git/Infra notifications> [apps] S​alamandar pushed 1 commit to main: Apps repo default branch is now main ([4b1e9868](https://github.com/YunoHost/apps/commit/4b1e9868248c90650cc09c4df8f6d225c9ff4f0b))
[18:02:52] <Yunohost Git/Infra notifications> [apps] S​alamandar deleted branch now_main
[18:02:52] <Yunohost Git/Infra notifications> [apps] S​alamandar merged [pull request #2928](https://github.com/YunoHost/apps/pull/2928): Apps repo default branch is now main
[18:05:27] <Yunohost Git/Infra notifications> [apps_tools] S​alamandar pushed 1 commit to main: Apps repo default branch is now main ([8e259f9e](https://github.com/YunoHost/apps_tools/commit/8e259f9e26536af544200adb9c81501b8e584b8b))
[18:07:28] <Yunohost Git/Infra notifications> [package_linter] S​alamandar pushed 1 commit to main: Apps repo default branch is now main ([8619cf7f](https://github.com/YunoHost/package_linter/commit/8619cf7fba2ae78a62acbb621b5ae1fc379a9d31))
[18:08:05] <Salamandar> 😬
[18:08:16] <Aleks (he/him/il/lui)> u wot m8
[18:08:39] <Salamandar> (i recreated it just after, but renaming master -> main via the github web ui updated the pull requests on the repo, that was nice)
[18:08:56] <Salamandar> now, gotta ssh on the infra and change the different clones of the apps repo
[18:18:04] <Yunohost Git/Infra notifications> [package_check] S​alamandar pushed 1 commit to main: Update comments about master branch ([c0729ac5](https://github.com/YunoHost/package_check/commit/c0729ac5133ef8553cf890cfad5915b50978f4b7))
[18:29:47] <Yunohost Git/Infra notifications> [nextcloud_ynh] S​alamandar deleted branch main
[18:36:12] <Salamandar> looks like it's ok
[18:50:29] <Yunohost Git/Infra notifications> [apps] S​alamandar deleted branch master
[18:50:48] <Salamandar> master branch is now deleted
[19:03:10] <Yunohost Git/Infra notifications> Autoupdater just ran, here are the results:

- 43 pending update PRs
- 11 new apps PRs
- 11 failed apps updates: autobd, dokos, funkwhale, khatru-pyramid, ofbiz, opencloud, phpldapadmin, phpmyadmin, pixelfedglitch, snweb, stremio

See the full log here: https://paste.yunohost.org/raw/lohahequno
[19:55:47] <Yunohost Git/Infra notifications> [piped_ynh] o​rhtej2 merged [pull request #201](https://github.com/YunoHost-Apps/piped_ynh/pull/201): Upgrade sources
[19:55:51] <Yunohost Git/Infra notifications> [piped_ynh] o​rhtej2 deleted branch ci-auto-update-sources-250421
[20:08:44] <Yunohost Git/Infra notifications> [apps] D​eMiro5001 opened [pull request #2929](https://github.com/YunoHost/apps/pull/2929): Add pretalx to catalog
[20:48:44] <Yunohost Git/Infra notifications> [apps] e​ricgaspar approved [pull request #2929](https://github.com/YunoHost/apps/pull/2929#pullrequestreview-2782218684) Add pretalx to catalog
[20:48:48] <Yunohost Git/Infra notifications> [apps] e​ricgaspar merged [pull request #2929](https://github.com/YunoHost/apps/pull/2929): Add pretalx to catalog
[20:48:48] <Yunohost Git/Infra notifications> [apps] e​ricgaspar pushed 1 commit to main: Add pretalx to catalog (#2929) * pretalx logo * Add pretalx to catalog ([8b588597](https://github.com/YunoHost/apps/commit/8b5885979d418781271509ceb832f0f817aa60df))
[22:20:24] <Yunohost Git/Infra notifications> Autoupdater just ran, here are the results:

- 34 pending update PRs
- 2 new apps PRs
- 10 failed apps updates: dokos, funkwhale, khatru-pyramid, ofbiz, opencloud, phpldapadmin, phpmyadmin, pixelfedglitch, snweb, stremio

See the full log here: https://paste.yunohost.org/raw/oyikikegom
[22:47:15] <Yunohost Git/Infra notifications> [borg_ynh] S​alamandar merged [pull request #201](https://github.com/YunoHost-Apps/borg_ynh/pull/201): Upgrade to v1.4.1
[22:48:39] <Salamandar> readme generator seems ko
[22:59:11] <Paprika> Just tried new readme generator, seems okay, but haven't checked tests. Just reworked badge generation for my yunorunner_ynh repo in accordance to the newest readme generation. Looking solid so far.
[23:00:13] <Yunohost Git/Infra notifications> [nextcloud_ynh] M​elolonn [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/801#issuecomment-2819090429) on [issue #801](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/801) Nextcloud gone after trying to upgrade: I now tried restoring with two different backup, and the result is different depending of using command line or admin UI...
[23:01:00] <Yunohost Git/Infra notifications> [nextcloud_ynh] M​elolonn [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/801#issuecomment-2819090429) on [issue #801](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/801) Nextcloud gone after trying to upgrade: I now tried restoring with two different backup, and the result is different depending of using command line or admin UI...