Skip to content

Commit

Permalink
Tessellator was broken for years! Revert-fixing.
Browse files Browse the repository at this point in the history
* Due to a typo in the cfg flag, tessellator was never built properly
  (we can even remove it as it is clearly not being used by anyone).
* This is a known issues in the rust compiler: rust-lang/cargo#10554
* I reverted Lyon to 0.16.2 and it worked fine.

Do we want to keep this and spend the effort to upgrade to the latest
Lyon crate, or is this a useless feature and should be removed?
  • Loading branch information
nyurik committed Apr 3, 2023
1 parent 0300a5d commit d205c38
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion geozero/Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,7 @@ geos = { version = "8.1", optional = true }
gdal = { version = "0.14", default-features = false, optional = true }
gdal-sys = { version = "0.8", optional = true }
gpx = { version = "0.8", default-features = false, optional = true }
lyon = { version = "1.0", optional = true }
lyon = { version = "0.16.2", optional = true }
log = "0.4.17"
scroll = { version = "0.11", optional = true }
sqlx = { version = "0.6", default-features = false, optional = true }
Expand Down
2 changes: 1 addition & 1 deletion geozero/src/lib.rs
Original file line number Diff line number Diff line change
Expand Up @@ -105,7 +105,7 @@ pub mod svg;
#[cfg(feature = "with-svg")]
pub use crate::svg::conversion::*;

#[cfg(feature = "with-tesselator")]
#[cfg(feature = "with-tessellator")]
pub mod tessellator;

#[cfg(feature = "with-wkb")]
Expand Down
2 changes: 1 addition & 1 deletion geozero/src/tessellator/tessellator.rs
Original file line number Diff line number Diff line change
Expand Up @@ -135,7 +135,7 @@ impl VertexOutput for ObjWriter {
#[cfg(test)]
mod test {
use super::*;
use crate::geojson_reader::read_geojson;
use crate::geojson::geojson_reader::read_geojson;

#[test]
fn point_geom() {
Expand Down

0 comments on commit d205c38

Please sign in to comment.