search

machines-in-motion / real_time_tools

BSD 3-Clause "New" or "Revised" License
9 stars 6 forks source link

vector include bug resolved #10

Closed amsabyaneh closed 5 years ago

amsabyaneh commented 5 years ago

catkin_make fails due to this missing include.

MaximilienNaveau commented 5 years ago

Why did the continuous integration not see this?

luator commented 5 years ago

Ah, thanks for the PR! (I had this fixed locally already but forgot to make a PR...)

@MaximilienNaveau I think due to the OS-depending ifdefs, this only fails when building on a real-time computer (I also only had this on the robot PC, not on my laptop).

luator commented 5 years ago

@amsabyaneh Since everyone approved, you can merge this PR

amsabyaneh commented 5 years ago

Done.

On Mon, 30 Sep 2019 at 13:00, Felix Widmaier notifications@github.com wrote:

@amsabyaneh https://github.com/amsabyaneh Since everyone approved, you can merge this PR

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/machines-in-motion/real_time_tools/pull/10?email_source=notifications&email_token=AMWXTCVA24ZICDZAHXJ6K6LQMHL4BA5CNFSM4I2XGEB2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD75IK7A#issuecomment-536511868, or mute the thread https://github.com/notifications/unsubscribe-auth/AMWXTCU5SADVTQKOKIVFEDDQMHL4BANCNFSM4I2XGEBQ .