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

Remove Bionic from Garden #6

Merged
merged 2 commits into from
Mar 11, 2022
Merged

Remove Bionic from Garden #6

merged 2 commits into from
Mar 11, 2022

Conversation

chapulina
Copy link
Contributor

@chapulina chapulina commented Mar 11, 2022

Update

Removing all of Bionic


$ diff -u ubuntu/debian/control bionic/debian/control 
--- ubuntu/debian/control       2022-03-10 16:00:50.794942191 -0800
+++ bionic/debian/control       2022-03-10 15:58:11.524577257 -0800
@@ -5,15 +5,11 @@                                                     
 Priority: extra
 Build-Depends: cmake,       
                debhelper (>= 11),
-               dh-python,                                            
                doxygen,                                              
                libeigen3-dev,                                        
                libignition-cmake2-dev (>= 2.8.0) | libignition-cmake-dev (>= 2.8.0),                                   
                libignition-utils1-dev,                           
                python3,
-               python3-dev,
-               python3-distutils,
-               python3-pybind11,
                ruby-dev,
                ruby-ronn,
                swig
@@ -21,7 +17,6 @@
 Vcs-Git: https://github.com/ignition-release/ign_math-release
 Homepage: https://github.com/ignition-release/ign_math
 XS-Ruby-Versions: all
-X-Python3-Version: >= 3.2
  
 Package: libignition-math7
 Architecture: any
@@ -73,25 +68,6 @@
  .
  The package contains the development files relative to Eigen
  
-Package: python3-ignition-math7
-Architecture: any
-Depends: libignition-math7 (= ${binary:Version}),
-         python3-distutils,
-         python3-pybind11,
-         ${misc:Depends},
-         ${python3:Depends}
-Enhances: libignition-math7
-Description: Ignition Robotics Math Library - Python3 bindings
- A small, fast, and high performance math library. This library is a
- self-contained set of classes and functions suitable for robot applications.
- .
- Ignition Robotics is a set of simple libraries that provide useful
- functionality to bootstrap robot applications. The included libraries
- encapsulate all the essentials, such as common math data types, console
- logging, 3D mesh management, and asynchronous message passing.
- .
- The package contains the Python3 bindings.
-
 Package: ruby-ignition-math7
 Architecture: any
 XB-Ruby-Versions: ${ruby:Versions} 

@chapulina chapulina requested a review from scpeters March 11, 2022 00:02
@chapulina
Copy link
Contributor Author

Test build: Build Status

@scpeters
Copy link
Contributor

Test build: Build Status

dh_missing: usr/lib/ruby/vendor_ruby/ignition/math.so exists in debian/tmp but is not installed to anywhere

I think we also need a bionic symlink to ubuntu/debian/ruby-ignition-math.install

Signed-off-by: Louise Poubel <[email protected]>
@chapulina
Copy link
Contributor Author

chapulina commented Mar 11, 2022

Ahh I remember now what I was thinking when I did this. Garden is dropping Bionic, so we can remove it altogether: 2390535

@chapulina chapulina changed the title Remove python package from Bionic Remove Bionic Mar 11, 2022
@chapulina chapulina changed the title Remove Bionic Remove Bionic from Garden Mar 11, 2022
@chapulina chapulina merged commit 77bca64 into main Mar 11, 2022
@chapulina chapulina deleted the chpaulina/python/bionic branch March 11, 2022 02:06
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.

2 participants