[06:54:57]
<eric_G> > <@Alekswag:matrix.org> sooooo i reworked the script to get rid of the remaining sed, on my side the script does patch this (though it ends up with PATH=PATH= because of the initial incorrect PATH= of the original conf..)
great! thanks for the work. The only warning left I get is:
```
python3 ./convert_to_helpers_2.1.py /Users/eric/Documents/GitHub/2FAuth_ynh
/Users/eric/Documents/./convert_to_helpers_2.1.py:95: SyntaxWarning: invalid escape sequence '\['
('\[\s+"?\$upgrade_type"?\s+==\s+"?UPGRADE_APP"? ]', "ynh_app_upstream_version_changed"),
```
[07:28:32]
<eric_G> I got this error on logrotate : https://ci-apps-dev.yunohost.org/ci/job/17080
[07:33:32]
<Thomas> and me this one : https://ci-apps-dev.yunohost.org/ci/job/17077 ^^
[07:33:40]
<Thomas> unbound variable too
[07:34:41]
<Thomas> For helpers 2.1
[07:39:36]
<eric_G> Thomas: the error on the postgresql helper was fixed yesterday (but needs to be released)
[07:39:56]
<Thomas> Okay thanks!
[07:40:39]
<eric_G> I this error when building the app with Node: https://ci-apps-dev.yunohost.org/ci/job/17081
[10:18:25]
<Yunohost Git/Infra notifications> [issues] fflorent opened [issue #2413](https://github.com/YunoHost/issues/issues/2413): App packaging: Email not present in headers since Yunohost 1.12.*?
[10:18:25]
<Yunohost Git/Infra notifications> [issues] fflorent labeled :space_invader: bug on [issue #2413](https://github.com/YunoHost/issues/issues/2413): App packaging: Email not present in headers since Yunohost 1.12.*?
[10:53:39]
<Yunohost Git/Infra notifications> [issues] alexAubin [commented](https://github.com/YunoHost/issues/issues/2413#issuecomment-2184941991) on [issue #2413](https://github.com/YunoHost/issues/issues/2413) App packaging: Email not present in headers since Yunohost 1.12.*?: (can you mention which app needs this exactly?)
[10:54:12]
<Yunohost Git/Infra notifications> [issues] alexAubin edited [issue #2413](https://github.com/YunoHost/issues/issues/2413): Bookworm: SSO/App packaging: Email not present in headers since Yunohost 12.x?
[10:54:17]
<Yunohost Git/Infra notifications> [issues] alexAubin labeled :door: SSO on [issue #2413](https://github.com/YunoHost/issues/issues/2413): Bookworm: SSO/App packaging: Email not present in headers since Yunohost 12.x?
[11:47:37]
<Yunohost Git/Infra notifications> [issues] fflorent [commented](https://github.com/YunoHost/issues/issues/2413#issuecomment-2184956604) on [issue #2413](https://github.com/YunoHost/issues/issues/2413) Bookworm: SSO/App packaging: Email not present in headers since Yunohost 12.x?: @alexAubin : The app is Grist.
[12:05:41]
<Aleks (he/him/il/lui)> > <@ericg:matrix.org> I got this error on logrotate : https://ci-apps-dev.yunohost.org/ci/job/17080
ah yes the helper should default to /var/log/$app/*.log by default, will fix that
[12:07:03]
<Aleks (he/him/il/lui)> ah it already does but it should be `${1:-}` instead of `$1`
[12:08:45]
<Yunohost Git/Infra notifications> [yunohost] alexAubin pushed 1 commit to dev: helpers2.1: fix ynh_config_add_logrotate when no arg is passed ([3942ea12](https://github.com/YunoHost/yunohost/commit/3942ea12aea990fae9e10217b8f7839aa957bb4b))
[12:15:26]
<Yunohost Git/Infra notifications> 🏗️ Starting build for yunohost/11.2.15+202406231215 for bullseye/unstable/all ...
[12:16:50]
<Yunohost Git/Infra notifications> ✔️ Completed build for yunohost/11.2.15+202406231215 for bullseye/unstable/all.
[12:25:03]
<Aleks (he/him/il/lui)> zgrmblf
[12:25:43]
<Aleks (he/him/il/lui)> `ynh_systemd_action` / `ynh_systemctl` has a 300s timeout waiting to find the match_line/wait_until pattern thing, which is way too long, and then at the end of the timeout it just assume it worked anyway
[12:26:06]
<Aleks (he/him/il/lui)> even if the service is in obvious failed state ~_~
[12:34:36]
<Salamandar> zblerg
[12:41:05]
<Aleks (he/him/il/lui)> > <@ericg:matrix.org> I this error when building the app with Node: https://ci-apps-dev.yunohost.org/ci/job/17081
ffs it doesn't work when using `sudo -E -u $app ./tiddlywiki.js` but it does with `sudo -E -u $app env PATH=$appropriatepath ./tiddlywiki.js`
[12:41:09]
<Aleks (he/him/il/lui)> despite PATH being exported
[12:41:09]
<Aleks (he/him/il/lui)> wth
[12:42:49]
<Aleks (he/him/il/lui)> zzzz `-E` works for a random var like `$FOO` but not `$PATH` ?
[12:44:29]
<Aleks (he/him/il/lui)> (╯°□°）╯︵ ┻━┻
[12:50:33]
<Aleks (he/him/il/lui)> and theeeeeen turns out that in systemd conf even if you have Environment=PATH=/some/path, binaries are search before interpreting env so we gotta put the full path anyway yay
[12:52:04]
<Yunohost Git/Infra notifications> [yunohost] alexAubin pushed 5 commits to dev ([3942ea12aea9...5f6df6a859f0](https://github.com/YunoHost/yunohost/compare/3942ea12aea9...5f6df6a859f0))
[12:52:04]
<Yunohost Git/Infra notifications> [yunohost/dev] helpers2.1: display 100 lines instead of 20 in CI context when service fails to start - Alexandre Aubin
[12:52:14]
<Yunohost Git/Infra notifications> [yunohost] ✖️ Pipeline [#1344238008](https://gitlab.com/YunoHost/yunohost/-/pipelines/1344238008) canceled on branch dev
[13:00:06]
<eric_G> 😱 I think I have to set myself a proper Debian system for packaging : https://github.com/YunoHost-Apps/yarr_ynh/pull/5/files
[13:00:38]
<Yunohost Git/Infra notifications> 🏗️ Starting build for yunohost/11.2.15+202406231300 for bullseye/unstable/all ...
[13:00:45]
<Aleks (he/him/il/lui)> https://github.com/YunoHost-Apps/yarr_ynh/blob/59380993aa8a759a1aad47a70619a329a6d1e80b/scripts/_common.sh
[13:00:55]
<Aleks (he/him/il/lui)> oopsies
[13:01:03]
<eric_G> and the output from last script:
```
python3 ./convert_to_helpers_2.1.py /Users/eric/Documents/GitHub/yarr_ynh
/Users/eric/Documents/./convert_to_helpers_2.1.py:95: SyntaxWarning: invalid escape sequence '\s'
('[\s+"?\$upgrade_type"?\s+==\s+"?UPGRADE_APP"? ]', "ynh_app_upstream_version_changed"),
```
[13:01:21]
<Aleks (he/him/il/lui)> uuuuh wtf haha
[13:01:38]
<Aleks (he/him/il/lui)> hmmmokay let me fix this again haha
[13:01:40]
<Yunohost Git/Infra notifications> ✔️ Completed build for yunohost/11.2.15+202406231300 for bullseye/unstable/all.
[13:21:40]
<Yunohost Git/Infra notifications> [yunohost] alexAubin pushed 1 commit to dev: helpers2.1: var rename / cosmetic etc for nodejs/ruby/go version and install directories ([2b1f7426](https://github.com/YunoHost/yunohost/commit/2b1f74268f1a52a420459d11607c11461da528e2))
[13:22:12]
<Yunohost Git/Infra notifications> [yunohost] ✖️ Pipeline [#1344252054](https://gitlab.com/YunoHost/yunohost/-/pipelines/1344252054) canceled on branch dev
[13:24:57]
<Aleks (he/him/il/lui)> alright i'm gonna pretend i've never read `ynh_remove_nodejs` and `ynh_remove_ruby`
[13:25:00]
<Aleks (he/him/il/lui)> let's release
[13:30:26]
<Yunohost Git/Infra notifications> 🏗️ Starting build for yunohost/11.2.15+202406231330 for bullseye/unstable/all ...
[13:32:03]
<Yunohost Git/Infra notifications> ✔️ Completed build for yunohost/11.2.15+202406231330 for bullseye/unstable/all.
[13:33:14]
<Yunohost Git/Infra notifications> [yunohost] alexAubin created new tag debian/11.2.16
[13:33:22]
<Yunohost Git/Infra notifications> [yunohost] alexAubin pushed 1 commit to dev: Update changelog for 11.2.16 ([0e4495f1](https://github.com/YunoHost/yunohost/commit/0e4495f11e3d74b97241e83345236df7c247a14e))
[13:34:02]
<Yunohost Git/Infra notifications> 🏗️ Starting build for yunohost/11.2.16 for bullseye/stable/all ...
[13:34:40]
<Yunohost Git/Infra notifications> [yunohost] ✖️ Pipeline [#1344266547](https://gitlab.com/YunoHost/yunohost/-/pipelines/1344266547) canceled on branch dev
[13:35:10]
<Yunohost Git/Infra notifications> ✔️ Completed build for yunohost/11.2.16 for bullseye/stable/all.
[13:38:38]
<eric_G> the script doesn't output error anymore! thanks
[13:40:24]
<Aleks (he/him/il/lui)> https://i.imgflip.com/8uqk2b.jpg
[13:45:19]
<Yunohost Git/Infra notifications> 🏗️ Starting build for yunohost/11.2.16+202406231345 for bullseye/unstable/all ...
[13:46:46]
<Yunohost Git/Infra notifications> ✔️ Completed build for yunohost/11.2.16+202406231345 for bullseye/unstable/all.
[14:14:41]
<Aleks (he/him/il/lui)> (appci now runs 11.2.16 👍️)
[14:27:30]
<eric_G> I have Go app failling... https://ci-apps-dev.yunohost.org/ci/job/17082
[14:28:49]
<Aleks (he/him/il/lui)> ```
2024-06-23 14:14:39,517: DEBUG - ++ goenv latest --print 1.20
2024-06-23 14:14:39,543: WARNING - goenv: no versions installed
```
[14:28:50]
<Aleks (he/him/il/lui)> 🤔
[14:28:57]
<Aleks (he/him/il/lui)> Salamandar: do you remember what this is about ?
[14:29:07]
<Salamandar> AGAIN?!
[14:29:10]
<Aleks (he/him/il/lui)> maybe we forgot to propagate some fix from helpers v1 to 2.1
[14:29:16]
<Salamandar> yes possibly
[14:29:22]
<Salamandar> i called the plugin directly
[14:29:26]
<Salamandar> let me see
[14:30:33]
<Salamandar> `local final_go_version=$("$goenv_latest_dir/bin/goenv-latest" --print "$go_version")`
[14:30:40]
<Salamandar> it's this line
[14:30:44]
<Salamandar> the original one calls goenv
[14:30:44]
<Salamandar> we now need to call goenv-latest
[14:30:45]
<Salamandar> https://github.com/go-nv/goenv/issues/386
[14:30:46]
<Salamandar> no reply…
[14:36:09]
<Yunohost Git/Infra notifications> [yunohost] alexAubin pushed 1 commit to dev: helpers2.1: forgot to propagate the goenv latest fix from helpers v1 ([d8c3ff4c](https://github.com/YunoHost/yunohost/commit/d8c3ff4c8ac1bd2c4bb9cd2b93a1c14f11ae7e33))
[14:37:04]
<Yunohost Git/Infra notifications> [yunohost] ✖️ Pipeline [#1344270171](https://gitlab.com/YunoHost/yunohost/-/pipelines/1344270171) canceled on branch dev
[14:39:00]
<Aleks (he/him/il/lui)> > <@ericg:matrix.org> I this error when building the app with Node: https://ci-apps-dev.yunohost.org/ci/job/17081
(seems to be fixed 👍️)
[14:41:59]
<Salamandar> I've been rate limited when creating PRs :D
[14:43:31]
<Aleks (he/him/il/lui)> 😬
[14:45:18]
<Yunohost Git/Infra notifications> 🏗️ Starting build for yunohost/11.2.16+202406231445 for bullseye/unstable/all ...
[14:46:31]
<Yunohost Git/Infra notifications> ✔️ Completed build for yunohost/11.2.16+202406231445 for bullseye/unstable/all.
[14:47:55]
<Salamandar> weird because the github api says i'm using only 10%
[16:25:41]
<Émy - OniriCorpe> > <@Salamandar:matrix.org> I've been rate limited when creating PRs :D
me too lol
[16:25:45]
<Émy - OniriCorpe> i restarted the script
[16:33:49]
<tituspijean> Which script(s) do you use?
[16:48:12]
<Émy - OniriCorpe> autopatch
[16:52:28]
<Émy - OniriCorpe> sinon c'est quoi l’intérêt de faire une PR testing -> master ? j'ai pas tilté
[16:52:40]
<Émy - OniriCorpe> c'était pas faire l'inverse le projet ?
[16:54:45]
<Salamandar> en fait c'était juste de lister toutes les branches testing u'on jamais été mergées
[16:54:57]
<Salamandar> ah ma touche entre p et r ne fonctionne plus
[16:55:02]
<Salamandar> q
[16:55:03]
<Salamandar> ah si
[16:55:05]
<Salamandar> chelou
[16:56:06]
<Salamandar> ya un paquet de branches testing qu'on pas été mergées depuis des mois, et qu'on pas de pr associée
[17:11:30]
<Émy - OniriCorpe> ça m'a fait une PR sur le paquet GTS alors que les branches sont à jour mais désynchronisées
[17:13:01]
<Salamandar> Comment ça désync ?
[17:13:14]
<Salamandar> mais oui ya des faux positifs dans le tas, c'est pas grave
[17:27:42]
<Yunohost Git/Infra notifications> [issues] chri2 labeled :birthday: feature on [issue #2414](https://github.com/YunoHost/issues/issues/2414): improve handling of app_senders_login_maps
[17:27:42]
<Yunohost Git/Infra notifications> [issues] chri2 opened [issue #2414](https://github.com/YunoHost/issues/issues/2414): improve handling of app_senders_login_maps
[18:04:29]
<Yunohost Git/Infra notifications> [yunohost] alexAubin pushed 1 commit to dev: helpers2.1: fix bad syntax in ynh_app_upstream_version ([30467f8b](https://github.com/YunoHost/yunohost/commit/30467f8bc3f55f366b3c4ad957ccb5a4f09df5e9))
[18:15:20]
<Yunohost Git/Infra notifications> 🏗️ Starting build for yunohost/11.2.16+202406231815 for bullseye/unstable/all ...
[18:16:44]
<Yunohost Git/Infra notifications> ✔️ Completed build for yunohost/11.2.16+202406231815 for bullseye/unstable/all.
[19:20:51]
<eric_G> some new warning popping up the script:
```
python3 ./convert_to_helpers_2.1.py /Users/eric/Documents/GitHub/planka_ynh
/Users/eric/Documents/./convert_to_helpers_2.1.py:384: SyntaxWarning: invalid escape sequence '\.'
raw_manifest = re.sub('yunohost = ">= 11\..*"', 'yunohost = ">= 11.2.16"', raw_manifest)
```