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

Can't use this within a project with wasm-bindgen due to cyclic dependency #29

Open
kdy1 opened this issue May 10, 2022 · 2 comments
Open

Comments

@kdy1
Copy link

kdy1 commented May 10, 2022

  • import_map enables serde_json/preserve_order
  • serde_json/preserve_order depends on indexmap (https://docs.rs/crate/serde_json/latest/features)
  • indexmap depends on hashbrown
  • hashbrown depends on ahash by default.
  • ahash depends on const-random

and the result is

❯ cargo check
    Updating crates.io index
error: cyclic package dependency: package `ahash v0.7.6` depends on itself. Cycle:
package `ahash v0.7.6`
    ... which satisfies dependency `ahash = "^0.7.0"` of package `hashbrown v0.11.2`
    ... which satisfies dependency `hashbrown = "^0.11"` of package `indexmap v1.8.0`
    ... which satisfies dependency `indexmap = "^1.5"` of package `serde_json v1.0.79`
    ... which satisfies dependency `serde_json = "^1.0"` of package `wasm-bindgen v0.2.79`
    ... which satisfies dependency `wasm-bindgen = "^0.2.79"` of package `js-sys v0.3.56`
    ... which satisfies dependency `js-sys = "^0.3"` of package `getrandom v0.2.3`
    ... which satisfies dependency `getrandom = "^0.2.0"` of package `const-random-macro v0.1.13`
    ... which satisfies dependency `const-random-macro = "^0.1.13"` of package `const-random v0.1.13`
    ... which satisfies dependency `const-random = "^0.1.12"` of package `ahash v0.7.6`

@kdy1
Copy link
Author

kdy1 commented May 10, 2022

It works if I use indexmap@~1.6.2

But it requires import_map@0.3.3.

Can you reduce the restriction of indexmap dependency to 1.6 or lower? (and publish 😄 )

@dsherret
Copy link
Member

dsherret commented Jun 1, 2022

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