Skip to content
This repository has been archived by the owner on Dec 5, 2019. It is now read-only.

don't skip cleanup #1

Closed
wants to merge 1 commit into from
Closed

don't skip cleanup #1

wants to merge 1 commit into from

Conversation

jaakkor2
Copy link

@jaakkor2 jaakkor2 commented Mar 9, 2019

Try to remove interference between the platform builds, could help for JuliaGeo/LibGEOS.jl#56

Try to remove interference between the platform builds, could help for JuliaGeo/LibGEOS.jl#56
@jaakkor2
Copy link
Author

jaakkor2 commented Mar 9, 2019

Close for now

@jaakkor2 jaakkor2 closed this Mar 9, 2019
@visr
Copy link
Member

visr commented Mar 10, 2019

Thanks for thinking along! Has this been a problem for other Builders as well? The reason the new GEOS build is not working on Win64 now is also reported here: JuliaPackaging/BinaryBuilder.jl#407.

@jaakkor2
Copy link
Author

I was planning to test make clean as well, but I could not get Travis to generate binaries for my fork. Would trying only Win64 build make sense?

@visr
Copy link
Member

visr commented Mar 12, 2019

Yes you can run only the Win64 build. I don't think the reason for the issues is contamination between the environments, but rather issues with mixing compiler versions. We could for instance try to split out the GCC versions in the GEOSBuilder, and try using GCC7, like in JuliaPackaging/BinaryProvider.jl#152

For development it is also convenient to build locally on Linux, with

julia --color=yes build_tarballs.jl --verbose --debug x86_64-w64-mingw32

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants