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

fix: compile on Ubuntu 21.10 and Unreal 4.27.2 #4365

Closed
wants to merge 2 commits into from

Conversation

russkel
Copy link

@russkel russkel commented Feb 22, 2022

Fixes: #

  • Not compiling on Ubuntu 21.10 and Unreal 4.27.2

About

How Has This Been Tested?

Successfully compiled on the machine described above.

Screenshots (if appropriate):

Work for ROS2 UE4 Camera Pub [sc-2974]
@ghost
Copy link

ghost commented Feb 22, 2022

CLA assistant check
All CLA requirements met.

@jonyMarino jonyMarino marked this pull request as ready for review April 7, 2022 17:19
@jonyMarino jonyMarino closed this Apr 7, 2022
@jonyMarino jonyMarino reopened this Apr 7, 2022
Copy link
Contributor

@zimmy87 zimmy87 left a comment

Choose a reason for hiding this comment

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

there's a build break and some commented out code that needs to be removed before this is ready for merging

Comment on lines +111 to +113
// needed when packaging
// PublicAdditionalLibraries.Add("stdc++");
// PublicAdditionalLibraries.Add("supc++");
Copy link
Contributor

Choose a reason for hiding this comment

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

commented out code should be deleted; it can be found in source history if need be

@@ -38,17 +38,17 @@ else #linux
software-properties-common \
wget \
libvulkan1 \
vulkan-utils
vulkan-tools
Copy link
Contributor

Choose a reason for hiding this comment

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

this change in dependencies is breaking the ubuntu 18.04 build:

+ sudo apt-get -y install --no-install-recommends lsb-release rsync software-properties-common wget libvulkan1 vulkan-tools
Reading package lists...
Building dependency tree...
Reading state information...
E: Unable to locate package vulkan-tools
Error: Process completed with exit code 100.

Can you make this logic conditional or change it to a dependency that both ubuntu 21.10 and 18.04 share?

@russkel
Copy link
Author

russkel commented Jun 11, 2022

Closing this in favour of the work done by @zimmy87 in #4561

@russkel russkel closed this Jun 11, 2022
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.

3 participants