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

Clean Up Dependencies if Possible #305

Open
zicklag opened this issue Jan 12, 2024 · 1 comment
Open

Clean Up Dependencies if Possible #305

zicklag opened this issue Jan 12, 2024 · 1 comment

Comments

@zicklag
Copy link
Member

zicklag commented Jan 12, 2024

Lately, especially after re-adding networking I think, we've taken on a lot of extra dependencies. A partial contributor is the windows API crates, which we are including at least 3 different versions of due to different dependencies. For example, ping-rs, dashmap, and bevy all depend on different versions of the same windows API crates I think.

Building in jumpy right now leads to 589 dependencies, which would be good to bring down if at all possible.

@nelson137
Copy link
Contributor

nelson137 commented Mar 24, 2024

@zicklag I've been trying to cut down on duplicate dependencies, but a lot is blocked by crates using different (old) versions of crates. I was able to get a couple by making upgrades but I don't think it's very impactful. In the process I upgraded to bevy 0.12 in hopes it would help... but it actually made things slightly worse.

I'll PR that since it may be useful, but it seems like our only option for movement here is to start forking things and make the changes to consolidate our dependency tree.

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

No branches or pull requests

2 participants