Improve ergonomics of export_package

Signed-off-by: J Robert Ray <jrray@jrray.org>

crates/spk-build/src/archive_test.rs

@@ -28,7 +28,7 @@ async fn test_archive_create_parents() {
         .await
         .unwrap();
     let filename = rt.tmpdir.path().join("deep/nested/path/archive.spk");
-    export_package::<_, _, NormalizedTagStrategy>(&spec.ident().to_any(), filename)
+    export_package::<NormalizedTagStrategy>(&spec.ident().to_any(), filename)
         .await
         .expect("export should create dirs as needed");
 }

crates/spk-cli/group3/src/cmd_export.rs

@@ -76,9 +76,9 @@ impl Run for Export {
         // TODO: this doesn't take the repos as an argument, but probably
         // should. It assumes/uses 'local' and 'origin' repos internally.
         let res = if self.legacy_spk_version_tags_for_writes {
-            storage::export_package::<_, _, VerbatimTagStrategy>(&pkg, &filename).await
+            storage::export_package::<VerbatimTagStrategy>(&pkg, &filename).await
         } else {
-            storage::export_package::<_, _, NormalizedTagStrategy>(&pkg, &filename).await
+            storage::export_package::<NormalizedTagStrategy>(&pkg, &filename).await
         };
         if let Err(spk_storage::Error::PackageNotFound(_)) = res {
             tracing::warn!("Ensure that you are specifying at least a package and");

crates/spk-cli/group3/src/cmd_export_test.rs

@@ -71,7 +71,7 @@ async fn test_export_works_with_missing_builds() {
 
     let filename = rt.tmpdir.path().join("archive.spk");
     filename.ensure();
-    spk_storage::export_package::<_, _, NormalizedTagStrategy>(
+    spk_storage::export_package::<NormalizedTagStrategy>(
         red_spec.ident().clone().to_version().to_any(None),
         &filename,
     )

crates/spk-cli/group3/src/cmd_import_test.rs

@@ -30,7 +30,7 @@ async fn test_archive_io() {
 
     let filename = rt.tmpdir.path().join("archive.spk");
     filename.ensure();
-    spk_storage::export_package::<_, _, NormalizedTagStrategy>(spec.ident().to_any(), &filename)
+    spk_storage::export_package::<NormalizedTagStrategy>(spec.ident().to_any(), &filename)
         .await
         .expect("failed to export");
     let mut actual = Vec::new();

crates/spk-storage/src/storage/archive.rs

@@ -11,10 +11,8 @@ use spk_schema::{AnyIdent, BuildIdent, VersionIdent};
 use super::{Repository, SpfsRepository};
 use crate::{Error, NameAndRepositoryWithTagStrategy, Result};
 
-pub async fn export_package<I, P, S>(pkg: I, filename: P) -> Result<()>
+pub async fn export_package<S>(pkg: impl AsRef<AnyIdent>, filename: impl AsRef<Path>) -> Result<()>
 where
-    I: AsRef<AnyIdent>,
-    P: AsRef<Path>,
     S: TagPathStrategy + Send + Sync,
 {
     let pkg = pkg.as_ref();