Monday, September 09, 2024
apps@conference.yunohost.org
September
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
           

[02:07:18] <Yunohost Git/Infra notifications> App borgwarehouse rises from level 7 to 8 in job [#30371](https://ci-apps.yunohost.org/ci/job/30371) !
[02:07:26] <Yunohost Git/Infra notifications> Autoupdater just ran, here are the results:

- 33 pending update PRs
- 18 new apps PRs
- 7 failed apps updates: anarchism, epicyon, fluffychat, mlmmj, mobilizon, onlyoffice, pleroma

See the full log here: http://paste.yunohost.org/raw/orewutoxun
[08:39:54] <Yunohost Git/Infra notifications> App jellyfin-vue rises from level 7 to 8 in job [#30375](https://ci-apps.yunohost.org/ci/job/30375) !
[08:49:30] <Yunohost Git/Infra notifications> App joomla rises from level 6 to 7 in job [#30376](https://ci-apps.yunohost.org/ci/job/30376) !
[10:07:11] <Yunohost Git/Infra notifications> App django-for-runners rises from level 0 to 6 in job [#30380](https://ci-apps.yunohost.org/ci/job/30380) !
[11:33:56] <eric_G> I don't understand this warning `2089 WARNING ./remove: line 27: mysqlshow: command not found` -> https://ci-apps-bookworm-dev.yunohost.org/ci/job/2841
[11:34:40] <eric_G> cf the code -> https://github.com/YunoHost-Apps/mattermost_ynh/blob/d9d2970e8c80fced023a7407231d46562439a4e1/scripts/remove#L27-L31
[11:43:09] <eric_G> ah... maybe because mysql is not installed 🫠
[11:56:38] <orhtej2> > <@ericg:matrix.org> cf the code -> https://github.com/YunoHost-Apps/mattermost_ynh/blob/d9d2970e8c80fced023a7407231d46562439a4e1/scripts/remove#L27-L31

nice catch 22 I guess, does the upgrade work ? will mariadb not be deprovisioned prior to running `upgrade`?
[12:17:21] <Yunohost Git/Infra notifications> [package_check] alexAubin pushed 1 commit to master: Allow to define TEST_CONTEXT from the env passed to the script, such that yunorunner may now about this var too and inc... ([2e8af94d](https://github.com/YunoHost/package_check/commit/2e8af94d8258699890fefecca59376bc8db7b1e7))
[12:43:06] <Yunohost Git/Infra notifications> [apps] Salamandar created new branch alert_local_changes
[12:43:06] <Yunohost Git/Infra notifications> [apps] Salamandar pushed 1 commit to alert_local_changes: Send message to maintainers chan if local changes are present ([2d4daef3](https://github.com/YunoHost/apps/commit/2d4daef37302a0de1d5a60ceb4b6970a0faf2002))
[12:43:26] <Yunohost Git/Infra notifications> [apps] Salamandar opened [pull request #2572](https://github.com/YunoHost/apps/pull/2572): Send message to maintainers chan if local changes are present
[12:43:36] <Yunohost Git/Infra notifications> [apps] Salamandar pushed 1 commit to master: Send message to maintainers chan if local changes are present ([1e7cbf91](https://github.com/YunoHost/apps/commit/1e7cbf91641378ed51f13159e6d57dad122941be))
[12:43:37] <Yunohost Git/Infra notifications> [apps] Salamandar deleted branch alert_local_changes
[12:43:41] <Yunohost Git/Infra notifications> [apps] Salamandar merged [pull request #2572](https://github.com/YunoHost/apps/pull/2572): Send message to maintainers chan if local changes are present
[13:04:23] <Émy - OniriCorpe> > <@yunohostinfra:matrix.org> [apps] Salamandar pushed 1 commit to alert_local_changes: Send message to maintainers chan if local changes are present ([2d4daef3](https://github.com/YunoHost/apps/commit/2d4daef37302a0de1d5a60ceb4b6970a0faf2002))

Oh thanks
[13:55:42] <Yunohost Git/Infra notifications> Job [#30383](https://ci-apps.yunohost.org/ci/job/30383) for kiwix failed miserably :(
[15:51:21] <eric_G> question à la con:
https://github.com/YunoHost-Apps/piwigo_ynh/blob/93b16abd3e4e9585c146d8818037f3b9175b3fc6/scripts/upgrade#L29-L31
Line 31 `full_replace` is likely to erase what was installed on line 30 right?
[15:54:10] <eric_G> and line 30 we should not need `--full_replace` as `install_dir` was cleaned on line 29?
[16:03:57] <Salamandar> gniiii but why https://docs.olivetin.app/upgrade-notes.html#_navigation_change_subpaths_no_longer_supported
[16:04:34] <Salamandar> > <@ericg:matrix.org> question à la con:
> https://github.com/YunoHost-Apps/piwigo_ynh/blob/93b16abd3e4e9585c146d8818037f3b9175b3fc6/scripts/upgrade#L29-L31
> Line 31 `full_replace` is likely to erase what was installed on line 30 right?

yeah we should swap the two lines indeed
[16:04:51] <Salamandar> > <@ericg:matrix.org> and line 30 we should not need `--full_replace` as `install_dir` was cleaned on line 29?

agreed
[16:13:16] <Yunohost Git/Infra notifications> App cusdis failed all tests in job [#30384](https://ci-apps.yunohost.org/ci/job/30384) :(
[16:37:17] <Yunohost Git/Infra notifications> App pgweb rises from level 0 to 7 in job [#30387](https://ci-apps.yunohost.org/ci/job/30387) !
[16:45:57] <Yunohost Git/Infra notifications> App redlib rises from level 0 to 7 in job [#30388](https://ci-apps.yunohost.org/ci/job/30388) !
[16:55:36] <Yunohost Git/Infra notifications> App screego rises from level 0 to 7 in job [#30389](https://ci-apps.yunohost.org/ci/job/30389) !
[17:04:47] <Yunohost Git/Infra notifications> App zoraxy rises from level 0 to 1 in job [#30390](https://ci-apps.yunohost.org/ci/job/30390) !
[17:37:11] <Yunohost Git/Infra notifications> App cusdis rises from level 0 to 1 in job [#30391](https://ci-apps.yunohost.org/ci/job/30391) !
[18:26:15] <tituspijean> https://github.com/YunoHost/apps/pull/2574

uwu

(We didn't get the notification here I think btw)
[19:09:49] <Yunohost Git/Infra notifications> App zoraxy rises from level 0 to 1 in job [#30393](https://ci-apps.yunohost.org/ci/job/30393) !
[19:19:32] <Salamandar> > <@titus:pijean.ovh> https://github.com/YunoHost/apps/pull/2574
>
> uwu
>
> (We didn't get the notification here I think btw)

yeh and wtf :D
[19:21:13] <m606> Hello, 2 questions when packaging an app:
1. should I run readme_generator manually to generate the initial readme, or will it be automatically generated by the YNH infra at some point when submitting the app to the catalog ?
2. if autoupdate strategy is set to "latest_github_commit", source URL in the manifest should be GITHUB_REPO_URL/archive/refs/heads/main.zip, right?
[19:23:08] <Salamandar> It will be run automatically :)
[19:23:19] <Salamandar> But you can run it manually if you want
[19:24:10] <Salamandar> > if autoupdate strategy is set to "latestgithubcommit", source URL in the manifest should be GITHUBREPOURL/archive/refs/heads/main.zip, right?

More or less. We point to a fixed commit (not main) to ensure we can test new commits before final users use them
[19:24:52] <Salamandar> See https://github.com/YunoHost-Apps/jellyfin-vue_ynh/blob/dc83648dc56338a2f0a17a7228345dbc0e22063e/manifest.toml#L48
[19:27:04] <m606> ok thanks
[20:09:06] <orhtej2> > <@titus:pijean.ovh> https://github.com/YunoHost/apps/pull/2574
>
> uwu
>
> (We didn't get the notification here I think btw)

time for hard forking ynh as yuwunohost based on uwuntu?
[20:32:27] <Émy - OniriCorpe> I prefer debuwuan
[20:33:48] <Émy - OniriCorpe> Yuwunowost