move-language / move

Apache License 2.0
2.26k stars 689 forks source link

sui-move: [12/x][move-package/lock] Use DependencyGraph in download_dependency_repos #859

Closed amnn closed 1 year ago

amnn commented 1 year ago

Another occasion where the previous implementation recursively traversed the transitive dependency graph -- the new approach discovers the graph and then iterates through its nodes to download them.

The second step is currently technically unnecessary, but becomes relevant in the presence of externally resolved dependencies where third-party resolvers can introduce sub-graphs into the DependencyGraph without having to explore them locally.

Test Plan

move/language/tools/move-package$ cargo nextest

Stack

See also: #788 for main.