[04:29:54]
<Yunohost Git/Infra notifications> [yunohost] OniriCorpe approved [pull request #1851](https://github.com/YunoHost/yunohost/pull/1851#pullrequestreview-2081713533) helpers: Add a --jinja option to ynh_add_config
[06:56:33]
<ljf> jinja 2, the return !!!
[07:31:24]
<Salamandar> yeah
[07:31:29]
<Salamandar> might be the future too
[07:34:43]
<Yunohost Git/Infra notifications> [yunohost] Salamandar [commented](https://github.com/YunoHost/yunohost/pull/1851#issuecomment-2134536619) on [issue #1851](https://github.com/YunoHost/yunohost/pull/1851) helpers: Add a --jinja option to ynh_add_config: Actually I got a request, here in pseudocode : if filename.endswith(".j2"): jinja = True That would help usi...
[08:53:24]
<ChriChri> Ha! First time flohmarkt installed using my `ynh_handle_getopts_args` - learned a lot about bash...
[08:58:15]
<ChriChri> I don't want to know what happens when I submit to CI...
[08:58:31]
<Salamandar> fyi @chrichri:librem.one https://github.com/YunoHost/issues/issues/2403
[09:06:02]
<Yunohost Git/Infra notifications> [yunohost] chri2 [commented](https://github.com/YunoHost/yunohost/pull/1716#issuecomment-2134708857) on [issue #1716](https://github.com/YunoHost/yunohost/pull/1716) Add --follow_symlink to ynh_secure_remove: A symlink is just a file. My expectation (beside an additional --follow-symlinks) would be that the file is deleted no...
[09:07:25]
<Aleks (he/him/il/lui)> > <@chrichri:librem.one> I don't want to know what happens when I submit to CI...
https://c.tenor.com/CXojMvlk4nQAAAAC/tenor.gif
[09:13:39]
<Yunohost Git/Infra notifications> [yunohost] alexAubin [commented](https://github.com/YunoHost/yunohost/pull/1851#issuecomment-2134724305) on [issue #1851](https://github.com/YunoHost/yunohost/pull/1851) helpers: Add a --jinja option to ynh_add_config: >That would help using jinja2 for nginx/systemd/etc without having to add --jinja arguments to all the relevant helpers....
[09:17:30]
<Yunohost Git/Infra notifications> [yunohost] Salamandar [commented](https://github.com/YunoHost/yunohost/pull/1716#issuecomment-2134732277) on [issue #1716](https://github.com/YunoHost/yunohost/pull/1716) Add --follow_symlink to ynh_secure_remove: @chri2 Yes, I fully agree. The issue described here is not that. The current issue is that if a symlink is broken, tryin...
[09:29:16]
<Yunohost Git/Infra notifications> [yunohost] alexAubin pushed 11 commits to add-jinja-support-to-ynh-add-config ([4a9a3ba13837...65abffa1ced1](https://github.com/YunoHost/yunohost/compare/4a9a3ba13837...65abffa1ced1))
[09:29:21]
<Yunohost Git/Infra notifications> [yunohost/add-jinja-support-to-ynh-add-config] Fix helpers: actually that might be a bad idea to read arguments as if source helpers has no arguments passed, @ is t... - Félix Piédallu
[09:46:18]
<Yunohost Git/Infra notifications> [issues] chri2 [commented](https://github.com/YunoHost/issues/issues/2403#issuecomment-2134792949) on [issue #2403](https://github.com/YunoHost/issues/issues/2403) Helpers 2.1 wishlist: > * Remove the need to pass --app=app to ynh_app_settings_set/get (or is it already optional?) I think it is already o...
[09:46:53]
<ChriChri> > <@Alekswag:matrix.org> https://c.tenor.com/CXojMvlk4nQAAAAC/tenor.gif
One of the best movies ever...
[10:23:03]
<ChriChri> > <@Alekswag:matrix.org> https://c.tenor.com/CXojMvlk4nQAAAAC/tenor.gif
https://ci-apps-bookworm-dev.yunohost.org/ci/job/1164 😅
[10:32:39]
<Yunohost Git/Infra notifications> [yunohost] 🔴 Pipeline [#1308188616](https://gitlab.com/YunoHost/yunohost/-/pipelines/1308188616) failed on branch add-jinja-support-to-ynh-add-config
[10:54:05]
<Yunohost Git/Infra notifications> [yunohost] chri2 opened [pull request #1856](https://github.com/YunoHost/yunohost/pull/1856): getopts
[11:16:13]
<Aleks (he/him/il/lui)> ChriChri: uuuh this sounds interesting but ugh could you provide some examples of what could not be achieved before this and what can now be done ?
[11:28:51]
<Aleks (he/him/il/lui)> https://aria.im/_matrix/media/v1/download/matrix.org/RtionBXKgaceUCpviOOhnHHp
[11:55:08]
<ChriChri> > <@Alekswag:matrix.org> ChriChri: uuuh this sounds interesting but ugh could you provide some examples of what could not be achieved before this and what can now be done ?
e.g. you want to parse a command line like `--header="Accept: text/html,application/xhtml+xml,application/xml;q=0.9,*/*;q=0.8" --header='Forwarded: by="10.0.0.1"; for="192.168.0.1:4711"; proto=http; host="www.example.com:8080'`. Those parameters to `--header` contain ';' and it would complicated to split them correctly into their parts if they would be concatenated by ';'.
[11:59:48]
<ChriChri> ```
# usage: ynh_local_curl "page" "key1=value1" "key2=value2" ...
# | arg: -l --line_match: check answer against an extended regex
# | arg: -m --method: request method to use: POST (default), PUT, GET, DELETE
# | arg: -H --header: add a header to the request (can be used multiple times)
# | arg: -d --data: data to be PUT or POSTed. Can be used multiple times.
```
…
```
local -A args_array=( [l]=line_match= [m]=method= [H]=header= [n]=no_sleep [L]=location= [d]=data= [u]=user= [p]=password= )
local method
local -a header
local location
local user
local password
local -a data
```
[12:00:08]
<Salamandar> FYI the next version of the helpers will completely change the way arguments are handled
[12:00:27]
<Salamandar> my suggestion is to use bash-modules, i really like the code
[12:01:08]
<Yunohost Git/Infra notifications> [yunohost] chri2 [commented](https://github.com/YunoHost/yunohost/pull/1856#issuecomment-2135044873) on [issue #1856](https://github.com/YunoHost/yunohost/pull/1856) getopts: what it could be good for: Lets say you want to parse a command line like --header="Accept: text/html,application/xht...
[12:13:42]
<ChriChri> > <@Salamandar:matrix.org> FYI the next version of the helpers will completely change the way arguments are handled
No ynh_getopts no more?
[12:14:34]
<Salamandar> do you mean `ynh_handle_getopts_args` ?
[12:15:11]
<Salamandar> tl;dr here is my current branch :
[12:15:17]
<Salamandar> https://aria.im/_matrix/media/v1/download/matrix.org/mrmOCGTcDIcTQtuCrNPNulgE
[12:20:08]
<ChriChri> Looks nice and clean :)
[12:23:49]
<Salamandar> my idea too ^^
[12:32:33]
<ChriChri> Is `ynh_handle_getopts` meanwhile still interesting? Otherwise I'll just use it for my own version of `ynh_local_curl` to be able to do stuff on couchdb or `curl` the requests directly.
[12:34:54]
<John Livingston> Hello. Is your new mascot already public? Is there an online file we can download?
[12:35:00]
<John Livingston> (and share)
[12:37:44]
<Tag> > Hello. Is your new mascot already public? Is there an online file we can download?
Hey!! Par ici : https://forum.yunohost.org/uploads/default/original/2X/d/d1f46ba56aac14ecaca292dd6533d5e8462da0ad.jpeg
https://forum.yunohost.org/t/yunohost-passe-a-lechelle-ca-va-scaler/29148
[12:55:38]
<Salamandar> > <@chrichri:librem.one> Is `ynh_handle_getopts` meanwhile still interesting? Otherwise I'll just use it for my own version of `ynh_local_curl` to be able to do stuff on couchdb or `curl` the requests directly.
yes of course it is. Helpers v2 (or v2.1, not sure yet) won't be available for some time yet. It'll be a complete revamp of the helpers, and we might drop a lot of legacy/unwanted helpers.
[15:21:14]
<ChriChri> > <@Salamandar:matrix.org> yes of course it is. Helpers v2 (or v2.1, not sure yet) won't be available for some time yet. It'll be a complete revamp of the helpers, and we might drop a lot of legacy/unwanted helpers.
I'd strongly suggest to drop legacy as much as possible (I guess it is a huge amount of work to rewrite all helpers and it will probably even more work to adopt all the scripts contained in the app repositories).
[15:24:24]
<Salamandar> Yes that's the plan
[15:24:37]
<Salamandar> that's why we merged https://github.com/YunoHost/yunohost/pull/1717
[15:25:17]
<Salamandar> that way, apps can move to new helpers at their own pace
[15:26:14]
<Aleks (he/him/il/lui)> (i was thinking about creating an auto-patch script for https://github.com/YunoHost/yunohost/pull/1855 similar to the v1->v2 script and then we can create a PR on all apps like the --full-replace story)
[15:48:03]
<Salamandar> you sure about the auto patch script ? :p
[15:48:03]
<Salamandar> hmmm
[15:48:18]
<Tag> John Livingston: la qualité est suffisante ? je pense qu'on peut te fournir mieux au besoin
[15:49:37]
<John Livingston> Tag: à quel sujet ?
[15:49:49]
<Tag> la mascotte :)
[15:50:07]
<John Livingston> ah oui, parfait. C'était pour une personne qui demandait sur mastodon
[15:50:22]
<Aleks (he/him/il/lui)> > <@Salamandar:matrix.org> you sure about the auto patch script ? :p
yeah idk, why ? it's just about auto-replacing the old syntaxs with the new ones, e.g. removing the unecesasry `--app="$app"` in setting helpers call, etc
[15:51:27]
<Salamandar> yeah i guess
[16:14:26]
<kayou> > <@Alekswag:matrix.org> (i was thinking about creating an auto-patch script for https://github.com/YunoHost/yunohost/pull/1855 similar to the v1->v2 script and then we can create a PR on all apps like the --full-replace story)
> +8,452 −0
[16:14:28]
<kayou> HAAAAA
[16:26:48]
<Aleks (he/him/il/lui)> Yeah you should read the commitd individually :|
[16:30:18]
<Salamandar> > <@Alekswag:matrix.org> yeah idk, why ? it's just about auto-replacing the old syntaxs with the new ones, e.g. removing the unecesasry `--app="$app"` in setting helpers call, etc
but it could not be automerged
[16:50:57]
<Aleks (he/him/il/lui)> Yes, that's the "create a PR like the --full-replace story" part
[16:52:06]
<Salamandar> yeah
[16:52:09]
<Salamandar> ok
[16:52:46]
<Salamandar> didn't get that part
[17:45:13]
<Yunohost Git/Infra notifications> [yunohost] chri2 [commented](https://github.com/YunoHost/yunohost/pull/1717#issuecomment-2135797367) on [issue #1717](https://github.com/YunoHost/yunohost/pull/1717) Create versionned directories of the helpers: Im a big fan of allowing small steps in development. Id go that far to allow each helper to be called with an option ...
[18:00:02]
<eric_G> we need to bring the mascot files to the artwork repo https://github.com/YunoHost/yunohost-artwork
[18:13:39]
<Yunohost Git/Infra notifications> [yunohost-artwork] ericgaspar pushed 1 commit to main: add mascot ([8c2745cf](https://github.com/YunoHost/yunohost-artwork/commit/8c2745cfbf26f3c24338680de66c08c07d858d1d))
[18:14:19]
<Yunohost Git/Infra notifications> [yunohost-artwork] ericgaspar pushed 1 commit to main: Update README.md ([e5311b00](https://github.com/YunoHost/yunohost-artwork/commit/e5311b00e3428af9fb4dbbf6de18da2f107cb7a7))
[18:14:39]
<Yunohost Git/Infra notifications> [yunohost-artwork] ericgaspar pushed 1 commit to main: Update README.md ([0e929311](https://github.com/YunoHost/yunohost-artwork/commit/0e929311f6b40601dbcfa7ad60f889de54407d7b))
[18:30:09]
<eric_G> Voilà. we also need to credit the mascot artwork and maybe give some legal terme of use.
[18:57:51]
<mathieuw> > <@ericg:matrix.org> Voilà. we also need to credit the mascot artwork and maybe give some legal terme of use.
I'm on my mobile phone, so I can't commit, but this is the credit and licence I wrote for the leaflet:
* Credits : [kaerhon](https://www.instagram.com/kaerhon/)
* Licence : [CC by-NC-SA 4.0](https://creativecommons.org/licenses/by-nc-sa/4.0/deed.en) - see https://pad.lqdn.fr/p/yunohost-2024-1eravril-merch-opossum-violet, line 80
poke John Livingston
[19:03:34]
<Yunohost Git/Infra notifications> [yunohost-artwork] ericgaspar pushed 1 commit to main: add credit ([b6e564a8](https://github.com/YunoHost/yunohost-artwork/commit/b6e564a8268d1159eee55d3e377b9488be7d77af))
[19:07:33]
<Yunohost Git/Infra notifications> [yunohost-artwork] ericgaspar pushed 1 commit to main: add artworks ([a14d8943](https://github.com/YunoHost/yunohost-artwork/commit/a14d894308b2a26fc7e105c15f46592d45966bb9))
[19:08:25]
<eric_G> do we need to link the pad?
[19:10:48]
<eric_G> Fursona being the name of the mascot right?
[19:12:17]
<mathieuw> > <@ericg:matrix.org> do we need to link the pad?
Maybe not: it's for our eyes only (but I wanted to link the source for us)
[19:12:36]
<mathieuw> > <@ericg:matrix.org> Fursona being the name of the mascot right?
Claude is the name.
[19:40:45]
<Émy - OniriCorpe> > <@ericg:matrix.org> Fursona being the name of the mascot right?
https://fr.m.wikipedia.org/wiki/Fandom_furry#Fursona
[20:54:38]
<Yunohost Git/Infra notifications> [yunohost] Josue-T [commented](https://github.com/YunoHost/yunohost/pull/1851#issuecomment-2136088073) on [issue #1851](https://github.com/YunoHost/yunohost/pull/1851) helpers: Add a --jinja option to ynh_add_config: From my point of view Im to migrate progressively everything to jinja for theses raison: - Migrating to jinja for an ap...