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

Update dependencies and bindings #102

Merged
merged 5 commits into from
Oct 5, 2023
Merged

Conversation

abrown
Copy link
Contributor

@abrown abrown commented Sep 19, 2023

This set of changes fixes some CI issues:

  • the Rust dependencies were out of date
  • the Rust bindings then needed to be updated
  • the C build task was failing because the GitHub runner failed to download some 32-bit packages needed by gcc-multilib

This removes some problematic dependencies:
- `ansi_term` and `atty` are deprecated/unmaintained (used by `bindgen`)
- we had duplicate `bitflags` versions
Updating the `bindgen` crate forces us to regenerate these bindings.
For some time, Ubuntu has been phasing out support for 32-bit packages
(e.g., [here]). `ittapi` still builds a 32-bit version of the library,
which requires `gcc-multilib` to be installed (it provides
`/usr/include/bits/libc-header-start.h` when compiling with `-m32`). But
some GitHub inconsistency is getting in the way: `gcc-multilib` fails to
install on the GitHub runner though it can be installed locally.

[here]: https://canonical.com/blog/statement-on-32-bit-i386-packages-for-ubuntu-19-10-and-20-04-lts

While we figure out the 32-bit support story for `ittapi`, this change
disables the 32-bit build by forcing it to be 64-bit exclusively. Once
we decide whether to (a) remove 32-bit support or (b) fix the GitHub
runners to allow 32-bit builds, we can remove the TODO.
@jlb6740 jlb6740 merged commit 976312d into intel:master Oct 5, 2023
9 checks passed
@abrown abrown deleted the update-dependencies branch October 5, 2023 21:25
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