Skip to content

Commit 1e1ef79

Browse files
authored
Merge pull request #148 from rust-osdev/dev3
workspace: use workspace dependencies
2 parents 09fd4ce + 95a9a1a commit 1e1ef79

File tree

5 files changed

+17
-11
lines changed

5 files changed

+17
-11
lines changed

Cargo.lock

Lines changed: 1 addition & 0 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

Cargo.toml

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,10 @@ members = [
55
"multiboot2-header",
66
]
77

8+
[workspace.dependencies]
9+
bitflags = "2"
10+
derive_more = { version = "0.99", default-features = false, features = ["display"] }
11+
log = { version = "0.4", default-features = false }
812

913
# This way, the "multiboot2" dependency in the multiboot2-header crate can be
1014
# referenced by version, while still the repository version is used

multiboot2-header/Cargo.toml

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -40,5 +40,11 @@ builder = ["alloc"]
4040
unstable = []
4141

4242
[dependencies]
43+
# Not yet used.
44+
# bitflags.workspace = true
45+
derive_more.workspace = true
46+
# Not yet used.
47+
# log.workspace = true
48+
4349
# used for MBI tags
4450
multiboot2 = "0.16.0"

multiboot2-header/src/header.rs

Lines changed: 2 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ use crate::{
55
RelocatableHeaderTag,
66
};
77
use core::convert::TryInto;
8-
use core::fmt::{Debug, Display, Formatter};
8+
use core::fmt::{Debug, Formatter};
99
use core::mem::size_of;
1010

1111
/// Magic value for a [`Multiboot2Header`], as defined in spec.
@@ -200,7 +200,7 @@ impl<'a> Debug for Multiboot2Header<'a> {
200200

201201
/// Errors that can occur when parsing a header from a slice.
202202
/// See [`Multiboot2Header::find_header`].
203-
#[derive(Copy, Clone, Debug, PartialEq, Eq, PartialOrd, Ord, Hash)]
203+
#[derive(Copy, Clone, Debug, derive_more::Display, PartialEq, Eq, PartialOrd, Ord, Hash)]
204204
pub enum LoadError {
205205
/// The checksum does not match the data.
206206
ChecksumMismatch,
@@ -212,12 +212,6 @@ pub enum LoadError {
212212
TooSmall,
213213
}
214214

215-
impl Display for LoadError {
216-
fn fmt(&self, f: &mut Formatter<'_>) -> core::fmt::Result {
217-
write!(f, "{:?}", self)
218-
}
219-
}
220-
221215
#[cfg(feature = "unstable")]
222216
impl core::error::Error for LoadError {}
223217

multiboot2/Cargo.toml

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -41,8 +41,9 @@ builder = ["alloc"]
4141
unstable = []
4242

4343
[dependencies]
44-
bitflags = "2"
45-
derive_more = { version = "0.99", default-features = false, features = ["display"] }
46-
log = { version = "0.4", default-features = false }
44+
bitflags.workspace = true
45+
derive_more.workspace = true
46+
log.workspace = true
47+
4748
uefi-raw = { version = "0.2.0", default-features = false }
4849
ptr_meta = { version = "0.2.0", default-features = false }

0 commit comments

Comments
 (0)