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

Upgrade to bstr 1.0.0 #2173

Merged
merged 2 commits into from
Sep 10, 2022
Merged

Upgrade to bstr 1.0.0 #2173

merged 2 commits into from
Sep 10, 2022

Conversation

lopopolo
Copy link
Member

See:

serde appears in the lockfile despite being unused and not being built due to:

I am expecting to send a followup PR that drops the std feature from bstr dep in spinoso-env in favor of replacing it with scolapasta-path.

@lopopolo lopopolo added the A-deps Area: Source and library dependencies. label Sep 10, 2022
@lopopolo lopopolo merged commit cbd198b into trunk Sep 10, 2022
@lopopolo lopopolo deleted the lopopolo/bstr-1.0.0 branch September 10, 2022 20:10
lopopolo added a commit that referenced this pull request Sep 10, 2022
The "system env" backend requires converting between bytes and platform
strings. Previously this was done with routines which require the
`bstr/std` feature. This feature is a bit heavy though ans the
`OsString` specific bits are available in `scolapasta-path`.

This commit also includes changes to get `spinoso-env` to build and test
with no default features.

Followup to #2173.
lopopolo added a commit that referenced this pull request Sep 10, 2022
The "system env" backend requires converting between bytes and platform
strings. Previously this was done with routines which require the
`bstr/std` feature. This feature is a bit heavy though ans the
`OsString` specific bits are available in `scolapasta-path`.

This commit also includes changes to get `spinoso-env` to build and test
with no default features.

Followup to #2173.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-deps Area: Source and library dependencies.
Development

Successfully merging this pull request may close these issues.

1 participant