Thursday, May 02, 2024
apps@conference.yunohost.org
May
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
31
   
             

[05:26:27] <Yunohost Git/Infra notifications> App rspamdui stays at level 3 in job [#26065](https://ci-apps.yunohost.org/ci/job/26065)
[05:26:27] <Yunohost Git/Infra notifications> App tvheadend rises from level 0 to 6 in job [#26069](https://ci-apps.yunohost.org/ci/job/26069) !
[05:26:27] <lapineige> The alternative is to hit clone and use a folder path instead of the URL
[05:45:22] <Yunohost Git/Infra notifications> App h5ai failed all tests in job [#26073](https://ci-apps.yunohost.org/ci/job/26073) :(
[05:45:22] <Yunohost Git/Infra notifications> App yourls goes down from level 8 to 6 in job [#26071](https://ci-apps.yunohost.org/ci/job/26071)
[06:11:36] <Yunohost Git/Infra notifications> App ombi goes down from level 7 to 6 in job [#26074](https://ci-apps.yunohost.org/ci/job/26074)
[11:39:17] <Yunohost Git/Infra notifications> App invidious goes down from level 8 to 6 in job [#26086](https://ci-apps.yunohost.org/ci/job/26086)
[12:18:57] <ChriChri> Do I have X like in -ynhX defined for the actual and the version to be upgraded to when running the upgrade script?
I want to do something like
```
if actual_ynh_version < 5 and new_ynh_version is > 5; then
do something that is only needed once between changing from ynh-version lower than 5 to a ynh-version bigger than 5
[12:22:35] <ChriChri> I already found `ynh_app_package_version` which I understood would be for the actual installed version of the app package if no option is given.
But how do I get the ynh version of the package that is about to be installed when the upgrade script is running? What do I put into `--manifest="manifest.json"` as `manifest.json` for the package about to be installed?
Is it correct anyway that `ynh_app_package_version` gives me the version of the package still running at the beginning of the upgrade-script?
[12:29:55] <Aleks (he/him/il/lui)> you probably want something like https://github.com/YunoHost-Apps/nextcloud_ynh/blob/master/scripts/upgrade#L22
[12:31:42] <ChriChri> > <@Alekswag:matrix.org> you probably want something like https://github.com/YunoHost-Apps/nextcloud_ynh/blob/master/scripts/upgrade#L22

Yes, but I only want to take the `~ynhX` part (package version) into account.
[12:34:17] <Aleks (he/him/il/lui)> uuuh
[12:34:34] <Aleks (he/him/il/lui)> but why would you want that x_x
[12:42:23] <ChriChri> Don't hit me…
I decided to change the paths and filenames now that I'm trying to make it possible to install the same app on different URIs multiple times on the same yunohost.
I just thought it would be good to have this as a not so important case to learn to provide upgrade steps that are only needed on certain version jumps upstream **or certain version jumps in the package ~ynh version.**
[12:43:04] <ChriChri> I guess nobody will care since there are not many installations out there, yet, but who knows...
[12:44:38] <orhtej2> > <@Alekswag:matrix.org> but why would you want that x_x

yeah one would assume you want to check the whole version string as the components are not trully independent?
[12:47:35] <ChriChri> Hm, yes, that is true. Actually the upstream version is irrelevant, because there is none (it's 0.00 at the moment). So if the upstream version didn't change anyway the jump from ynhX to ynhX+1 should always be correctly `ynh_check_app_version_changed`, right?
[12:52:36] <Aleks (he/him/il/lui)> > <@chrichri:librem.one> Don't hit me…
> I decided to change the paths and filenames now that I'm trying to make it possible to install the same app on different URIs multiple times on the same yunohost.
> I just thought it would be good to have this as a not so important case to learn to provide upgrade steps that are only needed on certain version jumps upstream **or certain version jumps in the package ~ynh version.**

the syntax linked in nextcloud previously does exactly that, but maybe you're confused because "ynhX" is only every incremented, but in reality, the "ynhX" is just here to differentiate between several release of the same upstream version, but then when we re-update the upstream version usually the "ynhX" goes back down to "ynh1"
[12:52:48] <orhtej2> > <@chrichri:librem.one> Hm, yes, that is true. Actually the upstream version is irrelevant, because there is none (it's 0.00 at the moment). So if the upstream version didn't change anyway the jump from ynhX to ynhX+1 should always be correctly `ynh_check_app_version_changed`, right?

if you use rolling release consider using date as version number
[12:53:04] <Aleks (he/him/il/lui)> eg you can have 1.0~ynh1, 1.0~ynh2, 1.0~ynh3, 1.1~ynh1, 1.1~ynh2, etc
[12:53:31] <Aleks (he/him/il/lui)> (or you can keep the same suffix and only increment it, why not, eg 1.0~ynh1, 1.0~ynh2, 1.0~ynh3, 1.1~ynh3, 1.1~ynh4)
[12:58:23] <Aleks (he/him/il/lui)> but anyway, if you want some specific upgrade step to happen when jumping over a specific version, you'd just write `if ynh_compare_current_package_version --comparison lt --version X.Y~ynhZ` and voila
[12:59:44] <ChriChri> > if you use rolling release consider using date as version number

No, no tag/release, yet ...
[13:00:00] <niklas> when I remove an apt package from an apps "resources.apt.packages" will it get uninstalled in the upgrade process?
[13:00:26] <ChriChri> But all that helps a lot! THANKS again! I'll stick to `ynh_check_app_version_changed` .
[13:03:50] <Aleks (he/him/il/lui)> > <@niklas:freiburg.social> when I remove an apt package from an apps "resources.apt.packages" will it get uninstalled in the upgrade process?

short answer is yes, long answer is : it depends, it will only be really uninstalled if no other piece of software depend on it on the system, and maybe "apt autoremove" is called somehow
[13:04:16] <Aleks (he/him/il/lui)> but the "dependency link" between the app and the apt package will be removed, yes
[13:05:28] <niklas> so ynh apps somehow register their dependencies with apt?
[13:06:13] <Aleks (he/him/il/lui)> yes using a "dummy", "virtual" apt dependency
[13:06:26] <Aleks (he/him/il/lui)> you can see those if you run for example `dpkg --list | grep ynh-deps`
[13:06:31] <Aleks (he/him/il/lui)> it's created using equivs
[13:06:55] <niklas> very cool, thanks
[13:41:55] <Émy - OniriCorpe> > <@ngill:nortel.cloud> Hi - trying to create a custom app for Yunohost but the app install command doesn't seem to like my git URL. Same issue when trying from the web UI.
> root@yunotest:~# yunohost app install https://git.nortel.cloud/nortel-cloud/userdirs
> Error: https://git.nortel.cloud/nortel-cloud/userdirs is not a valid app url: app url are expected to look like https://domain.tld/path/to/repo_ynh
> Error: Unknown app

Love the idea of this package ^w^
[13:42:49] <Yunohost Git/Infra notifications> [apps] alexAubin created new branch store-simplify-query-args-handling
[13:42:49] <Yunohost Git/Infra notifications> [apps] alexAubin pushed 1 commit to store-simplify-query-args-handling: store: simplify query args handling because request is available in jinja templates ([72f2c821](https://github.com/YunoHost/apps/commit/72f2c8217ae3e74fbecfabd772ec4cd16908325f))
[13:42:56] <Yunohost Git/Infra notifications> [apps] alexAubin opened [pull request #2286](https://github.com/YunoHost/apps/pull/2286): store: simplify query args handling because request is available in jinja templates
[13:43:01] <Yunohost Git/Infra notifications> [apps] alexAubin merged [pull request #2286](https://github.com/YunoHost/apps/pull/2286): store: simplify query args handling because request is available in jinja templates
[13:43:02] <Yunohost Git/Infra notifications> [apps] alexAubin pushed 1 commit to master: store: simplify query args handling because request is available in jinja templates (#2286) ([aa13a238](https://github.com/YunoHost/apps/commit/aa13a2389a662eaab434867ef721521de30fdbcf))
[13:43:05] <Yunohost Git/Infra notifications> [apps] alexAubin deleted branch store-simplify-query-args-handling
[13:43:58] <Wylel> eric_G: Does ynh5 of YOURLS fix the password issue? Curious, if so, what the issue was.
[15:18:33] <ChriChri> I just realised that flohmarkts logo is missing in the catalogue. I can't remember having read where to do that. Any help (didn't find it for the github repo neither, set the social media logo there...)
[15:32:12] <Aleks (he/him/il/lui)> it should be added to https://github.com/YunoHost/apps/tree/master/logos
[15:33:05] <Aleks (he/him/il/lui)> (though please try to keep in mind to keep a reasonable size such as idk 50ish kB max ? And ideally should be square-ish)
[15:48:22] <Salamandar> 50k is huuuuuuuuuuuuge
[15:48:27] <Salamandar> 10k is a grand max i'd say
[15:50:10] <Yunohost Git/Infra notifications> [apps] chri2 opened [pull request #2287](https://github.com/YunoHost/apps/pull/2287): flohmarkt on yunohost logo
[15:50:21] <ChriChri> > <@Alekswag:matrix.org> (though please try to keep in mind to keep a reasonable size such as idk 50ish kB max ? And ideally should be square-ish)

https://github.com/YunoHost/apps/pull/2287 25kb :) - thank you!
[15:50:40] <ChriChri> > <@Salamandar:matrix.org> 10k is a grand max i'd say

uhuhhhh
[15:51:14] <Salamandar> :|
[15:51:14] <Salamandar> i'm looking into the directory and some are already huge
[15:51:52] <Salamandar> but they are rarely shown at a "huge" size on-screen, so i think some could be sanely reduced, dunno…
[15:52:07] <Aleks (he/him/il/lui)> yup
[15:52:12] <Aleks (he/him/il/lui)> ideally we could even have a github workflow checking the size is decent
[15:52:22] <Salamandar> 311k for owncast-emojiwall wtf
[15:53:43] <Aleks (he/him/il/lui)> > <@yunohostinfra:matrix.org> [apps] chri2 opened [pull request #2287](https://github.com/YunoHost/apps/pull/2287): flohmarkt on yunohost logo

uuuuh but why add "yunohost" around the official logo though
[15:54:28] <ChriChri> haha - good one! It's the one I found first ;-) - but you're right.
[15:55:00] <Aleks (he/him/il/lui)> ô.o
[15:55:01] <Yunohost Git/Infra notifications> [apps] chri2 closed [pull request #2287](https://github.com/YunoHost/apps/pull/2287): flohmarkt on yunohost logo
[15:55:11] <Salamandar> logos size is the same as wealth distribution: 10% of logos make up for 50% of total size :D
[15:59:12] <ChriChri> > <@yunohostinfra:matrix.org> [apps] chri2 closed [pull request #2287](https://github.com/YunoHost/apps/pull/2287): flohmarkt on yunohost logo

O.k. - this should be better :)
[16:22:36] <Yunohost Git/Infra notifications> [apps] ballinger [commented](https://github.com/YunoHost/apps/pull/2244#issuecomment-2090951852) on [issue #2244](https://github.com/YunoHost/apps/pull/2244) Add apache superset to wishlist: the app is currently labeled as broken
[16:28:49] <Yunohost Git/Infra notifications> [apps] OniriCorpe [commented](https://github.com/YunoHost/apps/pull/2244#issuecomment-2090964973) on [issue #2244](https://github.com/YunoHost/apps/pull/2244) Add apache superset to wishlist: thats another problem
[16:29:12] <Yunohost Git/Infra notifications> [apps] OniriCorpe [commented](https://github.com/YunoHost/apps/pull/2280#issuecomment-2090975672) on [issue #2280](https://github.com/YunoHost/apps/pull/2280) Add Carre to wishlist: > Ce projet est archivé. anyway its a deprecated software :/
[16:29:28] <Yunohost Git/Infra notifications> [apps] OniriCorpe [commented](https://github.com/YunoHost/apps/pull/2280#issuecomment-2090975672) on [issue #2280](https://github.com/YunoHost/apps/pull/2280) Add Carre to wishlist: > Ce projet est archivé. anyway its a deprecated software :/ edit: my bad i checked the LQDN repo, so not the right o...
[16:29:54] <Yunohost Git/Infra notifications> [apps] OniriCorpe [commented](https://github.com/YunoHost/apps/pull/2280#issuecomment-2090975672) on [issue #2280](https://github.com/YunoHost/apps/pull/2280) Add Carre to wishlist: > Ce projet est archivé. anyway its a deprecated software :/ edit: my bad i checked the LQDN repo, so not the right o...
[16:58:27] <Yunohost Git/Infra notifications> [apps] chri2 opened [pull request #2288](https://github.com/YunoHost/apps/pull/2288): flohmarkt logo
[16:58:55] <ChriChri> > <@yunohostinfra:matrix.org> [apps] chri2 opened [pull request #2288](https://github.com/YunoHost/apps/pull/2288): flohmarkt logo

O.k. I meant this should be better :)
[17:00:39] <Yunohost Git/Infra notifications> [apps] OniriCorpe approved [pull request #2288](https://github.com/YunoHost/apps/pull/2288#pullrequestreview-2036325550) flohmarkt logo
[17:02:12] <Yunohost Git/Infra notifications> [apps] OniriCorpe pushed 1 commit to master: flohmarkt logo for the app catalog - no "yuno" and "host" - 5kb only *eg* ([8ec24d5b](https://github.com/YunoHost/apps/commit/8ec24d5b1d41d3c4f5961a07919d651ef7162733))
[17:03:29] <Gavi> hey i apologize if this isnt the right place to ask but i was curious about this issue with the conduit app package. specifically if there was anything specifically stopping the upgrade and if there was any way i could contribute to helping fix it? https://github.com/YunoHost-Apps/conduit_ynh/issues/27
[17:07:39] <Yunohost Git/Infra notifications> [apps] OniriCorpe pushed 1 commit to readme-logo: readme: add instructions for the app logo ([34bd9eb0](https://github.com/YunoHost/apps/commit/34bd9eb0935b352f907f0c3dd5354a09ae07c23b))
[17:09:21] <Yunohost Git/Infra notifications> [apps] OniriCorpe created new branch readme-logo
[17:09:39] <Yunohost Git/Infra notifications> [apps] OniriCorpe opened [pull request #2289](https://github.com/YunoHost/apps/pull/2289): readme: add instructions for the app logo
[17:11:29] <Émy - OniriCorpe> yunohost packages are not using docker, so it should be fine i guess
[17:11:56] <Émy - OniriCorpe> https://aria.im/_matrix/media/v1/download/im.emelyne.eu/FHkujbzmUDeFJnplNaPRsRGY
[17:12:46] <Émy - OniriCorpe> uhuh 🙃
[17:13:01] <Gavi> then the db config variable would be the only new thing that applies i assume, since it requires you to specify rocksdb or sqlite now
[17:13:47] <Gavi> im unsure what the previous default without the new config variable was, rocksdb i think?
[17:14:52] <Émy - OniriCorpe> since it's not documented and i don't use this package, i can't do anything
the former maintainer was eric_G maybe he could say if it's okay or not?
[17:16:37] <Gavi> https://github.com/YunoHost-Apps/conduit_ynh/blob/020656d07c467477868934d646e681dc43abe62e/conf/conduit.toml#L20 it seems the backend already is set to use rocksdb
[17:21:39] <Yunohost Git/Infra notifications> App conduit failed all tests in job [#26091](https://ci-apps.yunohost.org/ci/job/26091) :(
[17:53:45] <Gavi> is there a way for me to test it for you?:neocat_thinking:
[17:53:45] <eric_G> Gavi: Just updated the package to the last version (https://github.com/YunoHost-Apps/conduit_ynh/tree/0.7.0) but it is not tested and I have no idea if it's usable 😅
[17:57:22] <eric_G> you should be able to update with `sudo yunohost app upgrade conduit -u https://github.com/YunoHost-Apps/conduit_ynh/tree/0.7.0`
[17:57:33] <Émy - OniriCorpe> > <@ericg:matrix.org> you should be able to update with `sudo yunohost app upgrade conduit -u https://github.com/YunoHost-Apps/conduit_ynh/tree/0.7.0`

you're faster than me :"3
[18:00:33] <Gavi> if it fails, i can just restore it right?
[18:02:03] <thatoo> Thanks to help of orhtej2 , there is now a working first version of a Dokos Yunohost package : https://github.com/Thatoo/Dokos_ynh/tree/test
Some part clearly need to be cleaned and improved but at least a based exists. I'm happy.
[18:03:55] <Émy - OniriCorpe> usually yes, in rare cases, no
but yes it should be fine and restore automatically the previous install if the update fail
[18:10:50] <eric_G> I’m not sure about the Conduit version. I don't know which one to use Stable/Main or Latest/Next -> https://docs.conduit.rs/deploying/generic.html
[18:11:19] <Gavi> stable should be the latest release (0.0.7) and the latest/next should be the beta rolling release
[18:13:29] <Gavi> > <@ericg:matrix.org> I’m not sure about the Conduit version. I don't know which one to use Stable/Main or Latest/Next -> https://docs.conduit.rs/deploying/generic.html

stable should be the latest release (0.0.7) and the latest/next should be the beta rolling release
[18:27:10] <Yunohost Git/Infra notifications> [apps] ballinger [commented](https://github.com/YunoHost/apps/pull/2244#issuecomment-2091217291) on [issue #2244](https://github.com/YunoHost/apps/pull/2244) Add apache superset to wishlist: Aye, sorry. Seems like you two are the human contributers to this app. Any chance of fixing it?
[18:28:04] <ChriChri> Thinking about the upgrade script - how does it work?
* the upgrade is downloaded and the new upgrade script from the just downloaded upgrade is started
* the upgrade is downloaded and the upgrade script of the already installed and running app version is started
[18:32:18] <Émy - OniriCorpe> the first one
[18:51:47] <Gavi> where is the config file for that app located?? ive been trying to find it for ages :BlobCat_Think:
[18:53:24] <Émy - OniriCorpe> why? you must not edit it by hand
[18:53:56] <Gavi> i wanted to add the coturn config to it
[18:53:59] <Émy - OniriCorpe> anyway it's here `/etc/yunohost/apps/app/settings.yml`
[18:54:54] <Émy - OniriCorpe> ah, you're talking about this ? https://github.com/YunoHost-Apps/conduit_ynh/blob/master/conf/conduit.toml
[18:55:24] <Gavi> yeah that one, its in that directory
[18:55:51] <Émy - OniriCorpe> it's in `/var/www/conduit/conduit.toml`
[18:56:14] <Émy - OniriCorpe> but it's replaced after each update
[18:56:40] <Émy - OniriCorpe> so it's better to contribute to the package to add the coturn settings and it will profit to everyone
[19:30:38] <Yunohost Git/Infra notifications> [apps] alexAubin pushed 1 commit to master: readme: add instructions for the app logo (#2289) ([cfbefd70](https://github.com/YunoHost/apps/commit/cfbefd70352134d0e123ded567150a57094f7c03))
[19:32:31] <Yunohost Git/Infra notifications> [apps] alexAubin deleted branch readme-logo
[19:36:18] <Émy - OniriCorpe> i'm packaging an unlicensed software (there is no license in the repo and no mention of any license anywhere) for my personnal use :"3
https://github.com/SideStore/omnisette-server
[19:40:31] <tituspijean> > anisette servers

🍹
[19:41:09] <Gavi> update: the package works flawlessly!
[19:41:56] <Gavi> https://aria.im/_matrix/media/v1/download/matrix.sleepy.ink/n7I6EwsJuAVKVHKznizf7BpEeRML5zmw
[19:50:11] <Émy - OniriCorpe> > <@oniricorpe:im.emelyne.eu> i'm packaging an unlicensed software (there is no license in the repo and no mention of any license anywhere) for my personnal use :"3
> https://github.com/SideStore/omnisette-server

🎉 https://codeberg.org/OniriCorpe/omnisette-server_ynh
[20:49:31] <Yunohost Git/Infra notifications> App zeronet rises from level 6 to 8 in job [#26094](https://ci-apps.yunohost.org/ci/job/26094) !
[20:53:35] <Émy - OniriCorpe>
```
! manifest
! The license id 'null' is not registered in https://spdx.org/licenses/.
```

thats... correct lmao

[20:53:35] <orhtej2> > <@yunohostinfra:matrix.org> App zeronet rises from level 6 to 8 in job [#26094](https://ci-apps.yunohost.org/ci/job/26094) !

speaking of the devil, can we move the app to graveyard? Upstream is unmaintained for past 3 years and `bookworm` life support involves random bumps to frozen requirements
[20:53:35] <orhtej2> it's the same story with mygpo, if someone wants to play this dangerous game of 'this passes CI today but does not 2 months later' sure, be my guest, but IMO we should just get rid of it
[21:13:35] <Salamandar> Yeah, i completely agree with your position
[21:13:37] <Salamandar> I just merged my long-living packagingv2 branch
[21:13:56] <Salamandar> but that is all i guess
[21:35:41] <Émy - OniriCorpe> yay 🎉
[21:36:00] <Émy - OniriCorpe> https://aria.im/_matrix/media/v1/download/im.emelyne.eu/sDQpDzvigtcLJwvQWMNiKsEM
[21:37:45] <Émy - OniriCorpe> than you very much tituspijean for your CI server
[22:13:40] <Yunohost Git/Infra notifications> App crabfit rises from level 6 to 7 in job [#26097](https://ci-apps.yunohost.org/ci/job/26097) !
[22:17:34] <orhtej2> > <@yunohostinfra:matrix.org> App crabfit rises from level 6 to 7 in job [#26097](https://ci-apps.yunohost.org/ci/job/26097) !

Noice!
[22:25:02] <Émy - OniriCorpe> technically a level 7 :"3
[22:26:22] <Émy - OniriCorpe> https://aria.im/_matrix/media/v1/download/im.emelyne.eu/CCXfENdbsADOxDDyhOEZtVtC
[22:28:48] <orhtej2> nice, did you start with YoloGen?
[22:29:45] <Émy - OniriCorpe> nope ^^"
[22:30:21] <Émy - OniriCorpe> I haven't thought about it x)
[22:30:21] <orhtej2> there's more than way to cook a something
[22:30:40] <orhtej2> ah, not cook
[22:30:46] <orhtej2> and the rest is gruesome
[22:31:03] <Émy - OniriCorpe> tim cook
[22:36:15] <orhtej2> > and the rest is gruesome

apple devices, gruesome, math checks out ;P
[22:36:41] <orhtej2> it's the new app sideloading thing?
[22:39:29] <Émy - OniriCorpe> nope, that's not the new DMA thing (which is alstore pal)
[22:45:48] <orhtej2> https://aria.im/_matrix/media/v1/download/circledsquareroot.ovh/818da1eecee4bc4c324f3c08c23a7c2d44d39f94446cb5903632d038cee52c74
[22:45:49] <Émy - OniriCorpe> anyway, now that this software is packaged, i'm now able to completely remove docker from my yunohost server 😌
[22:45:49] <orhtej2> wait, you guys have vpses capable of running docker?
[22:45:49] <Émy - OniriCorpe> there are ynh servers unable to run docker? .__.
[22:45:50] <Émy - OniriCorpe> oh i see
[22:45:50] <Émy - OniriCorpe> i'm using a computer at home as my ynh server
[22:45:50] <orhtej2> Mine costs 2€/month, it's a pile of junk xD
[23:27:56] <Émy - OniriCorpe> > <@gavi:matrix.sleepy.ink> update: the package works flawlessly!

Eric has merged the update, so everyone now have access to it, thank you for the test! ^W^