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

Extend Cmake for building hpx as distribution packages (refs #3575) #3585

Merged
merged 3 commits into from Dec 19, 2018

Conversation

diehlpk
Copy link
Member

@diehlpk diehlpk commented Dec 7, 2018

Fixes #3575

Proposed Changes

  • Add HPX_WITH_BUILD_DISTRIBUTION, which disables rdsctp optimization. This option needs to be turned when we build hpx as an rpm package

Any background context you want to provide?

Building the rpm packages Fedora's build infrastructure or openSuse's build infrastructure requires to disable some optimization, since we do not know of the target architecture provide it.

CMakeLists.txt Show resolved Hide resolved
Copy link

@bmwiedemann bmwiedemann left a comment

Choose a reason for hiding this comment

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

  1. There is a typo in "infrastructures" in the commit message,
  2. once everyone is happy, the commits probably should be squashed together to make the git history more readable - use git rebase --interactive origin/master
  3. it is still "openSUSE", not openSuse (in the PR description ;-)

CMakeLists.txt Outdated Show resolved Hide resolved
@diehlpk
Copy link
Member Author

diehlpk commented Dec 8, 2018

it is still "openSUSE", not openSuse (in the PR description ;-)
@junghans Could you update this? Thanks

junghans
junghans previously approved these changes Dec 9, 2018
@junghans
Copy link
Contributor

junghans commented Dec 9, 2018

it is still "openSUSE", not openSuse (in the PR description ;-)
@junghans Could you update this? Thanks

@bmwiedemann, from your comment in #3575 (comment) I thought "openSUSE" is the correct capitalization, right?

CMakeLists.txt Outdated Show resolved Hide resolved
@bmwiedemann
Copy link

@junghans yes, openSUSE is the correct way, but look at this PR's description...

@junghans
Copy link
Contributor

I see @diehlpk can edit this!

Copy link
Member

@hkaiser hkaiser left a comment

Choose a reason for hiding this comment

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

LGTM,thanks!

@diehlpk
Copy link
Member Author

diehlpk commented Dec 19, 2018

@msimberg Again it is failing for cuda ans my changes within cmake should not affect this. Could you please merge the branch?

@msimberg msimberg merged commit 9bc1db1 into STEllAR-GROUP:master Dec 19, 2018
@msimberg msimberg added this to the 1.3.0 milestone Dec 19, 2018
@diehlpk diehlpk deleted the openSUSEpatch branch December 19, 2018 14:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants