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.

Sign up for GitHub

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

Jump to bottom

Support C++11 IDL Mapping #9

Open
iguessthislldo opened this issue Feb 6, 2020 · 1 comment
Open

Support C++11 IDL Mapping #9

iguessthislldo opened this issue Feb 6, 2020 · 1 comment
Labels
OpenDDS C++ API Parity Features/behavior in OpenDDS C++ that isn't in PyOpenDDS
Milestone
v0.4.0

Comments

@iguessthislldo
Copy link
Member

Will need to get logic with deciding on what mapping is being used. Maybe also add that information into OpenDDS is not already there, so PyOpenDDS can select automatically.
@iguessthislldo iguessthislldo changed the title Support C++11 IDL Mappings Support C++11 IDL Mapping Feb 6, 2020
@iguessthislldo iguessthislldo added this to the v0.3 milestone Feb 6, 2020
@mitza-oci
Copy link
Member

AFAIK it is not yet in the OpenDDS CMake module, so that would be a good starting point.

@iguessthislldo iguessthislldo added the OpenDDS C++ API Parity Features/behavior in OpenDDS C++ that isn't in PyOpenDDS label Feb 20, 2020
@iguessthislldo iguessthislldo mentioned this issue Jun 3, 2020
Closed
iguessthislldo added a commit to iguessthislldo/OpenDDS that referenced this issue Jun 9, 2020
@iguessthislldo
Support for C++11 Mapping in the CMake Module
b901737 
Fix for OpenDDS#1714

Support for the C++11 mapping in the CMake module. `-Lc++11` can now be
passed with `OPENDDS_IDL_OPTIONS` in `OPENDDS_TARGET_SOURCES`.

Also added a property to the target of `OPENDDS_TARGET_SOURCES` called
`OPENDDS_LANGUAGE_MAPPING` that states what mapping was used. This will
be exported if supported by CMake. Fix for
OpenDDS/pyopendds#9.
@iguessthislldo iguessthislldo mentioned this issue Jun 9, 2020
Merged
iguessthislldo added a commit that referenced this issue Jun 19, 2020
@iguessthislldo
Initial Support for C++11 Mapping
2583f5a 
For #9, Uses OpenDDS/OpenDDS#1728 to
automatically switch the CPython mapping to use the IDL-to-C++11 mapping
instead of the IDL-to-C++03 mapping

Still need to refine it and setup CI testing.
iguessthislldo added a commit to iguessthislldo/OpenDDS that referenced this issue Oct 19, 2020
@iguessthislldo
Support for C++11 Mapping in the CMake Module
756d10b 
Fix for OpenDDS#1714

Support for the C++11 mapping in the CMake module. `-Lc++11` can now be
passed with `OPENDDS_IDL_OPTIONS` in `OPENDDS_TARGET_SOURCES`.

Also added a property to the target of `OPENDDS_TARGET_SOURCES` called
`OPENDDS_LANGUAGE_MAPPING` that states what mapping was used. This will
be exported if supported by CMake. Fix for
OpenDDS/pyopendds#9.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
OpenDDS C++ API Parity Features/behavior in OpenDDS C++ that isn't in PyOpenDDS
Projects
None yet
Milestone
v0.4.0
Development

No branches or pull requests
2 participants
@iguessthislldo @mitza-oci