Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Dev infra #8438

Open
wants to merge 14 commits into
base: master
Choose a base branch
from
Open

Dev infra #8438

wants to merge 14 commits into from

Conversation

tuta-sudipg
Copy link
Contributor

No description provided.

@tuta-sudipg tuta-sudipg force-pushed the dev-infra branch 3 times, most recently from 9011447 to 5abab26 Compare January 30, 2025 15:12
@ganthern ganthern force-pushed the dev-infra branch 2 times, most recently from a90f301 to 926eb4f Compare January 30, 2025 16:32
"uniffi-bindgen",
"generate",
"--library",
"${layout.buildDirectory.asFile.get()}/rustJniLibs/android/${dir}/libtutasdk.so",
Copy link
Contributor

@ganthern ganthern Jan 30, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

these paths are still wrong, probably because of the changed target directory. https://github.com/tutao/tutanota/actions/runs/13056825592 will probably fail because of that

abdulkadir-tutao and others added 13 commits January 31, 2025 11:01
the progress estimation for failed imports was not stopped.
this gave the impression that the import was still ongoing.

we now reset the state if an error happens during import preparation
and will pause the running import if an async error happens.
this will allow the user to try to resume it and see the snackbar or
notification if the error happens again.

also adds some local state cleanup if the import fails before being
set up on the server
previously, we didn't increment the counter for all failure cases,
leading to
the finished import showing 0 failed mails even though some actually
failed.

we still leave some edge cases where we fail to import a mail but also
fail to mark it as failed.
- do not depend on un-needed tokio features
- move rand to Cargo.toml
- use latest uniffi-bindgen from crates.io
commands like `cargo test`, `cargo clippy` or `cargo check` now require
the `--all` or `--package` flags to choose which workspace members they
should run for.
this prevents the mobile apps from building node-mimimi
mlock is used by sqlcipher but is not allowed on our test containers
see sqlcipher/sqlcipher-android#48 (comment)
see systemd/systemd#9414 (comment)
@ganthern ganthern force-pushed the dev-infra branch 2 times, most recently from 0e6871b to e39bf8b Compare January 31, 2025 13:30
- build tutasdk object file in generateBinding task

note: not tested on ci/jenkins
@tuta-sudipg tuta-sudipg force-pushed the dev-infra branch 6 times, most recently from 99a1d2e to b8d442a Compare January 31, 2025 16:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants