Skip to content
This repository has been archived by the owner on Feb 3, 2025. It is now read-only.

Backport qwt 6.2 build fix to gazebo9 #3058

Merged
merged 2 commits into from
Aug 11, 2021

Conversation

scpeters
Copy link
Member

@scpeters scpeters commented Aug 7, 2021

Backport #3047 to gazebo9 to fix build:

In qwt 6.2, they have renamed all member variables,
so create helper functions to minimize the ifdefs we will need
to support both versions.

  • PlotCurve: add ifdefs to support qwt 6.2.0

Remember to use Rebase and Merge.

* IncrementalPlot.cc: include qwt_scale_map.h
* Use Qt::MiddleButton to fix deprecation warning
* PlotCurve: add private helper for renamed members

In qwt 6.2, they have renamed all member variables,
so create helper functions to minimize the ifdefs we will need
to support both versions.

* PlotCurve: add ifdefs to support qwt 6.2.0

Signed-off-by: Steven Peters <scpeters@openrobotics.org>
@scpeters scpeters requested a review from WilliamLewww August 7, 2021 09:16
We currently don't support 2021 and newer, see gazebosim#2867.

Signed-off-by: Steven Peters <scpeters@openrobotics.org>
@scpeters
Copy link
Member Author

scpeters commented Aug 8, 2021

I just back-ported #3037 as well

@scpeters scpeters requested a review from iche033 August 9, 2021 04:05
@scpeters
Copy link
Member Author

scpeters commented Aug 9, 2021

CI looks ok to me

@scpeters scpeters requested a review from mjcarroll August 11, 2021 17:06
@scpeters scpeters merged commit f8b4d09 into gazebosim:gazebo9 Aug 11, 2021
@scpeters scpeters deleted the backport_qwt62_9 branch August 11, 2021 18:37
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants