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

Add unversioned symlinks for engine plugins #141

Merged
merged 2 commits into from
Oct 2, 2020

Conversation

JShep1
Copy link

@JShep1 JShep1 commented Sep 23, 2020

Another step towards getting full support for custom render engines from command line. This PR symlinks binaries that don't contain the version number (same as how physics is doing for dartsim and tpe) so the user will not have to specify the version of ignition-rendering they are currently using when specifying an engine binary name. More progress towards getting #100 resolved

Signed-off-by: John Shepherd [email protected]

@JShep1 JShep1 requested a review from iche033 as a code owner September 23, 2020 21:34
@github-actions github-actions bot added the 🏰 citadel Ignition Citadel label Sep 23, 2020
@codecov
Copy link

codecov bot commented Sep 23, 2020

Codecov Report

Merging #141 into ign-rendering3 will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@               Coverage Diff               @@
##           ign-rendering3     #141   +/-   ##
===============================================
  Coverage           50.41%   50.41%           
===============================================
  Files                 129      129           
  Lines               11682    11682           
===============================================
  Hits                 5889     5889           
  Misses               5793     5793           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update dd6fa83...f532ca7. Read the comment docs.

ogre/src/CMakeLists.txt Outdated Show resolved Hide resolved
Signed-off-by: John Shepherd <[email protected]>
Copy link
Contributor

@iche033 iche033 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

new windows build looks good

@JShep1 JShep1 merged commit e0c1096 into ign-rendering3 Oct 2, 2020
@JShep1 JShep1 deleted the jshep1/add_unversioned_simlinks branch October 2, 2020 17:10
@scpeters
Copy link
Member

we need to update the debian metadata so that the engine-plugins folder will be included in our debian packages: gazebo-release/ign-rendering4-release#2

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🏰 citadel Ignition Citadel
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants