[09:42:19]
<Yunohost Git/Infra notifications> [apps] ericgaspar approved [pull request #2899](https://github.com/YunoHost/apps/pull/2899#pullrequestreview-2744850527) add PassED to catalog
[09:42:23]
<Yunohost Git/Infra notifications> [apps] ericgaspar pushed 1 commit to master: add PassED to catalog (#2899) ([4956055e](https://github.com/YunoHost/apps/commit/4956055e126b83edaa768ff80fc2c94e909b84c6))
[09:42:23]
<Yunohost Git/Infra notifications> [apps] ericgaspar merged [pull request #2899](https://github.com/YunoHost/apps/pull/2899): add PassED to catalog
[13:08:56]
<Yunohost Git/Infra notifications> [collabora_ynh] rodinux [commented](https://github.com/YunoHost-Apps/collabora_ynh/pull/101#issuecomment-2780716644) on [issue #101](https://github.com/YunoHost-Apps/collabora_ynh/pull/101) Testing: tesme
[13:08:59]
<Yunohost Git/Infra notifications> [collabora_ynh] rodinux [commented](https://github.com/YunoHost-Apps/collabora_ynh/pull/101#issuecomment-2780716644) on [issue #101](https://github.com/YunoHost-Apps/collabora_ynh/pull/101) Testing: tesme
[13:19:03]
<Yunohost Git/Infra notifications> [cryptpad_ynh] rodinux [commented](https://github.com/YunoHost-Apps/cryptpad_ynh/pull/242#issuecomment-2780720810) on [issue #242](https://github.com/YunoHost-Apps/cryptpad_ynh/pull/242) Update oudated information, fix #241: I agree with this PR, should we merge ?
[13:20:15]
<Yunohost Git/Infra notifications> [cryptpad_ynh] rodinux approved [pull request #242](https://github.com/YunoHost-Apps/cryptpad_ynh/pull/242#pullrequestreview-2744897110) Update oudated information, fix #241: LGTM
[13:25:39]
<m606> Hello, in an install script I use a helper that reads a var in which we define which python version we want and install the required version if it is not already installed.
Now I need to remove that python version in the remove script.
However I wonder how could I make sure removing that python version will not break other apps, or even YNH core (if python version is the same).
One idea could be to set a setting (with `ynh_app_setting`) - e.g. a boolean that tells whether required python version preexists to the app installation. Then at remove script, if that setting value is `true`, do not uninstall python.
However I guess that would not cover the cases where previous use of that python version have changed (e.g. python version for YNH core or given app was updated, so that eventually only current app was left using that python version). Any thoughts on this ?
[13:25:45]
<m606> Hello, in an install script I use a helper that reads a var in which we define which python version we want and install the required version if it is not already installed.
Now I need to remove that python version in the remove script.
However I wonder how could I make sure removing that python version will not break other apps, or even YNH core (if python version is the same).
One idea could be to set a setting (with `ynh_app_setting`) - e.g. a boolean that tells whether required python version preexists to that app installation. Then at remove script, if that setting value is `true`, do not uninstall python.
However I guess that would not cover the cases where previous use of that python version have changed (e.g. python version for YNH core or given app was updated, so that eventually only current app was left using that python version). Any thoughts on this ?
[13:29:31]
<Yunohost Git/Infra notifications> [cryptpad_ynh] rodinux [commented](https://github.com/YunoHost-Apps/cryptpad_ynh/pull/242#issuecomment-2780725140) on [issue #242](https://github.com/YunoHost-Apps/cryptpad_ynh/pull/242) Update oudated information, fix #241: @ericgaspar is all right for you ?
[13:31:12]
<Yunohost Git/Infra notifications> [collabora_ynh] rodinux merged [pull request #101](https://github.com/YunoHost-Apps/collabora_ynh/pull/101): Testing
[13:43:00]
<m606> Other idea would be to add to the core a kind of registry which saves says which version is used by what. Maybe it already exists ? Maybe packaging v3 ?
| python_version| app |
|----------------|-----------------|
| 3.11.2 | core, myapp3 |
| 3.12.4 | myapp1 |
| 3.9.1 | myapp2 |
[13:56:44]
<Salamandar> that's a bit what we do with node and golang
[13:57:04]
<Salamandar> but we aren't doing the same stuff with python because python as more stable
[14:01:23]
<m606> > <@Salamandar:matrix.org> that's a bit what we do with node and golang
hmm so should I just never uninstall python (at least it wont break anything, and may create only limited overrhead if versions do not change too often)?
[14:03:16]
<m606> not removing it is actually what are doing some apps using python in the catalog:
- https://github.com/YunoHost-Apps/indico_ynh/blob/1a4be95bcfa0aac73252e648d3d5d692cc0c4bd9/scripts/remove
- https://github.com/YunoHost-Apps/django_example_ynh/blob/master/scripts/remove
[14:15:37]
<Aleks (he/him/il/lui)> imho you should just never install a custom python version in the first place but meh ...
[14:23:26]
<Yunohost Git/Infra notifications> [apps] yunohost-bot opened [pull request #2905](https://github.com/YunoHost/apps/pull/2905): Update app levels according to CI results
[14:24:28]
<Aleks (he/him/il/lui)> ah yes, the linter / README story
[14:38:40]
<Yunohost Git/Infra notifications> [package_linter] alexAubin pushed 1 commit to master: Fix linter / accept new autogenerated README ([86ae6b98](https://github.com/YunoHost/package_linter/commit/86ae6b989add4672e1242e39ddf0340a9aec1d89))
[14:39:04]
<Aleks (he/him/il/lui)> i'm relaunching the affected jobs
[14:52:30]
<m606> > <@Alekswag:matrix.org> imho you should just never install a custom python version in the first place but meh ...
why so?
the app currently requires python >=3.9 so it is satisfied by version installed for YNH I guess (at least in YNH 12.x), but it was just in case at some point there would be a mismatch between app requirements and YNH default version
[14:53:10]
<orhtej2> I'd vote against futureproofing
[14:53:17]
<orhtej2> Trixie is right around the corner
[14:57:05]
<Aleks (he/him/il/lui)> because "every app installs its own copy of super-specific version of everything" is madness and has ramifications in terms of ecology and complexity
[14:57:45]
<Aleks (he/him/il/lui)> we do it for stuff like nodejs possibly because there's no other choice / it's kinda designed in the way the nodejs ecosystem works
[14:57:59]
<Aleks (he/him/il/lui)> we also do it for ruby but idk, i'm thinking maybe some harmonisation could be achieved
[14:58:46]
<Aleks (he/him/il/lui)> but for python honestly the language is pretty stable and things should just work with any recent-ish python version, unless the developers specificly chose to use some brand new syntax sugar not yet available in debian stable or whatver
[14:59:26]
<m606> Ok so I will just check if python is matching requirements (>=3.9) and keep installing the app if so, or echo something like "failed due to python mismatch" if not
[16:12:18]
<Yunohost Git/Infra notifications> [apps] yunohost-bot opened [pull request #2906](https://github.com/YunoHost/apps/pull/2906): Add Peergos to wishlist
[16:12:19]
<Yunohost Git/Infra notifications> [apps] yunohost-bot labeled Wishlist on [pull request #2906](https://github.com/YunoHost/apps/pull/2906): Add Peergos to wishlist
[16:16:26]
<Yunohost Git/Infra notifications> [apps] alexAubin deleted branch add-to-wishlist-filen-webdav
[16:16:26]
<Yunohost Git/Infra notifications> [apps] alexAubin merged [pull request #2845](https://github.com/YunoHost/apps/pull/2845): Add Filen Webdav to wishlist
[16:16:27]
<Yunohost Git/Infra notifications> [apps] alexAubin pushed 1 commit to master: Add Filen Webdav to wishlist (#2845) ([4c910e1b](https://github.com/YunoHost/apps/commit/4c910e1b91890eec484fdb8268d253e903052724))
[16:16:49]
<Yunohost Git/Infra notifications> [apps] alexAubin merged [pull request #2849](https://github.com/YunoHost/apps/pull/2849): Add Pollen to wishlist
[16:16:50]
<Yunohost Git/Infra notifications> [apps] alexAubin deleted branch add-to-wishlist-pollen
[16:16:53]
<Yunohost Git/Infra notifications> [apps] alexAubin pushed 1 commit to master: Add Pollen to wishlist (#2849) ([40c76f64](https://github.com/YunoHost/apps/commit/40c76f642da548c8d9e20dda4c8cc39eab5a8880))
[16:18:48]
<Yunohost Git/Infra notifications> [apps] alexAubin pushed 1 commit to master: Add BTW to wishlist (#2850) ([51750d79](https://github.com/YunoHost/apps/commit/51750d794a32d4317ca83fe3e4e73d32a86bc066))
[16:18:48]
<Yunohost Git/Infra notifications> [apps] alexAubin merged [pull request #2850](https://github.com/YunoHost/apps/pull/2850): Add BTW to wishlist
[16:18:48]
<Yunohost Git/Infra notifications> [apps] alexAubin deleted branch add-to-wishlist-btw
[16:21:08]
<Yunohost Git/Infra notifications> [apps] alexAubin merged [pull request #2857](https://github.com/YunoHost/apps/pull/2857): Add chatmail to wishlist
[16:21:08]
<Yunohost Git/Infra notifications> [apps] alexAubin pushed 1 commit to master: Add chatmail to wishlist (#2857) ([afb0d1e0](https://github.com/YunoHost/apps/commit/afb0d1e05bce23c9062fcad7fae12546dcb84949))
[16:21:09]
<Yunohost Git/Infra notifications> [apps] alexAubin deleted branch add-to-wishlist-chatmail
[16:21:09]
<Yunohost Git/Infra notifications> [apps] alexAubin [commented](https://github.com/YunoHost/apps/pull/2857#issuecomment-2780898403) on [issue #2857](https://github.com/YunoHost/apps/pull/2857) Add chatmail to wishlist: Urhg yeah it sounds indeed not trivial at all to retroengineer the whole thing considering it wants to configure postfix...
[16:24:05]
<Yunohost Git/Infra notifications> [apps] alexAubin [commented](https://github.com/YunoHost/apps/pull/2873#discussion_r2029911073) on pull request #2873 Add Urlaubsverwaltung to wishlist: suggestion description = "Web application to manage absences / vacations"
[16:24:23]
<Yunohost Git/Infra notifications> [apps] alexAubin pushed 1 commit to master: Add Urlaubsverwaltung to wishlist (#2873) * Add Urlaubsverwaltung to wishlist * Update wishlist.toml --------- Co-au... ([ae1f6378](https://github.com/YunoHost/apps/commit/ae1f637867c710a81e361b3e8c86cf9d0245f215))
[16:24:23]
<Yunohost Git/Infra notifications> [apps] alexAubin pushed 1 commit to add-to-wishlist-urlaubsverwaltung: Update wishlist.toml ([c717dbbc](https://github.com/YunoHost/apps/commit/c717dbbc3656d32c13d17106b1d27dd15fc559f8))
[16:24:23]
<Yunohost Git/Infra notifications> [apps] alexAubin merged [pull request #2873](https://github.com/YunoHost/apps/pull/2873): Add Urlaubsverwaltung to wishlist
[16:24:24]
<Yunohost Git/Infra notifications> [apps] alexAubin deleted branch add-to-wishlist-urlaubsverwaltung
[16:26:12]
<Yunohost Git/Infra notifications> [apps] alexAubin [commented](https://github.com/YunoHost/apps/pull/2877#issuecomment-2780913585) on [issue #2877](https://github.com/YunoHost/apps/pull/2877) Add Signal API to wishlist: Dunno what to think of this ... its just an API so that sounds very power-userish ... what are the actual use cases x_x
[16:27:42]
<Yunohost Git/Infra notifications> [apps] alexAubin pushed 1 commit to master: Add Pinka to wishlist (#2891) ([6a658f90](https://github.com/YunoHost/apps/commit/6a658f90b2bbbe0a1867ffe5fdc39836006a53bc))
[16:27:42]
<Yunohost Git/Infra notifications> [apps] alexAubin merged [pull request #2891](https://github.com/YunoHost/apps/pull/2891): Add Pinka to wishlist
[16:27:44]
<Yunohost Git/Infra notifications> [apps] alexAubin deleted branch add-to-wishlist-pinka
[16:28:37]
<Yunohost Git/Infra notifications> [apps] alexAubin pushed 1 commit to master: Add Reactive Resume to wishlist (#2897) ([8e05371b](https://github.com/YunoHost/apps/commit/8e05371beb218a85902fed6f437c76e033f9e820))
[16:28:37]
<Yunohost Git/Infra notifications> [apps] alexAubin merged [pull request #2897](https://github.com/YunoHost/apps/pull/2897): Add Reactive Resume to wishlist
[16:28:38]
<Yunohost Git/Infra notifications> [apps] alexAubin deleted branch add-to-wishlist-reactive-resume
[16:31:22]
<Yunohost Git/Infra notifications> [apps] alexAubin pushed 1 commit to add-to-wishlist-guppe: Update wishlist.toml ([a076b026](https://github.com/YunoHost/apps/commit/a076b0268529a2d17ab192be0ae3b07ea31cdaf8))
[16:31:23]
<Yunohost Git/Infra notifications> [apps] alexAubin [commented](https://github.com/YunoHost/apps/pull/2898#discussion_r2029912615) on pull request #2898 Add Guppe to wishlist: suggestion description = "Social groups for the fediverse. Connect and meet new people based on shared interests"
[16:31:26]
<Yunohost Git/Infra notifications> [apps] alexAubin merged [pull request #2898](https://github.com/YunoHost/apps/pull/2898): Add Guppe to wishlist
[16:31:26]
<Yunohost Git/Infra notifications> [apps] alexAubin pushed 1 commit to master: Add Guppe to wishlist (#2898) * Add Guppe to wishlist * Update wishlist.toml --------- Co-authored-by: Alexandre Aub... ([cf6eb1a0](https://github.com/YunoHost/apps/commit/cf6eb1a0304baa43f90a8998d07eca05df798115))
[16:31:27]
<Yunohost Git/Infra notifications> [apps] alexAubin deleted branch add-to-wishlist-guppe
[16:33:48]
<Yunohost Git/Infra notifications> [apps] alexAubin deleted branch add-to-wishlist-nepenthes
[16:33:48]
<Yunohost Git/Infra notifications> [apps] alexAubin pushed 1 commit to master: Add Nepenthes to wishlist (#2900) ([7e083b90](https://github.com/YunoHost/apps/commit/7e083b904d7dfff7f76023cc8ffd865daac12683))
[16:33:49]
<Yunohost Git/Infra notifications> [apps] alexAubin merged [pull request #2900](https://github.com/YunoHost/apps/pull/2900): Add Nepenthes to wishlist
[16:34:31]
<Yunohost Git/Infra notifications> [apps] alexAubin pushed 1 commit to add-to-wishlist-restreamer: Update wishlist.toml ([615dedbc](https://github.com/YunoHost/apps/commit/615dedbc53b8169235345e564b387b65096efcd0))
[16:34:36]
<Yunohost Git/Infra notifications> [apps] alexAubin pushed 1 commit to master: Add Restreamer to wishlist (#2901) * Add Restreamer to wishlist * Update wishlist.toml --------- Co-authored-by: Ale... ([a17cdaa6](https://github.com/YunoHost/apps/commit/a17cdaa683dbd046a25da7f59389d0c376463e45))
[16:34:37]
<Yunohost Git/Infra notifications> [apps] alexAubin deleted branch add-to-wishlist-restreamer
[16:34:38]
<Yunohost Git/Infra notifications> [apps] alexAubin merged [pull request #2901](https://github.com/YunoHost/apps/pull/2901): Add Restreamer to wishlist
[16:35:17]
<Yunohost Git/Infra notifications> [apps] alexAubin [commented](https://github.com/YunoHost/apps/pull/2902#discussion_r2029913215) on pull request #2902 Add Dawarich Location History to wishlist: suggestion description = "Alternative to Google Location History/Maps Timeline, allowing import and location visualiz...
[16:35:23]
<Yunohost Git/Infra notifications> [apps] alexAubin pushed 1 commit to add-to-wishlist-dawarich-location-history: Update wishlist.toml ([f7c72ec7](https://github.com/YunoHost/apps/commit/f7c72ec707e0c9678d55c58609e903aabd049dba))
[16:35:34]
<Yunohost Git/Infra notifications> [apps] alexAubin pushed 1 commit to master: Add Dawarich Location History to wishlist (#2902) * Add Dawarich Location History to wishlist * Update wishlist.toml ... ([e669fc32](https://github.com/YunoHost/apps/commit/e669fc32b90676f7fbbb18f1ea0aa47d0d510a97))
[16:35:34]
<Yunohost Git/Infra notifications> [apps] alexAubin merged [pull request #2902](https://github.com/YunoHost/apps/pull/2902): Add Dawarich Location History to wishlist
[16:35:35]
<Yunohost Git/Infra notifications> [apps] alexAubin deleted branch add-to-wishlist-dawarich-location-history
[16:41:12]
<Yunohost Git/Infra notifications> [apps] alexAubin pushed 1 commit to master: Add Typst to wishlist (#2903) ([29bc8981](https://github.com/YunoHost/apps/commit/29bc898140ae44ee3db02d72319fbc97e7cfbd52))
[16:41:12]
<Yunohost Git/Infra notifications> [apps] alexAubin merged [pull request #2903](https://github.com/YunoHost/apps/pull/2903): Add Typst to wishlist
[16:41:13]
<Yunohost Git/Infra notifications> [apps] alexAubin deleted branch add-to-wishlist-typst
[16:43:44]
<Yunohost Git/Infra notifications> [apps] alexAubin closed [pull request #2337](https://github.com/YunoHost/apps/pull/2337): Regenerate existing READMEs even if translation status has dropped from a lower value than 100%
[18:09:28]
<Yunohost Git/Infra notifications> [appstore] alexAubin pushed 1 commit to main: Add v2 ci level badges for the new REAME ([62dba414](https://github.com/YunoHost/appstore/commit/62dba414a6540659c6e91b36876635a335e03f90))
[18:21:48]
<Yunohost Git/Infra notifications> [apps_tools] alexAubin pushed 1 commit to main: AutoREADME: re-add the level/ci badge in the Developer info section ([6d3bd641](https://github.com/YunoHost/apps_tools/commit/6d3bd6414ac23c097069e57b2cc2046825e59386))
[18:22:36]
<Yunohost Git/Infra notifications> [apps_tools] alexAubin pushed 1 commit to main: Fix tests ? (followup previous commit) ([09bdb238](https://github.com/YunoHost/apps_tools/commit/09bdb2383813fbbca60466cd823ce5a1d6b7005f))
[18:25:20]
<Yunohost Git/Infra notifications> [apps_tools] alexAubin pushed 1 commit to main: Fix tests ? (followup previous commit) ([cefd5f15](https://github.com/YunoHost/apps_tools/commit/cefd5f1547efffc3e46654c1e80820fbd529014a))
[18:27:32]
<Yunohost Git/Infra notifications> [apps_tools] alexAubin [commented](https://github.com/YunoHost/apps_tools/pull/18#issuecomment-2781031494) on [issue #18](https://github.com/YunoHost/apps_tools/pull/18) Simpler READMEs: (CI level / badge re-added in developer section in https://github.com/YunoHost/apps_tools/commit/6d3bd6414ac23c097069e57...
[18:54:34]
<miro5001> > <@m606:matrix.org> that's actually what are doing some apps using python in the catalog:
> - https://github.com/YunoHost-Apps/indico_ynh/blob/1a4be95bcfa0aac73252e648d3d5d692cc0c4bd9/scripts/remove
> - https://github.com/YunoHost-Apps/django_example_ynh/blob/master/scripts/remove
Good point.
I fact, in the indico remove script, there is a remove python function at line 24.
I have used 'make altinstall' to install it as an alternative version, so it doesn't replace the existing version to avoid breaking yunohost. Calling that version requires the version number appended. The remove function will erase the folder of the installed alternative version and remove it from the alternative versions.
Removing the new version will certainly break other instances of the app. So yes, keep it, it will be available as an alternative version that can be used by other apps.
[20:49:44]
<Yunohost Git/Infra notifications> [my_webapp_ynh] orhtej2 deleted branch test
[20:49:44]
<Yunohost Git/Infra notifications> [my_webapp_ynh] orhtej2 merged [pull request #152](https://github.com/YunoHost-Apps/my_webapp_ynh/pull/152): Remove FPM configuration mentions
[20:50:33]
<Yunohost Git/Infra notifications> [my_webapp_ynh] orhtej2 pushed 1 commit to restore_php_version: Restore PHP version selection ([d9079a47](https://github.com/YunoHost-Apps/my_webapp_ynh/commit/d9079a473d9e57ba52b01108afe9ff183c7389de))
[20:50:45]
<Yunohost Git/Infra notifications> [my_webapp_ynh] orhtej2 merged [pull request #153](https://github.com/YunoHost-Apps/my_webapp_ynh/pull/153): Restore PHP version selection
[20:50:48]
<Yunohost Git/Infra notifications> [my_webapp_ynh] orhtej2 deleted branch restore_php_version
[20:51:42]
<Yunohost Git/Infra notifications> [my_webapp_ynh] orhtej2 opened [pull request #154](https://github.com/YunoHost-Apps/my_webapp_ynh/pull/154): Fixup conf script + restore PHP version selection
[20:51:50]
<Yunohost Git/Infra notifications> [my_webapp_ynh] orhtej2 [commented](https://github.com/YunoHost-Apps/my_webapp_ynh/pull/154#issuecomment-2781090181) on [issue #154](https://github.com/YunoHost-Apps/my_webapp_ynh/pull/154) Fixup conf script + restore PHP version selection: bump
[20:54:21]
<orhtej2> this one is bugged, it bumped revision but didn't regenerate the readmes 🤷
[21:04:54]
<orhtej2> unless there's some logic that filters out pushes from yunohost-bot from triggering the hooks perhaps?
[21:07:34]
<Yunohost Git/Infra notifications> [apps_tools] orhtej2 opened [pull request #28](https://github.com/YunoHost/apps_tools/pull/28): Regenerate README on version bump
[21:07:52]
<orhtej2> in such a case ^
[22:09:06]
<orhtej2> https://aria.im/_bifrost/v1/media/download/Ad_S6SsShVJ5iQWaitbrt7ccnGtWIzfCGRZa-k-EMMuq2wNICRqsd_OesxmjlderWtaLjdY4Rn2WR5spVH4Nf7tCeWDSZ2VAAGNpcmNsZWRzcXVhcmVyb290Lm92aC9jMTU2MTY2OTYxMTgwZThlMTUxZjkwMzUzMzY4MTBjNWYyMmQwOGQwYmQ3YWQ3MzE4NTRlODEyYzEyYWM5MWY2
[22:09:32]
<orhtej2> IDK, is [this](https://github.com/YunoHost-Apps/dumbbudget_ynh/blob/407dfe0827cf7b074b0b1eadecbd03bc24a420ec/manifest.toml#L49) somehow wrong or if this is ynh-admin bug?
[22:09:55]
<orhtej2> FF 137@Win11
[22:24:19]
<Yunohost Git/Infra notifications> Autoupdater just ran, here are the results:
- 9 pending update PRs
- 8 new apps PRs
- 15 failed apps updates: appflowy, khatru-pyramid, kiwix, languagetool, lemmy, localai, misskey, ofbiz, opencloud, phpldapadmin, phpmyadmin, pixelfedglitch, snweb, stremio, swingmusic
See the full log here: https://paste.yunohost.org/raw/aqanikumaq
[22:44:28]
<Yunohost Git/Infra notifications> [mastodon_ynh] mills1975 [commented](https://github.com/YunoHost-Apps/mastodon_ynh/issues/424#issuecomment-2781120946) on [issue #424](https://github.com/YunoHost-Apps/mastodon_ynh/issues/424) Upgrade takes forever: Hi
Ive got a similar problem.
Started the upgrade via CLI, and the step "Upgrading source files..." is taking many hou...
[23:16:40]
<Émy – OniriCorpe> > <@yunohostinfra:matrix.org> [mastodon_ynh] mills1975 [commented](https://github.com/YunoHost-Apps/mastodon_ynh/issues/424#issuecomment-2781120946) on [issue #424](https://github.com/YunoHost-Apps/mastodon_ynh/issues/424) Upgrade takes forever: Hi
> Ive got a similar problem.
> Started the upgrade via CLI, and the step "Upgrading source files..." is taking many hou...
Why is this package not using the data_dir system? 😭