[22:17:14]
<Aleks (he/him/il/lui)> > <@tag:lostpod.me> hmm tests ynhtest_config_write_php and ynhtest_config_read_php are failing because php is not installed anymore
> buuut it should be installed if the app requires it sooo it's ok ?
😬 yeah we gotta figure something about thi
[22:17:14]
<Aleks (he/him/il/lui)> > <@tag:lostpod.me> and we should install php-cli for the tests ?
maybe yep
[22:17:18]
<Aleks (he/him/il/lui)> > <@tag:lostpod.me> Should we drop the support of bind = "config.php" in config_panel.toml ? I don't think any app uses it...
eeeh are we sure about this ? Pretty sure maybe vpnclient uses it, but granted that there's not that many apps with that use case ...
[22:17:18]
<Aleks (he/him/il/lui)> https://github.com/YunoHost-Apps/vpnclient_ynh/blob/master/config_panel.toml#L31
[22:17:18]
<Tag> that's why we tests ynhtest_config_write_php and ynhtest_config_read_php
[22:17:18]
<Tag> I was referring to bind = something**.php**
[22:17:22]
<Tag> but I guess we should keep this feature and just install php-cli before running the tests
[22:17:28]
<Aleks (he/him/il/lui)> ah
[22:17:28]
<Aleks (he/him/il/lui)> yeah idk xD
[22:17:32]
<Aleks (he/him/il/lui)> the point is both to test the helper on php file and the fact that the ynh_read_var_stuff should be somewhat format-agnostic
[22:17:32]
<Aleks (he/him/il/lui)> (sort-of)
[22:17:38]
<Tag> wokay then, I'll add php-cli in the CI
[22:17:56]
<Tag> 🤔 CI crashed ?
[22:18:01]
<Aleks (he/him/il/lui)> yeah LXC commands keep crashing randomly ..
[22:18:03]
<Tag> ahhh... c'est lié aux soucis d'IO ?
[22:18:05]
<Aleks (he/him/il/lui)> oui apriori
[22:18:05]
<Aleks (he/him/il/lui)> enfin aussi au fait que LXC a l'air un peu designé avec le cul, ou en tout cas crashe souvent dès que y'a des délais un peu long x_x
[22:18:09]
<Tag> bon en rajoutant php-cli le test php passe o/
[22:18:09]
<Tag> par contre cette histoire de YNH_APP_INSTANCE_NAME non initialisée c'est dommage...
[22:37:11]
<Yunohost Git/Infra notifications> [yunohost] @Tagadda pushed 1 commit to bookworm: mariadb-client needed by tests/test_app_resources.py ([ea7bdb62](https://github.com/YunoHost/yunohost/commit/ea7bdb62ed3d37b3054abcf52b8e5ac10d159ee2))
[22:45:19]
<Yunohost Git/Infra notifications> 🏗️ Starting build for yunohost/12.0.0+202305052245 for bookworm/unstable/all ...
[22:46:03]
<Yunohost Git/Infra notifications> ✔️ Completed build for yunohost/12.0.0+202305052245 for bookworm/unstable/all.
[23:13:06]
<Yunohost Git/Infra notifications> [issues] @Tagadda edited [issue #2050](https://github.com/YunoHost/issues/issues/2050): Support Debian 12 "Bookworm"