[00:32:06]
<m606> > <@m606:matrix.org> could that be a bug in the 2.1 helpers? https://github.com/YunoHost/yunohost/blob/9982a775820798aaf2d7920205e3c9c7495f2e58/helpers/helpers.v2.1.d/config#L31
Shouldn't it be `old[$short_setting]="$(ynh_app_setting_get --app=$app --key=$short_setting)"` instead? https://doc.yunohost.org/en/packaging_apps_helpers#ynh-app-setting-get
Though I can't manage to test it successfully - editing `/usr/share/yunohost/helpers.v2.1.d/config` still runs the same command
[06:43:17]
<Yunohost Git/Infra notifications> [stirling-pdf_ynh] ericgaspar deleted branch ci-auto-update-0.45.5
[06:43:17]
<Yunohost Git/Infra notifications> [stirling-pdf_ynh] ericgaspar merged [pull request #50](https://github.com/YunoHost-Apps/stirling-pdf_ynh/pull/50): Upgrade to v0.45.5
[06:43:25]
<Yunohost Git/Infra notifications> [stirling-pdf_ynh] ericgaspar opened [pull request #51](https://github.com/YunoHost-Apps/stirling-pdf_ynh/pull/51): Testing
[06:43:28]
<Yunohost Git/Infra notifications> [stirling-pdf_ynh] ericgaspar merged [pull request #51](https://github.com/YunoHost-Apps/stirling-pdf_ynh/pull/51): Testing
[06:53:16]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar pushed 1 commit to 30.0.9: Update manifest.toml ([4715594a](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/4715594aaead78eeebfbc2f70e96dfaf13e28c29))
[06:53:16]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar created new branch 30.0.9
[06:54:01]
<Yunohost Git/Infra notifications> [nextcloud_ynh/30.0.9] Update manifest.toml - eric_G
[06:54:01]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar pushed 2 commits to 30.0.9 ([4f3b4fb03314...e52761e52fee](https://github.com/YunoHost-Apps/nextcloud_ynh/compare/4f3b4fb03314...e52761e52fee))
[06:54:02]
<Yunohost Git/Infra notifications> [nextcloud_ynh/30.0.9] Merge branch 30.0.9 of https://github.com/YunoHost-Apps/nextcloud_ynh into 30.0.9 - eric_G
[06:54:13]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar opened [pull request #797](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/797): 30.0.9
[07:11:02]
<Thomas> Could someone please reopen https://forum.yunohost.org/t/funkwhale-install-cant-find-server/35668 so we can share the update that Funkwhale is available again? Thanks!
[07:19:34]
<Yunohost Git/Infra notifications> Thovi98 transferred repository garmin-to-fittrackee_ynh https://github.com/YunoHost-Apps/garmin-to-fittrackee_ynh
[07:26:21]
<Yunohost Git/Infra notifications> [apps] Thovi98 opened [pull request #2916](https://github.com/YunoHost/apps/pull/2916): Add garmin-to-fittrackee to catalog
[08:10:09]
<Yunohost Git/Infra notifications> ericgaspar created repository gomft_ynh https://github.com/YunoHost-Apps/gomft_ynh
[08:12:22]
<Yunohost Git/Infra notifications> ericgaspar edited repository gomft_ynh: GoMFT package for YunoHost https://github.com/YunoHost-Apps/gomft_ynh
[08:12:22]
<Yunohost Git/Infra notifications> ericgaspar edited repository gomft_ynh: GoMFT package for YunoHost https://github.com/YunoHost-Apps/gomft_ynh
[08:12:36]
<Yunohost Git/Infra notifications> ericgaspar edited repository gomft_ynh: GoMFT package for YunoHost https://github.com/YunoHost-Apps/gomft_ynh
[08:25:34]
<Yunohost Git/Infra notifications> [apps] ericgaspar pushed 1 commit to gomft: Update apps.toml ([b17a8795](https://github.com/YunoHost/apps/commit/b17a8795093b77fab7e7c3da7e29abeae5c9f065))
[08:25:34]
<Yunohost Git/Infra notifications> [apps] ericgaspar created new branch gomft
[08:28:05]
<Yunohost Git/Infra notifications> [apps] ericgaspar pushed 1 commit to gomft: Create gomft.png ([0b70f517](https://github.com/YunoHost/apps/commit/0b70f5175e3643188ed12f63e41af5c8fefd67a3))
[08:28:12]
<Yunohost Git/Infra notifications> [apps] ericgaspar opened [pull request #2917](https://github.com/YunoHost/apps/pull/2917): Add Gomft
[08:54:40]
<limezy> Hi everybody ! I still can't manage to install Dex app on my server, would you have any idea why ? Since a few weeks I've been struggling with the go helpers refactoring, the app is now shown as working and the CI seems to install it without any issue, but it fails on my two Yunohost servers (one is up to date on main branch, one is op to date on testing branch). Thanks for your help ! https://paste.yunohost.org/raw/kivuyogodo
[09:41:15]
<Yunohost Git/Infra notifications> [apps] ericgaspar merged [pull request #2917](https://github.com/YunoHost/apps/pull/2917): Add Gomft
[09:41:16]
<Yunohost Git/Infra notifications> [apps] ericgaspar deleted branch gomft
[09:41:16]
<Yunohost Git/Infra notifications> [apps] ericgaspar pushed 1 commit to master: Add Gomft (#2917) * Update apps.toml * Create gomft.png ([d46c2e5b](https://github.com/YunoHost/apps/commit/d46c2e5bdf6a37cc3f4472cd673e618c412c8c95))
[09:41:40]
<Yunohost Git/Infra notifications> [apps] ericgaspar approved [pull request #2916](https://github.com/YunoHost/apps/pull/2916#pullrequestreview-2767589742) Add garmin-to-fittrackee to catalog
[09:41:49]
<Yunohost Git/Infra notifications> [apps] ericgaspar pushed 1 commit to master: Add garmin-to-fittrackee to catalog (#2916) ([1f70bd9a](https://github.com/YunoHost/apps/commit/1f70bd9a1bc3b0c7af7daacd71d26630e28a50ed))
[09:41:50]
<Yunohost Git/Infra notifications> [apps] ericgaspar merged [pull request #2916](https://github.com/YunoHost/apps/pull/2916): Add garmin-to-fittrackee to catalog
[10:16:22]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar assigned kay0u on [pull request #797](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/797): 30.0.9
[10:16:34]
<Yunohost Git/Infra notifications> WARNING: unknown pull_request action: unassigned
[10:46:26]
<Yunohost Git/Infra notifications> [synapse_ynh] Josue-T pushed 1 commit to element_call: fix tests ([17c8a1dd](https://github.com/YunoHost-Apps/synapse_ynh/commit/17c8a1dd0496db90f5f4f61645244dcf952dfa75))
[12:16:16]
<Salamandar> 😬
[12:17:00]
<Salamandar> https://aria.im/_bifrost/v1/media/download/AXny50IHz0mGPKV27AqYPIFMaUzn3tr5Iw0BPoZDQ5cGjldOgkP7jjgdHWA9KLbGIV4Q6DeEWjc9dY4tfOjL1hxCeWPof46gAG1hdHJpeC5vcmcveGtPSFVibXJ3Tktpb1pJWmtaTGdaeHht
[12:18:18]
<Salamandar> uuuuh ctrl-shift-r ? Looks like my background color change but partially
[12:18:27]
<Salamandar> oh humm by the way in mobile mode the dark/light button isn't present
[12:25:05]
<Salamandar> @josue:tille.ch uuuuuuuuh
[12:25:15]
<Salamandar> https://aria.im/_bifrost/v1/media/download/AfUjzdfDPgeU8c4eEjJ6JTuO4DdR_AvyQbyF9pFWpjLSSC9cpWIZ7Du4f3NBlSXR4EutkH1KwEUiBwmIudLme2FCeWPo-D0gAG1hdHJpeC5vcmcvcUpkdFd4WnZWaklYY0FmVGxkR1NGakFr
[12:25:25]
<Salamandar> it's running infinitely
[12:25:49]
<Salamandar> ah no not infinitely but it ran 38 times for a server with like 8 accounts
[12:26:48]
<Josué> ok and for how many organisation on forgejo ?
[12:29:29]
<Salamandar> hmmmm
[12:29:32]
<Salamandar> not many imho
[12:30:02]
<Salamandar> just one actually
[12:30:15]
<Salamandar> 13 users actually
[12:30:20]
<Josué> Can you report the issue with the full log I'll check it this evening.
[12:30:29]
<Salamandar> yeah
[12:32:26]
<Salamandar> https://paste.yunohost.org/raw/nuyeyanilu
[12:32:31]
<Salamandar> it's running for every installed app o.O
[12:34:10]
<m606> how can I test a change in helper file on a YNH instance?
I tried editing `/usr/share/yunohost/helpers.v2.1.d/config` but debug log shows that it still use original version for some reasons ? https://paste.yunohost.org/qagijaceke.txt (I expected to see `ynh_app_setting_get --app=marl --key=archives_paths` on line 122)
[12:38:07]
<Josué> Ok thanks, I'll check this when I've some time.
[12:46:56]
<orhtej2> > <@Salamandar:matrix.org> oh humm by the way in mobile mode the dark/light button isn't present
Yup
[12:50:08]
<m606> > <@m606:matrix.org> how can I test a change in helper file on a YNH instance?
> I tried editing `/usr/share/yunohost/helpers.v2.1.d/config` but debug log shows that it still use original version for some reasons ? https://paste.yunohost.org/qagijaceke.txt (I expected to see `ynh_app_setting_get --app=marl --key=archives_paths` on line 122)
and now it works... 🪄
[13:27:31]
<m606> Lately after merging PRs from `testing` to `master` on Github, it says that `testing` is late on `master` with commits such as `Merge pull request #50 from YunoHost-Apps/testing` (tracks of PR merge). Is it a good practice to merge them from `master` to `testing`? Is there a way to avoid it?
[13:37:58]
<Aleks (he/him/il/lui)> zmlerf
[13:38:28]
<Aleks (he/him/il/lui)> not sure how to avoid it but if it piles up too much then yeah just merge master in testing
[13:46:47]
<orhtej2> > <@m606:matrix.org> Lately after merging PRs from `testing` to `master` on Github, it says that `testing` is late on `master` with commits such as `Merge pull request #50 from YunoHost-Apps/testing` (tracks of PR merge). Is it a good practice to merge them from `master` to `testing`? Is there a way to avoid it?
There's a hook that should automatically merge back
[13:46:47]
<Aleks (he/him/il/lui)> oooh indeed i forgot about that :O
[14:55:49]
<m606> does it need to be enabled on my side somehow ? I am under the impression that this is kind of new (several weeks maybe)
[15:06:23]
<m606> is there a way to prevent logging replacing strings by stars (even temporarily) ? I may have a bug there but I would need to see the strings.
`2025-04-15 14:58:56,254: DEBUG - + ynh_replace '--match=^\s{2}"**********.*' '--replace= "**********": false' --file=/var/www/marl/js/config.js`
[15:07:08]
<Aleks (he/him/il/lui)> i guess you could comment the corresponding code
[15:07:08]
<Aleks (he/him/il/lui)> somewhere in there https://github.com/YunoHost/yunohost/blob/dev/src/log.py#L552-L561
[15:07:19]
<Aleks (he/him/il/lui)> on a live system the corresponding file is in uruuuguh something like /usr/lib/python3/dist-packages/yunohost/log.py
[15:07:51]
<Aleks (he/him/il/lui)> (and then you need to restart yunohost-api if you want it to propagate to the webadmin)
[15:08:59]
<m606> perfect thanks
[15:12:25]
<m606> interesting it's because the variable name includes `key` i'll rather change that
[16:00:40]
<Yunohost Git/Infra notifications> ericgaspar renamed repository homebox_ynh: HomeBox package for YunoHost https://github.com/YunoHost-Apps/homebox_ynh
[16:01:48]
<Yunohost Git/Infra notifications> [nextcloud_ynh] kay0u approved [pull request #797](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/797#pullrequestreview-2768987057) 30.0.9
[16:06:44]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar pushed 4 commits to v2_PostgreSQL ([522f619ab827...b713f30e3989](https://github.com/YunoHost-Apps/nextcloud_ynh/compare/522f619ab827...b713f30e3989))
[18:58:34]
<Yunohost Git/Infra notifications> [apps] ericgaspar pushed 1 commit to homebox: Update apps.toml ([e6e6b443](https://github.com/YunoHost/apps/commit/e6e6b4439325f40560ef300b53f2e8f2246b28d8))
[18:58:35]
<Yunohost Git/Infra notifications> [apps] ericgaspar created new branch homebox
[18:58:35]
<Yunohost Git/Infra notifications> [apps] ericgaspar opened [pull request #2918](https://github.com/YunoHost/apps/pull/2918): Add Homebox
[18:58:35]
<Yunohost Git/Infra notifications> [apps] ericgaspar pushed 1 commit to master: Add Homebox (#2918) * Update apps.toml * Create homebox.png * Update wishlist.toml ([962ca052](https://github.com/YunoHost/apps/commit/962ca0529f8767ea6eee11596e4b469eede4f3b6))
[18:58:35]
<Yunohost Git/Infra notifications> [apps] ericgaspar pushed 1 commit to homebox: Update wishlist.toml ([91743adf](https://github.com/YunoHost/apps/commit/91743adfc625c602e0c1dc1f21d47e34c3705fdc))
[18:58:35]
<Yunohost Git/Infra notifications> [apps] ericgaspar pushed 1 commit to homebox: Create homebox.png ([5b4caa5a](https://github.com/YunoHost/apps/commit/5b4caa5a8e169f03d3cd03bc77351a5ba52c6a9b))
[18:58:35]
<Yunohost Git/Infra notifications> [apps] ericgaspar deleted branch homebox
[18:58:35]
<Yunohost Git/Infra notifications> [apps] ericgaspar merged [pull request #2918](https://github.com/YunoHost/apps/pull/2918): Add Homebox
[19:01:05]
<Yunohost Git/Infra notifications> ericgaspar created repository yeetfile_ynh https://github.com/YunoHost-Apps/yeetfile_ynh
[19:01:09]
<Yunohost Git/Infra notifications> ericgaspar edited repository yeetfile_ynh: YeetFile package for YunoHost https://github.com/YunoHost-Apps/yeetfile_ynh
[19:01:09]
<Yunohost Git/Infra notifications> ericgaspar edited repository yeetfile_ynh: YeetFile package for YunoHost https://github.com/YunoHost-Apps/yeetfile_ynh
[19:01:09]
<Yunohost Git/Infra notifications> ericgaspar edited repository yeetfile_ynh https://github.com/YunoHost-Apps/yeetfile_ynh
[19:05:45]
<Yunohost Git/Infra notifications> [synapse_ynh] Josue-T edited [pull request #528](https://github.com/YunoHost-Apps/synapse_ynh/pull/528): Element call backend
[19:05:46]
<Yunohost Git/Infra notifications> [apps] ericgaspar pushed 2 commits to yeetfile ([b8f0bf3a6993^...9c240ca532a4](https://github.com/YunoHost/apps/compare/b8f0bf3a6993^...9c240ca532a4))
[19:05:46]
<Yunohost Git/Infra notifications> [apps/yeetfile] Update apps.toml - eric_G
[19:05:46]
<Yunohost Git/Infra notifications> [apps] ericgaspar created new branch yeetfile
[19:05:47]
<Yunohost Git/Infra notifications> [apps] ericgaspar opened [pull request #2919](https://github.com/YunoHost/apps/pull/2919): Add Yeetfile
[19:05:47]
<Yunohost Git/Infra notifications> [apps/yeetfile] Create yeetfile.png - eric_G
[19:07:00]
<m606> I am confused again - in upgrade script, should we initialize settings variables (`archives_paths="$(ynh_app_setting_get --app=$app --key=archives_paths)"`) for them to replace config template tags (`ynh_add_config`) ?
[19:07:01]
<m606> or managed by YNH magic?
[19:11:10]
<m606> > <@m606:matrix.org> or managed by YNH magic?
magic I guess!
> During other scripts, all app settings are also loaded and automatically available.
https://doc.yunohost.org/en/packaging_scripts
[20:18:22]
<Josué> Salamandar, did you upgrade forgejo from testing or is it the master version ?
[20:27:43]
<m606> > <@m606:matrix.org> magic I guess!
> > During other scripts, all app settings are also loaded and automatically available.
> https://doc.yunohost.org/en/packaging_scripts
or maybe not ?
```
> Validating that the app marl can/can't be accessed with its URL...
URL : https://sub.domain.tld/
Title : MARL - Mastodon Archive Reader Lite
Content extract:
JavaScript is required for this app to run.
Assets :
- https://sub.domain.tld/_astro/index.D5XH6sEB.css
- https://sub.domain.tld/js/config.js?v=2.8.1
OK
========
903 WARNING ./config: line 57: archives_paths: unbound variable
1505 ERROR Failed to read config panel values.
```
https://ci-apps-dev.yunohost.org/ci/job/8878
[20:27:57]
<Aleks (he/him/il/lui)> https://aria.im/_bifrost/v1/media/download/AQSL3G5oPSJlDxAEm9HL1E1M68o1tgVn1w1Xk2OAjrZ7bVjmDEJO-ugnbDv1DgY8OvOt6xdu6TH0fR71wS1sHp5CeWQElxoQAG1hdHJpeC5vcmcvUGhlV2pVQlRwdUlURHVYQnZCRUdLQkRj
[20:28:26]
<Aleks (he/him/il/lui)> why is the OK displayed in red omg xD
[20:29:23]
<mathieuw> UX hell.
Or communist propaganda ?
[20:41:49]
<m606> > <@m606:matrix.org> or maybe not ?
> ```
> > Validating that the app marl can/can't be accessed with its URL...
>
> URL : https://sub.domain.tld/
> Title : MARL - Mastodon Archive Reader Lite
> Content extract:
> JavaScript is required for this app to run.
> Assets :
> - https://sub.domain.tld/_astro/index.D5XH6sEB.css
> - https://sub.domain.tld/js/config.js?v=2.8.1
> OK
> ========
> 903 WARNING ./config: line 57: archives_paths: unbound variable
> 1505 ERROR Failed to read config panel values.
> ```
> https://ci-apps-dev.yunohost.org/ci/job/8878
So that's a migration issue...
In previous package version such variable was not initialized/stored. Where should I initialize it in new package version ? In upgrade script ? when upgrading a package YNH runs the upgrade file of the new package ?
[20:42:30]
<Aleks (he/him/il/lui)> yes : https://github.com/YunoHost-Apps/nextcloud_ynh/blob/master/scripts/upgrade#L28
[20:43:11]
<Aleks (he/him/il/lui)> also if you're using helpers 2.1, you may want to use the `ynh_app_setting_default`(?) instead of the complex "if -z ... define the setting"
[21:40:28]
<m606> FYI connection to YNH infra was lost for a short moment, and CI (or/and yunobot?) is unresponsive: can't restart that job nor trigger a new `!testme`
[21:40:32]
<m606> https://ci-apps-dev.yunohost.org/ci/job/8879
[22:24:00]
<Yunohost Git/Infra notifications> Autoupdater just ran, here are the results:
- 12 pending update PRs
- 17 new apps PRs
- 18 failed apps updates: appflowy, dokos, funkwhale, jupyterlab, khatru-pyramid, kiwix, languagetool, lemmy, localai, ofbiz, opencloud, phpldapadmin, phpmyadmin, pixelfedglitch, redlib, snweb, stremio, swingmusic
See the full log here: https://paste.yunohost.org/raw/lexumeluzo
[23:01:42]
<miro5001> > <@yunohostinfra:matrix.org> Autoupdater just ran, here are the results:
>
> - 12 pending update PRs
> - 17 new apps PRs
> - 18 failed apps updates: appflowy, dokos, funkwhale, jupyterlab, khatru-pyramid, kiwix, languagetool, lemmy, localai, ofbiz, opencloud, phpldapadmin, phpmyadmin, pixelfedglitch, redlib, snweb, stremio, swingmusic
>
> See the full log here: https://paste.yunohost.org/raw/lexumeluzo
```
Checking main ...
Ignoring tag RELEASE_5_2_2, doesn't look like a version number
```
Then
```
RuntimeError: No tags were found after sanity filtering!
```
[23:03:27]
<miro5001> Phpmyadmin
[23:03:59]
<Aleks (he/him/il/lui)> at some point we can't just be expected to handle every stupid random naming scheme people come up with ... version are supposed to be integer separated by dots ...
[23:05:15]
<miro5001> Yep
Some apps versions mix dots and underscores
[23:05:31]
<Aleks (he/him/il/lui)> @_@
[23:06:13]
<miro5001> And 3_4_0-8
[23:06:17]
<Aleks (he/him/il/lui)> jesus
[23:06:28]
<miro5001> https://github.com/pkp/ojs/releases/tag/3_4_0-8
[23:14:54]
<orhtej2> https://pypi.org/project/tariff/ 🤔
[23:14:55]
<Aleks (he/him/il/lui)> lolwut
[23:25:58]
<miro5001> tariff_ynh
[23:51:37]
<Aleks (he/him/il/lui)> on the other hand if i remember correctly it's possible to define a regex in the app to version matching ?
[23:53:13]
<Aleks (he/him/il/lui)> for example : https://github.com/YunoHost-Apps/dokuwiki_ynh/blob/master/manifest.toml#L65