Monday, March 03, 2025
apps@conference.yunohost.org
March
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
           

[06:13:33] <Yunohost Git/Infra notifications> [searxng_ynh] g​ithub-actions[bot] opened [pull request #376](https://github.com/YunoHost-Apps/searxng_ynh/pull/376): Upgrade master from testing
[06:13:33] <Yunohost Git/Infra notifications> [searxng_ynh] e​willy merged [pull request #375](https://github.com/YunoHost-Apps/searxng_ynh/pull/375): Upgrade to v2025.03.02
[06:13:33] <Yunohost Git/Infra notifications> [searxng_ynh] e​willy merged [pull request #376](https://github.com/YunoHost-Apps/searxng_ynh/pull/376): Upgrade master from testing
[06:14:08] <Yunohost Git/Infra notifications> [searxng_ynh] e​willy deleted branch master-promotion
[06:14:08] <Yunohost Git/Infra notifications> [searxng_ynh] e​willy deleted branch ci-auto-update-2025.03.02
[12:18:20] <Felix 3> I'm working on packaging a PHP app that uses a Postgres DB, and since I'm pretty new to Yunohost I used the [package generator](https://appgenerator.yunohost.org/) as recommended by some folks here.
I just checked the backup script, and noticed that the section for the DB only contains the `ynh_print_info` call and nothing else. Shouldn't there be something that actually creates a DB dump (and declares it part of the backup)? Or am I missing something?

Here's the section from `scripts/backup` I'm talking about:

```
#=================================================
# BACKUP THE DATABASE
#=================================================
ynh_print_info "Backing up the postgresql database..."

### (However, things like MySQL dumps *do* take some time to run, though the
### copy of the generated dump to the archive still happens later)
```

EDIT: Just checked, and looks the same for MySQL
[12:19:44] <Yunohost Git/Infra notifications> [synapse_ynh] J​osue-T merged [pull request #517](https://github.com/YunoHost-Apps/synapse_ynh/pull/517): Testing
[12:23:44] <Felix 3> Hmm, checking the sources, it looks like there actually is a call to `ynh_Xsql_dump_db` in the template that should be included, but isn't. This part of the template is checking the value of `data.use_db`, unlike other parts which check `data.database`.
Could that be a leftover of a rename?
[12:35:24] <miro5001> Here are some examples of apps using postgresql
https://github.com/YunoHost-Apps/mastodon_ynh/blob/master/scripts/backup

https://github.com/YunoHost-Apps/peertube_ynh/blob/master/scripts/backup

https://github.com/YunoHost-Apps/indico_ynh/blob/master/scripts/backup
[12:58:00] <Paprika> Installing Flarum fails with `2025-03-03 14:53:39,068: WARNING - swapon: /swap_flarum: skipping - it appears to have holes.`
Perhaps someone has encountered this before?
I feel like I faced this a couple years ago and I don't remember what was the solution (if there was any).
[12:59:11] <Felix 3> Thanks! I think I found the issue that causes the backup script to not have the dump command. I'm opening a PR to fix it as we speak 😁

[13:00:08] <Paprika> My bad, meant to post this in support room.
[13:06:38] <Yunohost Git/Infra notifications> [appgenerator] f​elurx opened [pull request #14](https://github.com/YunoHost/appgenerator/pull/14): Fix backup.j2 to include DB dump
[13:12:53] <Yunohost Git/Infra notifications> [apps] y​unohost-bot opened [pull request #2846](https://github.com/YunoHost/apps/pull/2846): Add Agorakit to wishlist
[13:12:53] <Yunohost Git/Infra notifications> [apps] y​unohost-bot labeled Wishlist on [pull request #2846](https://github.com/YunoHost/apps/pull/2846): Add Agorakit to wishlist
[13:18:03] <Yunohost Git/Infra notifications> [borg_ynh] d​maroulidis [commented](https://github.com/YunoHost-Apps/borg_ynh/issues/159#issuecomment-2694364666) on [issue #159](https://github.com/YunoHost-Apps/borg_ynh/issues/159) Borg fails when file changes on disk during archive creation: > I also wonder if it would be a good idea to add a function to the borg app to executed a command before the borg creat...
[15:11:33] <Yunohost Git/Infra notifications> e​ricgaspar created repository agorakit_ynh https://github.com/YunoHost-Apps/agorakit_ynh
[15:14:23] <Yunohost Git/Infra notifications> e​ricgaspar edited repository agorakit_ynh https://github.com/YunoHost-Apps/agorakit_ynh
[15:15:14] <Yunohost Git/Infra notifications> e​ricgaspar edited repository agorakit_ynh: Agorakit package for YunoHost https://github.com/YunoHost-Apps/agorakit_ynh
[15:15:15] <Yunohost Git/Infra notifications> e​ricgaspar edited repository agorakit_ynh: Agorakit package for YunoHost https://github.com/YunoHost-Apps/agorakit_ynh
[15:21:14] <Yunohost Git/Infra notifications> [apps] e​ricgaspar created new branch agorakit
[15:21:14] <Yunohost Git/Infra notifications> [apps] e​ricgaspar pushed 1 commit to agorakit: Update apps.toml ([d015294a](https://github.com/YunoHost/apps/commit/d015294af2365225abe96828d0c9cb9714d860b4))
[15:21:58] <Yunohost Git/Infra notifications> [apps] e​ricgaspar pushed 1 commit to agorakit: Create agorakit.png ([cf4d8bf8](https://github.com/YunoHost/apps/commit/cf4d8bf8bff1ec2b756652f08a694a502601d099))
[15:22:04] <Yunohost Git/Infra notifications> [apps] e​ricgaspar opened [pull request #2847](https://github.com/YunoHost/apps/pull/2847): Add Agorakit
[15:22:47] <Yunohost Git/Infra notifications> [apps] e​ricgaspar closed [pull request #2846](https://github.com/YunoHost/apps/pull/2846): Add Agorakit to wishlist
[15:22:49] <Yunohost Git/Infra notifications> [apps] e​ricgaspar [commented](https://github.com/YunoHost/apps/pull/2846#issuecomment-2694753001) on [issue #2846](https://github.com/YunoHost/apps/pull/2846) Add Agorakit to wishlist: packaged https://github.com/YunoHost-Apps/agorakit_ynh
[15:29:34] <Yunohost Git/Infra notifications> e​ricgaspar edited repository agorakit_ynh: Agorakit package for YunoHost https://github.com/YunoHost-Apps/agorakit_ynh
[15:35:03] <Yunohost Git/Infra notifications> [apps] e​ricgaspar pushed 1 commit to master: Add Agorakit (#2847) * Update apps.toml * Create agorakit.png ([60c9741e](https://github.com/YunoHost/apps/commit/60c9741ec140ec973886cd3f9c17dd27c315f5ed))
[15:35:04] <Yunohost Git/Infra notifications> [apps] e​ricgaspar deleted branch agorakit
[15:35:05] <Yunohost Git/Infra notifications> [apps] e​ricgaspar merged [pull request #2847](https://github.com/YunoHost/apps/pull/2847): Add Agorakit
[17:00:33] <Felix 3> For the PHP app I'm packaging, I'd like to give admins the option to override stuff from its `config.inc.php`, while still generating that file from the settings in the Yunohost admin interface.
My idea would be to place a `config-custom.inc.php` somewhere for the admin to edit as they please, and tell PHP to include that file in the `config.inc.php` provided by the package.
Does that idea sound sane?

The [Grist app seems to do something similiar](https://github.com/YunoHost-Apps/grist_ynh/blob/134b880bdec9c8d0b9cfec29a13d3947e063a15d/scripts/install#L68), with a `/var/www/grist/default.env` that's provided and a `/home/yunohost.app/grist/custom.env` for the admin to edit.
In my case, my app doesn't need a data dir. Where should I put the `config-custom.inc.php` in that case? Just into `/var/www/<my_app>/`?
[17:02:10] <Aleks (he/him/il/lui)> yup
[17:02:39] <Felix 3> Yay, thanks!
[17:06:49] <Felix 3> Other question: Do I just create a GitHub repo under my own personal account, do my testing with that, and get the transfer to the yunohost-apps org done when the app works well? Anything I need to keep in mind now to not create problems later?
[17:07:14] <Aleks (he/him/il/lui)> yup that sounds good
[17:32:54] <Yunohost Git/Infra notifications> [apps] C​odeShakingSheep opened [pull request #2848](https://github.com/YunoHost/apps/pull/2848): Add Appflowy logo
[18:02:47] <Yunohost Git/Infra notifications> [apps] e​ricgaspar approved [pull request #2848](https://github.com/YunoHost/apps/pull/2848#pullrequestreview-2654861192) Add Appflowy logo
[18:02:52] <Yunohost Git/Infra notifications> [apps] e​ricgaspar merged [pull request #2848](https://github.com/YunoHost/apps/pull/2848): Add Appflowy logo
[18:02:53] <Yunohost Git/Infra notifications> [apps] e​ricgaspar pushed 1 commit to master: Add Appflowy logo (#2848) * Add Appflowy logo * Update appflowy.png --------- Co-authored-by: eric_G <46165813+ericg... ([254bcbda](https://github.com/YunoHost/apps/commit/254bcbda146e4ae7bde5e557db7b0a0ee973e8b7))
[18:40:01] <Yunohost Git/Infra notifications> [piped_ynh] y​alh76 merged [pull request #188](https://github.com/YunoHost-Apps/piped_ynh/pull/188): Upgrade to v2025.02.27
[18:40:02] <Yunohost Git/Infra notifications> [piped_ynh] y​alh76 deleted branch ci-auto-update-2025.02.27
[18:40:23] <Yunohost Git/Infra notifications> [piped_ynh] y​alh76 opened [pull request #189](https://github.com/YunoHost-Apps/piped_ynh/pull/189): Upgrade to v2025.02.27
[19:44:13] <Yunohost Git/Infra notifications> [nextcloud_ynh] e​ricgaspar approved [pull request #790](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/790#pullrequestreview-2655123445) fix(user_ldap): retrieve from LDAP first "email shaped" value instead…
[19:44:16] <Yunohost Git/Infra notifications> [nextcloud_ynh] e​ricgaspar merged [pull request #790](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/790): fix(user_ldap): retrieve from LDAP first "email shaped" value instead…
[19:44:17] <Yunohost Git/Infra notifications> [nextcloud_ynh] e​ricgaspar pushed 1 commit to 31.0.0: fix(user_ldap): retrieve from LDAP first "email shaped" value instead… (#790) Because this PR has been merged upstream ... ([cb4def68](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/cb4def6856e665509b73fc76a2a4e67cabf494c4))
[19:44:32] <Yunohost Git/Infra notifications> [nextcloud_ynh] e​ricgaspar edited [pull request #787](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/787): 31.0.0