[00:01:16]
<Yunohost Git/Infra notifications> [Apps tools error] [List builder] Failed to fetch the CI apps dev result : (Connection broken: IncompleteRead(130556 bytes read, 1305012 more expected), IncompleteRead(130556 bytes read, 1305012 more expected))
[00:02:03]
<Yunohost Git/Infra notifications> [Apps tools error] [List builder] Failed to fetch the CI apps dev result : (Connection broken: IncompleteRead(540156 bytes read, 895412 more expected), IncompleteRead(540156 bytes read, 895412 more expected))
[00:02:25]
<Yunohost Git/Infra notifications> [issues] DeMiro5001 [commented](https://github.com/YunoHost/issues/issues/2818#issuecomment-4940251078) on [issue #2818](https://github.com/YunoHost/issues/issues/2818) env_var combines unit file variables in app shell: Argh, show doesnt output json. Other systemctl options do.
[00:02:57]
<Yunohost Git/Infra notifications> [issues] DeMiro5001 [commented](https://github.com/YunoHost/issues/issues/2818#issuecomment-4940336676) on [issue #2818](https://github.com/YunoHost/issues/issues/2818) env_var combines unit file variables in app shell: This handles key value pairs and preserves quoted strings
https://docs.python.org/3.12/library/shlex.html
Ill try it th...
[06:38:51]
<Yunohost Git/Infra notifications> [issues] tituspijean [commented](https://github.com/YunoHost/issues/issues/2818#issuecomment-4943132205) on [issue #2818](https://github.com/YunoHost/issues/issues/2818) env_var combines unit file variables in app shell: @DeMiro5001 here is something that seems to work:
eval "(python3 -c
import sys, shlex
s=sys.argv[1]
out=[]
for i...
[06:51:54]
<selfhoster1312> oh, the xmpp gateway is back, hello friends :)
[06:53:35]
<selfhoster1312> i read on nlnet website about the manifest v3 (pun intended) packaging and i was wondering if there's a plan to make a proper working group / specifications (beyond a github issue)
[06:59:46]
<Yunohost Git/Infra notifications> [yunohost] tituspijean pushed to fix-2818: fix(helpers): app_shell parsing Environment options ([a6319ec7](https://github.com/YunoHost/yunohost/commit/a6319ec7810c9303bfd1c135d507aa08cea80840))
[07:10:22]
<Yunohost Git/Infra notifications> [yunohost] tituspijean opened [pull request #2320](https://github.com/YunoHost/yunohost/pull/2320): fix(helpers): app_shell parsing Environment options
[07:10:47]
<Yunohost Git/Infra notifications> [yunohost] tituspijean [commented](https://github.com/YunoHost/yunohost/pull/2320#issuecomment-4943420540) on [issue #2320](https://github.com/YunoHost/yunohost/pull/2320) fix(helpers): app_shell parsing Environment options: @zamentur, you seemed wary about using eval in the original issue. Is that implementation OK?
[07:17:13]
<tituspijean[m]> @_bifrost_selfhoster1312=2fdev=40conference.yunohost.org:aria-net.org hi! So far no, I think the matter is for now mostly powered by Aleks, who is the lead on that task, with the one issue used as a wishlist/braindump log.
Feel free to discuss here if he's available, and we also have the meetings if a live oral discussion becomes necessary.
[07:18:42]
<selfhoster1312> thanks! and good luck on #2818 which is one good showcase of why we need better packaging to get rid of bash entirely :D :D :D
[07:24:17]
<tituspijean[m]> oh yeah xD
[07:29:57]
<selfhoster1312> https://xmpp-upload.kl.netlib.re/upload/019f5015-3ace-7bd6-bb73-67bb995ef31c/bash-vars.png
[08:54:51]
<Yunohost Git/Infra notifications> [yunohost] DeMiro5001 [commented](https://github.com/YunoHost/yunohost/pull/2320#issuecomment-4944111062) on [issue #2320](https://github.com/YunoHost/yunohost/pull/2320) fix(helpers): app_shell parsing Environment options: The docs of shlex say > Changed in version 3.12: Passing None for s argument now raises an exception, rather than readin...
[12:47:47]
<Yunohost Git/Infra notifications> [yunohost] tituspijean approved [pull request #2314](https://github.com/YunoHost/yunohost/pull/2314#pullrequestreview-4677801083) fix : exclude second port used by app
[12:52:33]
<Yunohost Git/Infra notifications> [yunohost] tituspijean [commented](https://github.com/YunoHost/yunohost/pull/2320#issuecomment-4946034995) on [issue #2320](https://github.com/YunoHost/yunohost/pull/2320) fix(helpers): app_shell parsing Environment options: > Did you test apps that do not have environment variables? I mean when "env_var" is empty Looks ok, no warnings displ...
[16:05:42]
<Yunohost Git/Infra notifications> [issues] zamentur [commented](https://github.com/YunoHost/issues/issues/2816#issuecomment-4947438989) on [issue #2816](https://github.com/YunoHost/issues/issues/2816) Baikal upgrade internal error: Maybe your system is full can you validate the error is still here AND run
df -h
Have you some error in your d...
[16:06:50]
<Yunohost Git/Infra notifications> [issues] zamentur [commented](https://github.com/YunoHost/issues/issues/2816#issuecomment-4947452513) on [issue #2816](https://github.com/YunoHost/issues/issues/2816) Baikal upgrade internal error: Are you able to fin the log related to baikal upgrade ?
Webadmin > Tools > Logs > Upgrade baikal (could you share it w...
[16:26:46]
<Yunohost Git/Infra notifications> [webhooks] tituspijean pushed to enh-links: enh(comments): add a zero-width space between links and the colon ([7370cefa](https://github.com/YunoHost/webhooks/commit/7370cefa958981389a116832d8be9dc5bc2a2e1a))
[16:27:37]
<Yunohost Git/Infra notifications> [webhooks] tituspijean opened [pull request #13](https://github.com/YunoHost/webhooks/pull/13): enh(comments): add a zero-width space between links and the colon
[16:39:25]
<Yunohost Git/Infra notifications> [webhooks] tituspijean pushed to enh-links: enh(comments): zero-width spaces are stripped, switch to normal spaces ([388ee107](https://github.com/YunoHost/webhooks/commit/388ee1072492902f7a19d450324f950e0f709865))
[16:39:27]
<Yunohost Git/Infra notifications> [webhooks] tituspijean edited [pull request #13](https://github.com/YunoHost/webhooks/pull/13) : enh(comments): add a space between links and the colon
[16:39:34]
<Yunohost Git/Infra notifications> [webhooks] tituspijean edited [pull request #13](https://github.com/YunoHost/webhooks/pull/13) : enh(comments): add a space between links and the colon
[16:43:43]
<Yunohost Git/Infra notifications> [yunohost] zamentur [commented](https://github.com/YunoHost/yunohost/pull/2320#issuecomment-4947802823) on [issue #2320](https://github.com/YunoHost/yunohost/pull/2320) fix(helpers): app_shell parsing Environment options: > @zamentur, you seemed wary about using eval in the original issue. Is that implementation OK? I think eval should a...
[16:46:25]
<Yunohost Git/Infra notifications> [webhooks] tituspijean pushed to add-unassigned: fix(comments): unassigned action handling for PRs ([e3e98434](https://github.com/YunoHost/webhooks/commit/e3e984344ef1b1a70e1362459d1b8573eb59d0fa))
[16:46:26]
<Yunohost Git/Infra notifications> [webhooks] tituspijean opened [pull request #14](https://github.com/YunoHost/webhooks/pull/14) : fix(comments): unassigned action handling for PRs
[16:46:40]
<Yunohost Git/Infra notifications> [webhooks] tituspijean merged [pull request #14](https://github.com/YunoHost/webhooks/pull/14) : fix(comments): unassigned action handling for PRs
[16:46:44]
<Yunohost Git/Infra notifications> [webhooks] tituspijean pushed to main: fix(comments): unassigned action handling for PRs ([b80d89dc](https://github.com/YunoHost/webhooks/commit/b80d89dc0ec17c9e4a24f8af65b636c93ca964d9))
[16:46:57]
<Yunohost Git/Infra notifications> [webhooks] tituspijean deleted branch add-unassigned
[16:49:24]
<Yunohost Git/Infra notifications> [webhooks] tituspijean merged [pull request #13](https://github.com/YunoHost/webhooks/pull/13) : enh(comments): add a space between links and the colon
[16:49:26]
<Yunohost Git/Infra notifications> [webhooks] tituspijean pushed to main: enh(comments): add a zero-width space between links and the colon ([9ddf61fd](https://github.com/YunoHost/webhooks/commit/9ddf61fd8cc747cab55cff571b7583e44d48a07e))
[16:49:28]
<Yunohost Git/Infra notifications> [webhooks] tituspijean pushed to main: enh(comments): zero-width spaces are stripped, switch to normal spaces ([607ae721](https://github.com/YunoHost/webhooks/commit/607ae721456b02387b0c5fbdc4b172fed009a76a))
[16:49:29]
<Yunohost Git/Infra notifications> [webhooks] tituspijean deleted branch enh-links
[16:59:22]
<Yunohost Git/Infra notifications> [webhooks] tituspijean [commented](https://github.com/YunoHost/webhooks/pull/10#discussion_r3564634319) on pull request #10 Generate MOST_POPULAR_APPS based on catalog contents: suggestion
# for apps repo, only notify for apps that are in the most popular apps
[16:59:24]
<Yunohost Git/Infra notifications> [webhooks] tituspijean [commented](https://github.com/YunoHost/webhooks/pull/10#discussion_r3564637227) on pull request #10 Generate MOST_POPULAR_APPS based on catalog contents: Removed because unused?
[16:59:30]
<Yunohost Git/Infra notifications> [webhooks] tituspijean [commented](https://github.com/YunoHost/webhooks/pull/10#discussion_r3564648025) on pull request #10 Generate MOST_POPULAR_APPS based on catalog contents: These @cache will keep the values in cache for as long as the service is running. Maybe we could implement a monthly r...
[17:49:19]
<Yunohost Git/Infra notifications> [yunohost] DeMiro5001 [commented](https://github.com/YunoHost/yunohost/pull/2320#issuecomment-4948100987) on [issue #2320](https://github.com/YunoHost/yunohost/pull/2320) fix(helpers): app_shell parsing Environment options: We can use https://www.freedesktop.org/software/systemd/man/latest/systemd-run.html (from what I understood, it gets env...
[21:30:47]
<Yunohost Git/Infra notifications> [yunohost] kay0u pushed to backup-tar-rework: backup: Factorize archive extension resolution helper in backup.py ([8308c594](https://github.com/YunoHost/yunohost/commit/8308c5946760be32bc3b0998a8e6df844403a272))
[21:30:48]
<Yunohost Git/Infra notifications> [yunohost] kay0u pushed to backup-tar-rework: backup: Read tar archives in a single sequential stream pass ([7b82a6fe](https://github.com/YunoHost/yunohost/commit/7b82a6feb4dc41bf3aec3dcb16859da4a110bbdb))
[21:30:48]
<Yunohost Git/Infra notifications> [yunohost] kay0u pushed to backup-tar-rework: backup: Compress tar.gz archives with pigz to use all cores ([2ad1db37](https://github.com/YunoHost/yunohost/commit/2ad1db37d18cb257a889f1cd50a7bba975c46ec0))
[21:31:02]
<Yunohost Git/Infra notifications> [yunohost] tituspijean pushed to fix-2818: fix(helpers): do not use eval for app_shell ([14f30580](https://github.com/YunoHost/yunohost/commit/14f3058066f178243cab33a6592d54481e7917a3))
[21:37:34]
<Yunohost Git/Infra notifications> [yunohost] kay0u opened [pull request #2321](https://github.com/YunoHost/yunohost/pull/2321) : backup: Rework tar archive reading (single pass) and compress with pigz
[21:43:27]
<Yunohost Git/Infra notifications> [yunohost] tituspijean [commented](https://github.com/YunoHost/yunohost/pull/2320#issuecomment-4948851872) on [issue #2320](https://github.com/YunoHost/yunohost/pull/2320) fix(helpers): app_shell parsing Environment options: systemd-run seemed overly complex to bend to our use case (one main issue was that it would create the app shell based...
[21:52:52]
<Yunohost Git/Infra notifications> [yunohost] tituspijean pushed to fix-2818: fix(helpers): typo in app_shell ([f51cd9dd](https://github.com/YunoHost/yunohost/commit/f51cd9dda1edcc92934369efc1e1120c8073bcfa))
[22:01:43]
<Yunohost Git/Infra notifications> [yunohost] kay0u pushed to fix-sse-during-upgrade-2: Another way to fix the sse, keep only one socket for all operations, no need to open and close it everytime ([299762a2](https://github.com/YunoHost/yunohost/commit/299762a255230511831c64dcbbf3d5058765a13f))
[22:01:48]
<Yunohost Git/Infra notifications> [yunohost] kay0u pushed to fix-sse-during-upgrade-2: functools.cache instead of global variable, reconnect the socket if needed ([6fdbf394](https://github.com/YunoHost/yunohost/commit/6fdbf3949ec1db5179822f0813bbedc10507c029))
[22:01:50]
<Yunohost Git/Infra notifications> [yunohost] kay0u pushed to fix-sse-during-upgrade-2: sse: share a single zmq context across clients instead of one per connection ([67380bea](https://github.com/YunoHost/yunohost/commit/67380bead1fbd213a4ce1ed72c160148123ac88a))
[22:01:52]
<Yunohost Git/Infra notifications> [yunohost] kay0u edited [pull request #2291](https://github.com/YunoHost/yunohost/pull/2291) : Another way to fix SSE during upgrade