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

Updated dependencies #71

Merged
merged 1 commit into from
Feb 2, 2021
Merged

Updated dependencies #71

merged 1 commit into from
Feb 2, 2021

Conversation

jbreitbart
Copy link
Contributor

No description provided.

@jbreitbart
Copy link
Contributor Author

I am using perfcnt and as a dependency also this crate in one of my projects. Using raw-cpuid < 0.9.0 results in an error when running cargo audit (RUSTSEC-2021-0013).

I cannot run cargo test (even without my changes) as it reports

  --> /home/jbreitbart/.cargo/registry/src/github.com-1ecc6299db9ec823/x86-0.21.0/src/lib.rs:96:5
   |
96 |     asm!("rdpid $0" : "=r"(pid));
   |     ----^^^^^^^^^^^^^^^^^^^^^^^^^
   |     |
   |     help: replace with: `llvm_asm!`
   |
   = note: consider migrating to the new asm! syntax specified in RFC 2873
   = note: alternatively, switch to llvm_asm! to keep your code working as it is

error: aborting due to 70 previous errors

I am not sure why this is happening. Are the tests working on your machine?

stlankes added a commit to stlankes/kernel that referenced this pull request Feb 2, 2021
The workaround should solve issue hermit-os#164 and
is only necessry until gz/rust-x86#71 is accepted.
@gz gz merged commit c8a844e into gz:master Feb 2, 2021
@gz
Copy link
Owner

gz commented Feb 2, 2021

Thanks for submitting this!

@jbreitbart
Copy link
Contributor Author

@gz Thanks for merging. Would you be interested in a dependabot config that keeps your dependencies updated automatically? The bot would create PR like jbreitbart/criterion-perf-events#3

@gz
Copy link
Owner

gz commented Feb 3, 2021

@jbreitbart yes that sounds great, is there a way to sign up for this?

bors bot added a commit to hermit-os/kernel that referenced this pull request Feb 3, 2021
166: clearify license text, add workround to use raw-cpuid r=stlankes a=stlankes

The workaround should solve issue #164 and is only necessry until gz/rust-x86#71 is accepted.

Co-authored-by: Stefan Lankes <slankes@eonerc.rwth-aachen.de>
@jbreitbart
Copy link
Contributor Author

It's part of Github Actions. I'll send a PR with a config file for the bot later today/tomorrow. IIRC that is all you need to have the bot monitor the repository.

bors bot added a commit to hermit-os/kernel that referenced this pull request Feb 3, 2021
166: clearify license text, add workround to use raw-cpuid r=stlankes a=stlankes

The workaround should solve issue #164 and is only necessry until gz/rust-x86#71 is accepted.

Co-authored-by: Stefan Lankes <slankes@eonerc.rwth-aachen.de>
simonschoening pushed a commit to simonschoening/libhermit-rs that referenced this pull request Aug 26, 2021
The workaround should solve issue hermit-os#164 and
is only necessry until gz/rust-x86#71 is accepted.
simonschoening pushed a commit to simonschoening/libhermit-rs that referenced this pull request Aug 26, 2021
166: clearify license text, add workround to use raw-cpuid r=stlankes a=stlankes

The workaround should solve issue hermit-os#164 and is only necessry until gz/rust-x86#71 is accepted.

Co-authored-by: Stefan Lankes <slankes@eonerc.rwth-aachen.de>
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