-
Notifications
You must be signed in to change notification settings - Fork 69
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
feat: Use opendal
in rattler-index
and add executable
#1076
Changes from 37 commits
d0085a3
8c58912
f326733
76afe88
87b4d84
a347827
dd01d4f
ac8ea01
554f00b
8a99102
ab1b0a3
59a0bdc
563c66e
6b8a13a
52fae88
1344176
131138a
7341218
efe483c
39713a8
b1a5bb2
63a09e0
b1425d2
6e7add6
e38dc2d
7b7261a
c7785b8
bf8445d
d010241
a869c2d
c592135
57d52c3
867a49d
e2829f0
cdb5daa
0133a55
ded2ee7
8fc824c
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,2 @@ | ||
pixi.lock merge=binary linguist-language=YAML linguist-generated=true | ||
Cargo.lock merge=binary linguist-generated=true |
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. we still need to do the whole native-tls / rustls-tls and native-roots dance There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. done in
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -3,23 +3,64 @@ name = "rattler_index" | |
version = "0.20.13" | ||
edition.workspace = true | ||
authors = [] | ||
description = "A crate that indexes directories containing conda packages to create local conda channels" | ||
description = "A crate to index conda channels and create a repodata.json file." | ||
categories.workspace = true | ||
homepage.workspace = true | ||
repository.workspace = true | ||
license.workspace = true | ||
readme.workspace = true | ||
default-run = "rattler-index" | ||
|
||
[features] | ||
default = ["rustls-tls"] | ||
native-tls = [ | ||
"reqwest/native-tls", | ||
"reqwest/native-tls-alpn", | ||
"rattler_package_streaming/native-tls", | ||
"rattler_networking/native-tls", | ||
"tools/native-tls", | ||
] | ||
rustls-tls = [ | ||
"reqwest/rustls-tls", | ||
"reqwest/rustls-tls-native-roots", | ||
"rattler_package_streaming/rustls-tls", | ||
"rattler_networking/rustls-tls", | ||
"tools/rustls-tls", | ||
] | ||
|
||
[[bin]] | ||
name = "rattler-index" | ||
path = "src/main.rs" | ||
Comment on lines
+31
to
+33
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. i would suggest we don't publish binaries for rattler-index in this repo as this will probably complicate the release-plz workflow in this repo quite a bit. instead, we can only build binaries on conda-forge (and other package managers like brew), imo this should be enough |
||
|
||
[dependencies] | ||
anyhow = { workspace = true } | ||
clap = { workspace = true, features = ["derive", "env"] } | ||
clap-verbosity-flag = { workspace = true } | ||
console = { workspace = true } | ||
fs-err = { workspace = true } | ||
rattler_conda_types = { path="../rattler_conda_types", version = "0.31.0", default-features = false } | ||
rattler_digest = { path="../rattler_digest", version = "1.0.6", default-features = false } | ||
rattler_package_streaming = { path="../rattler_package_streaming", version = "0.22.29", default-features = false } | ||
fxhash = { workspace = true } | ||
futures = { workspace = true } | ||
indicatif = { workspace = true } | ||
opendal = { workspace = true, features = [ | ||
"services-s3", | ||
"services-fs", | ||
], default-features = false } | ||
rattler_networking = { path = "../rattler_networking", version = "0.22.4", default-features = false } | ||
rattler_conda_types = { path = "../rattler_conda_types", version = "0.31.0" } | ||
rattler_digest = { path = "../rattler_digest", version = "1.0.6", default-features = false } | ||
rattler_package_streaming = { path = "../rattler_package_streaming", version = "0.22.29", default-features = false } | ||
reqwest = { workspace = true, default-features = false, features = [ | ||
"http2", | ||
"macos-system-configuration", | ||
"charset", | ||
] } | ||
serde_json = { workspace = true } | ||
tempfile = { workspace = true } | ||
tokio = { workspace = true, features = ["full"] } | ||
tracing = { workspace = true } | ||
walkdir = { workspace = true } | ||
tracing-subscriber = { workspace = true, features = ["env-filter", "fmt"] } | ||
tracing-log = { workspace = true } | ||
url = { workspace = true } | ||
|
||
[dev-dependencies] | ||
tempfile = { workspace = true } | ||
tools = { path = "../tools" } | ||
tools = { path = "../tools", default-features = false } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is
default-features = false
okay here? otherwise this will pull in reqwest stuff