[08:59:19]
<nicofrand> Bonjour ! Avez-vous une estimation de sortie pour Yunohost 13 ? Le paquet Kresus est incompatible, du fait de l'installation de yarn, et on réfléchit à virer yarn pour npm (et faciliter l'installation du paquet au passage), mais va falloir un peu de temps
[09:11:38]
<Thomas> Quel est le problème avec yarn ? De nombreuses apps tournent sur Trixie sans problème avec yarn
[09:16:12]
<Salamandar> Oui pour info yarn n'est plus dans les dépôts mais c'est toujours possible de l'utiliser
[09:17:09]
<nicofrand> oui mais faut revoir comment l'installer, et la doc de ynh que j'ai pu voir mentionne encore l'ancienne
[09:19:29]
<nicofrand> je vais essayer de retrouver le souci
[09:21:55]
<nicofrand> https://ci-apps-trixie.yunohost.org/ci/job/11565 le warning
[09:22:33]
<Thomas> > oui mais faut revoir comment l'installer, et la doc de ynh que j'ai pu voir mentionne encore l'ancienne
https://github.com/YunoHost-Apps/garden-party_ynh/blob/f9876fd9ca19427437a2dbd0e4d5f5389fc813eb/scripts/install#L44
[09:24:45]
<Thomas> Utiliser ces commandes et retirer yarn du manifest devrait faire l'affaire pour yarn
[09:26:02]
<nicofrand> pour yarn 1 ?
[09:27:50]
<nicofrand> et https://doc.yunohost.org/fr/dev/packaging/resources/#example
[09:28:52]
<nicofrand> mais ça change pas la question de se débarrasser de yarn dans kresus. Si on part là-dessus, on aurait combien de temps avant que YNH 13 passe en stable ?
[09:29:20]
<Thomas> > et https://doc.yunohost.org/fr/dev/packaging/resources/#example
Oui ça c'est plus à jour pour Trixie, yarn devant être installé via Corepack
[09:29:48]
<orhtej2> `npm i yarn@v1` iirc works
[09:32:23]
<orhtej2> Called problems with `/tmp` 😢
https://forum.yunohost.org/t/languagetool-server/41277/3
[09:38:06]
<Salamandar> j'pense qu'on peut supprimer complètement la doc autour de yarn puisqu'il faut juste appeler le helper (?)
[09:38:30]
<Salamandar> hmm je sais pas trop, ya besoin d'aide pour corriger les derniers bugs levés sur github.com/yunohost/issues
[09:39:22]
<Thomas> > <@Salamandar:matrix.org> j'pense qu'on peut supprimer complètement la doc autour de yarn puisqu'il faut juste appeler le helper (?)
Vu que la doc concerne apt, je changerais juste l'exemple
[09:39:29]
<nicofrand> le helper? corepack ? ça dépend de la version de yarn non ?
[09:39:41]
<Salamandar> hmmm pourquoi ça dépendrai de la version de yarn ?
[09:39:49]
<Salamandar> en tout cas a priori faut plus utiliser yarn v1 :)
[09:40:25]
<nicofrand> oui bah on va pas changer toute notre app pour ça. Donc au mieux on vire yarn
[09:40:47]
<nicofrand> me semblait qu'il y avait des subtilités justement pour la v1
[10:02:07]
<Salamandar> comment ça pour ça ? v1 est déprécié et plus maintenu depuis 2019, je disais ça indépendament de yunohost
[10:02:33]
<nicofrand> ça n'empêche que Kresus utilise la v1…
[10:03:22]
<nicofrand> la v1 est compatible avec npm globalement, ensuite non
[10:14:55]
<Salamandar> hmmm ok
[12:04:39]
<thatoo> Is there something like a `` pre_app_upgrade `` ?
```
```
I'm asking with the idea of running this before upgrading nextcloud. Currently, it is within the upgrade script which means that backup is done, and if it fails, it has to remove and restore.
```
```
It would be nice to do it before anything and avoid backup remove and restore...
```
function list_installed_apps_not_compatible_with_future_version()
{
local nextcloud_destination_version="$1"
local nextcloud_current_version="$(grep OC_VersionString "$install_dir/version.php" | cut -d' -f2)"
local installed_apps=$(mktemp)
local core_apps_in_current_version=$(mktemp)
local nextcloud_destination_appcatalog=$(mktemp)
# Run a first "dummy" command just to make sure we have the appropriate php dependencies installed,
# otherwise this creates funky stuff when tweaking the apt deps because the next command is piped into jq ...
exec_occ -V >/dev/null
# List installed apps
exec_occ app:list --output json | jq -r ".enabled | keys[]" | sort > $installed_apps
# Fetch Nextcloud list of core apps from their github repo for the current version
curl -s https://raw.githubusercontent.com/nextcloud/server/v$nextcloud_current_version/core/shipped.json | jq -r '.shippedApps[]' | sort > $core_apps_in_current_version
# Fetch Nextcloud app catalog (doesnt contain core app) for the future version
curl -s https://apps.nextcloud.com/api/v1/platform/$nextcloud_destination_version.0.0/apps.json | jq -r '.[] | .id' | sort > $nextcloud_destination_appcatalog
# Compute set complement, cf https://catonmat.net/set-operations-in-unix-shell
# We want to list the installed apps which are neither core apps nor in the destination catalog
comm -23 <(comm -23 $installed_apps $core_apps_in_current_version) $nextcloud_destination_appcatalog
}
current_version=$(grep OC_VersionString "$install_dir/version.php" | cut -d' -f2)
current_major_version=${current_version%%.*}
last_version=$(ynh_read_manifest "resources.sources.main.url" | grep -Eo '[0-9]+.[0-9]+.[0-9]+')
last_major_version=${last_version%%.*}
if [[ "$last_major_version" != "$current_major_version" ]]
then
installed_apps_not_compatible_with_future_version="$(list_installed_apps_not_compatible_with_future_version $last_major_version)"
if [[ -n "$installed_apps_not_compatible_with_future_version" ]]
then
ynh_die "The following apps are not (yet?) compatible with Nextcloud $last_major_version. You should make sure to upgrade the app, or disable it, or wait for it to become compatible before running this upgrade : $installed_apps_not_compatible_with_future_version"
fi
fi
```
[15:16:59]
<Aleks (he/him/il/lui)> nope, there's not
[16:34:23]
<Yunohost Git/Infra notifications> [yunohost] Gredin67 just made [pull request #2230](https://github.com/YunoHost/yunohost/pull/2230) ready for review: helpers: add support for .mount in ynh_config_add_systemd
[16:36:02]
<Yunohost Git/Infra notifications> [yunohost] Gredin67 [commented](https://github.com/YunoHost/yunohost/pull/2230#issuecomment-3729720633) on [issue #2230](https://github.com/YunoHost/yunohost/pull/2230) helpers: add support for .mount in ynh_config_add_systemd: @alexAubin @zamentur should we merge ? I need to find a way to test https://github.com/YunoHost-Apps/garage_ynh/pull/49
[16:38:27]
<Yunohost Git/Infra notifications> [yunohost] alexAubin [commented](https://github.com/YunoHost/yunohost/pull/2230#discussion_r2676861977) on pull request #2230 helpers: add support for .mount in ynh_config_add_systemd: We probably want the symetric thing in ynh_config_remove_systemd
[17:06:31]
<Yunohost Git/Infra notifications> [yunohost] Gredin67 [commented](https://github.com/YunoHost/yunohost/pull/2230#discussion_r2676948933) on pull request #2230 helpers: add support for .mount in ynh_config_add_systemd: done, to be tested
[17:16:44]
<Yunohost Git/Infra notifications> [yunohost] alexAubin merged [pull request #2230](https://github.com/YunoHost/yunohost/pull/2230): helpers: add support for .mount in ynh_config_add_systemd
[17:16:48]
<Yunohost Git/Infra notifications> [yunohost] alexAubin pushed to dev: helpers: add support for .mount in ynh_config_add_systemd (#2230) * add_systemd mount * [fix] Missing indent Co-autho... ([b371a797](https://github.com/YunoHost/yunohost/commit/b371a797c4a00cea4c633fe88f568947f6deb9da))
[17:22:11]
<Yunohost Git/Infra notifications> [yunohost] Gredin67 [commented](https://github.com/YunoHost/yunohost/pull/2230#issuecomment-3729879049) on [issue #2230](https://github.com/YunoHost/yunohost/pull/2230) helpers: add support for .mount in ynh_config_add_systemd: @alexAubin how can I test the helper without upgrading my yunohost ?
[17:24:58]
<Yunohost Git/Infra notifications> [yunohost] alexAubin [commented](https://github.com/YunoHost/yunohost/pull/2230#issuecomment-3729884862) on [issue #2230](https://github.com/YunoHost/yunohost/pull/2230) helpers: add support for .mount in ynh_config_add_systemd: @Gredin67 : i suppose you could savagely copypasta https://github.com/YunoHost/yunohost/blob/dev/helpers/helpers.v2.1.d/...
[17:30:06]
<Yunohost Git/Infra notifications> 🏗️ Starting build for yunohost/12.1.38+202601091830 for bookworm/unstable/all...
[17:31:32]
<Yunohost Git/Infra notifications> ✔️ Completed build for yunohost/12.1.38+202601091830 for bookworm/unstable/all.
[17:31:37]
<Yunohost Git/Infra notifications> ✔️ Completed distribution for yunohost/12.1.38+202601091830 for bookworm/unstable.
[17:44:12]
<Yunohost Git/Infra notifications> [yunohost] Thovi98 opened [pull request #2255](https://github.com/YunoHost/yunohost/pull/2255): disable proxy_set_header Authorization in proxy_params_no_auth
[17:53:42]
<Yunohost Git/Infra notifications> [yunohost] alexAubin [commented](https://github.com/YunoHost/yunohost/pull/2255#discussion_r2677097332) on pull request #2255 disable proxy_set_header Authorization in proxy_params_no_auth: suggestion # The Authorization header cannot be force-cleared here, because some apps do have auth mechanism that dep...
[17:53:59]
<Yunohost Git/Infra notifications> [yunohost] alexAubin pushed to dev: disable proxy_set_header Authorization in proxy_params_no_auth (#2255) * disable proxy_set_header Authorization in prox... ([8fd58018](https://github.com/YunoHost/yunohost/commit/8fd58018cec5c30470070382ac286988012723bc))
[17:53:59]
<Yunohost Git/Infra notifications> [yunohost] alexAubin merged [pull request #2255](https://github.com/YunoHost/yunohost/pull/2255): disable proxy_set_header Authorization in proxy_params_no_auth
[18:00:07]
<Yunohost Git/Infra notifications> 🏗️ Starting build for yunohost/12.1.38+202601091900 for bookworm/unstable/all...
[18:00:37]
<Yunohost Git/Infra notifications> [yunohost] alexAubin merged [pull request #2248](https://github.com/YunoHost/yunohost/pull/2248): Translations update from Weblate
[18:00:39]
<Yunohost Git/Infra notifications> [yunohost] alexAubin pushed to dev: Translated using Weblate (Kabyle) Currently translated at 14.5% (133 of 912 strings) Translation: YunoHost/Core Transl... ([826b7896](https://github.com/YunoHost/yunohost/commit/826b789617973e366504bd54426fd94ea73fe4b6))
[18:00:43]
<Yunohost Git/Infra notifications> [yunohost] alexAubin pushed to dev: Translated using Weblate (Kabyle) Currently translated at 15.3% (140 of 912 strings) Translation: YunoHost/Core Transl... ([9cfcaaeb](https://github.com/YunoHost/yunohost/commit/9cfcaaeba99d61a0a966e7363e53633891a6b6ac))
[18:00:50]
<Yunohost Git/Infra notifications> [yunohost] alexAubin pushed to dev: Translated using Weblate (Kabyle) Currently translated at 15.4% (141 of 912 strings) Translation: YunoHost/Core Transl... ([cbc25610](https://github.com/YunoHost/yunohost/commit/cbc256106ce32cf4d8c92d266c94a3dcab9d84fa))
[18:00:52]
<Yunohost Git/Infra notifications> [yunohost] alexAubin pushed to dev: Translations update from Weblate (#2248) * Translated using Weblate (Kabyle) Currently translated at 14.5% (133 of 912... ([452d3556](https://github.com/YunoHost/yunohost/commit/452d3556a4417288c74774058bd537202cfece7d))
[18:01:34]
<Yunohost Git/Infra notifications> ✔️ Completed build for yunohost/12.1.38+202601091900 for bookworm/unstable/all.
[18:01:38]
<Yunohost Git/Infra notifications> ✔️ Completed distribution for yunohost/12.1.38+202601091900 for bookworm/unstable.
[18:06:00]
<Yunohost Git/Infra notifications> [yunohost] alexAubin pushed to dev: Update changelog for 12.1.39 ([3572fc9d](https://github.com/YunoHost/yunohost/commit/3572fc9dff10b9f4f4ab30efe88a9b145867c241))
[18:06:06]
<Yunohost Git/Infra notifications> [yunohost] alexAubin created new t𝚊g debian/12.1.39
[18:06:13]
<Yunohost Git/Infra notifications> [repository] github-actions[bot] published [new release #debian/12.1.39](https://github.com/YunoHost/yunohost/releases/tag/debian/12.1.39) YunoHost 12.1.39
[18:06:21]
<Yunohost Git/Infra notifications> 🏗️ Starting build for yunohost/12.1.39 for bookworm/stable/all...
[18:07:51]
<Yunohost Git/Infra notifications> ✔️ Completed build for yunohost/12.1.39 for bookworm/stable/all.
[18:07:52]
<Yunohost Git/Infra notifications> ✔️ Completed distribution for yunohost/12.1.39 for bookworm/stable.
[18:15:13]
<Yunohost Git/Infra notifications> 🏗️ Starting build for yunohost/12.1.39+202601091915 for bookworm/unstable/all...
[18:16:36]
<Yunohost Git/Infra notifications> ✔️ Completed build for yunohost/12.1.39+202601091915 for bookworm/unstable/all.
[18:16:38]
<Yunohost Git/Infra notifications> ✔️ Completed distribution for yunohost/12.1.39+202601091915 for bookworm/unstable.
[20:38:49]
<Yunohost Git/Infra notifications> [yunohost] Josue-T [commented](https://github.com/YunoHost/yunohost/pull/2152#issuecomment-3730497855) on [issue #2152](https://github.com/YunoHost/yunohost/pull/2152) Upgrade helper version for hook in version 2.1: @alexAubin do you think we could merge it maybe in Trixie ?
[20:45:28]
<Yunohost Git/Infra notifications> [yunohost] alexAubin approved [pull request #2152](https://github.com/YunoHost/yunohost/pull/2152#pullrequestreview-3645523200) Upgrade helper version for hook in version 2.1: Im gonna say it looks okay to me to merge in current dev but we may get some surprises even though we pretty much car...