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

Choose subversion of tracing crate in wasix #3907

Merged

Conversation

kajacx
Copy link
Contributor

@kajacx kajacx commented May 24, 2023

The wasix crate needs version at least 0.1.36, because of method "record" here: https://docs.rs/crate/tracing/0.1.36/source/src/span.rs

    pub fn record<Q: ?Sized, V>(&self, field: &Q, value: V) -> &Self
    where
        Q: field::AsField,
        V: field::Value,
    { ... }

but sys-utils is using 0.1.37, so might as well use that.

@Michael-F-Bryan Michael-F-Bryan enabled auto-merge May 24, 2023 14:49
The wasix crate needs version at least 0.1.36, because of method "record" here: https://docs.rs/crate/tracing/0.1.36/source/src/span.rs

 pub fn record<Q: ?Sized, V>(&self, field: &Q, value: V) -> &Self
    where
        Q: field::AsField,
        V: field::Value,
    {

but sys-utils is using 0.1.37, so might as well use that.
@theduke theduke force-pushed the choose-subersion-of-tracing-lib-in-wasix branch from fe1058b to d94820b Compare May 24, 2023 15:41
@theduke
Copy link
Contributor

theduke commented May 24, 2023

Thanks!

@ptitSeb ptitSeb disabled auto-merge May 24, 2023 16:06
@ptitSeb ptitSeb merged commit c6138c2 into wasmerio:master May 24, 2023
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.

4 participants