X Tutup
Skip to content

Fix missing include#5833

Merged
sturnclaw merged 1 commit intopioneerspacesim:masterfrom
impaktor:fix-include
May 18, 2024
Merged

Fix missing include#5833
sturnclaw merged 1 commit intopioneerspacesim:masterfrom
impaktor:fix-include

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
@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

X Tutup