[01:49:21]
<Yunohost Git/Infra notifications> [element_ynh] yunohost-bot opened [pull request #210](https://github.com/YunoHost-Apps/element_ynh/pull/210): Upgrade to v1.11.98
[01:52:00]
<Yunohost Git/Infra notifications> [synapse_ynh] centralscrutinizer21 [commented](https://github.com/YunoHost-Apps/synapse_ynh/pull/528#issuecomment-2822605748) on [issue #528](https://github.com/YunoHost-Apps/synapse_ynh/pull/528) Element call backend: Tested and working also for me 👍
[01:53:07]
<Yunohost Git/Infra notifications> Autoupdater just ran, here are the results:
- 28 pending update PRs
- 15 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/giquyusubo
[02:03:32]
<m606> > <@rodinux:matrix.org> hello, I have tried add a config_panel for node_exporter_ynh, but I still have difficulties... I try add a `_ynh_app_config_apply` but can't get it working... Still figthing ...
hello, I encountered the same issue recently. Using `ynh_app_config_apply` in `script/config` does overwrite the default helper as mentioned in the doc however `_ynh_app_config_apply` does not seem to overwrite its equivalent (this one is not mentioned in the doc but I had thought too that somehow it could have worked - I haven't digged why it does not though).
[02:03:34]
<m606> > <@m606:matrix.org> hello, I encountered the same issue recently. Using `ynh_app_config_apply` in `script/config` does overwrite the default helper as mentioned in the doc however `_ynh_app_config_apply` does not seem to overwrite its equivalent (this one is not mentioned in the doc but I had thought too that somehow it could have worked - I haven't digged why it does not though).
actually I am doubting what I just said... I already kind of forgot what happened exactly. But I ended up successfully overwriting `ynh_app_config_apply_one` https://github.com/YunoHost-Apps/marl_ynh/blob/62d863d13b81d0c197995fb4207088894ec124ca/scripts/config#L80C1-L80C21
[02:03:36]
<m606> > <@rodinux:matrix.org> If you use a `ynh_app_config_apply() {
> }` you need also put a `ynh_app_config_run $1` ??
if you mean in `scripts/config`, yes you need to call it at the end of the file because "run" is the helper which triggers all the other config helpers (including "apply"): https://github.com/YunoHost/yunohost/blob/f8f77ade6950590ef2771d4e0b550a8a591e5cd7/helpers/helpers.v2.1.d/config#L299
[02:20:13]
<rodinux> Ok, I finally found how resolve... I would like to do better but it works like this... Thanks m606
[02:20:13]
<Yunohost Git/Infra notifications> [apps_tools] oleole39 opened [pull request #33](https://github.com/YunoHost/apps_tools/pull/33): readme_generator: check that logo exists
[07:30:57]
<Yunohost Git/Infra notifications> [apps_tools] tituspijean [commented](https://github.com/YunoHost/apps_tools/pull/33#discussion_r2055429025) on pull request #33 readme_generator: check that logo exists: Would raising an exception stop the execution of the whole script? If so, maybe only a warning should be output, and the...
[10:16:41]
<Yunohost Git/Infra notifications> [apps] zeroheure opened [pull request #2933](https://github.com/YunoHost/apps/pull/2933): Put "redirect" in small utilities category
[11:36:52]
<Yunohost Git/Infra notifications> [apps_tools] oleole39 [commented](https://github.com/YunoHost/apps_tools/pull/33#discussion_r2055857201) on pull request #33 readme_generator: check that logo exists: I wondered about that too, but couldnt imagine a case where that would be an issue that the script stops if requests ...
[11:37:34]
<Yunohost Git/Infra notifications> [apps_tools] oleole39 edited a [comment](https://github.com/YunoHost/apps_tools/pull/33#discussion_r2055857201) on pull request #33 readme_generator: check that logo exists: I wondered about that too, but couldnt imagine a case where that would be an issue that the script stops if requests ...
[11:43:38]
<Yunohost Git/Infra notifications> [apps_tools] oleole39 edited a [comment](https://github.com/YunoHost/apps_tools/pull/33#discussion_r2055857201) on pull request #33 readme_generator: check that logo exists: I wondered about that too, but couldnt imagine a case where that would be an issue that the script stops if requests ...
[12:53:41]
<Yunohost Git/Infra notifications> [my_webapp_ynh] orhtej2 [commented](https://github.com/YunoHost-Apps/my_webapp_ynh/issues/151#issuecomment-2824203950) on [issue #151](https://github.com/YunoHost-Apps/my_webapp_ynh/issues/151) db_access.txt is not created anymore: The issue is in wording, db connection details should now be presented as a part of POST_INSTALL page (also visible on a...
[12:54:46]
<Yunohost Git/Infra notifications> [my_webapp_ynh] orhtej2 [commented](https://github.com/YunoHost-Apps/my_webapp_ynh/issues/149#issuecomment-2824207236) on [issue #149](https://github.com/YunoHost-Apps/my_webapp_ynh/issues/149) sed: -e expression #1, char 17: unterminated s command --- when SFTP enabled: This one should be resolved on testing branch already
[13:56:18]
<Yunohost Git/Infra notifications> [nextcloud_ynh] tio-trom [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/798#issuecomment-2824395936) on [issue #798](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/798) Many errors after the latest update: I fixed the LDAP error.
The "Your web server is not yet properly set up to allow file synchronization, because the Web...
[13:56:48]
<Yunohost Git/Infra notifications> [nextcloud_ynh] tio-trom [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/798#issuecomment-2824395936) on [issue #798](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/798) Many errors after the latest update: I fixed the LDAP error.
But this:
> "Your web server is not yet properly set up to allow file synchronization, becau...
[16:06:22]
<Yunohost Git/Infra notifications> [package_linter] Salamandar created new branch ynh_setup_source_install_dir
[16:06:22]
<Yunohost Git/Infra notifications> [package_linter] Salamandar pushed 1 commit to ynh_setup_source_install_dir: Detect apps running ynh_setup_source --dest_dir=install_dir ([0b733046](https://github.com/YunoHost/package_linter/commit/0b7330461172922c1b594d172f4ddcf213980c23))
[16:10:24]
<Yunohost Git/Infra notifications> [package_linter] Salamandar opened [pull request #177](https://github.com/YunoHost/package_linter/pull/177): Detect apps running ynh_setup_source --dest_dir=install_dir
[16:19:40]
<Aleks (he/him/il/lui)> uuuuuuuuuuuuh is it not like the majority of apps ? x_x
[16:21:08]
<Yunohost Git/Infra notifications> [package_linter] Salamandar pushed 1 commit to ynh_setup_source_install_dir: Detect apps running ynh_setup_source --dest_dir=install_dir ([b8f3060c](https://github.com/YunoHost/package_linter/commit/b8f3060caa512baa5ce869bd9f9fed04e6874609))
[16:21:33]
<Salamandar> it probably is
[16:22:26]
<Salamandar> but still, it's bad practice, we discussed that months ago
[16:22:26]
<Salamandar> also that's why i only put it as info
[16:23:01]
<Salamandar> using a subdir also allow backing up without the "build cache" of nodejs
[16:23:52]
<Aleks (he/him/il/lui)> ```
$ grep -Enr 'ynh_setup_source.*dest_dir="?\$install_dir"?$' */scripts/install | wc -l
354
```
[16:25:42]
<Aleks (he/him/il/lui)> agree there's an issue but to me that shounds unrealistic to change 350+ apps without a clear recommendation of what should be done instead and/or some autopatch mechanism ...
we could also dig other solutions such as forbidding classic homedir files or ignoring build stuff from backups idk
[16:27:18]
<Aleks (he/him/il/lui)> otherwise people will be like "ok i'm gonna do everything in $install_dir/app" including npm build or whatever (i don't even know how/when exactly the build cache of nodejs is created) and we're back to square one with even more of a mess
[17:16:36]
<Salamandar> that would still be better
[18:00:43]
<Aleks (he/him/il/lui)> how would it still be better, we'd have exactly the same issues but probably with less standard practices so it would be even messier to fix ...
[21:34:49]
<Yunohost Git/Infra notifications> [synapse_ynh] 32bitx64bit [commented](https://github.com/YunoHost-Apps/synapse_ynh/pull/528#issuecomment-2825550915) on [issue #528](https://github.com/YunoHost-Apps/synapse_ynh/pull/528) Element call backend: Upgraded my existing instance to the element call branch. Doesnt seem to work. Still getting the error "MISSING_MATRIX_...
[21:54:50]
<Yunohost Git/Infra notifications> [synapse_ynh] centralscrutinizer21 [commented](https://github.com/YunoHost-Apps/synapse_ynh/pull/528#issuecomment-2825580719) on [issue #528](https://github.com/YunoHost-Apps/synapse_ynh/pull/528) Element call backend: @32bitx64bit did you upgrade with a command line or via admin interface?
[22:10:05]
<Yunohost Git/Infra notifications> [element_ynh] yunohost-bot opened [pull request #211](https://github.com/YunoHost-Apps/element_ynh/pull/211): Upgrade to v1.11.99
[22:27:41]
<Yunohost Git/Infra notifications> Autoupdater just ran, here are the results:
- 38 pending update PRs
- 14 new apps PRs
- 5 failed apps updates: dokos, funkwhale, khatru-pyramid, pixelfedglitch, snweb
See the full log here: https://paste.yunohost.org/raw/rumeruhefa
[23:00:12]
<Yunohost Git/Infra notifications> [synapse_ynh] Josue-T pushed 9 commits to matrix_v2 ([11f8d2102287...117665e25f0e](https://github.com/YunoHost-Apps/synapse_ynh/compare/11f8d2102287...117665e25f0e))
[23:00:13]
<Yunohost Git/Infra notifications> [synapse_ynh/matrix_v2] Improve livekit config - Josué Tille
[23:00:14]
<Yunohost Git/Infra notifications> [synapse_ynh] Josue-T merged [pull request #528](https://github.com/YunoHost-Apps/synapse_ynh/pull/528): Element call backend
[23:00:14]
<Yunohost Git/Infra notifications> [synapse_ynh/matrix_v2] Auto-update READMEs - yunohost-bot
[23:00:14]
<Yunohost Git/Infra notifications> [synapse_ynh/matrix_v2] Put SFU in subpath - Josué Tille
[23:01:14]
<Yunohost Git/Infra notifications> [synapse_ynh] Josue-T deleted branch element_call
[23:03:07]
<Yunohost Git/Infra notifications> [synapse_ynh] Josue-T edited [pull request #439](https://github.com/YunoHost-Apps/synapse_ynh/pull/439): Matrix v2
[23:06:22]
<Yunohost Git/Infra notifications> [synapse_ynh] Josue-T [commented](https://github.com/YunoHost-Apps/synapse_ynh/pull/439#issuecomment-2825723297) on [issue #439](https://github.com/YunoHost-Apps/synapse_ynh/pull/439) Matrix v2: Hello, Ive a good and a bad news 😄 The good one is that element call backend is now supported in matrix_v2 and I pla...
[23:06:33]
<Yunohost Git/Infra notifications> [synapse_ynh] Josue-T just made [pull request #439](https://github.com/YunoHost-Apps/synapse_ynh/pull/439) ready for review: Matrix v2
[23:29:49]
<Yunohost Git/Infra notifications> [synapse_ynh] 32bitx64bit [commented](https://github.com/YunoHost-Apps/synapse_ynh/pull/528#issuecomment-2825760400) on [issue #528](https://github.com/YunoHost-Apps/synapse_ynh/pull/528) Element call backend: > @32bitx64bit did you upgrade with a command line or via admin interface? Via command line.
[23:33:26]
<Yunohost Git/Infra notifications> [synapse_ynh] 32bitx64bit [commented](https://github.com/YunoHost-Apps/synapse_ynh/pull/528#issuecomment-2825760400) on [issue #528](https://github.com/YunoHost-Apps/synapse_ynh/pull/528) Element call backend: > @32bitx64bit did you upgrade with a command line or via admin interface? Via command line. Maybe I did something wr...