Skip to content

fb/logger: bump noto-sans-mono-bitmap + improvements #267

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

Merged

Conversation

phip1611
Copy link
Member

@phip1611 phip1611 commented Oct 1, 2022

Hi,

Update

This is now ready for merge.


Original Post

I'm the author of the noto-sans-mono-bitmap-crate and about to release 0.2.0 with breaking changes and many improvements (CHANGELOG).

As the bootloader crate is the only/the biggest user, I'd like to leave room for API discussions before I release the crate.

This MR bumps the noto-sans-mono-bitmap to 0.2.0 (unreleased yet) and performs several other small improvements to the framebuffer logger.

Any comments? If there are no comments, I'll release the crate in 2-4 days and switch this MR from draft state to ready

What's new?

  • more unicode ranges and they are selectable as cargo features
  • the internal structure was refactored by a lot which enables crate users to better specify which features they want and which they do not want
  • the API changed along with the documentation to better fit the use case

Example
Please note the fallback character in the BIOS boot logging message, which is also new.
image

@phip1611 phip1611 force-pushed the fb-logger-noto-sans-mono-bitmap-bump branch from 3ca83e2 to 508adeb Compare October 1, 2022 09:30
@phip1611 phip1611 force-pushed the fb-logger-noto-sans-mono-bitmap-bump branch 5 times, most recently from 0d9995b to 2cb3a8f Compare October 1, 2022 10:44
@asensio-project
Copy link
Contributor

Looks great!

@phip1611 phip1611 force-pushed the fb-logger-noto-sans-mono-bitmap-bump branch from 2cb3a8f to 113faf6 Compare October 1, 2022 14:02
Copy link
Member

@phil-opp phil-opp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me, I like the new crate features!

@phip1611 phip1611 force-pushed the fb-logger-noto-sans-mono-bitmap-bump branch from 113faf6 to ecd9e39 Compare October 7, 2022 11:25
@phip1611 phip1611 changed the title Draft: fb/logger: bump noto-sans-mono-bitmap + improvements fb/logger: bump noto-sans-mono-bitmap + improvements Oct 7, 2022
@phip1611
Copy link
Member Author

phip1611 commented Oct 7, 2022

Looks good to me, I like the new crate features!

this is now ready to merge!

@@ -94,6 +100,6 @@ default-target = "x86_64-unknown-linux-gnu"
[package.metadata.release]
dev-version = false
pre-release-replacements = [
{ file="Changelog.md", search="# Unreleased", replace="# Unreleased\n\n# {{version}} – {{date}}", exactly=1 },
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yikes, my IDE did this automatically

optional = true

[build-dependencies]
llvm-tools-build = { version = "0.1", optional = true, package = "llvm-tools" }
toml = { version = "0.5.1", optional = true }
serde = { version = "1.0", features = ["derive"], optional = true}
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yikes, my IDE did this automatically

Copy link
Member

@phil-opp phil-opp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great, thanks!,

@phil-opp phil-opp merged commit b1f442b into rust-osdev:main Oct 10, 2022
@phip1611 phip1611 deleted the fb-logger-noto-sans-mono-bitmap-bump branch October 10, 2022 07:23
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.

3 participants