[01:19:50]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @zamentur [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/637#discussion_r1461177898) on pull request #637 [enh] Faster permissions setter on big datadir: If you have a lot of files, the find method could trigger a message saying there is too many destination listed (if i re...
[01:20:07]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @zamentur edited a [comment](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/637#discussion_r1461177898) on pull request #637 [enh] Faster permissions setter on big datadir: If you have a lot of files, the find method could trigger a message saying there is too many destination listed (if i re...
[01:20:39]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @zamentur pushed 1 commit to enh-configpanel: Maj in brand name Co-authored-by: eric_G <46165813+ericgaspar@users.noreply.github.com> ([3092bb46](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/3092bb461572b12693790e89dc062f573db30b56))
[01:22:02]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @zamentur pushed 1 commit to enh-configpanel: Maj in product name Co-authored-by: eric_G <46165813+ericgaspar@users.noreply.github.com> ([d0fcac28](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/d0fcac2802b45a2f26bd490c2449f72cab0ea93e))
[01:22:17]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @zamentur pushed 1 commit to enh-configpanel: __DATA_DIR__ instead of longer path Co-authored-by: eric_G <46165813+ericgaspar@users.noreply.github.com> ([cfeddfa3](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/cfeddfa329326b6dd35f60877bb9372980b674a7))
[01:23:02]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @zamentur pushed 1 commit to enh-configpanel: data_dir instead of longer path Co-authored-by: eric_G <46165813+ericgaspar@users.noreply.github.com> ([66ec3e8e](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/66ec3e8ee0cdfa178faf7c89db30024f51be1b9a))
[01:23:49]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @zamentur pushed 1 commit to enh-configpanel: data_dir instead of longer path Co-authored-by: eric_G <46165813+ericgaspar@users.noreply.github.com> ([b2890650](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/b289065015dd8924c79e5c3896c75855c2ccdf9f))
[01:23:55]
<Aleks (he/him/il/lui)> > It's very strange, I don't understand why it would include \r and not \n
Probable related to jinja templating, {% vs {-% etc
[01:24:08]
<Yunohost Git/Infra notifications> App digiscreen rises from level 0 to 7 in job [#22448](https://ci-apps.yunohost.org/ci/job/22448) !
[01:24:20]
<Yunohost Git/Infra notifications> [apps] @OniriCorpe [commented](https://github.com/YunoHost/apps/pull/1500#issuecomment-1902838726) on [issue #1500](https://github.com/YunoHost/apps/pull/1500) add Minecraft: arent Spigot and Paper open-source minecraft servers? maybe we can package them but the official server?
[01:25:15]
<Émy - OniriCorpe> eric_G: why installing pufferpanel using a deb repo and not the deb provided in the github releases?
https://github.com/YunoHost-Apps/pufferpanel\_ynh
https://github.com/pufferpanel/pufferpanel/releases
[06:59:03]
<Raphael> ok, that what I thought, but was not sure if it was better to recompile it in restore script to reduce the archive size
[07:27:42]
<Yunohost Git/Infra notifications> App fider stays at level 3 in job [#22452](https://ci-apps.yunohost.org/ci/job/22452)
[07:27:42]
<Yunohost Git/Infra notifications> App grav goes down from level 8 to 1 in job [#22455](https://ci-apps.yunohost.org/ci/job/22455)
[07:27:50]
<Yunohost Git/Infra notifications> [apps] @tituspijean [commented](https://github.com/YunoHost/apps/pull/1500#issuecomment-1903292927) on [issue #1500](https://github.com/YunoHost/apps/pull/1500) add Minecraft: IIRC they both require the Minecraft server jar file to be patched to generate their own jar files.
And both require th...
[07:28:55]
<lapineige> > <@Alekswag:matrix.org> Probable related to jinja templating, {% vs {-% etc
Yeah but nowhere there is \r 😅
[07:33:38]
<tituspijean> > <@yunohostinfra:matrix.org> App grav goes down from level 8 to 1 in job [#22455](https://ci-apps.yunohost.org/ci/job/22455)
Restarting the job... 🤞
[08:43:52]
<Yunohost Git/Infra notifications> [apps] @OniriCorpe [commented](https://github.com/YunoHost/apps/pull/1500#issuecomment-1903475224) on [issue #1500](https://github.com/YunoHost/apps/pull/1500) add Minecraft: I see 😓
[13:17:41]
<lejocelyn> Hey, do you think it would be possible to sponsor the update of a package?
[13:18:23]
<lejocelyn> an existing one, but a bit old
[13:19:10]
<lejocelyn> this app, Overleaf: https://github.com/YunoHost-Apps/overleaf_ynh/tree/master
[13:20:47]
<lejocelyn> I'd like the Yunohost package to have the review feature, and to be able to work on the project with other users
[13:29:42]
<lejocelyn> sorry, I guess my question is not so simple, it's not like I can provide anything close to a salary for someone to develop this feature, it's just that I would really be happy if I could work with other colleagues using Yunohost/Overleaf
[13:31:34]
<lejocelyn> and I don't think it has to do with something I misconfigured
[13:58:05]
<Thomas> As you can see here (https://github.com/YunoHost-Apps/overleaf_ynh/pull/21) I'm trying to make it work again. If you have some insight about the history-v1 issue I have (referenced in the issue), I would be glad :)
[14:03:47]
<Aleks (he/him/il/lui)> `at connectionStringHasValidScheme (/var/www/overleaf/live/node_modules/mongodb-connection-string-url/lib/index.js:9:30)` sounds like it doesn't like the mongodb url or something ?
[14:04:13]
<Aleks (he/him/il/lui)> this maybe ? `SHARELATEX_MONGO_URL="mongodb://127.0.0.1:27017/__DB_NAME__"` (it was `localhost` before) #randomguess
[14:05:11]
<Aleks (he/him/il/lui)> i'm more concerned about the `FileNotFoundError: [Errno 2] No such file or directory: '/tmp/tmpj7lfjao5/stdreturn'` later though, it's like for some reason something deleted the entire /tmp/ maybe o_O
[14:07:42]
<lejocelyn> > <@thovi98:matrix.org> As you can see here (https://github.com/YunoHost-Apps/overleaf_ynh/pull/21) I'm trying to make it work again. If you have some insight about the history-v1 issue I have (referenced in the issue), I would be glad :)
Hum, I must say I don't have any clue about this issue. I'm taking a look at the logs in case
[14:08:39]
<lejocelyn> Also, I don't have this issue on my server. Overleaf works well for 1 user on my server
[14:16:46]
<lejocelyn> here, the problems are: Overleaf does not send emails to the people we want to share a project with (it does not seem I have any problem with emails or DNS configuration according to Yunohost test services)
[14:17:33]
<lejocelyn> and I cannot other people contribute to any project I have, which kills a bit the interest of overleaf for me
[14:17:56]
<Thomas> > <@Alekswag:matrix.org> `at connectionStringHasValidScheme (/var/www/overleaf/live/node_modules/mongodb-connection-string-url/lib/index.js:9:30)` sounds like it doesn't like the mongodb url or something ?
Thanks for your suggestions! This was my guess too, but installing mongo-express showed the changes in mongo when I created a user so I understand that the mongo_url is correct
[14:18:06]
<Thomas> Will have a look again when possible
[14:33:11]
<Yunohost Git/Infra notifications> [apps] @alexAubin deleted branch add-to-wishlist-linkwarden
[14:33:13]
<Yunohost Git/Infra notifications> [apps] @alexAubin pushed 2 commits to master ([115aaceee9b2...89c68a958c59](https://github.com/YunoHost/apps/compare/115aaceee9b2...89c68a958c59))
[14:33:19]
<Yunohost Git/Infra notifications> [apps] @alexAubin merged [pull request #1969](https://github.com/YunoHost/apps/pull/1969): Add Linkwarden to wishlist
[14:41:17]
<Aleks (he/him/il/lui)> trying to debug the cockpit and transmission regression in https://github.com/YunoHost/apps/pull/1968
[14:41:19]
<Aleks (he/him/il/lui)> turns out apps that can be installed using `apt` are really not convenient ...
[14:41:31]
<Aleks (he/him/il/lui)> cf https://github.com/YunoHost-Apps/transmission_ynh/blob/master/scripts/remove#L30 `ynh_secure_remove --file=/usr/share/transmission`
[14:42:05]
<Aleks (he/him/il/lui)> thing is that if you remove and reinstall the app, it may not necessarily `apt remove` the transmission package (because it may be flagged as manually installed for some reason), and won't recreate the /usr/share, and then the reinstall crashes
[14:42:07]
<Aleks (he/him/il/lui)> ~_~
[14:46:19]
<Aleks (he/him/il/lui)> (and it's also not convenient because we have no control over which version is installed because upgrades are effectively shipped via apt, and we have no control on the postinst script like for collabora or onlyoffice where they though it'd be a good idea to have literally`systemctl restart nginx` in the middle of nowhere)
[14:46:46]
<Aleks (he/him/il/lui)> anyway, guess i'll write a stupid hack to workaround the issue
[15:00:59]
<Yunohost Git/Infra notifications> [package_check] @alexAubin pushed 1 commit to master: Fix boring issue with preinstalling apt dependencies in the context of transmission/cockpit: ignore apt dependencies sta... ([7cdffc74](https://github.com/YunoHost/package_check/commit/7cdffc745ce183e2dded9507aa1b691120150481))
[15:01:16]
<Yunohost Git/Infra notifications> Job [#22391](https://ci-apps.yunohost.org/ci/job/22391) for transmission failed miserably :(
[15:34:13]
<Yunohost Git/Infra notifications> [package_check] @Jakski opened [pull request #149](https://github.com/YunoHost/package_check/pull/149): Allow overriding is_webapp parameter in tests.toml
[15:36:08]
<Yunohost Git/Infra notifications> [package_check] @Salamandar [commented](https://github.com/YunoHost/package_check/pull/149#issuecomment-1904260912) on [issue #149](https://github.com/YunoHost/package_check/pull/149) Allow overriding is_webapp parameter in tests.toml: IMHO this configuration would be better in the tests.toml :)
[15:36:58]
<Yunohost Git/Infra notifications> [package_check] @Jakski edited [pull request #149](https://github.com/YunoHost/package_check/pull/149): Allow overriding is_webapp parameter in tests.toml
[15:38:54]
<Yunohost Git/Infra notifications> [package_check] @Jakski [commented](https://github.com/YunoHost/package_check/pull/149#issuecomment-1904266535) on [issue #149](https://github.com/YunoHost/package_check/pull/149) Allow overriding is_webapp parameter in tests.toml: I agree, but for now Ive left auto-detection using grep in order to not break compatibility with already existing appli...
[15:40:46]
<Yunohost Git/Infra notifications> [package_check] @Salamandar [commented](https://github.com/YunoHost/package_check/pull/149#issuecomment-1904270473) on [issue #149](https://github.com/YunoHost/package_check/pull/149) Allow overriding is_webapp parameter in tests.toml: Ah oops, i misread, now I see you already did that :)
[15:49:20]
<Yunohost Git/Infra notifications> [package_check] @alexAubin [commented](https://github.com/YunoHost/package_check/pull/149#issuecomment-1904288702) on [issue #149](https://github.com/YunoHost/package_check/pull/149) Allow overriding is_webapp parameter in tests.toml: We could also just grep _common.sh in addition to install TBH we tend to discourage to "factorize" this kind of stu...
[15:56:17]
<Yunohost Git/Infra notifications> App cockpit goes down from level 8 to 6 in job [#22213](https://ci-apps.yunohost.org/ci/job/22213)
[16:37:01]
<Yunohost Git/Infra notifications> [my_webapp_ynh] @alexAubin approved [pull request #130](https://github.com/YunoHost-Apps/my_webapp_ynh/pull/130#pullrequestreview-1836892355) Remove dependencies and reinstall them when changing php version
[16:37:02]
<Yunohost Git/Infra notifications> [my_webapp_ynh] @alexAubin pushed 3 commits to testing ([25ec30c16632...7d592faabe62](https://github.com/YunoHost-Apps/my_webapp_ynh/compare/25ec30c16632...7d592faabe62))
[16:37:02]
<Yunohost Git/Infra notifications> [my_webapp_ynh/testing] explicitly call ynh_remove_app_dependencies when changing phpversion - Florent
[16:37:02]
<Yunohost Git/Infra notifications> [my_webapp_ynh] @alexAubin merged [pull request #130](https://github.com/YunoHost-Apps/my_webapp_ynh/pull/130): Remove dependencies and reinstall them when changing php version
[16:38:14]
<Yunohost Git/Infra notifications> [apps] @alexAubin pushed 1 commit to update_app_levels: Update apps.toml: transmission and cockpit back to level 8 ([2f05d3cd](https://github.com/YunoHost/apps/commit/2f05d3cd139525979296e411c8a6b046552ef081))
[16:38:46]
<Yunohost Git/Infra notifications> [apps] @alexAubin edited [pull request #1968](https://github.com/YunoHost/apps/pull/1968): Update app levels according to CI results
[16:38:57]
<Yunohost Git/Infra notifications> [apps] @alexAubin approved [pull request #1968](https://github.com/YunoHost/apps/pull/1968#pullrequestreview-1836896392) Update app levels according to CI results
[16:39:29]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @alexAubin [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/651#issuecomment-1904386395) on [issue #651](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/651) Filter boring occ warnings during upgrade: (Imho this is done can be merged to testing)
[16:39:37]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @alexAubin labeled ready to merge on [pull request #651](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/651): Filter boring occ warnings during upgrade
[16:39:42]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @alexAubin [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/652#issuecomment-1904386865) on [issue #652](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/652) Fix PHP/APT dependency confusion when upgrading through many versions: (Imho this is done can be merged to testing)
[16:39:48]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @alexAubin labeled ready to merge on [pull request #652](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/652): Fix PHP/APT dependency confusion when upgrading through many versions
[16:40:09]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @alexAubin [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/653#issuecomment-1904387693) on [issue #653](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/653) Use new source format in manifest.toml + stop supporting upgrades from before 22.2: (Imho this is done can be merged to testing)
[16:40:13]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @alexAubin labeled ready to merge on [pull request #653](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/653): Use new source format in manifest.toml + stop supporting upgrades from before 22.2
[17:30:04]
<Yunohost Git/Infra notifications> [package_check] @alexAubin merged [pull request #144](https://github.com/YunoHost/package_check/pull/144): Publish logs with YunoPaste on failure
[17:30:05]
<Yunohost Git/Infra notifications> [package_check] @alexAubin deleted branch yunopaste-on-failure
[17:30:06]
<Yunohost Git/Infra notifications> [package_check] @alexAubin pushed 2 commits to master ([7cdffc745ce1...dbe2441b0cff](https://github.com/YunoHost/package_check/compare/7cdffc745ce1...dbe2441b0cff))
[17:55:34]
<Yunohost Git/Infra notifications> Job [#22474](https://ci-apps.yunohost.org/ci/job/22474) for icecoder failed miserably :(
[18:00:45]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @JimboJoe [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/637#issuecomment-1902782881) on [issue #637](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/637) [enh] Faster permissions setter on big datadir: There have been some work discussed in the past in #174, if that can help. Moreover, theres something that could be don...
[18:04:29]
<Yunohost Git/Infra notifications> App icecoder rises from level 0 to 7 in job [#22474](https://ci-apps.yunohost.org/ci/job/22474) !
[22:30:36]
<Krakotte> how to deal in v2 with a new param in the manifest.toml between 2 version of a package, when this param is also used as a variable in the manifest (__PARAM__)? Previously I could just make a small if [ ! -z $param]; then blabla fi in the upgrade script, but now the provisionning take the lead, cannot charge the param and just fail, leaving the app broken?
[22:31:30]
<Krakotte> specifically, I'm talking about https://github.com/YunoHost-Apps/domoticz_ynh/blob/testing/manifest.toml where __MQTT_DOMAIN__ may in some case not exists in settings.yml of the app
[22:31:59]
<Krakotte> and the upgrade break before I can do anything with a script
[22:32:22]
<Krakotte> so now, I have a warning for the admin to manually add the param before upgrading, but meeeh 😕
[22:40:50]
<Aleks (he/him/il/lui)> i guess you're looking for `if [ -z "${param:-}" ]` instead of `if [ -z "$param" ]`
[22:41:11]
<Aleks (he/him/il/lui)> cf for example https://github.com/YunoHost-Apps/nextcloud_ynh/blob/master/scripts/upgrade#L17
[22:41:43]
<Aleks (he/him/il/lui)> `${foobar:-}` *obviously* means "foobar's value or empty string if foobar is not defined"
[22:41:47]
<Aleks (he/him/il/lui)> #JustBashThings
[22:45:45]
<Émy - OniriCorpe> > <@lejocelyn:sans-nuage.fr> here, the problems are: Overleaf does not send emails to the people we want to share a project with (it does not seem I have any problem with emails or DNS configuration according to Yunohost test services)
the mail config is currently empty, this will be fixed when Thomas' work is done ^w^
cf: https://github.com/YunoHost-Apps/overleaf_ynh/pull/21/files#diff-4454b760602aaa8b6f640dbb448fb4761d156a2c8f09cde00fca824fb5166160R58-R62
[22:45:45]
<lejocelyn> > <@oniricorpe:im.emelyne.eu> the mail config is currently empty, this will be fixed when Thomas' work is done ^w^
> cf: https://github.com/YunoHost-Apps/overleaf_ynh/pull/21/files#diff-4454b760602aaa8b6f640dbb448fb4761d156a2c8f09cde00fca824fb5166160R58-R62
thanks for the info