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

Remove extreneous image features and dependencies #1425

Merged
merged 7 commits into from
Feb 28, 2023
Merged

Conversation

emilk
Copy link
Member

@emilk emilk commented Feb 27, 2023

Part of #1316

cargo clean && cargo build --timings -p rerun

Total time: 161s -> 146s (-15s / -9%)
Total crates: 482 -> 466 (-16)
image features: bmp, dds, default, dxt, exr, farbfeld, gif, hdr, ico, jpeg, jpeg_rayon, openexr, png, pnm, scoped_threadpool, tga, tiff, webp -> jpeg, png, tiff

@emilk emilk added the 🧑‍💻 dev experience developer experience (excluding CI) label Feb 27, 2023
@emilk emilk mentioned this pull request Feb 27, 2023
@emilk emilk changed the title Remove extreneous image dependencies Remove extreneous image features and dependencies Feb 28, 2023
@teh-cmc teh-cmc self-requested a review February 28, 2023 07:40
@emilk emilk merged commit a6ae6cd into main Feb 28, 2023
@emilk emilk deleted the emilk/fewer-deps branch February 28, 2023 08:41
emilk added a commit that referenced this pull request Mar 2, 2023
* Cut out all image dependencies except for png and jpeg

* Small refactor of re_log: split out a file

* deny.toml: remove recursive exceptions for `ring` and `winit`

* ci: cargo deny check all four target platforms

* fix cargo deny CI snafu

* Still I messed up the CI command
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🧑‍💻 dev experience developer experience (excluding CI)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants