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

Only build test thread_pool_executor_1114_test if HPX_LOCAL_SCHEDULER is set #1118

Merged

Conversation

andreasbuhr
Copy link
Contributor

Hi,

in the current master with default configuration, "make tests" fails.

This is because "thread_pool_executor_1114" requires local_queue_executor, which is disabled if HPX_LOCAL_SCHEDULER is set to false, which is the default. The error message is:

CMakeFiles/thread_pool_executor_1114_test_exe.dir/thread_pool_executor_1114.cpp.o: In function hpx_startup::user_main()': thread_pool_executor_1114.cpp:(.text+0xd9e): undefined reference tohpx::threads::executors::local_queue_executor::local_queue_executor()'

The proposed fix builds thread_pool_executor_1114 only if HPX_LOCAL_SCHEDULER is set to ON

hkaiser added a commit that referenced this pull request Apr 20, 2014
…14_test

only build test thread_pool_executor_1114_test if HPX_LOCAL_SCHEDULER is set
@hkaiser hkaiser merged commit dc39499 into STEllAR-GROUP:master Apr 20, 2014
@hkaiser
Copy link
Member

hkaiser commented Apr 20, 2014

Thanks!

@hkaiser hkaiser added this to the 0.9.9 milestone Apr 20, 2014
@hkaiser hkaiser self-assigned this Apr 20, 2014
@andreasbuhr andreasbuhr deleted the fix_thread_pool_executor_1114_test branch April 20, 2014 07:55
@hkaiser hkaiser changed the title only build test thread_pool_executor_1114_test if HPX_LOCAL_SCHEDULER is set Only build test thread_pool_executor_1114_test if HPX_LOCAL_SCHEDULER is set May 27, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category: examples difficulty: easy Good issues for starting out with HPX development type: enhancement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants