Skip to content

Conversation

@EduPonz
Copy link

@EduPonz EduPonz commented Nov 28, 2023

eProsima/Fast-DDS#4045 introduced new == operators for ReturnCode_t, removing the existing template ones. This has caused a redefinition of _wrap___eq__ on the python bindings generated code. This PR fixes that issue by ignoring the new operators, thus keeping the same API in the bindings.

Fast DDS docs will not build until this PR is merged.

Signed-off-by: EduPonz <eduardoponz@eprosima.com>
@EduPonz EduPonz changed the title [20051: Fix redefinition of _wrap___eq__ [20051] Fix redefinition of _wrap___eq__ Nov 28, 2023
@EduPonz EduPonz marked this pull request as ready for review November 28, 2023 07:16
@EduPonz EduPonz merged commit 94fa8a9 into main Nov 28, 2023
@EduPonz EduPonz deleted the hotfix/operator_eq_redefinition branch November 28, 2023 10:01
EduPonz added a commit that referenced this pull request Dec 12, 2023
Signed-off-by: EduPonz <eduardoponz@eprosima.com>
@EduPonz
Copy link
Author

EduPonz commented Dec 12, 2023

@Mergifyio backport 1.1.x 1.2.x 1.3.x

@mergify
Copy link

mergify bot commented Dec 12, 2023

backport 1.1.x 1.2.x 1.3.x

✅ Backports have been created

mergify bot pushed a commit that referenced this pull request Dec 12, 2023
Signed-off-by: EduPonz <eduardoponz@eprosima.com>
(cherry picked from commit 94fa8a9)
mergify bot pushed a commit that referenced this pull request Dec 12, 2023
Signed-off-by: EduPonz <eduardoponz@eprosima.com>
(cherry picked from commit 94fa8a9)
mergify bot pushed a commit that referenced this pull request Dec 12, 2023
Signed-off-by: EduPonz <eduardoponz@eprosima.com>
(cherry picked from commit 94fa8a9)
EduPonz added a commit that referenced this pull request Dec 13, 2023
Signed-off-by: EduPonz <eduardoponz@eprosima.com>
(cherry picked from commit 94fa8a9)
Signed-off-by: EduPonz <eduardoponz@eprosima.com>
EduPonz added a commit that referenced this pull request Dec 13, 2023
Signed-off-by: EduPonz <eduardoponz@eprosima.com>
(cherry picked from commit 94fa8a9)
Signed-off-by: EduPonz <eduardoponz@eprosima.com>
EduPonz added a commit that referenced this pull request Dec 13, 2023
Signed-off-by: EduPonz <eduardoponz@eprosima.com>
(cherry picked from commit 94fa8a9)
Signed-off-by: EduPonz <eduardoponz@eprosima.com>
EduPonz added a commit that referenced this pull request Dec 14, 2023
* Fix redefinition of _wrap___eq__ (#73)

Signed-off-by: EduPonz <eduardoponz@eprosima.com>
(cherry picked from commit 94fa8a9)
Signed-off-by: EduPonz <eduardoponz@eprosima.com>

* Refactor workflow

Signed-off-by: EduPonz <eduardoponz@eprosima.com>

---------

Signed-off-by: EduPonz <eduardoponz@eprosima.com>
Co-authored-by: Eduardo Ponz Segrelles <eduardoponz@eprosima.com>
EduPonz added a commit that referenced this pull request Dec 14, 2023
* Fix redefinition of _wrap___eq__ (#73)

Signed-off-by: EduPonz <eduardoponz@eprosima.com>
(cherry picked from commit 94fa8a9)
Signed-off-by: EduPonz <eduardoponz@eprosima.com>

* Refactor workflow

Signed-off-by: EduPonz <eduardoponz@eprosima.com>

* Add /bigobj flag in windows

Signed-off-by: EduPonz <eduardoponz@eprosima.com>

---------

Signed-off-by: EduPonz <eduardoponz@eprosima.com>
Co-authored-by: Eduardo Ponz Segrelles <eduardoponz@eprosima.com>
EduPonz added a commit that referenced this pull request Dec 14, 2023
* Fix redefinition of _wrap___eq__ (#73)

Signed-off-by: EduPonz <eduardoponz@eprosima.com>
(cherry picked from commit 94fa8a9)
Signed-off-by: EduPonz <eduardoponz@eprosima.com>

* Refactor workflow

Signed-off-by: EduPonz <eduardoponz@eprosima.com>

* Add /bigobj flag in windows

Signed-off-by: EduPonz <eduardoponz@eprosima.com>

---------

Signed-off-by: EduPonz <eduardoponz@eprosima.com>
Co-authored-by: Eduardo Ponz Segrelles <eduardoponz@eprosima.com>
JesusPoderoso pushed a commit that referenced this pull request Feb 27, 2024
Signed-off-by: EduPonz <eduardoponz@eprosima.com>
JesusPoderoso pushed a commit that referenced this pull request Feb 27, 2024
Signed-off-by: EduPonz <eduardoponz@eprosima.com>
JesusPoderoso pushed a commit that referenced this pull request Feb 27, 2024
Signed-off-by: EduPonz <eduardoponz@eprosima.com>
JesusPoderoso added a commit that referenced this pull request Feb 27, 2024
Signed-off-by: EduPonz <eduardoponz@eprosima.com>
Co-authored-by: Eduardo Ponz Segrelles <eduardoponz@eprosima.com>
achintyamohan pushed a commit to boardwalkrobotics/Fast-DDS-python that referenced this pull request Aug 5, 2024
* Fix redefinition of _wrap___eq__ (eProsima#73)

Signed-off-by: EduPonz <eduardoponz@eprosima.com>
(cherry picked from commit 94fa8a9)
Signed-off-by: EduPonz <eduardoponz@eprosima.com>

* Refactor workflow

Signed-off-by: EduPonz <eduardoponz@eprosima.com>

* Add /bigobj flag in windows

Signed-off-by: EduPonz <eduardoponz@eprosima.com>

---------

Signed-off-by: EduPonz <eduardoponz@eprosima.com>
Co-authored-by: Eduardo Ponz Segrelles <eduardoponz@eprosima.com>
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