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

Create Raspirus #3263

Closed
wants to merge 4 commits into from
Closed

Create Raspirus #3263

wants to merge 4 commits into from

Conversation

Benji377
Copy link
Contributor

No description provided.

@probonopd
Copy link
Member

The test fails with:

(raspirus:4): dbind-WARNING **: 17:57:30.716: AT-SPI: Error retrieving accessibility bus address: org.freedesktop.DBus.Error.ServiceUnknown: The name org.a11y.Bus was not provided by any .service files
thread 'main' panicked at '`APPDIR` or `APPIMAGE` environment variable found but this application was not detected as an AppImage; this might be a security issue.', /home/runner/.cargo/registry/src/index.crates.io-6f17d22bba15001f/tauri-utils-1.4.0/src/lib.rs:205:11
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

This seems to be caused by

@Benji377
Copy link
Contributor Author

@probonopd is there anything I can do to fix this?

@probonopd
Copy link
Member

Hi @Benji377, maybe send a pull request to the tauri project that would remove the "validation" described in the link above? Then, once it is fixed in tauri, all applications made with it should benefit and start working.

@Benji377
Copy link
Contributor Author

Hi @Benji377, maybe send a pull request to the tauri project that would remove the "validation" described in the link above? Then, once it is fixed in tauri, all applications made with it should benefit and start working.

I will take a look and see if I can do something about it. Unfortunately, I don't exactly know where the verification takes place in the Tauri code and if it might be essential for other parts of the app.

@probonopd
Copy link
Member

Need to undo (reverse) this:
tauri-apps/tauri@6fbd6db

@Benji377
Copy link
Contributor Author

@probonopd I opened a PR to change the panic to a warning instead. This allows Tauri to keep the check, while also (hopefully) allowing the workflow here to pass the test

@probonopd
Copy link
Member

The thing is, the there is nothing to "warn" about.

@Benji377
Copy link
Contributor Author

The thing is, the there is nothing to "warn" about.

Well then I don't understand why they added it in the first place

@Benji377
Copy link
Contributor Author

Benji377 commented Sep 28, 2023

@probonopd the issue should be fixed now. Can I somehow request a new check, or do I need to wait for Tauri to release a patch version?

@probonopd
Copy link
Member

I triggered a re-run of the test by modifying the file.

@probonopd probonopd closed this Oct 28, 2023
probonopd added a commit that referenced this pull request Oct 28, 2023
Continuation of #3263
@probonopd probonopd mentioned this pull request Oct 28, 2023
@probonopd
Copy link
Member

Continues in #3273.

@Benji377 Benji377 deleted the patch-1 branch February 11, 2024 16:21
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.

None yet

2 participants