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

Parse without use of regex to work around dependency conflict #1

Closed
wants to merge 1 commit into from
Closed

Conversation

laanwj
Copy link

@laanwj laanwj commented May 31, 2020

The dependencies of the regex crate (especially memchr) clash with my runtime dependencies which need to be no_std. I don't think there's a general way to prevent feature clashes between build-time and run-time dependencies yet.

So the most straightforward workaround seems to be to write the parsing in this crate without relying on a regex. It hardly increases the length or complexity of the code.

The dependencies of the `regex` crate (especially `memchr`) clash
with my runtime dependencies which need to be `no_std`. I don't think
there's a general way to prevent feature clashes between build-time and
run-time dependencies yet.

So the most straightforward workaround seems to be to write the parsing
in this crate without relying on a regex.
@kallisti5
Copy link

kallisti5 commented Dec 19, 2020

@ilya-epifanov are there any plans to accept this one? It seems pretty clear that depending on std libraries isn't really portable for something so low level

@ilya-epifanov
Copy link
Owner

please try running cargo with -Zfeatures=itarget,build_dep.
https://doc.rust-lang.org/cargo/reference/unstable.html#features
It's gonna be stable pretty soon, it seems. rust-lang/cargo#8997

@laanwj
Copy link
Author

laanwj commented Feb 8, 2021

Thanks, yes, haven't tried yet (I'm not currently actively working on this project) but agree that solving the underlying issue is a better long-run solution.

@laanwj laanwj closed this Feb 8, 2021
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.

3 participants