Thursday, February 08, 2024
apps@conference.yunohost.org
February
Mon Tue Wed Thu Fri Sat Sun
      1
2 3 4
5 6 7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
     
             

[19:42:02] <Émy - OniriCorpe> mdr j'arrive déjà pas à faire la vaisselle
[20:06:40] <lapineige> club 😂
[20:09:47] <Chatpitaine Caverne> De mon côté, vais tenter de packager ceci [Gitlab - Garden Party](https://gitlab.com/experimentslabs/garden-party/garden-party).
Après vu la période que je traverse, ça risque d'être long. Pas non plus encore étudié s'il peut y avoir des impossibilité techniques. Mais ce dev est en quête de visibilité pour ce joli projet et Yunohost, ça peut aider.
[20:11:56] <lapineige> > <@mathieuw:tetaneutral.net> Allez, why not : ça semble bien fonctionner, et c'est d'une efficacité assez redoutable. Mais j'ai des scrupules à rajouter un peu légèrement une app au catalogue because on se dit que déjà y'en a beaucoup. Mais bon, c'est super utile, donc zou -> wishlist. Merci orhtej2 et Émy - OniriCorpe :-)

Intéressant comparé à Framadate en effet pour certains cas d'usages. Probablement pas accessible par contre
[20:12:27] <lapineige> pas horrible à packager a priori, mais pas hyper rapide non plus
[20:15:31] <Émy - OniriCorpe> > A package contains a precompiled assets and all the Ruby dependencies. Also, almost all the unnecessary files are removed.

https://gitlab.com/experimentslabs/garden-party/garden-party/-/blob/develop/docs/production.md

Oh that's nice for a ynh package!\
[20:15:31] <Émy - OniriCorpe> > A package contains a precompiled assets and all the Ruby dependencies. Also, almost all the unnecessary files are removed.

https://gitlab.com/experimentslabs/garden-party/garden-party/-/blob/develop/docs/production.md

Oh that's nice for a ynh package!
[20:36:33] <eric_G> reconnect with nature with Garden Party app... 🫠
[21:07:24] <Yunohost Git/Infra notifications> [apps] @Salamandar [commented](https://github.com/YunoHost/apps/pull/1999#discussion_r1483590423) on pull request #1999 Rework list_builder.py: okay :)
[21:19:36] <Yunohost Git/Infra notifications> [apps] @Salamandar [commented](https://github.com/YunoHost/apps/pull/1999#discussion_r1483601434) on pull request #1999 Rework list_builder.py: Done: logging_sender
[21:27:44] <Salamandar> fyi @Alekswag:matrix.org my PR is ready :)
[21:33:26] <Salamandar> https://github.com/Salamandar/apps/actions/runs/7836212052/job/21383331504
[21:33:48] <Salamandar> That's why you need `#!/usr/bin/env python3` and not just `#!/usr/bin/python3`
[21:39:10] <Yunohost Git/Infra notifications> [apps] @alexAubin pushed 11 commits to master ([44bcb8d481c1...eb39a26e0bb1](https://github.com/YunoHost/apps/compare/44bcb8d481c1...eb39a26e0bb1))
[21:39:11] <Yunohost Git/Infra notifications> [apps] @alexAubin merged [pull request #1999](https://github.com/YunoHost/apps/pull/1999): Rework list_builder.py
[21:39:12] <Yunohost Git/Infra notifications> [apps/master] Add test that builds the catalog - Félix Piédallu
[21:39:36] <Salamandar> o/
[21:52:59] <kayou> > <@Salamandar:matrix.org> That's why you need `#!/usr/bin/env python3` and not just `#!/usr/bin/python3`

what's the difference?
[21:53:30] <Salamandar> > <@kayou:matrix.org> what's the difference?

tl;dr if you have python3 in /usr/local/bin (and that is in your PATH), the second one won't work
[21:53:57] <Salamandar> /usr/bin/env will just execute the binary found in your path, instead of having the path hard-coded to /usr/bin
[21:54:06] <Salamandar> that also is the only way to play nice with virtualenvs
[21:54:21] <Salamandar> (without having to manually call venv/bin/python3 myscript
[21:54:52] <Salamandar> > ts - adds a timestamp to the beginning of each line of input.

OMG so useful
[21:55:36] <Salamandar> ```
~/t/Y/a/b/default ❯❯❯ echo toto | ts
f�vr. 08 22:55:10 toto
```
woops unicode
[21:56:35] <Aleks (he/him/il/lui)> > <@Salamandar:matrix.org> (without having to manually call venv/bin/python3 myscript

ah using `#!/usr/bin/env python3` will implicitly source the venv if there is one ? o_O
[21:56:45] <Aleks (he/him/il/lui)> ah nah nvm
[21:56:50] <Salamandar> If you sourced `activate` yeah
[21:57:03] <Aleks (he/him/il/lui)> yeah
[21:57:03] <Salamandar> hu
[21:57:03] <Salamandar> sorry misread
[21:57:38] <Aleks (he/him/il/lui)> i was hoping you could run e.g. `./manage.py` without sourcing the venv but you still need to source it
[21:57:52] <Salamandar> ah yeah no you still need to source it
[21:57:53] <Aleks (he/him/il/lui)> cf my comment/question in some app PR earlier
[21:58:07] <kayou> > <@Salamandar:matrix.org> If you sourced `activate` yeah

how? i don't understand
[21:58:33] <Salamandar> > <@kayou:matrix.org> how? i don't understand

source venv/bin/activate will add venv/bin to your PATH, so /usr/bin/env will find python3 there first, before the system python3
[21:59:50] <Aleks (he/him/il/lui)> hmyeah in the end it "just" allows to be able to write `./manage.py` but honestly in all cases it's easier to write `python3 manage.py` or even `venv/bin/python3 manage.py` (so that you don't have to care about sourcing the venv) rather than `./manage.py` because then you also need `chmod +x`
[22:00:30] <Aleks (he/him/il/lui)> (in a script at least, maybe if you call manage.py manually from CLI it helps yeah)
[22:01:24] <Salamandar> > because then you also need chmod +x

first step when i write a script is chmod +x :D
[22:01:31] <Salamandar> to each their own i guess :D
[22:02:09] <Yunohost Git/Infra notifications> [apps] @ericgaspar pushed 1 commit to open-web-calendar: Update apps.toml ([714ebd9c](https://github.com/YunoHost/apps/commit/714ebd9c7f5b4279ebfb48a778b1a7ba785fb250))
[22:02:13] <Yunohost Git/Infra notifications> [apps] @ericgaspar created new branch open-web-calendar
[22:04:11] <Yunohost Git/Infra notifications> [apps] @ericgaspar pushed 1 commit to open-web-calendar: Create open-web-calendar.png ([d99bfe87](https://github.com/YunoHost/apps/commit/d99bfe87463e5d2894de97a4422c63614a640c77))
[22:04:42] <Yunohost Git/Infra notifications> [apps] @ericgaspar opened [pull request #2003](https://github.com/YunoHost/apps/pull/2003): Add Open web calendar to catalog
[22:09:51] <Yunohost Git/Infra notifications> [apps] @Salamandar opened [pull request #2004](https://github.com/YunoHost/apps/pull/2004): Fixes
[22:10:40] <Yunohost Git/Infra notifications> [apps] @alexAubin merged [pull request #2004](https://github.com/YunoHost/apps/pull/2004): Fixes
[22:10:45] <Yunohost Git/Infra notifications> [apps] @alexAubin pushed 3 commits to master ([eb39a26e0bb1...b4f0a68807cb](https://github.com/YunoHost/apps/compare/eb39a26e0bb1...b4f0a68807cb))
[22:10:45] <Yunohost Git/Infra notifications> [apps/master] Wooops, revert wrong apps_repo repo - Félix Piédallu
[22:14:24] <Yunohost Git/Infra notifications> App nextcloud failed all tests in job [#23120](https://ci-apps.yunohost.org/ci/job/23120) :(
[23:02:12] <Émy - OniriCorpe> the fail with the sub_path test is really weird
https://ci-apps-dev.yunohost.org/ci/job/13382
[23:02:13] <orhtej2> > <@oniricorpe:im.emelyne.eu> the fail with the sub_path test is really weird
> https://ci-apps-dev.yunohost.org/ci/job/13382

is it? It means you were sent to ynh portal AKA the app was not where it was supposed to be
[23:03:46] <orhtej2> did it ever pass this test? I'm 37.9% sure [this line](https://github.com/YunoHost-Apps/kavita_ynh/blob/b5b843d79ba2cedc144c1f99322a58f3d92ffab8/conf/nginx.conf#L4) should end in `/` to strip the path from which NGINX did proxy pass
[23:04:54] <orhtej2> let's see.... https://github.com/YunoHost-Apps/kavita_ynh/pull/41
[23:17:27] <Yunohost Git/Infra notifications> [nextcloud_ynh] @rodinux opened [pull request #663](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/663): Update manifest.toml
[23:20:41] <orhtej2> > let's see.... https://github.com/YunoHost-Apps/kavita_ynh/pull/41

Sort of worked
[23:35:34] <Émy - OniriCorpe> > Sort of worked

yeah weird, thank you!
[23:46:08] <Yunohost Git/Infra notifications> [nextcloud_ynh] @rodinux [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/663#issuecomment-1935105849) on [issue #663](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/663) Update manifest.toml: testme
[23:46:14] <Yunohost Git/Infra notifications> [nextcloud_ynh] @yunohost-bot [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/663#issuecomment-1935105874) on [issue #663](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/663) Update manifest.toml: Fingers crossed
[[Test Badge](https://img.shields.io/endpoint?url=https://ci-apps-dev.yunohost.org/ci/api/job/13385/ba...
[23:48:45] <Yunohost Git/Infra notifications> [nextcloud_ynh] @rodinux edited [pull request #663](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/663): Update manifest.toml
[23:53:07] <Yunohost Git/Infra notifications> [nextcloud_ynh] @OniriCorpe [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/663#discussion_r1483716469) on pull request #663 Update manifest.toml: hi why have you commented this?
[23:58:06] <Yunohost Git/Infra notifications> [nextcloud_ynh] @rodinux [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/663#discussion_r1483718917) on pull request #663 Update manifest.toml: These lines was not here on v27.1.4, they have been added after a bug with the v28... https://github.com/nextcloud/serve...