Skip to content

Commit

Permalink
v2/manifest: Explicitly enumerate re-exported items from manifest_sch…
Browse files Browse the repository at this point in the history
…ema2

There is a name conflict when re-exporting v2::Config alongside
v2::manifest::manifest_schema2::Config

This fixes this clippy error:

error: ambiguous glob re-exports
Error:   --> src/v2/manifest/mod.rs:12:9
   |
3  | use crate::v2::*;
   |     ------------ but the name `Config` in the type namespace is also re-exported here
...
12 | pub use self::manifest_schema2::*;
   |         ^^^^^^^^^^^^^^^^^^^^^^^^^ the name `Config` in the type namespace is first re-exported here
   |
   = note: `-D ambiguous-glob-reexports` implied by `-D warnings`

error: could not compile `dkregistry` (lib) due to previous error
  • Loading branch information
jeckersb committed Jun 1, 2023
1 parent cdcf851 commit caaae79
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion src/v2/manifest/mod.rs
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,9 @@ mod manifest_schema1;
pub use self::manifest_schema1::*;

mod manifest_schema2;
pub use self::manifest_schema2::*;
pub use self::manifest_schema2::{
ConfigBlob, ManifestList, ManifestObj, ManifestSchema2, ManifestSchema2Spec, Platform,
};

impl Client {
/// Fetch an image manifest.
Expand Down

0 comments on commit caaae79

Please sign in to comment.