Monday, December 16, 2024
apps@conference.yunohost.org
December
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
31
         

[00:22:34] <Yunohost Git/Infra notifications> [vaultwarden_ynh] l​ooowizz [commented](https://github.com/YunoHost-Apps/vaultwarden_ynh/issues/4#issuecomment-2544229898) on [issue #4](https://github.com/YunoHost-Apps/vaultwarden_ynh/issues/4) Implement LDAP authentication: Currently trying to implement [vaultwarden_ldap](https://github.com/ViViDboarder/vaultwarden_ldap) on my fork [here](htt...
[01:33:10] <Yunohost Git/Infra notifications> [piped_ynh] o​rhtej2 merged [pull request #162](https://github.com/YunoHost-Apps/piped_ynh/pull/162): Upgrade to v2024.12.09
[01:33:16] <Yunohost Git/Infra notifications> [piped_ynh] o​rhtej2 deleted branch ci-auto-update-2024.12.09
[01:33:56] <Yunohost Git/Infra notifications> [piped_ynh] o​rhtej2 opened [pull request #163](https://github.com/YunoHost-Apps/piped_ynh/pull/163): v2024.12.09
[01:34:12] <Yunohost Git/Infra notifications> [piped_ynh] o​rhtej2 merged [pull request #163](https://github.com/YunoHost-Apps/piped_ynh/pull/163): v2024.12.09
[01:35:26] <Yunohost Git/Infra notifications> o​leole39 transferred repository fontcompare_ynh: An easy way to dynamically compare locally hosted fonts https://github.com/YunoHost-Apps/fontcompare_ynh
[07:13:34] <Yunohost Git/Infra notifications> [synapse_ynh] J​osue-T merged [pull request #505](https://github.com/YunoHost-Apps/synapse_ynh/pull/505): Testing
[08:55:00] <Yunohost Git/Infra notifications> e​ricgaspar edited repository fontcompare_ynh: An easy way to dynamically compare locally hosted fonts https://github.com/YunoHost-Apps/fontcompare_ynh
[12:30:40] <Yunohost Git/Infra notifications> [snappymail_ynh] e​ricgaspar [commented](https://github.com/YunoHost-Apps/snappymail_ynh/pull/190#discussion_r1886738749) on pull request #190 Testing: is this issue related to the path change for --keep option ? https://forum.yunohost.org/t/settings-for-snappymail/3255...
[14:41:00] <m606> > <@m606:matrix.org> So, this seems to get settings when yml setting file is defined: https://github.com/YunoHost/yunohost/blob/dev/src/app.py#L2056
> But in my current case, i havn't set "bind" properties in config-panel.toml so it is meant to default to /etc/yunohost/apps/$app/settings.yml (as per the doc). So it could rather be inside [`_get_AppConfigPanel`](https://github.com/YunoHost/yunohost/blob/dev/src/app.py#L1887-L2126) and following 2 functions. But what I don't get is where the value is stored ? Apparently not in `/etc/yunohost/apps/$app/settings.yml` as I can't find anything related, but definitely somewhere because if I change the getter to not output the starting `#`, `yunohost app config get fontcompare --full --debug` outputs `value: 78b7de` (instead of None with starting `#`). `grep -rli '78b7de' /etc/yunohost/` returns nothing.
> My objective being to find the code triggering the yaml ouput of `yunohost app config get fontcompare --full --debug` to see whether I could have the HTML color code value between quotes.

@Alekswag:matrix.org Eureka (or almost)!
https://github.com/YunoHost/issues/issues/2501
https://github.com/YunoHost/yunohost/pull/2016
[15:59:51] <Yunohost Git/Infra notifications> [mastodon_ynh] n​schiwy [commented](https://github.com/YunoHost-Apps/mastodon_ynh/issues/477#issuecomment-2546014056) on [issue #477](https://github.com/YunoHost-Apps/mastodon_ynh/issues/477) upgrade to 4.2.13~ynh1 is failing: Question for you both - how much RAM does your VPS have?
[16:45:29] <Yunohost Git/Infra notifications> [vaultwarden_ynh] e​ricgaspar pushed 1 commit to testing: Update manifest.toml ([0431759f](https://github.com/YunoHost-Apps/vaultwarden_ynh/commit/0431759f5cd0e64ab4cf3944bb4985f44baa5893))
[16:46:24] <Yunohost Git/Infra notifications> [joplin_ynh] e​ricgaspar pushed 1 commit to testing: Update manifest.toml ([a7c44163](https://github.com/YunoHost-Apps/joplin_ynh/commit/a7c4416333ad32c434bb61a674a7efa62285a70a))
[16:57:04] <Yunohost Git/Infra notifications> [apps] a​lexAubin merged [pull request #2745](https://github.com/YunoHost/apps/pull/2745): Fix errors in manifest with YNH version and digit name sources
[16:57:04] <Yunohost Git/Infra notifications> [apps] a​lexAubin pushed 1 commit to master: Fix errors in manifest with YNH version and digit sources (#2745) fix https://ci-apps-dev.yunohost.org/ci/job/4772 lint... ([556c1bec](https://github.com/YunoHost/apps/commit/556c1bec30bf6a33bfb0e1ca7fcdb0aca62374a4))
[16:57:06] <Yunohost Git/Infra notifications> [apps] a​lexAubin deleted branch ericgaspar-patch-1
[17:28:38] <m606> do we have any tool to edit given toml values without touching the format of manifest.toml, or to beautify resulting TOML markup as per standard manifest.toml presentation ?
the best tool I had come across to edit values is https://github.com/kislyuk/yq (`tomlq -t '.resources.sources.main.url = "test"' manifest.toml`)
But it removes all the original file's indentation... https://github.com/kislyuk/yq/issues/164#issuecomment-1620799292
[17:29:19] <m606> for the context, I'd like to use this in a Github workflow
[17:29:59] <Aleks (he/him/il/lui)> yeah hmpf toml's tooling ecosystem is a bit meh in that regard, there's no standard formatting, or at least no standard that includes indentation ...
[17:31:04] <m606> hmm, so should i `sed` instead ?
[17:31:27] <m606> less convenient, but it can work
[17:31:44] <Aleks (he/him/il/lui)> at the time of v1->v2 i used something like this which is not super convenient : https://github.com/YunoHost/apps/blob/4cab9ecc4565f559091fde2dc26b1e149995379d/tools/packaging_v2/convert_app_to_packaging_v2.py#L182
[17:32:10] <Aleks (he/him/il/lui)> (using tomlkit)
[17:34:25] <m606> hmm so I would edit using `tomlq` and then run such a beautifying python script calling `tomlkit`
[17:37:34] <m606> thanks I'll think about it. Somehow `tomlq` is convenient as it is flexible and would be reusable, but maybe it my current case where only 2 values have to be modified, it could all be simplified with `sed` with appropriate regex.
[17:41:00] <Aleks (he/him/il/lui)> random google search, i'm finding https://ota-meshi.github.io/eslint-plugin-toml/rules/indent.html
[17:41:40] <Aleks (he/him/il/lui)> maybe with keyValuePairs = 0 and subTables = 1 with 4 space is what we'd want
[17:42:03] <Aleks (he/him/il/lui)> ah but it doesnt autofix it i suppose
[17:42:31] <Yunohost Git/Infra notifications> [nextcloud_ynh] e​ricgaspar pushed 1 commit to v2_PostgreSQL: cleaning ([fb3b7924](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/fb3b792407b499209a9539411045896e51b12012))
[17:49:55] <Yunohost Git/Infra notifications> [searxng_ynh] e​willy merged [pull request #352](https://github.com/YunoHost-Apps/searxng_ynh/pull/352): Upgrade to v2024.12.06
[17:49:58] <Yunohost Git/Infra notifications> [searxng_ynh] e​willy deleted branch ci-auto-update-2024.12.06
[17:50:12] <Yunohost Git/Infra notifications> [searxng_ynh] g​ithub-actions[bot] opened [pull request #353](https://github.com/YunoHost-Apps/searxng_ynh/pull/353): Upgrade master from testing
[17:50:17] <Yunohost Git/Infra notifications> [searxng_ynh] e​willy merged [pull request #353](https://github.com/YunoHost-Apps/searxng_ynh/pull/353): Upgrade master from testing
[17:50:22] <Yunohost Git/Infra notifications> [searxng_ynh] e​willy deleted branch master-promotion
[18:07:52] <m606> > <@Alekswag:matrix.org> ah but it doesnt autofix it i suppose

what do you mean by autofix here ?

I found that which seems to be working too (not that straightforward but no deps...):
```
sed -e '@\[resources.sources.ynh_build\]@,@sha256@!b' \
-e '@url@s@=.*@= "https://example.com/new-file.zip"@' \
-e '@sha256@s@=.*@= "123abc456def"@' \
manifest.toml
```
[18:08:59] <Aleks (he/him/il/lui)> by "autofix" i mean i'm guessing that you can't run eslint-plugin-toml-whatever to magically fix the file, it just reports error and humans are supposed to fix it
[18:09:44] <m606> oh right, yes that's not helping in that case
[19:18:59] <Yunohost Git/Infra notifications> [mastodon_ynh] k​illthealias [commented](https://github.com/YunoHost-Apps/mastodon_ynh/issues/474#issuecomment-2546440981) on [issue #474](https://github.com/YunoHost-Apps/mastodon_ynh/issues/474) Mastodon is not starting after migration to Yunohost 12: I also am having this issue, trying to force a reinstall/update led to this result. Would prefer to find a solution to t...
[20:11:04] <Yunohost Git/Infra notifications> [nextcloud_ynh] e​ricgaspar pushed 1 commit to master: Update manifest.toml ([da79bc5d](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/da79bc5d3404da524a93a4f4a43f28ab7b8f0cfd))
[20:52:49] <Yunohost Git/Infra notifications> [nextcloud_ynh] e​ricgaspar pushed 3 commits to testing ([fc6f6d1d1512...c69a9183b0e0](https://github.com/YunoHost-Apps/nextcloud_ynh/compare/fc6f6d1d1512...c69a9183b0e0))
[20:52:53] <Yunohost Git/Infra notifications> [nextcloud_ynh/testing] Auto-update READMEs - yunohost-bot
[20:52:53] <Yunohost Git/Infra notifications> [nextcloud_ynh/testing] Update manifest.toml - eric_G
[20:52:57] <Yunohost Git/Infra notifications> [nextcloud_ynh/testing] Merge branch master into testing - eric_G
[21:03:03] <Yunohost Git/Infra notifications> [nextcloud_ynh] e​ricgaspar pushed 1 commit to v2_PostgreSQL: cleaning ([d2da1e9e](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/d2da1e9ebfa9736b9cc634f28ceb3d1cd3a53803))
[21:43:46] <Yunohost Git/Infra notifications> [nextcloud_ynh] e​ricgaspar pushed 4 commits to v2_PostgreSQL ([d2da1e9ebfa9...01c4e955d4ff](https://github.com/YunoHost-Apps/nextcloud_ynh/compare/d2da1e9ebfa9...01c4e955d4ff))
[21:43:54] <Yunohost Git/Infra notifications> [nextcloud_ynh/v2_PostgreSQL] Auto-update READMEs - yunohost-bot
[21:43:56] <Yunohost Git/Infra notifications> [nextcloud_ynh/v2_PostgreSQL] Merge branch master into testing - eric_G
[21:43:58] <Yunohost Git/Infra notifications> [nextcloud_ynh/v2_PostgreSQL] Merge branch testing into v2_PostgreSQL - eric_G
[22:19:17] <Yunohost Git/Infra notifications> [nextcloud_ynh] e​ricgaspar pushed 1 commit to v2_PostgreSQL: Update restore ([76389919](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/76389919e84e84a08c0d2c128ac0f78fc1f72bad))