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

Wizard recipe: KaHyPar-v1.2.1 #3111

Merged

Conversation

Roger-luo
Copy link
Contributor

This pull request contains a new build recipe I built using the BinaryBuilder.jl wizard:

  • Package name: KaHyPar
  • Version: v1.2.1

@staticfloat please review and merge.

@Roger-luo
Copy link
Contributor Author

I think unfortunately kahypar is written in a non cross compilable way for windows
https://github.com/kahypar/kahypar/blob/master/kahypar/application/command_line_options.h#L25

I'm not sure how previous build script build windows successfully without install things like VS2019

]

# Build the tarballs, and possibly a `build.jl` as well.
build_tarballs(ARGS, name, version, sources, script, platforms, products, dependencies; preferred_gcc_version = v"6.1.0")
build_tarballs(ARGS, name, version, sources, script, platforms, products, dependencies; julia_compat="1.6", preferred_gcc_version = v"10.2.0")
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't recommend at all using GCC 10. In general use the oldest version of GCC possible, for maximum compatibility. If GCC 6 is enough to build the library, we should stick to it

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OK, I think kahypar recommends GCC 9, should we use GCC 9 then?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Recommends or requires? Everybody recommends the latest compiler, but we want the largest compatibility possible, which is given by the oldest GCC possible

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

it says "requirements" is GCC 9 or higher so I guess it is required: https://github.com/kahypar/kahypar#requirements

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

but it seems GCC supports C++17 since GCC 5: https://gcc.gnu.org/projects/cxx-status.html#cxx17

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

also should we remove the julia compat, I ran the wizard on julia 1.6, but I guess this is not required.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OK GCC 7 seems to be the lowest GCC version that works

@Roger-luo Roger-luo requested a review from giordano May 31, 2021 18:54
@giordano
Copy link
Member

giordano commented Jun 1, 2021

Ok, #3115 is not going to happen anytime soon, so let's go on with this one

@giordano giordano merged commit cb0b035 into JuliaPackaging:master Jun 1, 2021
@giordano giordano removed their request for review July 17, 2021 23:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants