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

Fix missing include #5833

Merged
merged 1 commit into from
May 18, 2024
Merged

Fix missing include #5833

merged 1 commit into from
May 18, 2024

Conversation

impaktor
Copy link
Member

I've adopted the pioneer-git repo in AUR (Arch Linux User) repository, that tracks our master branch. As such, I note the package is currently not building:

[ 15%] Building CXX object CMakeFiles/pioneer-lib.dir/src/HyperspaceCloud.cpp.o
[ 16%] Building CXX object CMakeFiles/pioneer-lib.dir/src/Input.cpp.o
In file included from /tmp/src/pioneer-git/src/Input.cpp:4:
/tmp/src/pioneer-git/src/Input.h: In member function ‘void Input::Manager::GetMouseMotion(int*)’:
/tmp/src/pioneer-git/src/Input.h:206:22: error: ‘copy_n’ is not a member of ‘std’; did you mean ‘copy’?
  206 |                 std::copy_n(mouseMotion.data(), mouseMotion.size(), motion);
      |                      ^~~~~~
      |                      copy
make[2]: *** [CMakeFiles/pioneer-lib.dir/build.make:524: CMakeFiles/pioneer-lib.dir/src/Input.cpp.o] Error 1
make[1]: *** [CMakeFiles/Makefile2:316: CMakeFiles/pioneer-lib.dir/all] Error 2
make: *** [Makefile:136: all] Error 2
make: Leaving directory '/tmp/src/pioneer-git/build'
==> ERROR: A failure occurred in build().
    Aborting...

I dunno what our policy is on missing includes / implicit includes, but this fixes that.

@sturnclaw sturnclaw merged commit 8691843 into pioneerspacesim:master May 18, 2024
5 checks passed
@impaktor impaktor deleted the fix-include branch May 19, 2024 13:54
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