[00:46:51]
<Solrac> Pardon me, I'm pretty sure I asked ages ago but is there an example for an app that uses debian packages rather than a tarball? Looking to attempt Zoneminder
[00:51:10]
<Émy - OniriCorpe>
```
[resources.apt]
packages = "zoneminder"
extras.zoneminder.repo = "deb https://zmrepo.zoneminder.com bullseye main"
```
[00:51:10]
<Émy - OniriCorpe> in the manifest.toml
[01:01:59]
<Solrac> yes, plus database and the other bits too. Im currently mid-config and reading their docs
[01:03:38]
<Solrac> but also, considering it's an apt package, should config and the like still go to /var/<app config>?
[01:03:39]
<Aleks (he/him/il/lui)> yes
[01:03:56]
<Aleks (he/him/il/lui)> `/var/www/$app`
[01:05:36]
<Aleks (he/him/il/lui)> in packaging v2 you don't really need to think about it, just use the install_dir var
[01:46:36]
<Yunohost Git/Infra notifications> Autoupdater just ran, here are the results:
- 73 pending update PRs
- 14 new apps PRs
- 4 failed apps updates: elasticsearch8, focalboard, tvheadend, vikunja
See the full log here: http://paste.yunohost.org/raw/kirucujaja
[01:53:44]
<Aleks (he/him/il/lui)> hmm that's weird, i was thinking about opening the update PR to check them out, and for example : 2fauth (5.1.0 -> 5.1.1) see https://github.com/YunoHost-Apps/2FAuth_ynh/pull/65 -> but this is actually an unrelated testing PR ? x_X
[01:58:05]
<Aleks (he/him/il/lui)> actually the right link is https://github.com/YunoHost-Apps/2FAuth_ynh/pull/64
[01:58:06]
<Aleks (he/him/il/lui)> (checking the other link they seem to be okay)
[02:38:44]
<Yunohost Git/Infra notifications> Job [#25353](https://ci-apps.yunohost.org/ci/job/25353) for mopidy failed miserably :(
[02:39:59]
<Yunohost Git/Infra notifications> Job [#25354](https://ci-apps.yunohost.org/ci/job/25354) for osticket failed miserably :(
[02:40:00]
<Yunohost Git/Infra notifications> Job [#25355](https://ci-apps.yunohost.org/ci/job/25355) for owntracks failed miserably :(
[02:40:20]
<Yunohost Git/Infra notifications> Job [#25356](https://ci-apps.yunohost.org/ci/job/25356) for prometheus failed miserably :(
[02:40:53]
<Yunohost Git/Infra notifications> Job [#25357](https://ci-apps.yunohost.org/ci/job/25357) for tiddlywiki failed miserably :(
[02:41:31]
<Yunohost Git/Infra notifications> Job [#25358](https://ci-apps.yunohost.org/ci/job/25358) for wemawema failed miserably :(
[02:42:28]
<Yunohost Git/Infra notifications> Job [#25360](https://ci-apps.yunohost.org/ci/job/25360) for 20euros failed miserably :(
[02:42:28]
<Yunohost Git/Infra notifications> Job [#25359](https://ci-apps.yunohost.org/ci/job/25359) for yacy failed miserably :(
[02:43:06]
<Yunohost Git/Infra notifications> Job [#25361](https://ci-apps.yunohost.org/ci/job/25361) for navidrome failed miserably :(
[02:43:25]
<Yunohost Git/Infra notifications> Job [#25362](https://ci-apps.yunohost.org/ci/job/25362) for tube failed miserably :(
[02:44:04]
<Yunohost Git/Infra notifications> Job [#25363](https://ci-apps.yunohost.org/ci/job/25363) for influxdb_v2 failed miserably :(
[02:44:59]
<Solrac> > <@Alekswag:matrix.org> in packaging v2 you don't really need to think about it, just use the install_dir var
even if its an apt package? X_X I assume yunohost does it's thing
[02:45:20]
<Solrac> what about providing databases? I assume this could go in install script rather than manifest?
[02:50:59]
<Émy - OniriCorpe> You should use $install_dir, whatever the app uses to function
[02:51:57]
<Émy - OniriCorpe> > <@lsolrac:matrix.org> what about providing databases? I assume this could go in install script rather than manifest?
Initialize it using the manifest
[03:10:22]
<Solrac> understandable.
[03:15:06]
<Solrac> > <@oniricorpe:im.emelyne.eu> Initialize it using the manifest
how? The docs of the manifest only say what kind to provision
[03:16:06]
<Aleks (he/him/il/lui)> > <@lsolrac:matrix.org> even if its an apt package? X_X I assume yunohost does it's thing
well yeah you're right that with a .deb / apt package you dont usually have control over where it's installed, it will usually just extract stuff in /usr/bin and /etc/ for example, however you may have control over where to put the conf file somehow, and if you do have this control, ideally it should go in /var/www/$app i guess
[03:16:47]
<Solrac> > <@Alekswag:matrix.org> well yeah you're right that with a .deb / apt package you dont usually have control over where it's installed, it will usually just extract stuff in /usr/bin and /etc/ for example, however you may have control over where to put the conf file somehow, and if you do have this control, ideally it should go in /var/www/$app i guess
thats what I was thinking
[03:17:48]
<Aleks (he/him/il/lui)> but yeah having the "main" source of the app being a .deb / apt package may seem sort of convenient but actually doesn't fit very well for the yunohost paradigm ... for example if the package is just installed from a custom sources.list, you don't really have control over which version is installed and the version number provided in the manifest may or may not match the actual installed version depending if the upstream updated their package recently for example
[03:18:09]
<Aleks (he/him/il/lui)> there's a whole bunch of ambiguities like this
[03:19:28]
<Aleks (he/him/il/lui)> > <@lsolrac:matrix.org> how? The docs of the manifest only say what kind to provision
the database is automatically created with a user+password stored in `db_user` and `db_pwd`
[03:55:43]
<Solrac> seeing the AUR package files, the only have nginx, a conf file which could be moved, and the rest is /usr/bin/ and some perl files
[03:56:04]
<Solrac> https://aria.im/_matrix/media/v1/download/matrix.org/YzzQgcVjeuVnylHVHTawumYH
[03:56:44]
<Solrac> oh...and /usr/share/webapps/zoneminder which is full of php files
[07:47:26]
<Yunohost Git/Infra notifications> [nextcloud_ynh] rodinux opened [pull request #679](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/679): Update manifest.toml
[09:44:17]
<Yunohost Git/Infra notifications> [apps] yunohost-bot pushed 1 commit to add-to-wishlist-jan: Add JAN to wishlist ([36ce7ca2](https://github.com/YunoHost/apps/commit/36ce7ca22ac004de231348446e44b7559e5c0536))
[09:44:35]
<Yunohost Git/Infra notifications> [apps] yunohost-bot labeled Wishlist on [pull request #2230](https://github.com/YunoHost/apps/pull/2230): Add JAN to wishlist
[09:44:35]
<Yunohost Git/Infra notifications> [apps] yunohost-bot opened [pull request #2230](https://github.com/YunoHost/apps/pull/2230): Add JAN to wishlist
[09:44:35]
<Yunohost Git/Infra notifications> [apps] yunohost-bot created new branch add-to-wishlist-jan
[09:54:04]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/679#issuecomment-2042331036) on [issue #679](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/679) Update manifest.toml: testme
[09:54:04]
<Yunohost Git/Infra notifications> [nextcloud_ynh] yunohost-bot [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/679#issuecomment-2042331289) on [issue #679](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/679) Update manifest.toml: :bug:
[[Test Badge](https://img.shields.io/endpoint?url=https://ci-apps-bookworm-dev.yunohost.org/ci/api/job/436/badge)...
[09:54:04]
<Yunohost Git/Infra notifications> [nextcloud_ynh] yunohost-bot [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/679#issuecomment-2042331226) on [issue #679](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/679) Update manifest.toml: :sunflower:
[[Test Badge](https://img.shields.io/endpoint?url=https://ci-apps-dev.yunohost.org/ci/api/job/14882/badge)]...
[10:24:31]
<Yunohost Git/Infra notifications> [apps] ericgaspar pushed 2 commits to microbin ([e3858501a70a...61cac4bec522](https://github.com/YunoHost/apps/compare/e3858501a70a...61cac4bec522))
[10:57:23]
<Mateusz Szymański> `iceshrimp` is the newest `corepack` victim x_x
[11:04:53]
<lapineige> > <@orhtej2:matrix.org> `iceshrimp` is the newest `corepack` victim x_x
Ping oufmilo:
[12:41:32]
<Leonard> Hello, I don't update my application enough so I don't remember the process. I've push a commit in testing branch. To push my change to master, I should make a pull request, and then it's automatic if tests pass?
[12:41:55]
<Leonard> Here the link to the github repo I pushed on : https://github.com/YunoHost-Apps/FastAPI_ynh/
[12:43:44]
<Mateusz Szymański> > <@leolecolo:matrix.org> Hello, I don't update my application enough so I don't remember the process. I've push a commit in testing branch. To push my change to master, I should make a pull request, and then it's automatic if tests pass?
you create a PR, type `!testme` in the comment, wait for the CI to pass then marge manually
[17:35:01]
<Yunohost Git/Infra notifications> [apps] yunohost-bot labeled Wishlist on [pull request #2231](https://github.com/YunoHost/apps/pull/2231): Add Lobe Chat to wishlist
[17:35:02]
<Yunohost Git/Infra notifications> [apps] yunohost-bot opened [pull request #2231](https://github.com/YunoHost/apps/pull/2231): Add Lobe Chat to wishlist
[17:35:02]
<Yunohost Git/Infra notifications> [apps] yunohost-bot created new branch add-to-wishlist-lobe-chat
[17:35:02]
<Yunohost Git/Infra notifications> [apps] yunohost-bot pushed 1 commit to add-to-wishlist-lobe-chat: Add Lobe Chat to wishlist ([f3676b8e](https://github.com/YunoHost/apps/commit/f3676b8e0865b37bc863ae206d74b4a600cf5501))
[17:36:32]
<kayou> tituspijean: par rapport au problème de sonarr sur le forum, j'ai eu la même chose je crois: https://github.com/YunoHost-Apps/sonarr_ynh/pull/11#issuecomment-2005159590
[17:36:32]
<kayou> meh, j'ai plus les logs, parce que ça fait trop longtemps que j'ai upgrade :/
[17:36:35]
<kayou> j'avais creusé pour comprendre pourquoi j'avais eu un wipe de ma db, mais rien trouvé (peut être simplement le passage de packageV1 à packageV2 et un changement de install_dir?)
[17:39:11]
<tituspijean> Should we revert the package? Losing the DB seems quite inconvenient.
[17:39:11]
<Salamandar> > <@kayou:matrix.org> I didn't find out why the db was not migrated properly (maybe due to the packaging V1 to V2 and an install_dir change?)
Crap i thought that was fixed :/ my bad
[17:39:25]
<Yunohost Git/Infra notifications> [nextcloud_ynh] tituspijean opened [pull request #680](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/680): Make system address book access configurable
[17:39:25]
<Yunohost Git/Infra notifications> [nextcloud_ynh] yunohost-bot pushed 1 commit to config-addressbook: Auto-update READMEs ([1e9852a7](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/1e9852a7f2cc3d37c80b502f556f792481cf7c85))
[17:39:25]
<Yunohost Git/Infra notifications> [nextcloud_ynh] tituspijean created new branch config-addressbook
[17:39:25]
<Yunohost Git/Infra notifications> [nextcloud_ynh] tituspijean pushed 1 commit to config-addressbook: Make system address book access configurable ([ee0a22ca](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/ee0a22caba0a43757060de00c156d33263bed1ad))
[17:39:26]
<tituspijean> I let it merge because I really could not replicate the issue 😕
[17:39:27]
<Yunohost Git/Infra notifications> [nextcloud_ynh] tituspijean pushed 1 commit to config-addressbook: Make system address book access configurable: default value ([6a0e5ee9](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/6a0e5ee9de6dfd9eb55acd8cd97bdc85110324cd))
[17:39:27]
<Yunohost Git/Infra notifications> [nextcloud_ynh] tituspijean edited [pull request #680](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/680): Make system address book access configurable
[17:39:27]
<Yunohost Git/Infra notifications> [nextcloud_ynh] tituspijean [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/680#issuecomment-2043294093) on [issue #680](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/680) Make system address book access configurable: testme
[17:39:28]
<Yunohost Git/Infra notifications> [nextcloud_ynh] tituspijean edited [pull request #680](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/680): Make system address book access configurable
[17:39:28]
<Yunohost Git/Infra notifications> [nextcloud_ynh] yunohost-bot [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/680#issuecomment-2043294171) on [issue #680](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/680) Make system address book access configurable: :sunflower:
[[Test Badge](https://img.shields.io/endpoint?url=https://ci-apps-dev.yunohost.org/ci/api/job/14889/badge)]...
[17:39:28]
<Yunohost Git/Infra notifications> [nextcloud_ynh] yunohost-bot [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/680#issuecomment-2043294181) on [issue #680](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/680) Make system address book access configurable: Living in the future, are we?
[[Test Badge](https://img.shields.io/endpoint?url=https://ci-apps-bookworm-dev.yunohost.o...
[17:56:59]
<Yunohost Git/Infra notifications> [nextcloud_ynh] tituspijean [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/680#issuecomment-2043340398) on [issue #680](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/680) Make system address book access configurable: > Argument system_addressbook_exposed : you might want to simply use a boolean-type argument. No need to specify the c...
[19:45:49]
<Bram> oh, indeed
[19:45:52]
<Bram> do you want to become the maintainer then?
[19:45:53]
<Bram> (fyi, I'm not super up to date on how people become maintainers on apps so I might be missing something)
[19:45:53]
<Bram> eric_G: I think this is for you ^
[19:45:54]
<Salamandar> > <@thovi98:matrix.org> Could someone merge https://github.com/YunoHost-Apps/2FAuth_ynh/pull/65 please? Several persons confirm it’s working! i’ve still to fix the mail config, but I didn’t find a solution nor time for now :)
I added 2 comments (mere questions, you know the app better than me), but anyways LGTM. I added the review so you can merge this if you are part of the org i think.
[19:45:54]
<Salamandar> And yeah, if you want to maintain some apps (i think you intervened on some other apps as well), please do :D
[19:46:08]
<Aleks (he/him/il/lui)> otherwise LGTM
[19:46:08]
<Aleks (he/him/il/lui)> (CORS / Access-Control-Allow-Origin)
[19:46:08]
<Aleks (he/him/il/lui)> (i think nobody is clear with maintainership workflow/meaningfulness 😬)
[19:46:09]
<Aleks (he/him/il/lui)> Thomas: made a quick review, I'd advise having `chmod 640` on the .env instead of 644, and i'm wondering about the CORS header thing
[19:46:09]
<Thomas> Not sure https://github.com/YunoHost-Apps/2FAuth_ynh/commit/2f841883dd19ce7516a434a6c1d7f8af0feca777 ^^
[19:46:09]
<Thomas> > <@Salamandar:matrix.org> I added 2 comments (mere questions, you know the app better than me), but anyways LGTM. I added the review so you can merge this if you are part of the org i think.
Thank you! I cannot because I don’t have enough rights on that repository, but I will make some more testing soon
[19:46:09]
<Aleks (he/him/il/lui)> (i have no idea what this app is 😬)
[19:46:09]
<Aleks (he/him/il/lui)> ah the app is like some sort of API ?
[19:46:09]
<Thomas> Could someone merge https://github.com/YunoHost-Apps/2FAuth_ynh/pull/65 please? Several persons confirm it’s working! i’ve still to fix the mail config, but I didn’t find a solution nor time for now :)
[19:46:09]
<Thomas> Yess thank you!
[19:46:09]
<Thomas> Thanks for your input! :)
[19:46:09]
<Thomas> > <@Alekswag:matrix.org> ah the app is like some sort of API ?
No it’s a webapp where you can "register" your 2fa codes to access them from anywhere (without the fear of losing your device)
[20:56:05]
<lapineige> > <@Bram_:matrix.org> (fyi, I'm not super up to date on how people become maintainers on apps so I might be missing something)
They put their name in a file saying their maintainers (assuming they have write rights / PR approval).
Yeah, that's about it
[21:00:23]
<Aleks (he/him/il/lui)> yeah but what does it mean to have your name in the file 😬
[21:02:21]
<orhtej2> fame, glory, money
[21:02:21]
<Bram> Really? Where is my money then v_v?
[21:06:58]
<orhtej2> Oh element android 🫠
[21:06:59]
<Aleks (he/him/il/lui)> the face of app maintaners when they receive the check in the mail https://i.imgflip.com/8m24dm.jpg
[21:06:59]
<orhtej2> > <@Bram_:matrix.org> Really? Where is my money then v_v?
/shrug
[21:07:50]
<Yunohost Git/Infra notifications> [nextcloud_ynh] ericgaspar edited [pull request #680](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/680): Make system address book access configurable
[21:15:09]
<lapineige> > <@Alekswag:matrix.org> yeah but what does it mean to have your name in the file 😬
The value you put in there
[21:24:34]
<Aleks (he/him/il/lui)> so you mean it's just "self-crediting for the work" in some sense ? (that's a genuine question because this info could also be seen as 'I intend to maintain this app on the mid-term/long-term and I will look at issues, review PR and i'm okay to be pinged about this app')
[21:26:35]
<lapineige> In my view, it's only that :
> i'm okay to be pinged about this app
The rest, really depends on the situation and people
[21:26:56]
<lapineige> For instance I "maintain" a lot of apps where I push the green merge button on other people work and that's about it
[21:27:50]
<lapineige> So I really think it's nothing more than "I have my name listed on that app", as a common ground.
Then people add more expectations on their own side, depending on the case.
At least it's how I feel it
[21:33:39]
<Yunohost Git/Infra notifications> [apps] yunohost-bot created new branch add-to-wishlist-slskd
[21:33:40]
<Yunohost Git/Infra notifications> [apps] yunohost-bot pushed 1 commit to add-to-wishlist-slskd: Add Slskd to wishlist ([7e7286a5](https://github.com/YunoHost/apps/commit/7e7286a53ef1cca0a2612976a4b3869040a67b4c))
[21:33:42]
<Yunohost Git/Infra notifications> [apps] yunohost-bot labeled Wishlist on [pull request #2232](https://github.com/YunoHost/apps/pull/2232): Add Slskd to wishlist
[21:33:43]
<Yunohost Git/Infra notifications> [apps] yunohost-bot opened [pull request #2232](https://github.com/YunoHost/apps/pull/2232): Add Slskd to wishlist
[21:48:01]
<Yunohost Git/Infra notifications> [apps] OniriCorpe approved [pull request #2224](https://github.com/YunoHost/apps/pull/2224#pullrequestreview-1987456958) Microbin
[21:48:03]
<Yunohost Git/Infra notifications> [apps] ericgaspar merged [pull request #2224](https://github.com/YunoHost/apps/pull/2224): Microbin
[21:48:04]
<Yunohost Git/Infra notifications> [apps] ericgaspar pushed 1 commit to master: Microbin (#2224) * Update apps.toml * add logo ([de4ac3f4](https://github.com/YunoHost/apps/commit/de4ac3f4945667dbec1dc7c3559b4d0f00b0bdc4))
[21:48:04]
<Yunohost Git/Infra notifications> [apps] ericgaspar deleted branch microbin
[21:48:31]
<Yunohost Git/Infra notifications> [apps] github-actions[bot] pushed 1 commit to actions/black: :art: Format Python code with Black ([9712b89a](https://github.com/YunoHost/apps/commit/9712b89abe2a993e18b335bb1c0afeaf345e050c))