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

First pass at implementing configurable read timeout for X11 #46

Merged
merged 4 commits into from
Sep 20, 2024

Conversation

jasonterando
Copy link

Proposed solution for issue documented in issue 45.

@ChurchTao
Copy link
Owner

@jasonterando
Please take care of the formatting issues here and run basic fmt and clippy, otherwise the merged ci will not pass.

@ChurchTao
Copy link
Owner

@jasonterando
I modified a few #[cfg(target_os = "linux")] and the test passed. However, Linux does not have a CI test. Could you please run the test on a Linux machine? To ensure that everything is safe.

@jasonterando
Copy link
Author

I think we're good (running on Linux Mint / Ubuntu)...

$:~/projects/personal/clipboard-rs$ cargo run --example image
   Compiling clipboard-rs v0.2.0 (/home/jason/projects/personal/clipboard-rs)
    Finished `dev` profile [unoptimized + debuginfo] target(s) in 4.73s
     Running `target/debug/examples/image`
["image/png", "image/jpeg", "image/avif", "image/bmp", "image/x-bmp", "image/x-MS-bmp", "image/x-icon", "image/x-ico", "image/x-win-bitmap", "image/vnd.microsoft.icon", "application/ico", "image/ico", "image/icon", "text/ico", "image/jxl", "image/tiff", "image/webp", "audio/x-riff"]
$:~/projects/personal/clipboard-rs$ cargo test
   Compiling clipboard-rs v0.2.0 (/home/jason/projects/personal/clipboard-rs)
    Finished `test` profile [unoptimized + debuginfo] target(s) in 4.17s
     Running unittests src/lib.rs (target/debug/deps/clipboard_rs-b98f79e4aeb98b3d)

running 0 tests

test result: ok. 0 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out; finished in 0.00s

     Running tests/file_test.rs (target/debug/deps/file_test-7316e766406a54db)

running 1 test
test test_file ... ok

test result: ok. 1 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out; finished in 0.36s

     Running tests/image_test.rs (target/debug/deps/image_test-dd5e132e5a66ba70)

running 1 test
test test_image ... ok

test result: ok. 1 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out; finished in 0.85s

     Running tests/string_test.rs (target/debug/deps/string_test-2741cf6cc92c3633)

running 1 test
test test_string ... ok

test result: ok. 1 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out; finished in 0.72s

   Doc-tests clipboard_rs

running 0 tests

test result: ok. 0 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out; finished in 0.00s

$:~/projects/personal/clipboard-rs$ 

@ChurchTao ChurchTao merged commit 7c73b43 into ChurchTao:master Sep 20, 2024
6 checks passed
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.

2 participants