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

Make errors related to init_globally.cpp example while building HPX out of the box #2020

Closed
satyaki3794 opened this issue Mar 11, 2016 · 5 comments

Comments

@satyaki3794
Copy link
Contributor

After the latest commit (https://github.com/STEllAR-GROUP/hpx/pull/2005/files), I am not being able to build HPX out-of-the-box. I get make errors related to init_globally.cpp. Gist file is here: https://gist.github.com/satyaki3794/a7bf80a8678c55433065

@hkaiser hkaiser added this to the 0.9.12 milestone Mar 11, 2016
@hkaiser
Copy link
Member

hkaiser commented Mar 11, 2016

For now I have disabled the failing example. I'll leave this ticket open as a reminder that it still needs to be fixed properly.

@hkaiser
Copy link
Member

hkaiser commented Mar 11, 2016

Some background: the example fails to compile on different platforms for different reasons. The first reason is that Boost versions 1.55 and below don't support optional<>::emplace. The second reason is that older versions of gcc (and other compilers) don't support expanding template parameter packs inside a lambda expression where the pack is captured from the outer context (see for instance here: https://gcc.gnu.org/bugzilla/show_bug.cgi?id=47226)..

@satyaki3794
Copy link
Contributor Author

I am using Boost V 1.59 and gcc 4.8.2 (stock gcc with Ubuntu 14.04). Yes, older version of gcc may be the reason (I can see an error related to no support for expanding template parameter).

@hkaiser
Copy link
Member

hkaiser commented Mar 11, 2016

[07:07] hkaiser: satyaki3794: do you think you'd be able to fix #2020?
[07:08] hkaiser: satyaki3794: I think it's fairly easy
[07:08] hkaiser: the boost version could be handled like here: https://github.com/STEllAR-GROUP/hpx/blob/master/hpx/parallel/executors/executor_traits.hpp#L166, the lambda/parameter pack issue could be worked around by replacing the lambdas with hpx::util::bind
[07:09] satyaki3794: hkaiser: okay, i'll try it
[07:09] hkaiser: great!

@hkaiser
Copy link
Member

hkaiser commented Mar 14, 2016

This was fixed by merging #2023

@hkaiser hkaiser closed this as completed Mar 14, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants