Skip to content

Conversation

folkertdev
Copy link
Collaborator

No description provided.

@folkertdev folkertdev force-pushed the ci-cross-compile branch 4 times, most recently from ffbae7c to 7c981df Compare August 12, 2024 14:22
…st?); it often takes over 1 minute to 'post rust cache', building our dependencies is much, much faster than that
Comment on lines -82 to -85
- name: Rust cache
uses: Swatinem/rust-cache@3cf7f8cc28d1b4e7d01e3783be10a97d55d483c8
with:
shared-key: "${{matrix.rust}}-${{matrix.target}}"
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

this just took more time than it saved

Comment on lines -124 to +120
packages: qemu qemu-user qemu-user-static qemu-system-s390x gcc-s390x-linux-gnu gcc-i686-linux-gnu
packages: qemu-user qemu-user-static qemu-system-s390x gcc-s390x-linux-gnu gcc-i686-linux-gnu g++-s390x-linux-gnu
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

cmake needs the g++ compiler apparently. or it wants it anyway. fine

[workspace.dependencies]
libloading = "0.8.1"
libz-sys = { version = "1.1.12", default-features = false, features = ["zlib-ng"] } # use libz-ng in libz compat mode
libz-sys = { version = "1.1.19", default-features = false, features = ["zlib-ng"] } # use libz-ng in libz compat mode
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

this release fixes a bug with a missing file that caused issues for the s390x target rust-lang/libz-sys#205

strm.next_in = std::ptr::null_mut();
let mut ret = unsafe { inflateInit2_(&mut strm, -8, VERSION, STREAM_SIZE) };
let mut input = [0x63, 0x00];
let mut input = [0x63u8, 0x00];
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

was inferred as i32 which will fail on big endian systems.

@folkertdev folkertdev merged commit 6ee6cef into main Aug 12, 2024
16 checks passed
@folkertdev folkertdev deleted the ci-cross-compile branch August 12, 2024 19:09
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.

1 participant