Friday, June 30, 2023
apps@conference.yunohost.org
June
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
   
             

[07:12:41] <nicofrand> Hi! In the kresusapp we install rust for the cryptography python module needed by some woob banks modules and that seems to fail on raspbian. I don't want to remove the rust installation from the package but would there be a yunohost helperlike `ynh_exec_warn_less` to ignore errors in `ynh_exec_as`? In different words, I'd like it to fail silently and keep the app package installation
[07:19:11] <nicofrand> `ynh_abort_if_errors` is used but it would be nice to keep it and only try/catch the rust installation
[07:19:53] <nicofrand> also it seems to be enabled automatically in format 2+
[08:47:32] <Aleks (he/him/il/lui)> > Hi! In the kresusapp we install rust for the cryptography python module needed by some woob banks modules and that seems to fail on raspbian. I don't want to remove the rust installation from the package but would there be a yunohost helperlike `ynh_exec_warn_less` to ignore errors in `ynh_exec_as`? In different words, I'd like it to fail silently and keep the app package installation

not sure to understand, cant you just run `ynh_exec_warn_less ynh_exec_as $app the actual command` ?
[08:48:30] <Yunohost Git/Infra notifications> [gitlab_ynh] @kay0u pushed 1 commit to testing: 16.1.1, add bookworm support ([0c24304d](https://github.com/YunoHost-Apps/gitlab_ynh/commit/0c24304de8480793bcb2c52e1183b46c9ca5c9cf))
[08:48:32] <Yunohost Git/Infra notifications> [gitlab_ynh] @yunohost-bot pushed 1 commit to testing: Auto-update README ([7b4758b3](https://github.com/YunoHost-Apps/gitlab_ynh/commit/7b4758b323155c5b31287250dbe696cdf295a52c))
[08:48:51] <Yunohost Git/Infra notifications> [gitlab_ynh] @kay0u opened [pull request #223](https://github.com/YunoHost-Apps/gitlab_ynh/pull/223): Testing
[08:48:57] <Yunohost Git/Infra notifications> [gitlab_ynh] @kay0u pushed 3 commits to master ([df1e676766ff...7b5f5ced054b](https://github.com/YunoHost-Apps/gitlab_ynh/compare/df1e676766ff...7b5f5ced054b))
[08:48:57] <Yunohost Git/Infra notifications> [gitlab_ynh] @kay0u merged [pull request #223](https://github.com/YunoHost-Apps/gitlab_ynh/pull/223): Testing
[08:49:00] <Yunohost Git/Infra notifications> [gitlab_ynh/master] 16.1.1, add bookworm support - Kay0u
[08:49:04] <Yunohost Git/Infra notifications> [gitlab_ynh/master] Auto-update README - yunohost-bot
[08:49:07] <Yunohost Git/Infra notifications> [gitlab_ynh/master] Merge pull request #223 from YunoHost-Apps/testing Testing - Kayou
[08:51:59] <tituspijean[m]> > <@Alekswag:matrix.org> not sure to understand, cant you just run `ynh_exec_warn_less ynh_exec_as $app the actual command` ?

It might be possible that if the cryptography module cannot be installed then it returns an error.
If so, a good ol' `set +eu` then `set -eu` to revert back to normal behavior might solve the issue?
[08:52:00] <tituspijean[m]> We have `ynh_exec_warn` or even `ynh_exec_err`, why not `ynh_exec_success` 😅
[08:52:00] <nicofrand> > <@Alekswag:matrix.org> not sure to understand, cant you just run `ynh_exec_warn_less ynh_exec_as $app the actual command` ?

I thought this would hide the error but still would fail ?
[09:08:49] <tituspijean[m]> Actually, why don't we try fix the issue with cryptography? Is this your issue? https://stackoverflow.com/questions/69743936/problem-installing-cryptography-on-raspberry-pi
[09:08:50] <nicofrand> > <@titus:pijean.ovh> Actually, why don't we try fix the issue with cryptography? Is this your issue? https://stackoverflow.com/questions/69743936/problem-installing-cryptography-on-raspberry-pi

something like this yeah (https://framagit.org/kresusapp/kresus/-/issues/1214)
[09:08:50] <tituspijean[m]> Maybe this (if the error is the same, please share it): https://cryptography.io/en/latest/faq/#installing-cryptography-fails-with-error-can-not-find-rust-compiler
We'd basically need to make sure Rust in the $PATH
[09:08:50] <tituspijean[m]> (the proposed solution would need to be adapted I guess)
[09:09:34] <nicofrand> > <@titus:pijean.ovh> Maybe this (if the error is the same, please share it): https://cryptography.io/en/latest/faq/#installing-cryptography-fails-with-error-can-not-find-rust-compiler
> We'd basically need to make sure Rust in the $PATH

yes https://github.com/YunoHost-Apps/kresus_ynh/blob/master/scripts/install#L138
[09:09:55] <nicofrand> hm, I actually don't know where that would be installed though
[09:10:20] <nicofrand> it uses the `kresus` user but as far as I remember the user created by YNH does not have its own directory right?
[09:10:48] <nicofrand> but we use `CARGO_HOME="$final_path"/.cargo` to set it in the final path…
[09:42:35] <nicofrand> probably comes from https://github.com/rust-lang/rustup/issues/3342
[17:01:18] <Yunohost Git/Infra notifications> [apps] @yunohost-bot created new branch update_app_levels
[17:01:18] <Yunohost Git/Infra notifications> [apps] @yunohost-bot opened [pull request #1689](https://github.com/YunoHost/apps/pull/1689): Update app levels according to CI results
[17:01:18] <Yunohost Git/Infra notifications> [apps] @yunohost-bot pushed 1 commit to update_app_levels: Update app levels according to CI results ([bb073536](https://github.com/YunoHost/apps/commit/bb073536bcade416c00d0f113fc9b295a78eb1f6))
[23:05:12] <Yunohost Git/Infra notifications> App mailman failed all tests in job [#16757](https://ci-apps.yunohost.org/ci/job/16757) :(