Skip to content

Prototype code for seeing if FindPython3 is usable for rosidl_python#140

Merged
clalancette merged 10 commits intorollingfrom
use_FindPython3__prototype
Feb 11, 2024
Merged

Prototype code for seeing if FindPython3 is usable for rosidl_python#140
clalancette merged 10 commits intorollingfrom
use_FindPython3__prototype

Conversation

@sloretz
Copy link
Copy Markdown
Contributor

@sloretz sloretz commented Aug 4, 2021

This is a prototype intended to see how usable FindPython3 on non-linux platforms is. rosidl_generator_py makes the most use of FindPythonExtra.

Should inform ros2/python_cmake_module#7

@sloretz sloretz self-assigned this Aug 4, 2021
@sloretz
Copy link
Copy Markdown
Contributor Author

sloretz commented Aug 4, 2021

Full CI with build type None

  • Linux Build Status
  • Linux-aarch64 Build Status
  • macOS Build Status
  • Windows Build Status

Full CI with build type Release

  • Linux Build Status
  • Linux-aarch64 Build Status
  • macOS Build Status
  • Windows Build Status

Not doing Debug, because I would expect this to fail on windows as it's currently written

@sloretz sloretz changed the title Prototype code for seeing if FindPython3 is useable for rosidl_python Prototype code for seeing if FindPython3 is usable for rosidl_python Aug 4, 2021
@sloretz
Copy link
Copy Markdown
Contributor Author

sloretz commented Aug 4, 2021

Full CI with build type None

  • Linux Build Status
  • Linux-aarch64 Build Status
  • macOS Build Status (Edit: issue with buildfarm machine, failed to delete workspace)
  • Windows Build Status

@sloretz sloretz force-pushed the use_FindPython3__prototype branch from c73fd47 to 42df4d9 Compare August 5, 2021 17:26
@audrow audrow changed the base branch from master to rolling June 28, 2022 14:24
@ivanpauno
Copy link
Copy Markdown
Member

@sloretz I'm curious what were the issues with this, do you remember?
The CI jobs are too old so I cannot see the logs. Thanks!

@clalancette clalancette force-pushed the use_FindPython3__prototype branch from 42df4d9 to 191a804 Compare November 3, 2023 19:16
@clalancette clalancette force-pushed the use_FindPython3__prototype branch from 5351042 to a899bf7 Compare November 16, 2023 22:03
@clalancette clalancette force-pushed the use_FindPython3__prototype branch from 6ec3578 to 1b18ac0 Compare January 3, 2024 19:44
@clalancette clalancette marked this pull request as ready for review January 5, 2024 01:21
@clalancette
Copy link
Copy Markdown
Contributor

CI for this is over in ros2/python_cmake_module#7 (comment)

sloretz and others added 9 commits January 26, 2024 20:06
Signed-off-by: Chris Lalancette <clalancette@gmail.com>
Signed-off-by: Chris Lalancette <clalancette@gmail.com>
Signed-off-by: Chris Lalancette <clalancette@gmail.com>
Signed-off-by: Chris Lalancette <clalancette@gmail.com>
In the first stage, we just find the interpreter.  For
the second stage, we use the debug interpreter as
necessary to find NumPy.

Signed-off-by: Chris Lalancette <clalancette@gmail.com>
Signed-off-by: Chris Lalancette <clalancette@gmail.com>
Signed-off-by: Chris Lalancette <clalancette@gmail.com>
Signed-off-by: Chris Lalancette <clalancette@gmail.com>
@clalancette clalancette force-pushed the use_FindPython3__prototype branch from b195abb to 4b6622e Compare January 26, 2024 20:12
python_cmake_module is now only exporting that.

Signed-off-by: Chris Lalancette <clalancette@gmail.com>
Copy link
Copy Markdown
Contributor Author

@sloretz sloretz left a comment

Choose a reason for hiding this comment

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

LGTM!

@clalancette
Copy link
Copy Markdown
Contributor

CI for this is over in ros2/python_cmake_module#7 (comment)

@clalancette clalancette merged commit 1505ac0 into rolling Feb 11, 2024
@clalancette clalancette deleted the use_FindPython3__prototype branch February 11, 2024 17:42
wentasah added a commit to wentasah/nix-ros-overlay that referenced this pull request Aug 8, 2024
wentasah added a commit to wentasah/nix-ros-overlay that referenced this pull request Aug 8, 2024
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