Skip to content

Commit

Permalink
Remove theseus_path_std in favor of the new theseus_std::path mod…
Browse files Browse the repository at this point in the history
…ule (theseus-os#512)

The old `theseus_path_std` was a hacky approach to provide `std::path`-like `Path` and `PathBuf` types. It was hacky because we bypassed using `OsString`/`OsStr` and also didn't support any useful filesystem operations like `Path::open()`. 

The new `theseus_std` does provide those missing filesystem functions as well as `OsString` and `OsStr`, so we migrate to using those instead.

The primary crate affected is `ports/backtrace` (`backtrace-rs`).
  • Loading branch information
kevinaboos authored May 2, 2022
1 parent 7bd703c commit 44978b4
Show file tree
Hide file tree
Showing 8 changed files with 78 additions and 3,202 deletions.
73 changes: 71 additions & 2 deletions Cargo.lock

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -38,7 +38,7 @@ exclude = [
"applications/test_mutex_sleep",
"applications/test_wait_queue",
"applications/test_ixgbe",
"applications/test_backtrace",
# "applications/test_backtrace",

## libtheseus is currently built separately and not used
"libtheseus",
Expand Down
3 changes: 1 addition & 2 deletions applications/test_std_fs/src/lib.rs
Original file line number Diff line number Diff line change
Expand Up @@ -9,8 +9,7 @@ extern crate core2;
#[macro_use] extern crate terminal_print;

use alloc::{string::String, vec::Vec};
use core2::io::{self, Result, Error, Read, Write};
use theseus_std::fs::File;
use core2::io::{self, Write};


pub fn main(_args: Vec<String>) -> isize {
Expand Down
2 changes: 1 addition & 1 deletion ports/backtrace
10 changes: 0 additions & 10 deletions ports/path_std/Cargo.toml

This file was deleted.

Loading

0 comments on commit 44978b4

Please sign in to comment.