Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

GEOS 3.8 is also failing #991

Closed
robe2 opened this issue Nov 12, 2023 · 0 comments
Closed

GEOS 3.8 is also failing #991

robe2 opened this issue Nov 12, 2023 · 0 comments
Assignees
Labels
Blocker Do not release milestone until addressed.
Milestone

Comments

@robe2
Copy link
Member

robe2 commented Nov 12, 2023

This one seems to be caused by the compilers becoming more pedantic. Complaining about variables defined and not used etc. https://github.com/libgeos/geos/actions/runs/6837999164/job/18594608439

As far as I can tell, no code has changed since the last successful run.

[ 17%] Building CXX object CMakeFiles/geos.dir/src/geomgraph/Depth.cpp.o
In file included from /home/runner/work/geos/geos/src/geomgraph/Depth.cpp:24:
In file included from /home/runner/work/geos/geos/include/geos/geomgraph/Label.h:26:
/home/runner/work/geos/geos/include/geos/geomgraph/TopologyLocation.h:133:10: error: no type named 'uint8_t' in namespace 'std'
    std::uint8_t locationSize;
    ~~~~~^
In file included from /home/runner/work/geos/geos/src/geomgraph/Depth.cpp:24:
In file included from /home/runner/work/geos/geos/include/geos/geomgraph/Label.h:26:
In file included from /home/runner/work/geos/geos/include/geos/geomgraph/TopologyLocation.h:142:
/home/runner/work/geos/geos/include/geos/geomgraph/TopologyLocation.inl:109:21: warning: implicit conversion changes signedness: 'int' to 'std::array::size_type' (aka 'unsigned long') [-Wsign-conversion]
    return location[locIndex] == le.location[locIndex];
           ~~~~~~~~ ^~~~~~~~
/home/runner/work/geos/geos/include/geos/geomgraph/TopologyLocation.inl:109:46: warning: implicit conversion changes signedness: 'int' to 'std::array::size_type' (aka 'unsigned long') [-Wsign-conversion]
    return location[locIndex] == le.location[locIndex];
                                 ~~          ^~~~~~~~
2 warnings and 1 error generated.
make[2]: *** [CMakeFiles/geos.dir/build.make:1154: CMakeFiles/geos.dir/src/geomgraph/Depth.cpp.o] Error 1
make[2]: *** Waiting for unfinished jobs....
/home/runner/work/geos/geos/src/geom/util/SineStarFactory.cpp:59:38: warning: implicit conversion changes signedness: 'int' to 'std::vector::size_type' (aka 'unsigned long') [-Wsign-conversion]
    std::vector<Coordinate> pts(nPts + 1);
                            ~~~ ~~~~~^~~
/home/runner/work/geos/geos/src/geom/util/SineStarFactory.cpp:79:16: warning: implicit conversion changes signedness: 'int' to 'std::vector::size_type' (aka 'unsigned long') [-Wsign-conversion]
        pts[iPt++] = coord(x, y);
        ~~~ ~~~^~
/home/runner/work/geos/geos/src/geom/util/SineStarFactory.cpp:81:9: warning: implicit conversion changes signedness: 'int' to 'std::vector::size_type' (aka 'unsigned long') [-Wsign-conversion]
    pts[iPt] = pts[0];
    ~~~ ^~~
3 warnings generated.
make[1]: *** [CMakeFiles/Makefile2:438: CMakeFiles/geos.dir/all] Error 2
@robe2 robe2 added the Blocker Do not release milestone until addressed. label Nov 12, 2023
@robe2 robe2 added this to the 3.8.4 milestone Nov 12, 2023
@robe2 robe2 self-assigned this Nov 12, 2023
robe2 added a commit to robe2/geos that referenced this issue Nov 12, 2023
robe2 added a commit that referenced this issue Nov 12, 2023
Closes GH-991 for GEOS 3.8.4
@robe2 robe2 closed this as completed Nov 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Blocker Do not release milestone until addressed.
Projects
None yet
Development

No branches or pull requests

1 participant