search

fschutt / rust-fontconfig

Pure-Rust rewrite of the Linux fontconfig library (no system dependencies) - using ttf-parser and allsorts
MIT License
39 stars 8 forks source link

Support <include> element in fontconfig #5

Closed Riey closed 2 years ago

Riey commented 2 years ago

Currently this crate parse only dir but many conf file was exists in separate files with include

I've tested in NixOS

20211022133402

Riey commented 2 years ago

Build failed is come from encoding_rs-0.8.29 0.8.28 should be work

fschutt commented 2 years ago

Hi,

thanks for making the effort. I currently can't test it, but I'd be willing to merge it if it doesn't break anything.

Would it be possible to refactor your changes without running cargo fmt, so that I can see what actually changed? Right now the changeset is relatively large and a lot of it is just reformatting the existing code.

Riey commented 2 years ago

Sorry I just realized fontconfig use more complex config file(prefix and remap-dirs reset-dirs ...)

so I'm close this now but I'm making parsing process in other crate https://github.com/Riey/fontconfig-parser when this complete I'll try open another PR