Monday, October 23, 2023
dev@conference.yunohost.org
October
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
         

[04:18:23] <Yunohost Git/Infra notifications> Failed to run the source auto-update for : calibreweb, cinny, zwave-js-ui. Please run manually the `autoupdate_app_sources.py` script on these apps to debug what is happening! Debug log : http://paste.yunohost.org/raw/ivonezajum
[07:14:49] <ChriChri> https://github.com/YunoHost-Apps/phpldapadmin_ynh/issues/26
[07:15:24] <ChriChri> Anybody literate in php? ;-)
[07:31:45] <tituspijean> As answered on Github, it's most likely due to PHPLDAPadmin's own code. The issue you linked to offers a hotfix: https://github.com/leenooks/phpLDAPadmin/issues/237
[07:32:03] <tituspijean> (hopefully it's compatible with PHP8.1)
[07:42:11] <ChriChri> In my first attempt I didn't get it right somehow and it didn't work after applying the changes. Just tried it again and now it works :). Thanks for nudging!
[09:43:25] <ChriChri> > <@titus:pijean.ovh> (hopefully it's compatible with PHP8.1)

It is.
ynh migrates from php7.4 to php8.1? Is that the reason why php7.4 is still installed (and probably would work with phpldapadmin) and phpldapadmin-ynh-deps pulls in php8.1 and uses it?
However that is: as long as phpldapadmin isn't fixed for 8.x shouldn't it just use php7.4 if it works o.k. with it?

(This is an edge case - I guess not many people will use phpldapadmin to actually work on the ynh directory, right? Maybe it should just be documented as by the issue now until upstream is compatible with php8.x. Then the issue should get a 'wontfix' with reason we're waiting for upstream to integrate the patch).
[10:39:49] <tituspijean> Currently the default behavior of YunoHost is to install a default PHP version (7.4 for 11.1x, 8.0 for 12.x IIRC). However, each app can specify its own PHP version. They can run at the same time with no issue, we merely follow the requirements of each upstream app.
In the case of PHPLDAPadmin, they require 8.1 or 8.2, and the packager chose 8.1. 😉
[12:05:45] <lapineige> And hopefully when possible packagers upgrade to PHP 8.x the previous PHP 7.4 apps, as the former version is no longer supported
[12:17:14] <Yunohost Git/Infra notifications> [yunohost] @Axolotle pushed 3 commits to options-doc ([a0994af956a2...b3167ba2e88f](https://github.com/YunoHost/yunohost/compare/a0994af956a2...b3167ba2e88f))
[12:17:24] <Yunohost Git/Infra notifications> [yunohost/options-doc] doc:options: add documentation and generator for configpanel/manifest options - axolotle
[12:17:25] <Yunohost Git/Infra notifications> [yunohost/options-doc] doc:form: complete with ljf comments found in app example - axolotle
[12:17:29] <Yunohost Git/Infra notifications> [yunohost/options-doc] doc:configpanel: add ConfigPanel doc with part of lfj comment in app example_ynh - axolotle
[12:21:15] <Yunohost Git/Infra notifications> [yunohost] 🔴 Pipeline [#1046170717](https://gitlab.com/yunohost/yunohost/-/pipelines/1046170717) failed on branch options-doc
[13:06:47] <Aleks (he/him/il/lui)> https://github.com/bugen/pypipe o_o
[14:17:35] <lapineige> Impressive
[16:18:44] <Yunohost Git/Infra notifications> Failed to run the source auto-update for : calibreweb, cinny. Please run manually the `autoupdate_app_sources.py` script on these apps to debug what is happening! Debug log : http://paste.yunohost.org/raw/imemupuxox
[16:53:43] <Tag> > <@yunohostinfra:matrix.org> Failed to run the source auto-update for : calibreweb, cinny. Please run manually the `autoupdate_app_sources.py` script on these apps to debug what is happening! Debug log : http://paste.yunohost.org/raw/imemupuxox

Fixed
[17:05:58] <Yunohost Git/Infra notifications> [yunohost] @Axolotle pushed 2 commits to options-doc ([b3167ba2e88f...ee72d2f463d2](https://github.com/YunoHost/yunohost/compare/b3167ba2e88f...ee72d2f463d2))
[17:06:05] <Yunohost Git/Infra notifications> [yunohost/options-doc] doc:config: add ljfs advanced config panel doc - axolotle
[17:06:07] <Yunohost Git/Infra notifications> [yunohost/options-doc] doc: fix page routes and inconsistencies - axolotle
[17:09:51] <Yunohost Git/Infra notifications> [yunohost] 🔴 Pipeline [#1046559158](https://gitlab.com/yunohost/yunohost/-/pipelines/1046559158) failed on branch options-doc
[19:26:17] <Yunohost Git/Infra notifications> [issues] @zamentur [commented](https://github.com/YunoHost/issues/issues/2275#issuecomment-1775882319) on [issue #2275](https://github.com/YunoHost/issues/issues/2275) Error sending mail from an internal domain to another domain: Its probably due to the dnsmasq config. Could work if your remove the mx lines inside /etc/dnsmasq.d/DOMAIN.conf
[19:26:28] <Yunohost Git/Infra notifications> [issues] @zamentur labeled :mailbox: Email on [issue #2275](https://github.com/YunoHost/issues/issues/2275): Error sending mail from an internal domain to another domain
[19:26:41] <Yunohost Git/Infra notifications> [issues] @zamentur labeled :mailbox: Email on [issue #2273](https://github.com/YunoHost/issues/issues/2273): include rspamd repository
[19:40:35] <Krakinou> @tag: when you say "fixed", what did you do?
[19:41:17] <Krakinou> It seems that the maintainer of the upstream app does not release zip or targz file anymore, so I should change the path, but it seems you just change the update strategy from the testing branch? Or am I missing something?