Skip to content

Fixed namespace in typesystem.xml#201

Merged
dirk-thomas merged 2 commits intoros-visualization:melodic-develfrom
stertingen:fix_shiboken
Feb 29, 2020
Merged

Fixed namespace in typesystem.xml#201
dirk-thomas merged 2 commits intoros-visualization:melodic-develfrom
stertingen:fix_shiboken

Conversation

@stertingen
Copy link
Copy Markdown
Contributor

@dirk-thomas
Copy link
Copy Markdown
Contributor

I also had to remove the include of pyside2_global.h in order to successfully build on Ubuntu Focal: see 22b5a69.

@dirk-thomas
Copy link
Copy Markdown
Contributor

This should probably target only Melodic+ since in Kinetic the PySide version works with the non-patched code. I will create a new melodic-devel branch soon...

@dirk-thomas dirk-thomas changed the title WIP: Fixed namespace in typesystem.xml Fixed namespace in typesystem.xml Feb 29, 2020
@dirk-thomas
Copy link
Copy Markdown
Contributor

@ros-pull-request-builder retest this please

@dirk-thomas dirk-thomas changed the base branch from kinetic-devel to melodic-devel February 29, 2020 05:09
@dirk-thomas
Copy link
Copy Markdown
Contributor

Noetic passing except a compiler warning in Qt itself: http://build.ros.org/job/Npr__qt_gui_core__ubuntu_focal_amd64/11/warnings24Result/

@dirk-thomas dirk-thomas merged commit 4a82a5a into ros-visualization:melodic-devel Feb 29, 2020
dirk-thomas added a commit that referenced this pull request Mar 2, 2020
* Fixed namespace in typesystem.xml

* remove include of pyside2_global.h

Co-authored-by: Dirk Thomas <dirk-thomas@users.noreply.github.com>
@dirk-thomas
Copy link
Copy Markdown
Contributor

Cherry-picked to crystal-devel: 1248b9a.

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants