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

Merged
merged 3 commits into from
Nov 13, 2024
Merged

Conversation

mhelsley
Copy link
Contributor

@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
@keith-hall keith-hall merged commit 9bed8ad into sharkdp:master Nov 13, 2024
23 of 24 checks passed
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.

2 participants