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

Fix lessopen feature #3095

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

mhelsley
Copy link

@mhelsley mhelsley commented Oct 3, 2024

RawOsString API changed in os_str_bytes ~7.0.0 and bat no longer builds with the lessopen feature because ::assert_from_raw_vec() is no longer merely deprecated and now requires selecting the "conversions" feature of os_str_bytes.

Bonus: ::from_string() is now deprecated in favor of ::new() because ::new() no longer copies and thus made from_string() obsolete.

Replaces #2938

Matt Helsley and others added 2 commits October 3, 2024 16:32
RawOsString API changed in os_str_bytes 7.0.0 and bat no longer builds
with the lessopen feature because:

- ::from_string() is deprecated and says to use new().
  The documentation says that new(), like from_string() no longer
  needs to copy the string.

- ::assert_from_raw_vec() is no longer merely deprecated and now
  requires selecting the "conversions" feature of os_str_bytes.

Replaces PR#2938
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.

1 participant