Friday, November 15, 2024
apps@conference.yunohost.org
November
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
30
 
             

[08:40:59] <Yunohost Git/Infra notifications> [apps] a​lexAubin deleted branch add-to-wishlist-domain-watchdog
[08:41:31] <Yunohost Git/Infra notifications> [apps] a​lexAubin pushed 1 commit to add-to-wishlist-domain-watchdog: Add Domain Watchdog to wishlist ([e42e77c9](https://github.com/YunoHost/apps/commit/e42e77c90b86260b0dbd47768773713ee9e41399))
[08:41:32] <Yunohost Git/Infra notifications> [apps] a​lexAubin created new branch add-to-wishlist-domain-watchdog
[08:41:45] <Yunohost Git/Infra notifications> [apps] a​lexAubin deleted branch add-to-wishlist-domain-watchdog
[08:56:47] <Salamandar> > <@ericg:matrix.org> Salamandar: do we really need this annoying password policy? https://github.com/YunoHost-Apps/fab-manager_ynh/blob/cb5ae1be7bc28d29bd3b696e6eb4120d3dd9a5dc/scripts/_common.sh#L28-L48

if i'm the one who wrote that, that might be because the app itself does this check later
[09:31:17] <tituspijean> > <@Salamandar:matrix.org> if i'm the one who wrote that, that might be because the app itself does this check later

"This bash could have been a regex" 🙃
https://stackoverflow.com/a/21456918
[09:35:12] <Salamandar> > <@titus:pijean.ovh> "This bash could have been a regex" 🙃
> https://stackoverflow.com/a/21456918

it couldve :D
[09:35:40] <Yunohost Git/Infra notifications> [appgenerator] e​ricgaspar opened [pull request #10](https://github.com/YunoHost/appgenerator/pull/10): add Go
[09:59:12] <boss.bd.anime> Hi
[12:17:54] <Aleks (he/him/il/lui)> henlo
[12:19:38] <Yunohost Git/Infra notifications> [appgenerator] a​lexAubin merged [pull request #10](https://github.com/YunoHost/appgenerator/pull/10): add Go
[12:19:39] <Yunohost Git/Infra notifications> [appgenerator] a​lexAubin pushed 2 commits to main ([153bba5fdabf...f70c6103e08d](https://github.com/YunoHost/appgenerator/compare/153bba5fdabf...f70c6103e08d))
[12:19:42] <Yunohost Git/Infra notifications> [appgenerator/main] add Go - Éric Gaspar
[12:19:42] <Yunohost Git/Infra notifications> [appgenerator/main] Merge pull request #10 from ericgaspar/main add Go - Alexandre Aubin
[17:15:58] <Yunohost Git/Infra notifications> m​dtita forked apps to [mdtita/apps](https://github.com/mdtita/apps)
[17:38:31] <denisapain> heyo! I'm trying to make a chibisafe app for yunohost, but im getting a really confusing error

```
Info: INFO - [++..................] > --message=Installing dependencies...
Info: DEBUG - + ynh_exec_warn_less ynh_install_nodejs --nodejs_version=20
Info: WARNING - ./install: line 32: ynh_exec_warn_less: command not found
Info: DEBUG - + ynh_exit_properly
Warning: Removing the app after installation failure…
```

I see a bit of apps using "ynh\_exec\_warn\_less" so I'm not sure why it isn't working for me.

My install script is here https://git.qilk.de/denis/chibisafe\_ynh/src/branch/master/scripts/install
[17:49:17] <orhtej2> > <@denisapain:matrix.org> heyo! I'm trying to make a chibisafe app for yunohost, but im getting a really confusing error
>
> ```
> Info: INFO - [++..................] > --message=Installing dependencies...
> Info: DEBUG - + ynh_exec_warn_less ynh_install_nodejs --nodejs_version=20
> Info: WARNING - ./install: line 32: ynh_exec_warn_less: command not found
> Info: DEBUG - + ynh_exit_properly
> Warning: Removing the app after installation failure…
> ```
>
> I see a bit of apps using "ynh\_exec\_warn\_less" so I'm not sure why it isn't working for me.
>
> My install script is here https://git.qilk.de/denis/chibisafe\_ynh/src/branch/master/scripts/install

this was renamed in helpers 2.1, are you using these?
[17:50:36] <denisapain> > this was renamed in helpers 2.1, are you using these?

I based the app off Example_ynh and I think that uses helpers 2.1
[17:51:51] <orhtej2> https://yunohost.org/en/packaging_apps_helpers_v2.1#ynh-hide-warnings
[18:01:23] <denisapain> ```
Info: DEBUG - + apt-get --assume-yes --quiet -o=Acquire::Retries=3 -o=Dpkg::Use-Pty=0 --no-remove --option Dpkg::Options::=--force-confdef --option Dpkg::Options::=--force-confold install --fix-broken
Info: DEBUG - Reading package lists...
Info: DEBUG - Building dependency tree...
Info: DEBUG - Reading state information...
Info: DEBUG - 0 upgraded, 0 newly installed, 0 to remove and 1092 not upgraded.
Info: DEBUG - + rm --recursive --force /tmp/tmp.FIz3q5rYUZ
Info: DEBUG - + _ynh_apt_package_is_installed chibisafe-ynh-deps
Info: DEBUG - + local package=chibisafe-ynh-deps
Info: DEBUG - + dpkg-query --show '--showformat=${db:Status-Status}' chibisafe-ynh-deps
Info: DEBUG - + grep --quiet '^installed$'
Info: DEBUG - ++ _ynh_apt_package_is_installed postgresql-13
Info: DEBUG - ++ local package=postgresql-13
Info: DEBUG - ++ dpkg-query --show '--showformat=${db:Status-Status}' postgresql-13
Info: DEBUG - ++ grep --quiet '^installed$'
Info: DEBUG - ++ echo no
Info: DEBUG - + local psql_installed2=no
Info: DEBUG - + [[ no != \n\o ]]
Info: DEBUG - + ynh_exit_properly
```
[18:01:43] <denisapain> it works but now its failing due to postgres-13 not being installed.... which is weird since I never put it to be installed, doing a git grep search for postgres in my repo returns no results
[18:01:53] <denisapain> it works but now its failing due to postgresql-13 not being installed.... which is weird since I never put it to be installed, doing a git grep search for postgres in my repo returns no results
[18:11:40] <orhtej2> > <@denisapain:matrix.org> it works but now its failing due to postgresql-13 not being installed.... which is weird since I never put it to be installed, doing a git grep search for postgres in my repo returns no results

Which ynh version? 12 ships pg15
[18:12:00] <denisapain> > Which ynh version? 12 ships pg15

yeah my server is on ynh 12
[18:12:41] <denisapain> but why is it trying to install postgresql-13?
my resources.apt is ` packages = "ffmpeg"` in manifest.toml
[19:28:12] <Yunohost Git/Infra notifications> k​hulnasoft forked example_ynh to [khulnasoft/khulnasoft_ynh](https://github.com/khulnasoft/khulnasoft_ynh)
[19:49:03] <Émy – OniriCorpe> Wtf if khulnasoft??
*opens https://khulnasoft.com/*
Hm yeah ofc
[20:06:24] <Salamandar> ah okay i'm not the only one triggered by the name
[20:11:06] <Aleks (he/him/il/lui)> i'm totally confused
[20:11:39] <Aleks (he/him/il/lui)> i ended up on https://readyapi.khulnasoft.com/ which describes a super top-notch API blablabla called "Ready API" but ... the logo says Fast API ?
[20:11:51] <Aleks (he/him/il/lui)> did they just fork Fast API and just changed the name without changing the logo ?
[20:11:52] <Aleks (he/him/il/lui)> wth ?
[20:21:41] <Salamandar> > <@oniricorpe:im.emelyne.eu> Wtf if khulnasoft??
> *opens https://khulnasoft.com/*
> Hm yeah ofc

doesn't load for me
[20:31:20] <Aleks (he/him/il/lui)> (same)
[20:36:26] <Yunohost Git/Infra notifications> [apps] y​unohost-bot pushed 1 commit to update_app_levels: Update app levels according to CI results ([550e8cc2](https://github.com/YunoHost/apps/commit/550e8cc27f7a303bf04b76a85f61b504b341a071))
[20:36:26] <Yunohost Git/Infra notifications> [apps] y​unohost-bot created new branch update_app_levels
[20:36:28] <Yunohost Git/Infra notifications> [apps] y​unohost-bot opened [pull request #2704](https://github.com/YunoHost/apps/pull/2704): Update app levels according to CI results
[20:36:46] <Yunohost Git/Infra notifications> [apps] g​ithub-actions[bot] created new branch actions/toml
[20:36:46] <Yunohost Git/Infra notifications> [apps] g​ithub-actions[bot] pushed 1 commit to actions/toml: :art: Format TOML with Taplo ([214e4599](https://github.com/YunoHost/apps/commit/214e4599904edccf545a52b13c90617fcbb3e422))
[20:36:48] <Yunohost Git/Infra notifications> [apps] g​ithub-actions[bot] opened [pull request #2705](https://github.com/YunoHost/apps/pull/2705): Format TOML with Taplo
[20:42:09] <denisapain> > <@Alekswag:matrix.org> i ended up on https://readyapi.khulnasoft.com/ which describes a super top-notch API blablabla called "Ready API" but ... the logo says Fast API ?

real readyapi https://github.com/readyapi/readyapi
[20:42:32] <denisapain> surprised it even has an issue, and prs
[20:43:01] <denisapain> most of the prs being from github bots
[20:43:28] <denisapain> > <@denisapain:matrix.org> ```
> Info: DEBUG - + apt-get --assume-yes --quiet -o=Acquire::Retries=3 -o=Dpkg::Use-Pty=0 --no-remove --option Dpkg::Options::=--force-confdef --option Dpkg::Options::=--force-confold install --fix-broken
> Info: DEBUG - Reading package lists...
> Info: DEBUG - Building dependency tree...
> Info: DEBUG - Reading state information...
> Info: DEBUG - 0 upgraded, 0 newly installed, 0 to remove and 1092 not upgraded.
> Info: DEBUG - + rm --recursive --force /tmp/tmp.FIz3q5rYUZ
> Info: DEBUG - + _ynh_apt_package_is_installed chibisafe-ynh-deps
> Info: DEBUG - + local package=chibisafe-ynh-deps
> Info: DEBUG - + dpkg-query --show '--showformat=${db:Status-Status}' chibisafe-ynh-deps
> Info: DEBUG - + grep --quiet '^installed$'
> Info: DEBUG - ++ _ynh_apt_package_is_installed postgresql-13
> Info: DEBUG - ++ local package=postgresql-13
> Info: DEBUG - ++ dpkg-query --show '--showformat=${db:Status-Status}' postgresql-13
> Info: DEBUG - ++ grep --quiet '^installed$'
> Info: DEBUG - ++ echo no
> Info: DEBUG - + local psql_installed2=no
> Info: DEBUG - + [[ no != \n\o ]]
> Info: DEBUG - + ynh_exit_properly
> ```

why is it trying to install pg-13
[20:50:22] <orhtej2> > <@denisapain:matrix.org> why is it trying to install pg-13

share full log and revision you're trying to install please
[21:34:22] <Émy – OniriCorpe> > <@Salamandar:matrix.org> doesn't load for me

That’s the point lol
[21:37:07] <Yunohost Git/Infra notifications> [apps] e​ricgaspar pushed 1 commit to update_app_levels: Update apps.toml ([9824beb7](https://github.com/YunoHost/apps/commit/9824beb7eaf5beda644a75ebbdc909fb0496fb84))
[21:37:11] <Yunohost Git/Infra notifications> [apps] e​ricgaspar edited [pull request #2704](https://github.com/YunoHost/apps/pull/2704): Update app levels according to CI results
[21:37:24] <Yunohost Git/Infra notifications> [apps] g​ithub-actions[bot] pushed 1 commit to actions/toml: :art: Format TOML with Taplo ([2be9570a](https://github.com/YunoHost/apps/commit/2be9570a1c952379a0cb62bbfcbcec96252ff1da))
[21:40:40] <Yunohost Git/Infra notifications> [apps] e​ricgaspar edited [pull request #2704](https://github.com/YunoHost/apps/pull/2704): Update app levels according to CI results
[21:42:00] <Yunohost Git/Infra notifications> [apps] e​ricgaspar edited [pull request #2704](https://github.com/YunoHost/apps/pull/2704): Update app levels according to CI results
[21:43:51] <Yunohost Git/Infra notifications> [apps] e​ricgaspar edited [pull request #2704](https://github.com/YunoHost/apps/pull/2704): Update app levels according to CI results
[21:45:32] <Yunohost Git/Infra notifications> [apps] e​ricgaspar edited [pull request #2704](https://github.com/YunoHost/apps/pull/2704): Update app levels according to CI results
[21:48:22] <Yunohost Git/Infra notifications> [apps] e​ricgaspar edited [pull request #2704](https://github.com/YunoHost/apps/pull/2704): Update app levels according to CI results
[21:50:44] <Yunohost Git/Infra notifications> [apps] e​ricgaspar edited [pull request #2704](https://github.com/YunoHost/apps/pull/2704): Update app levels according to CI results
[21:52:21] <Yunohost Git/Infra notifications> [apps] e​ricgaspar edited [pull request #2704](https://github.com/YunoHost/apps/pull/2704): Update app levels according to CI results
[21:53:58] <Yunohost Git/Infra notifications> [apps] e​ricgaspar edited [pull request #2704](https://github.com/YunoHost/apps/pull/2704): Update app levels according to CI results
[21:55:41] <Yunohost Git/Infra notifications> [apps] e​ricgaspar edited [pull request #2704](https://github.com/YunoHost/apps/pull/2704): Update app levels according to CI results
[21:56:43] <Yunohost Git/Infra notifications> [apps] e​ricgaspar edited [pull request #2704](https://github.com/YunoHost/apps/pull/2704): Update app levels according to CI results
[21:57:27] <Yunohost Git/Infra notifications> [apps] e​ricgaspar pushed 1 commit to update_app_levels: Update apps.toml ([40f6d737](https://github.com/YunoHost/apps/commit/40f6d737fe21814b0bd81e87859f44ee819fbe11))
[21:57:43] <Yunohost Git/Infra notifications> [apps] g​ithub-actions[bot] pushed 1 commit to actions/toml: :art: Format TOML with Taplo ([224be212](https://github.com/YunoHost/apps/commit/224be2121b40911c4be745d5900d8b074ae486f9))
[22:00:40] <Yunohost Git/Infra notifications> [apps] e​ricgaspar pushed 1 commit to update_app_levels: :art: Format TOML with Taplo (#2705) Co-authored-by: ericgaspar <46165813+ericgaspar@users.noreply.github.com> ([8b741bef](https://github.com/YunoHost/apps/commit/8b741bef72d49dd19eb0c430260a0ad8d1f5299c))
[22:00:41] <Yunohost Git/Infra notifications> [apps] e​ricgaspar deleted branch actions/toml
[22:00:41] <Yunohost Git/Infra notifications> [apps] e​ricgaspar merged [pull request #2705](https://github.com/YunoHost/apps/pull/2705): Format TOML with Taplo
[22:31:21] <denisapain> > share full log and revision you're trying to install please

log: https://paste.yunohost.org/raw/itokesazof
revision - master branch: https://git.qilk.de/denis/chibisafe_ynh (i think that what revision means right)
[22:32:11] <denisapain> hm
[22:32:11] <denisapain> 1s
[22:32:17] <denisapain> I try do a thing.
[22:33:30] <orhtej2> > <@denisapain:matrix.org> log: https://paste.yunohost.org/raw/itokesazof
> revision - master branch: https://git.qilk.de/denis/chibisafe_ynh (i think that what revision means right)

ok so it quite clearly states that you should define `nodejs_version=20` in `_common.sh` and just trash the `--nodejs_version=20` param in `ynh_nodejs_install`, the syntax changed in helpers 2.1
[22:33:40] <denisapain> yeah I just noticed when I opened the log
[22:34:26] <orhtej2> > <@denisapain:matrix.org> yeah I just noticed when I opened the log

https://yunohost.org/en/packaging_apps_helpers_v2.1 is your friend, some helpers got different syntax now and copy-pasting from existing packages requires taking that into consideration
[22:34:57] <orhtej2> (line that makes your package use helpers 2.1: https://git.qilk.de/denis/chibisafe_ynh/src/commit/7ae46617729f87f0bc2ce5915fe63fbc8d19c774/manifest.toml#L34)
[23:05:12] <orhtej2> > https://git.qilk.de/denis/chibisafe_ynh/src/commit/1bcf87c5f6907ca81b89ff78494191a06d126c82/scripts/install#L159 this likely builds some database and you're requiring none, consider switching from sqlite before productionizing

Ah it's not configurable
[23:06:20] <orhtej2> > <@denisapain:matrix.org> yeah I just noticed when I opened the log

<del>https://git.qilk.de/denis/chibisafe_ynh/src/commit/1bcf87c5f6907ca81b89ff78494191a06d126c82/scripts/install#L159 this likely builds some database and you're requiring none, consider switching from sqlite before productionizing</del>
[23:47:49] <denisapain> only thing left is the systemd service since I need to start both backend and frontend, and the current yarn command only starts the backend, also I translated the readme to french with deepl, I hope its decent
[23:58:26] <denisapain> is this frowned upon or not... `yarn start:backend & yarn start:frontend`