Support specifying crate names and versions separated by forward slashes
rather than hyphens.  Since slashes are not valid in crate names, this
makes splitting the tokens trivial and free of regular expressions.
Effectively, the slash variant is roughly 180% faster:

```
 * CRATES with slashes
real  952 it/s
user  952 it/s
 * CRATES with hyphens
real  339 it/s
user  339 it/s
```

Signed-off-by: Michał Górny <mgo...@gentoo.org>
---
 eclass/cargo.eclass         |  24 +++++---
 eclass/tests/cargo-bench.sh | 111 +++++++++++++++++++-----------------
 2 files changed, 75 insertions(+), 60 deletions(-)

diff --git a/eclass/cargo.eclass b/eclass/cargo.eclass
index d97bb0df9348..54a1edd36c28 100644
--- a/eclass/cargo.eclass
+++ b/eclass/cargo.eclass
@@ -59,12 +59,16 @@ ECARGO_VENDOR="${ECARGO_HOME}/gentoo"
 # Bash string containing all crates that are to be downloaded.
 # It is used by cargo_crate_uris.
 #
+# Ideally, crate names and versions should be separated by a forward
+# slash.  A legacy syntax using hyphen is also supported but it is much
+# slower.
+#
 # Example:
 # @CODE
 # CRATES="
-# metal-1.2.3
-# bar-4.5.6
-# iron_oxide-0.0.1
+# metal/1.2.3
+# bar/4.5.6
+# iron_oxide/0.0.1
 # "
 # inherit cargo
 # ...
@@ -182,10 +186,16 @@ _cargo_set_crate_uris() {
        CARGO_CRATE_URIS=
        for crate in ${crates}; do
                local name version url
-               [[ $crate =~ $regex ]] || die "Could not parse name and version 
from crate: $crate"
-               name="${BASH_REMATCH[1]}"
-               version="${BASH_REMATCH[2]}"
-               
url="https://crates.io/api/v1/crates/${name}/${version}/download -> 
${crate}.crate"
+               if [[ ${crate} == */* ]]; then
+                       name=${crate%/*}
+                       version=${crate##*/}
+               else
+                       [[ ${crate} =~ ${regex} ]] ||
+                               die "Could not parse name and version from 
crate: ${crate}"
+                       name="${BASH_REMATCH[1]}"
+                       version="${BASH_REMATCH[2]}"
+               fi
+               
url="https://crates.io/api/v1/crates/${name}/${version}/download -> 
${name}-${version}.crate"
                CARGO_CRATE_URIS+="${url} "
        done
 
diff --git a/eclass/tests/cargo-bench.sh b/eclass/tests/cargo-bench.sh
index 11b740f8dfcd..cb346178f0a2 100755
--- a/eclass/tests/cargo-bench.sh
+++ b/eclass/tests/cargo-bench.sh
@@ -47,63 +47,68 @@ timeit() {
 
 # taken from cryptograpy-41.0.1
 CRATES="
-       Inflector-0.11.4
-       aliasable-0.1.3
-       asn1-0.15.2
-       asn1_derive-0.15.2
-       autocfg-1.1.0
-       base64-0.13.1
-       bitflags-1.3.2
-       cc-1.0.79
-       cfg-if-1.0.0
-       foreign-types-0.3.2
-       foreign-types-shared-0.1.1
-       indoc-1.0.9
-       libc-0.2.144
-       lock_api-0.4.9
-       memoffset-0.8.0
-       once_cell-1.17.2
-       openssl-0.10.54
-       openssl-macros-0.1.1
-       openssl-sys-0.9.88
-       ouroboros-0.15.6
-       ouroboros_macro-0.15.6
-       parking_lot-0.12.1
-       parking_lot_core-0.9.7
-       pem-1.1.1
-       pkg-config-0.3.27
-       proc-macro-error-1.0.4
-       proc-macro-error-attr-1.0.4
-       proc-macro2-1.0.59
-       pyo3-0.18.3
-       pyo3-build-config-0.18.3
-       pyo3-ffi-0.18.3
-       pyo3-macros-0.18.3
-       pyo3-macros-backend-0.18.3
-       quote-1.0.28
-       redox_syscall-0.2.16
-       scopeguard-1.1.0
-       smallvec-1.10.0
-       syn-1.0.109
-       syn-2.0.18
-       target-lexicon-0.12.7
-       unicode-ident-1.0.9
-       unindent-0.1.11
-       vcpkg-0.2.15
-       version_check-0.9.4
-       windows-sys-0.45.0
-       windows-targets-0.42.2
-       windows_aarch64_gnullvm-0.42.2
-       windows_aarch64_msvc-0.42.2
-       windows_i686_gnu-0.42.2
-       windows_i686_msvc-0.42.2
-       windows_x86_64_gnu-0.42.2
-       windows_x86_64_gnullvm-0.42.2
-       windows_x86_64_msvc-0.42.2
+       Inflector/0.11.4
+       aliasable/0.1.3
+       asn1/0.15.2
+       asn1_derive/0.15.2
+       autocfg/1.1.0
+       base64/0.13.1
+       bitflags/1.3.2
+       cc/1.0.79
+       cfg-if/1.0.0
+       foreign-types/0.3.2
+       foreign-types-shared/0.1.1
+       indoc/1.0.9
+       libc/0.2.144
+       lock_api/0.4.9
+       memoffset/0.8.0
+       once_cell/1.17.2
+       openssl/0.10.54
+       openssl-macros/0.1.1
+       openssl-sys/0.9.88
+       ouroboros/0.15.6
+       ouroboros_macro/0.15.6
+       parking_lot/0.12.1
+       parking_lot_core/0.9.7
+       pem/1.1.1
+       pkg-config/0.3.27
+       proc-macro-error/1.0.4
+       proc-macro-error-attr/1.0.4
+       proc-macro2/1.0.59
+       pyo3/0.18.3
+       pyo3-build-config/0.18.3
+       pyo3-ffi/0.18.3
+       pyo3-macros/0.18.3
+       pyo3-macros-backend/0.18.3
+       quote/1.0.28
+       redox_syscall/0.2.16
+       scopeguard/1.1.0
+       smallvec/1.10.0
+       syn/1.0.109
+       syn/2.0.18
+       target-lexicon/0.12.7
+       unicode-ident/1.0.9
+       unindent/0.1.11
+       vcpkg/0.2.15
+       version_check/0.9.4
+       windows-sys/0.45.0
+       windows-targets/0.42.2
+       windows_aarch64_gnullvm/0.42.2
+       windows_aarch64_msvc/0.42.2
+       windows_i686_gnu/0.42.2
+       windows_i686_msvc/0.42.2
+       windows_x86_64_gnu/0.42.2
+       windows_x86_64_gnullvm/0.42.2
+       windows_x86_64_msvc/0.42.2
 "
 
 inherit cargo
 
+einfo "CRATES with slashes"
+timeit
+
+einfo "CRATES with hyphens"
+CRATES=${CRATES//\//-}
 timeit
 
 texit
-- 
2.41.0


Reply via email to