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

The "attach-debugger" option is broken on the current master branch. #6282

Closed
JiakunYan opened this issue Jun 16, 2023 · 1 comment · Fixed by #6283
Closed

The "attach-debugger" option is broken on the current master branch. #6282

JiakunYan opened this issue Jun 16, 2023 · 1 comment · Fixed by #6283

Comments

@JiakunYan
Copy link
Contributor

Expected Behavior

The option is accepted by HPX.

Actual Behavior

HPX complaint

hpx::init: exception caught: Unknown/misspelled HPX command line option found: --hpx:attach-debugger=startup

Steps to Reproduce the Problem

  1. Compile the current master branch.
  2. Run ./fibonacci_futures_distributed --hpx:attach-debugger=startup

Specifications

  • HPX Version: The current master branch 734d37f039ca00338fff4711aba7804fb1e778a0
  • Platform (compiler, OS): Ubuntu 20.04. GCC 9.4.
@JiakunYan
Copy link
Contributor Author

The related code are pasted below.

#if defined(_POSIX_VERSION) || defined(HPX_WINDOWS)
            ("hpx:attach-debugger",
                value<std::string>()->implicit_value("startup"),
                "wait for a debugger to be attached, possible values: "
                "off, startup, exception or test-failure (default: startup)")
#endif

It seems _POSIX_VERSION never got defined because unistd.h isn't included when compiling that file.

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

Successfully merging a pull request may close this issue.

2 participants