[01:19:32]
<Aleks (he/him/il/lui)> annnnd oopsies it's 3am
[01:19:32]
<Yunohost Git/Infra notifications> [yunohost/helpers-2.1] apps: remove /var/log/app during app_remove if --purge is used - Alexandre Aubin
[01:19:32]
<Yunohost Git/Infra notifications> [yunohost] alexAubin pushed 6 commits to helpers-2.1 ([ef924f7a1786...fca26ead788f](https://github.com/YunoHost/yunohost/compare/ef924f7a1786...fca26ead788f))
[01:19:32]
<Aleks (he/him/il/lui)> anyway, i have a pretty good working prototype to magically patch apps for helpers 2.1
[01:29:51]
<Yunohost Git/Infra notifications> [yunohost] alexAubin edited [pull request #1855](https://github.com/YunoHost/yunohost/pull/1855): Helpers 2.1
[01:33:46]
<Yunohost Git/Infra notifications> [yunohost] alexAubin edited [pull request #1855](https://github.com/YunoHost/yunohost/pull/1855): Helpers 2.1
[02:36:19]
<Yunohost Git/Infra notifications> [yunohost] 🔴 Pipeline [#1326427274](https://gitlab.com/YunoHost/yunohost/-/pipelines/1326427274) failed on branch helpers-2.1
[08:04:31]
<Yunohost Git/Infra notifications> [yunohost] 🔴 Pipeline [#1325208401](https://gitlab.com/YunoHost/yunohost/-/pipelines/1325208401) failed on branch dev, bookworm, migrate-to-bookworm
[11:48:48]
<Aleks (he/him/il/lui)> (ti tus : followup of https://forum.yunohost.org/t/cant-install-anything-in-the-root-domain-due-to-conduit-well-known/29987, do you remember what's the status of https://github.com/YunoHost/yunohost/pull/1647 ? The approach sounds good ? (I didn't read the full discussion))
[13:00:08]
<Aleks (he/him/il/lui)> `ynh_foobar_load_path` :|
[13:00:18]
<Aleks (he/him/il/lui)> i'm discovering it's not just for node but also go and ruby T_T
[13:09:16]
<Aleks (he/him/il/lui)> and the epic `ynh_exec_warn_less ynh_exec_as $app $ynh_foobar_load_path $ynh_foo install`
[13:09:44]
<Aleks (he/him/il/lui)> and the epic `ynh_exec_warn_less ynh_exec_as $app env $ynh_foobar_load_path $ynh_foo install`
[13:10:52]
<tituspijean> The status was "ready to merge", though I would still try to install again a synapse and conduit app on the same server to check if that still works (it should, though). The approach is "allow an app to declare a path under /well-known, even though there is an app install on the root of the domain."
[13:11:03]
<tituspijean> The status was "ready to merge", though I would still try to install again a synapse and conduit app on the same server to check if that still works (it should, though). The approach is "allow an app to declare a path under /well-known, even though there is an app installed on the root of the domain."
[13:13:50]
<Aleks (he/him/il/lui)> > <@Alekswag:matrix.org> and the epic `ynh_exec_warn_less ynh_exec_as $app env $ynh_foobar_load_path $ynh_foo install`
anybody understands why we don't just add `-E` to the underlying sudo call to preserve env variables ? X_X
[13:14:40]
<tituspijean> I don't see why we should not add it
[13:15:27]
<tituspijean> oooor #madlad , let's use the app_shell helper? I dunno if it would be fully initialized, I don't think so, since it parses the service conf.
[13:20:04]
<Aleks (he/him/il/lui)> ah, variables gotta be exported to be propagated with `-E`
[13:21:14]
<Aleks (he/him/il/lui)> (or you can write `FOO=bar sudo -E -u $user` instead of `sudo -u $user env FOO=bar`)
[17:38:49]
<Yunohost Git/Infra notifications> [yunohost] alexAubin pushed 4 commits to helpers-2.1 ([fca26ead788f...47b2a5695f3a](https://github.com/YunoHost/yunohost/compare/fca26ead788f...47b2a5695f3a))
[17:38:52]
<Yunohost Git/Infra notifications> [yunohost/helpers-2.1] helpers2.1: rework the nodejs/ruby/go mess: export the appropriate extended PATH to be able to call npm/node/ruby/gem/go... - Alexandre Aubin
[18:34:21]
<Yunohost Git/Infra notifications> [yunohost] alexAubin edited [pull request #1855](https://github.com/YunoHost/yunohost/pull/1855): Helpers 2.1
[20:55:44]
<Yunohost Git/Infra notifications> [yunohost] alexAubin pushed 1 commit to helpers-2.1: helpers2.1: in ynh_setup_source, remove the underused sources/extra_files/sourceid/ mechanism (basically the only apps w... ([66f667e4](https://github.com/YunoHost/yunohost/commit/66f667e48a20aaf8694ef936f2ce5082254de1bf))
[21:08:58]
<Yunohost Git/Infra notifications> [yunohost] alexAubin edited [pull request #1855](https://github.com/YunoHost/yunohost/pull/1855): Helpers 2.1
[21:19:31]
<Yunohost Git/Infra notifications> [yunohost] ericgaspar [commented](https://github.com/YunoHost/yunohost/pull/1855#issuecomment-2161614323) on [issue #1855](https://github.com/YunoHost/yunohost/pull/1855) Helpers 2.1: Amazing cleanup job well done