[00:02:40]
<Paprika> Just a heads up - got it all running. Also set up matrix-commander within yunorunner too for notifications in Matrix.
[00:35:54]
<m606> Hello, bash question - can I use a YNH helper like this `find "$install_dir/" -maxdepth 1 -not -path './dist' -exec ynh_safe_rm {} \; #keep only dist folder` ?
Currently, install fails with several errors like `WARNING - find: ‘ynh_safe_rm’: No such file or directory`
[00:36:04]
<m606> Hello, bash question - can I use a YNH helper like this `find "$install_dir/" -maxdepth 1 -not -path './dist' -exec ynh_safe_rm {} \;` ?
Currently, install fails with several errors like `WARNING - find: ‘ynh_safe_rm’: No such file or directory`
[00:36:41]
<Aleks (he/him/il/lui)> probably not because `find` wants a real command not a function
[00:37:21]
<Aleks (he/him/il/lui)> only 95% sure though but i would be surprised that it doesnt run the stuff in a subshell
[00:38:49]
<m606> ok thanks. So I'd better stick with `rm -r` here ?
[00:39:35]
<m606> (i basically want to keep only given folder in the install folder)
[00:39:36]
<Aleks (he/him/il/lui)> what's the context, you want to remove eveything in the install dir except the "dist" folder ?
[00:40:07]
<Aleks (he/him/il/lui)> during what kind of operation ? upgrade ? why not use the `--keep` option for ynh_setup_source ?
[00:40:17]
<m606> install & upgrade
[00:40:57]
<m606> ```
--keep=: Space-separated list of files/folders that will be backup/restored in $dest_dir, such as a config file you don't want to overwrite. For example 'conf.json secrets.json logs' (no trailing / for folders)
```
[00:44:23]
<m606> i may be confused on what `--keep` is doing, but i understand it as if I wanted to keep initially installed files during upgrade process.
Here whether at install or upgrade, I want to unzip the upstream release file, which contains several folders (including `dist`), and I want `$install_dir` to contain only what's in `dist`.
```
find "$install_dir/" -maxdepth 1 -not -path './dist' -exec ynh_safe_rm {} \; #keep only dist folder
mv "$install_dir/dist/*" "$install_dir"
rmdir dist
```
[00:44:48]
<Aleks (he/him/il/lui)> ah yes i see hmpf
[00:45:40]
<Aleks (he/him/il/lui)> i think i saw a couple apps with a similar story, i think they end up doing the `ynh_setup_source` in a temporary location then moving the `dist` folder to `$install_dir`
[00:48:28]
<m606> oh, like `mktemp -d`?
but what would be the point here ? do `rm -r` in a safer way ?
[00:52:06]
<m606> that makes me think of an unrelated question about `ynh_setup_source` and `--full_replace=: Remove previous sources before installing new sources (can be 1 or 0, default to 0)`: if in upgrade script we use ynh_setup_source without specifying `--full_replace` param, does that mean that only files of different names will be copied from the new release to the `$install_dir` ?
[00:52:33]
<Aleks (he/him/il/lui)> just having a single `mv` (and then yes `rm` the temp dir) rather than a complex "delete everything except this specific folder"
[00:53:00]
<m606> > <@Alekswag:matrix.org> just having a single `mv` (and then yes `rm` the temp dir) rather than a complex "delete everything except this specific folder"
ah yes )
[00:53:12]
<Aleks (he/him/il/lui)> >does that mean that only files of different names will be copied from the new release to the $install_dir ?
no it means everything is kept as is, and then files are extracted "on top" of it
[00:53:27]
<Aleks (he/him/il/lui)> so if they already exist, they are overwritten
[00:53:37]
<Aleks (he/him/il/lui)> but if they're not in the archive, they're kept
[00:53:39]
<m606> got it thanks!
[00:59:19]
<m606> ```
temp_dir=$(mktemp -d)
ynh_setup_source --dest_dir="$temp_dir"
mv "$temp_dir/dist/*" "$install_dir" #keep only the content of `dist` folder
ynh_safe_rm $temp_dir
```
[01:20:08]
<m606> in case of a script with several sources (release, and dependencies for the release, all of them to be put in $install_dir), do you have in mind whether `ynh_setup_source --dest_dir="$install_dir" --source_id="ynh_build" --full-replace=1` will remove all $install_dir or that somehow the core has in memory what files belong to 'ynh_build' source_id ?
[01:20:57]
<m606> in case of a script with several sources (release, and dependencies for the release, all of them to be put in $install_dir), do you have in mind whether `ynh_setup_source --dest_dir="$install_dir" --source_id="ynh_build" --full-replace=1` will remove all $install_dir (what I want here) or that somehow the core has in memory what files belong to 'ynh_build' source_id ?
[01:44:13]
<Aleks (he/him/il/lui)> uuuh it will remove the `dest_dir` i suppose ?
[01:45:07]
<m606> yes that what's I was checking right now in the code
[01:45:21]
<m606> ```
if [ "$full_replace" -eq 1 ]; then
ynh_safe_rm "$dest_dir"
fi
```
[01:47:48]
<m606> last thing I would like to clarify now I understand better full_replace is: `ynh_setup_source --dest_dir="$install_dir" --full_replace`
Is that equivalent to --full_replace=0 as I guess, or would it trigget above snippet ?
[01:48:27]
<m606> we have `full_replace="${full_replace:-0}"` https://github.com/YunoHost/yunohost/blob/e8c175231e78683c98e84f7fd637d234fb31c2cc/helpers/helpers.v2.1.d/sources#L72C1-L72C38
[01:54:23]
<Aleks (he/him/il/lui)> hmpf so in helpers v2.0 the syntax is `--full_replace=1` because of some stupid typo long ago (it's a boolean flag so just having `--full-replace` should be enough like we do have for a lot of other options in helpers)
[01:54:35]
<Aleks (he/him/il/lui)> and in helpers 2.1 it was fixed so it's now just `--full-replace`
[02:02:04]
<Yunohost Git/Infra notifications> yalh76 edited repository invoiceninja5_ynh: self-hosted invoicing platform to create & email invoices, track payments and expenses, and time billable tasks & projects for clients. https://github.com/YunoHost-Apps/invoiceninja5_ynh
[02:02:41]
<Yunohost Git/Infra notifications> yalh76 edited repository airsonic_ynh: Web-based media streamer https://github.com/YunoHost-Apps/airsonic_ynh
[02:03:06]
<Yunohost Git/Infra notifications> yalh76 edited repository concrete5_ynh: modern CMS https://github.com/YunoHost-Apps/concrete5_ynh
[11:38:20]
<Yunohost Git/Infra notifications> [apps] Salamandar created new branch fix_schema
[11:38:21]
<Yunohost Git/Infra notifications> [apps] Salamandar pushed 1 commit to fix_schema: manifest schema: resources.apt.extras.X support packages_from_raw_bash too ([73cc6062](https://github.com/YunoHost/apps/commit/73cc60620c04036bcf221673b74b0eb484555b24))
[11:38:32]
<Yunohost Git/Infra notifications> [apps] Salamandar opened [pull request #2765](https://github.com/YunoHost/apps/pull/2765): manifest schema: resources.apt.extras.X support packages_from_raw_bash too
[11:38:38]
<Yunohost Git/Infra notifications> [apps] Salamandar merged [pull request #2765](https://github.com/YunoHost/apps/pull/2765): manifest schema: resources.apt.extras.X support packages_from_raw_bash too
[11:38:39]
<Yunohost Git/Infra notifications> [apps] Salamandar pushed 1 commit to master: manifest schema: resources.apt.extras.X support packages_from_raw_bash too ([750e93b0](https://github.com/YunoHost/apps/commit/750e93b02ea02582417697f3ec257b21ecf93647))
[11:38:40]
<Yunohost Git/Infra notifications> [apps] Salamandar deleted branch fix_schema
[11:46:00]
<Yunohost Git/Infra notifications> [apps] Salamandar created new branch fix_schema
[11:46:00]
<Yunohost Git/Infra notifications> [apps] Salamandar pushed 2 commits to fix_schema ([73cc60620c04^...e2228d3449e3](https://github.com/YunoHost/apps/compare/73cc60620c04^...e2228d3449e3))
[11:46:00]
<Yunohost Git/Infra notifications> [apps/fix_schema] manifest schema: resources.apt.extras.X support packages_from_raw_bash too - Félix Piédallu
[11:46:01]
<Yunohost Git/Infra notifications> [apps/fix_schema] manifest schema: apt requires either packages or packages_from_raw_bash - Félix Piédallu
[11:46:08]
<Yunohost Git/Infra notifications> [apps] Salamandar opened [pull request #2766](https://github.com/YunoHost/apps/pull/2766): manifest schema: resources.apt.extras.X support packages_from_raw_bas…
[11:46:20]
<Yunohost Git/Infra notifications> [apps] Salamandar pushed 1 commit to fix_schema: manifest schema: apt requires either packages or packages_from_raw_bash ([74734a3c](https://github.com/YunoHost/apps/commit/74734a3c4591e76dc1e2a3d7c29cc40fbb9b952a))
[11:46:26]
<Yunohost Git/Infra notifications> [apps] Salamandar merged [pull request #2766](https://github.com/YunoHost/apps/pull/2766): manifest schema: resources.apt.extras.X support packages_from_raw_bas…
[11:46:26]
<Yunohost Git/Infra notifications> [apps] Salamandar pushed 1 commit to master: manifest schema: apt requires either packages or packages_from_raw_bash ([f9274918](https://github.com/YunoHost/apps/commit/f9274918609305d34d1c9a706410e8eb859be0d5))
[11:46:28]
<Yunohost Git/Infra notifications> [apps] Salamandar deleted branch fix_schema
[13:27:18]
<limezy> Thanks I will try
[15:35:32]
<Yunohost Git/Infra notifications> [mastodon_ynh] tio-trom [commented](https://github.com/YunoHost-Apps/mastodon_ynh/issues/480#issuecomment-2580585442) on [issue #480](https://github.com/YunoHost-Apps/mastodon_ynh/issues/480) upgrade to 4.3.2~ynh1 is failing: Worked all fine for me. Yours is an issue with this 2025-01-06 13:29:21,696: DEBUG - 2025-01-06 13:29:21,696: DEBU...
[15:42:32]
<Yunohost Git/Infra notifications> [mastodon_ynh] nizarus [commented](https://github.com/YunoHost-Apps/mastodon_ynh/issues/480#issuecomment-2580601674) on [issue #480](https://github.com/YunoHost-Apps/mastodon_ynh/issues/480) upgrade to 4.3.2~ynh1 is failing: @tio-trom, On your instance, do you have users with 2FA enabled ? Not sure, but perhaps the issue is related to this "‼️...
[17:00:22]
<Yunohost Git/Infra notifications> [mastodon_ynh] tio-trom [commented](https://github.com/YunoHost-Apps/mastodon_ynh/issues/480#issuecomment-2580822708) on [issue #480](https://github.com/YunoHost-Apps/mastodon_ynh/issues/480) upgrade to 4.3.2~ynh1 is failing: > @tio-trom, On your instance, do you have users with 2FA enabled ? Not sure, but perhaps the issue is related to this "...
[17:45:59]
<Salamandar> https://aria.im/_bifrost/v1/media/download/AfQSVzcWfxPY681LVdBpA9NN8unwt16o6SwKfU1u40sCB9lYbnjkLH_gGEZ1dvfUDQaQu-xh9bkzkzys_tpp6NdCeUUVKpowAG1hdHJpeC5vcmcvUnduV2tITm5iZmFKV1Fva1hPUHBFZ0VG
[17:46:10]
<Salamandar> Tomorrow's catalog level update will be reeeaaaaaal nice
[17:46:31]
<orhtej2> > <@Salamandar:matrix.org> Tomorrow's catalog level update will be reeeaaaaaal nice
go helper fix is up?
[17:46:57]
<orhtej2> Dendrite is all on go
[17:47:09]
<Salamandar> That, but not only. I've worked hard at fixing some apps, and some other people too (You, i think, for mygpo at least)
[18:11:53]
<m606> hello! is there a way to ask YNH CI to only make, say... upgrade section of the usual tests?
[18:12:32]
<m606> like when you're trying to debug something in the upgrade script only
[18:21:12]
<orhtej2> > <@m606:matrix.org> hello! is there a way to ask YNH CI to only make, say... upgrade section of the usual tests?
No, such hackery will fail because of no passing install test
[18:21:48]
<orhtej2> you need at least install on root
[18:22:30]
<m606> oh but that would still be interesting, how can I skill all the others ?
[18:26:22]
<orhtej2> > <@m606:matrix.org> oh but that would still be interesting, how can I skill all the others ?
https://github.com/YunoHost/example_ynh/blob/1bdcd685cc6726307e2f38bdbf9a597a3be91b1e/tests.toml#L35
[18:42:22]
<m606> @Salamandar:matrix.org Would you agree to support custom path install again with Cinny ? https://github.com/YunoHost-Apps/cinny_ynh/pull/75
[18:51:21]
<orhtej2> > <@m606:matrix.org> @Salamandar:matrix.org Would you agree to support custom path install again with Cinny ? https://github.com/YunoHost-Apps/cinny_ynh/pull/75
This thing has absolute links hardcoded, it would require a rebuild
[18:51:31]
<orhtej2> Unless they fixed that
[19:00:59]
<Salamandar> Yeah he suggests pre-building a version of Cinny with path = `__YNH_a_token_we_can_grep_while_installing__`
[19:01:40]
<Salamandar> idk, on one hand, why not
[19:02:03]
<Salamandar> but on the other hand… grepping a built artifact… what if the build process measures the path variable string length ? then it's all broken
[19:57:45]
<m606> > Unless they fixed that
not an option currently - https://github.com/cinnyapp/cinny/issues/1843#issuecomment-2277998443
[19:59:32]
<m606> > <@Salamandar:matrix.org> but on the other hand… grepping a built artifact… what if the build process measures the path variable string length ? then it's all broken
why would they do so for instance?
[20:02:10]
<m606> In case you are willing to consider that direction and if I get write access to the repo I can set a workflow + update PR so that we can check that point
[20:35:28]
<Yunohost Git/Infra notifications> [nextcloud_ynh] stemy2 closed [issue #775](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/775): Environment not properly prepared
[20:35:29]
<Yunohost Git/Infra notifications> [nextcloud_ynh] stemy2 [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/775#issuecomment-2581198886) on [issue #775](https://github.com/YunoHost-Apps/nextcloud_ynh/issues/775) Environment not properly prepared: Closed because my nextcloud is now on a fresh install.
[21:13:32]
<Salamandar> tvheadend fails CI because it uses basic auth… but the CI test just sees a 401
[21:17:06]
<Salamandar> So how could I make it pass the CI ?
[21:18:55]
<Salamandar> (i manually confirmed it does work)
[21:49:37]
<Aleks (he/him/il/lui)> maybe using the stuff that act as logged in ? https://github.com/YunoHost-Apps/nextcloud_ynh/blob/master/tests.toml#L27
[21:54:08]
<orhtej2> > <@Alekswag:matrix.org> maybe using the stuff that act as logged in ? https://github.com/YunoHost-Apps/nextcloud_ynh/blob/master/tests.toml#L27
Tried that somewhere else, didn't work
[22:27:30]
<Salamandar> Ah, might try that on tvheadend
[22:47:44]
<Yunohost Git/Infra notifications> [apps] tituspijean changes_requested [pull request #2763](https://github.com/YunoHost/apps/pull/2763#pullrequestreview-2541094962) AppFlowy
[22:47:45]
<Yunohost Git/Infra notifications> [apps] tituspijean [commented](https://github.com/YunoHost/apps/pull/2763#discussion_r1909554849) on pull request #2763 AppFlowy: Lets add an anti-feature, since it relies on OpenAI.
[22:51:06]
<Yunohost Git/Infra notifications> [apps] tituspijean [commented](https://github.com/YunoHost/apps/pull/2763#discussion_r1909557287) on pull request #2763 AppFlowy: suggestion antifeatures = [ "non-free-network" ] category = "publishing"
[22:52:01]
<Yunohost Git/Infra notifications> [apps] tituspijean deleted a [comment](https://github.com/YunoHost/apps/pull/2763#discussion_r1909554849) on pull request #2763 AppFlowy: Lets add an anti-feature, since it relies on OpenAI.
[22:52:35]
<Yunohost Git/Infra notifications> [apps] tituspijean [commented](https://github.com/YunoHost/apps/pull/2763#discussion_r1909558309) on pull request #2763 AppFlowy: Antifeature since it relies on OpenAI, even optionally.
[23:05:39]
<Yunohost Git/Infra notifications> [apps] yalh76 pushed 1 commit to appflowy: Update apps.toml Co-authored-by: tituspijean <titus+yunohost@pijean.ovh> ([e5d5d203](https://github.com/YunoHost/apps/commit/e5d5d2039992e1ac531fbfdf80e329bcf3846413))
[23:13:36]
<Yunohost Git/Infra notifications> [joplin_ynh] yunohost-bot opened [pull request #72](https://github.com/YunoHost-Apps/joplin_ynh/pull/72): Upgrade to v3.2.9
[23:20:51]
<Yunohost Git/Infra notifications> [syncthing_ynh] yunohost-bot opened [pull request #198](https://github.com/YunoHost-Apps/syncthing_ynh/pull/198): Upgrade to v1.29.1
[23:25:43]
<Yunohost Git/Infra notifications> Autoupdater just ran, here are the results:
- 16 pending update PRs
- 14 new apps PRs
- 2 failed apps updates: flood, stremio
See the full log here: https://paste.yunohost.org/raw/dapovupoza
[23:28:54]
<Salamandar> indeed it doesnt work
[23:54:51]
<Aleks (he/him/il/lui)> loooooogs
[23:54:52]
<Salamandar> ?
[23:54:52]
<Salamandar> https://github.com/YunoHost-Apps/tvheadend_ynh/pull/22
[23:54:53]
<Salamandar> > 401 Unauthorized
[23:54:54]
<Aleks (he/him/il/lui)> the stuff is not in a `[default.curl_tests]` block :|
[23:54:54]
<Aleks (he/him/il/lui)> not sure why it still throws a 401 though
[23:54:54]
<Aleks (he/him/il/lui)> hmyup
[23:54:54]
<orhtej2> Like this, right? https://github.com/YunoHost-Apps/z-push_ynh/pull/102/files
[23:54:58]
<Aleks (he/him/il/lui)> maybe the 301 redirection is fucking things up and `/Microsoft-Server-ActiveSync/` should be tried directly but i kinda doubt it
[23:54:58]
<Aleks (he/him/il/lui)> maybe the 301 redirection is fucking things up and `/Microsoft-Server-ActiveSync/` should be tried directly but i kinda doubt it
[23:54:58]
<Aleks (he/him/il/lui)> does it works with just the SSO locally in your case ?
[23:54:59]
<orhtej2> > <@Alekswag:matrix.org> maybe the 301 redirection is fucking things up and `/Microsoft-Server-ActiveSync/` should be tried directly but i kinda doubt it
I poked it with curl locally, never got it to work
[23:54:59]
<orhtej2> No idea what makes it tick allright in a browser