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 Python API generation [20261] #647

Merged
merged 1 commit into from
Jan 17, 2024
Merged

Fix Python API generation [20261] #647

merged 1 commit into from
Jan 17, 2024

Conversation

richiware
Copy link
Member

@richiware richiware commented Jan 17, 2024

The inclusion of fastcdr/config.h in Fast DDS Python's fastdds.i created a regression on the RTD generated documentation, as the Python API reference generation done with SWIG could not find that fastcdr/config.h file. This PR fixes that issue by creating the expected file (in opposition to cloning Fast CDR and configuring the real config.h.

Signed-off-by: Ricardo González Moreno <[email protected]>
@richiware richiware changed the title Fix Python API generation Fix Python API generation [20261] Jan 17, 2024
@EduPonz EduPonz merged commit 5de9f3c into master Jan 17, 2024
4 checks passed
@EduPonz EduPonz deleted the bugfix/python-api branch January 17, 2024 11:53
@EduPonz
Copy link

EduPonz commented Jan 17, 2024

@Mergifyio backport 2.12.x

Copy link

mergify bot commented Jan 17, 2024

backport 2.12.x

✅ Backports have been created

mergify bot pushed a commit that referenced this pull request Jan 17, 2024
Signed-off-by: Ricardo González Moreno <[email protected]>
(cherry picked from commit 5de9f3c)
@EduPonz EduPonz added this to the v2.13.2 milestone Jan 17, 2024
elianalf pushed a commit that referenced this pull request Mar 13, 2024
Signed-off-by: Ricardo González Moreno <[email protected]>
(cherry picked from commit 5de9f3c)

Co-authored-by: Ricardo González <[email protected]>
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