bwasty / gltf-viewer

glTF 2.0 Viewer written in Rust
The Unlicense
188 stars 31 forks source link

Build failure with rustc 1.31.1 #56

Closed jansol closed 5 years ago

jansol commented 5 years ago

This occurs when trying to install gltf-viewer using cargo install.

   Compiling gltf-viewer v0.4.0
error[E0659]: `Texture` is ambiguous (glob import vs glob import in the same module)
 --> ~/.cargo/registry/src/github.com-1ecc6299db9ec823/gltf-viewer-0.4.0/src/render/root.rs:8:26
  |
8 | use render::{Mesh, Node, Texture, Material};
  |                          ^^^^^^^ ambiguous name
  |
note: `Texture` could refer to the struct imported here
 --> ~/.cargo/registry/src/github.com-1ecc6299db9ec823/gltf-viewer-0.4.0/src/render/mod.rs:18:9
  |
18| pub use self::texture::*;
  |         ^^^^^^^^^^^^^^^^
  = help: consider adding an explicit import of `Texture` to disambiguate
note: `Texture` could also refer to the struct imported here
 --> ~/.cargo/registry/src/github.com-1ecc6299db9ec823/gltf-viewer-0.4.0/src/render/mod.rs:13:9
  |
13| pub use self::primitive::*;
  |         ^^^^^^^^^^^^^^^^^^
  = help: consider adding an explicit import of `Texture` to disambiguate

error[E0659]: `Texture` is ambiguous (glob import vs glob import in the same module)
 --> ~/.cargo/registry/src/github.com-1ecc6299db9ec823/gltf-viewer-0.4.0/src/render/material.rs:7:21
  |
7 | use render::{ Root, Texture };
  |                     ^^^^^^^ ambiguous name
  |
note: `Texture` could refer to the struct imported here
 --> ~/.cargo/registry/src/github.com-1ecc6299db9ec823/gltf-viewer-0.4.0/src/render/mod.rs:18:9
  |
18| pub use self::texture::*;
  |         ^^^^^^^^^^^^^^^^
  = help: consider adding an explicit import of `Texture` to disambiguate
note: `Texture` could also refer to the struct imported here
 --> ~/.cargo/registry/src/github.com-1ecc6299db9ec823/gltf-viewer-0.4.0/src/render/mod.rs:13:9
  |
13| pub use self::primitive::*;
  |         ^^^^^^^^^^^^^^^^^^
  = help: consider adding an explicit import of `Texture` to disambiguate

error: aborting due to 2 previous errors
bwasty commented 5 years ago

Already fixed with #55. I'll try to do a release soon, until then you can use the binaries or build from git.