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

replaced deprecated --force-yes command #2595

Conversation

matteobarbera
Copy link
Contributor

Fix for Issue #2594

@gautierhattenberger
Copy link
Member

It seems that the problem is coming from gazebo package from ROS. The version for 18.04 bionic have been update the Oct 3, 2020, but the reference to dependency packages seems to be broken (9.15.0-1~bionic replaced by 9.15.0-2~bionic). Maybe it will solve by itself when they fix the problem.
Or we could use the version from ubuntu which is much older (9.0.0+dfsg5-3ubuntu1). Is it problem ? Since it is only for compilation tests, maybe it is good enough.

@dewagter
Copy link
Member

dewagter commented Oct 5, 2020

Since only for compilation tests, the old version is OK if it compiles.

@gautierhattenberger
Copy link
Member

@matteobarbera can you try to update your PR without adding the ROS/Gazebo repository to the travis config ?

@matteobarbera
Copy link
Contributor Author

@gautierhattenberger do you mean removing the Gazebo repository from the pre-install commands?

paparazzi/.travis.yml

Lines 10 to 11 in fab1253

- sudo sh -c 'echo "deb http://packages.osrfoundation.org/gazebo/ubuntu-stable `lsb_release -cs` main" > /etc/apt/sources.list.d/gazebo-stable.list'
- wget http://packages.osrfoundation.org/gazebo.key -O - | sudo apt-key add -

It seems that this is a recurring issue, and is being addressed gazebosim/gazebo-classic#2744 and gazebo-tooling/release-tools#307

@gautierhattenberger
Copy link
Member

yes, remove the Gazebo repository, it should use the (old) version from ubuntu

@matteobarbera
Copy link
Contributor Author

The Travis job log shows that the different Gazebo packages were taken from the much older Ubuntu versions as you expected
Get:101 http://archive.ubuntu.com/ubuntu bionic/universe amd64 gazebo9-common all 9.0.0+dfsg5-3ubuntu1 [34.4 MB]

Copy link
Member

@gautierhattenberger gautierhattenberger left a comment

Choose a reason for hiding this comment

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

it seems fine to use old gazebo for compilation tests

@gautierhattenberger gautierhattenberger merged commit cd9cb69 into paparazzi:master Oct 7, 2020
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.

None yet

3 participants