Tuesday, March 28, 2023
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
   
             

[01:32:26] <Yunohost Git/Infra notifications> App funkwhale rises from level 6 to 8 in job [#14652](https://ci-apps.yunohost.org/ci/job/14652) !
[06:58:04] <Yunohost Git/Infra notifications> [example_ynh] @ericgaspar approved [pull request #206](https://github.com/YunoHost/example_ynh/pull/206#pullrequestreview-1360332534) New source autoupdate mechanism
[06:58:26] <Yunohost Git/Infra notifications> [example_ynh] @ericgaspar merged [pull request #206](https://github.com/YunoHost/example_ynh/pull/206): New source autoupdate mechanism
[06:58:26] <Yunohost Git/Infra notifications> [example_ynh] @ericgaspar pushed 3 commits to master ([2b781a06d3bc...d6ba28df5bb1](https://github.com/YunoHost/example_ynh/compare/2b781a06d3bc...d6ba28df5bb1))
[06:58:28] <Yunohost Git/Infra notifications> [example_ynh] @ericgaspar deleted branch new-auto-updater
[06:58:28] <Yunohost Git/Infra notifications> [example_ynh/master] New source autoupdate mechanism - Alexandre Aubin
[06:58:32] <Yunohost Git/Infra notifications> [example_ynh/master] Actuall ... lets comment the autoupdate.strategy line so people explicitly enable it when they think its relevant - Alexandre Aubin
[06:58:35] <Yunohost Git/Infra notifications> [example_ynh/master] Merge pull request #206 from YunoHost/new-auto-updater New source autoupdate mechanism - Éric Gaspar
[07:03:19] <eric_G> so just to be sure, with the new auto-update mechanism, we don't need the `updater.sh` and `updater.yml` scripts in the `.github` folder
[07:08:57] <Aleks (he/him/il/lui)> Yup
[07:12:40] <eric_G> how would this work for something like Galène? (two repo, one from tag and the other one from commit) https://github.com/YunoHost-Apps/galene_ynh/blob/c135608074c7767b4058b8ffbf486b4b6f128fbc/manifest.toml#L66-L73
[07:19:03] <Aleks (he/him/il/lui)> The first one should be just about adding `autoupdate.strategy = "latest_gitbub_tag" `
[07:19:58] <Aleks (he/him/il/lui)> And the second one i gotta implement some "latest_commit" strategy + specifying the upstream repo which is different from the main upstream repo
[08:14:05] <carrabelloy> "Hello,

I am having trouble accessing the help service forum and I am not sure why. It is possible that there was an error in one of my posts, but I would appreciate it if I could be allowed back in so that I can ask for help from other users. English is not my first language, so I may need some assistance. I appreciate the work that the Yunohost user groups do and would like to be able to participate fully.

Thank you."

I hope this helps. If you need further assistance, please let me know.

[09:37:21] <tituspijean[m]> carrabelloy: it looks like you have been banned for spam. Even though English is not your first language, I often cannot understand your messages and they are usually disruptive to the flow of the conversation.
 
I can lift the ban once more, but I will not tolerate more disruptions. Next ban will relegate you to the forum.
 
Note, do not start your first messages by a picture, that may trigger the anti-spam.
[09:39:18] <tituspijean[m]> Ban lifted.
[10:15:20] <Aleks (he/him/il/lui)> https://botsin.space/@scream
[10:43:28] <tituspijean[m]> Am I regretting unbanning them? Yes. Kinda. 😭
[11:56:41] <Yunohost Git/Infra notifications> [apps] @alexAubin created new branch improve-auto-update-mechanism
[13:50:27] <Yunohost Git/Infra notifications> [apps] @ericgaspar pushed 1 commit to master: package maintained ([77cbc6ac](https://github.com/YunoHost/apps/commit/77cbc6ac8aa7046297c3dd90aace8d5ed6fcda64))
[13:54:59] <Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar pushed 1 commit to fix-security-warning: Update manifest.json ([6c5a7697](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/6c5a7697dd5c088bdc59403d8ef8e67881495521))
[13:55:08] <Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar pushed 1 commit to fix-security-warning: Nextcloud security warning X-Robots-Tag" HTTP header is not set as "noindex, nofollow" ([312c6cb1](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/312c6cb1d884313f23c57bd44365a4e5a2d4fa03))
[13:55:12] <Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar created new branch fix-security-warning
[13:55:22] <Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar opened [pull request #563](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/563): Fix security warning
[13:55:36] <Yunohost Git/Infra notifications> [nextcloud_ynh] @yunohost-bot pushed 1 commit to fix-security-warning: Auto-update README ([7ea580c0](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/7ea580c0a0281d9f1ae81941adbfcf1e7c0892da))
[13:56:01] <Yunohost Git/Infra notifications> [nextcloud_ynh] @yunohost-bot [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/563#issuecomment-1486938666) on [issue #563](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/563) Fix security warning: Meow :cat2:
[[Test Badge](https://img.shields.io/endpoint?url=https://ci-apps-dev.yunohost.org/ci/api/job/6440/badge)](...
[13:56:49] <Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar edited [pull request #563](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/563): Fix security warning
[16:08:50] <Yunohost Git/Infra notifications> [my_webapp_ynh] @ericgaspar pushed 1 commit to v2: Update manifest.toml ([58809e2b](https://github.com/YunoHost-Apps/my_webapp_ynh/commit/58809e2b1ef802ce8b32459d79f431775e8a120b))
[16:09:06] <Yunohost Git/Infra notifications> [my_webapp_ynh] @ericgaspar pushed 3 commits to testing ([c2a3f0ec5b7b...5a0072e44c38](https://github.com/YunoHost-Apps/my_webapp_ynh/compare/c2a3f0ec5b7b...5a0072e44c38))
[16:09:10] <Yunohost Git/Infra notifications> [my_webapp_ynh/testing] Merge pull request #106 from YunoHost-Apps/testing Testing - Éric Gaspar
[16:09:14] <Yunohost Git/Infra notifications> [my_webapp_ynh/testing] Forgot to bump version in last testing - Alexandre Aubin
[16:09:16] <Yunohost Git/Infra notifications> [my_webapp_ynh/testing] Auto-update README - yunohost-bot
[16:09:44] <Yunohost Git/Infra notifications> [my_webapp_ynh] @ericgaspar pushed 4 commits to v2 ([58809e2b1ef8...1c3d35bf2b0c](https://github.com/YunoHost-Apps/my_webapp_ynh/compare/58809e2b1ef8...1c3d35bf2b0c))
[16:09:49] <Yunohost Git/Infra notifications> [my_webapp_ynh/v2] Forgot to bump version in last testing - Alexandre Aubin
[16:09:49] <Yunohost Git/Infra notifications> [my_webapp_ynh] @yunohost-bot pushed 1 commit to v2: Auto-update README ([ba620f9f](https://github.com/YunoHost-Apps/my_webapp_ynh/commit/ba620f9fd10d08087febae1eca6db8e03bd4878c))
[16:09:50] <Yunohost Git/Infra notifications> [my_webapp_ynh/v2] Auto-update README - yunohost-bot
[16:09:54] <Yunohost Git/Infra notifications> [my_webapp_ynh/v2] Merge branch testing into v2 - Éric Gaspar
[16:21:10] <Yunohost Git/Infra notifications> [my_webapp_ynh] @ericgaspar pushed 2 commits to v2 ([ba620f9fd10d...28e96e88e261](https://github.com/YunoHost-Apps/my_webapp_ynh/compare/ba620f9fd10d...28e96e88e261))
[16:21:19] <Yunohost Git/Infra notifications> [my_webapp_ynh/v2] Merge branch v2 of https://github.com/YunoHost-Apps/my_webapp_ynh into v2 - Éric Gaspar
[16:21:19] <Yunohost Git/Infra notifications> [my_webapp_ynh/v2] Update manifest.toml - Éric Gaspar
[16:22:03] <eric_G> > <@yunohostinfra:matrix.org> [my_webapp_ynh] @ericgaspar pushed 2 commits to v2 ([ba620f9fd10d...28e96e88e261](https://github.com/YunoHost-Apps/my_webapp_ynh/compare/ba620f9fd10d...28e96e88e261))

is this nonsense?
[16:27:22] <Aleks (he/him/il/lui)> hmmm this sounds okay but shouldnt we test `with_mysql` instead ?
[16:28:58] <eric_G> https://aria.im/_matrix/media/v1/download/matrix.org/aLxytEFEJxnahPAMFhlCDEHC
[16:29:07] <eric_G> mmmh, it's against my philosophie 😅
[16:29:25] <Aleks (he/him/il/lui)> 😅
[16:30:38] <eric_G> ```
[resources.apt]
packages_from_raw_bash = """
if [[ "$with_mysql" = 1 ]]
then
echo "mariadb-server"
else
echo ""
fi
"""

```
[16:31:07] <Aleks (he/him/il/lui)> 👍️
[16:31:36] <Aleks (he/him/il/lui)> but shouldnt we in fact have something regarding php ..?
[16:31:55] <Aleks (he/him/il/lui)> like uh
[16:32:06] <eric_G> php-fpm ?
[16:32:36] <Aleks (he/him/il/lui)> yeah
[16:34:07] <Aleks (he/him/il/lui)> ```
if [[ "$phpversion" != none ]]
then
echo "php${phpversion}-fpm"
fi
```
[16:35:06] <eric_G> and also:
```
if [[ "$with_mysql" = 1 ]]
then
echo "mariadb-server php${phpversion}-mysql"
else
echo ""
fi

```
[16:35:51] <Aleks (he/him/il/lui)> hmmm
[16:36:11] <Aleks (he/him/il/lui)> well nope because phpversion could be `none` while `mysql` is enabled 😅
[16:36:30] <Aleks (he/him/il/lui)> but we can add it to the `if phpversion` part
[16:38:58] <Yunohost Git/Infra notifications> [my_webapp_ynh] @ericgaspar pushed 1 commit to v2: Update manifest.toml ([bafc4037](https://github.com/YunoHost-Apps/my_webapp_ynh/commit/bafc403789ef998e267eda6f99be8dc081a82899))
[16:39:35] <Yunohost Git/Infra notifications> [my_webapp_ynh] @ericgaspar pushed 1 commit to v2: Update tests.toml ([0305cc08](https://github.com/YunoHost-Apps/my_webapp_ynh/commit/0305cc08dc27eefe298a663678bf5b8a8d5dc68a))
[16:40:35] <Aleks (he/him/il/lui)> also in fact we can probably remove the `else echo ""`, it's not useful, sorry
[16:41:15] <Yunohost Git/Infra notifications> [my_webapp_ynh] @ericgaspar pushed 1 commit to v2: Update manifest.toml ([d4fa5807](https://github.com/YunoHost-Apps/my_webapp_ynh/commit/d4fa580701c9c8dbf5f9dbdd948294343d61a5ef))
[16:42:58] <Aleks (he/him/il/lui)> 👍️
[16:43:23] <eric_G> \o/
[16:44:24] <Yunohost Git/Infra notifications> [my_webapp_ynh] @ericgaspar pushed 1 commit to v2: Update config_panel.toml ([65eb6319](https://github.com/YunoHost-Apps/my_webapp_ynh/commit/65eb6319034904825682d737b6e51938ad882c5d))
[16:46:15] <eric_G> I suppose there is no easy way to get rid of this send email to admin... https://github.com/YunoHost-Apps/my_webapp_ynh/blob/65eb6319034904825682d737b6e51938ad882c5d/scripts/install#L164-L202
[16:48:52] <Aleks (he/him/il/lui)> ogod
[16:51:23] <Aleks (he/him/il/lui)> I think I would just add a "human readable" if, like :

"If you enabled the mysql database, you can find the corresponding database credentials in `__INSTALL_DIR__/db_access.txt`"
[16:52:03] <Aleks (he/him/il/lui)> "If you enabled SFTP, ..."
[17:14:51] <Yunohost Git/Infra notifications> [apps] @alexAubin pushed 1 commit to master: Improve new autoupdate mechanism, support latest_github_commit, be able to specify custom upstream to check, run in dry ... ([13b47e62](https://github.com/YunoHost/apps/commit/13b47e62a499662b1e1605e806b8ca03034eb399))
[17:16:16] <Aleks (he/him/il/lui)> so I improved a bit the script, cf the previous discussion regarding galene. You can now write something like:

```toml
[resources.sources.main]
url = "https://github.com/jech/galene/archive/refs/tags/galene-0.6.2.tar.gz"
sha256 = "7caedc[...]"

autoupdate.strategy = "latest_github_tag"

[resources.sources.ldap]
url = "https://github.com/jech/galene-ldap/archive/33b8a930372388b2b8b506d2564516fb3eaf895a.tar.gz"
sha256 = "00247712e4[...]"

autoupdate.upstream = "https://github.com/jech/galene-ldap"
autoupdate.strategy = "latest_github_commit"
```
[17:16:39] <Aleks (he/him/il/lui)> (note the `autoupdate.upstream` and `autoupdate.strategy = "latest_github_commit"` at the bottom
[17:17:52] <Aleks (he/him/il/lui)> and you can now run the script yourself on some app folder to test if the autoupdate thing is working like you expect, with `python3 autoupdate_app_sources.py /path/to/some/app`, in which case it will run in "dry run" mode and only check for updates, without requiring any github token or whatever, and won't create a pull request
[17:18:58] <Aleks (he/him/il/lui)> ```
> python3 autoupdate_app_sources.py ./galene_ynh/
Checking main ...
Current version in manifest: 0.6.2
Newest version on upstream: 0.6.2
Version is still 0.6.2, no update required for main
Checking ldap ...
URL is still up to date for asset ldap
```
[17:23:49] <Yunohost Git/Infra notifications> [my_webapp_ynh] @ericgaspar pushed 1 commit to v2: Update DESCRIPTION.md ([8eec20a4](https://github.com/YunoHost-Apps/my_webapp_ynh/commit/8eec20a4edcbf30e1c99cb7f63c86ca9715d7e82))
[17:23:53] <Yunohost Git/Infra notifications> [my_webapp_ynh] @yunohost-bot pushed 1 commit to v2: Auto-update README ([b136cc2b](https://github.com/YunoHost-Apps/my_webapp_ynh/commit/b136cc2b3d214f4a405a9f758d44fbeeeb94996a))
[17:24:45] <Yunohost Git/Infra notifications> [my_webapp_ynh] @ericgaspar pushed 2 commits to v2 ([b136cc2b3d21...f78dfe4da8f5](https://github.com/YunoHost-Apps/my_webapp_ynh/compare/b136cc2b3d21...f78dfe4da8f5))
[17:24:49] <Yunohost Git/Infra notifications> [my_webapp_ynh/v2] Update DESCRIPTION_fr.md - Éric Gaspar
[17:24:52] <Yunohost Git/Infra notifications> [my_webapp_ynh] @yunohost-bot pushed 1 commit to v2: Auto-update README ([6b39fca6](https://github.com/YunoHost-Apps/my_webapp_ynh/commit/6b39fca6873d7e1c02c65f49299c67907bd1e8fe))
[17:24:56] <Yunohost Git/Infra notifications> [my_webapp_ynh/v2] Merge branch v2 of https://github.com/YunoHost-Apps/my_webapp_ynh into v2 - Éric Gaspar
[17:28:30] <Yunohost Git/Infra notifications> [apps] @alexAubin pushed 1 commit to master: New source autoupdate: black ([2939e876](https://github.com/YunoHost/apps/commit/2939e876ff1f4afffaa565d12621ef64eaa975a1))
[17:29:00] <Yunohost Git/Infra notifications> [apps] @alexAubin deleted branch improve-auto-update-mechanism
[18:02:22] <Yunohost Git/Infra notifications> [nextcloud_ynh] @JimboJoe commented [pull request #563](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/563#pullrequestreview-1361653847) Fix security warning: LGTM (not tested) 👍
[18:02:32] <Yunohost Git/Infra notifications> [nextcloud_ynh] @JimboJoe approved [pull request #563](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/563#pullrequestreview-1361654206) Fix security warning
[18:10:23] <Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar merged [pull request #563](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/563): Fix security warning
[18:10:24] <Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar pushed 4 commits to master ([627a9ad27a3c...3e3147a32b2a](https://github.com/YunoHost-Apps/nextcloud_ynh/compare/627a9ad27a3c...3e3147a32b2a))
[18:10:29] <Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar deleted branch fix-security-warning
[18:10:31] <Yunohost Git/Infra notifications> [nextcloud_ynh/master] Update manifest.json - Éric Gaspar
[18:10:33] <Yunohost Git/Infra notifications> [nextcloud_ynh/master] Auto-update README - yunohost-bot
[18:10:37] <Yunohost Git/Infra notifications> [nextcloud_ynh/master] Merge pull request #563 from YunoHost-Apps/fix-security-warning - Éric Gaspar
[19:58:50] <Yunohost Git/Infra notifications> [my_webapp_ynh] @ericgaspar pushed 1 commit to v2: Update tests.toml ([60bc8fab](https://github.com/YunoHost-Apps/my_webapp_ynh/commit/60bc8fab646718f5d6671e33cfe745d05d0431f8))
[19:59:19] <Yunohost Git/Infra notifications> [my_webapp_ynh] @ericgaspar pushed 1 commit to v2: Update install ([94761c15](https://github.com/YunoHost-Apps/my_webapp_ynh/commit/94761c15d20c6827d393e26b0bc4ab4aa0e37010))
[19:59:37] <Yunohost Git/Infra notifications> [my_webapp_ynh] @ericgaspar [commented](https://github.com/YunoHost-Apps/my_webapp_ynh/pull/107#issuecomment-1487517795) on [issue #107](https://github.com/YunoHost-Apps/my_webapp_ynh/pull/107) V2: testme
[19:59:38] <Yunohost Git/Infra notifications> [my_webapp_ynh] @yunohost-bot [commented](https://github.com/YunoHost-Apps/my_webapp_ynh/pull/107#issuecomment-1487517834) on [issue #107](https://github.com/YunoHost-Apps/my_webapp_ynh/pull/107) V2: Meow :cat2:
[[Test Badge](https://img.shields.io/endpoint?url=https://ci-apps-dev.yunohost.org/ci/api/job/6443/badge)](...