Skip to content

Commit

Permalink
Check suprocess return codes on RDT generation (#795) (#796)
Browse files Browse the repository at this point in the history
* Check subprocess return codes on RDT generation (#795)

Signed-off-by: eduponz <[email protected]>
(cherry picked from commit 0974196)

* Refs #21114: Remove unsued Doxyfile EXAMPLE_PATH

Signed-off-by: eduponz <[email protected]>

* Refs #21114: Fix Fast DDS Python API reference warnings regarding Qos aliases

Signed-off-by: eduponz <[email protected]>

* Refs #21114: Fail generation when docutils errors

Signed-off-by: eduponz <[email protected]>

* Refs #21114: Use correct branches of Fast DDS and Fast DDS Python in RTD

Signed-off-by: eduponz <[email protected]>

---------

Signed-off-by: eduponz <[email protected]>
Co-authored-by: Eduardo Ponz Segrelles <[email protected]>
  • Loading branch information
2 people authored and JesusPoderoso committed Jul 3, 2024
1 parent 1e5c8dd commit 7016602
Show file tree
Hide file tree
Showing 10 changed files with 16 additions and 29 deletions.
2 changes: 1 addition & 1 deletion code/doxygen-config.in
Original file line number Diff line number Diff line change
Expand Up @@ -985,7 +985,7 @@ EXCLUDE_SYMBOLS =
# that contain example code fragments that are included (see the \include
# command).

EXAMPLE_PATH = @PROJECT_SOURCE_DIR@/code
EXAMPLE_PATH =

# If the value of the EXAMPLE_PATH tag contains directories, you can use the
# EXAMPLE_PATTERNS tag to specify one or more wildcard pattern (like *.cpp and
Expand Down
27 changes: 6 additions & 21 deletions docs/conf.py
Original file line number Diff line number Diff line change
Expand Up @@ -216,22 +216,13 @@ def configure_doxyfile(
fastdds_branch = os.environ.get('FASTDDS_BRANCH', None)

# First try to checkout to ${FASTDDS_BRANCH}
# Else try with current documentation branch
# Else checkout to master
# Else checkout to 2.13.x
if (fastdds_branch and
fastdds.refs.__contains__('origin/{}'.format(fastdds_branch))):
fastdds_branch = 'origin/{}'.format(fastdds_branch)
elif (docs_branch and
fastdds.refs.__contains__('origin/{}'.format(docs_branch))):
fastdds_branch = 'origin/{}'.format(docs_branch)
else:
print(
'Fast DDS does not have either "{}" or "{}" branches'.format(
fastdds_branch,
docs_branch
)
)
fastdds_branch = 'origin/master'
fastdds_branch = 'origin/2.13.x'
print(f'Fast DDS branch is not set by env var. Using "{fastdds_branch}"')

# Actual checkout
print('Checking out Fast DDS branch "{}"'.format(fastdds_branch))
Expand All @@ -248,20 +239,14 @@ def configure_doxyfile(
fastdds_python_branch = os.environ.get('FASTDDS_PYTHON_BRANCH', None)

# First try to checkout to ${FASTDDS_PYTHON_BRANCH}
# Else try with current documentation branch
# Else checkout to master
# Else checkout to 1.4.x
if (fastdds_python_branch and
fastdds_python.refs.__contains__(
'origin/{}'.format(fastdds_python_branch))):
fastdds_python_branch = 'origin/{}'.format(fastdds_python_branch)
elif (docs_branch and
fastdds_python.refs.__contains__('origin/{}'.format(docs_branch))):
fastdds_python_branch = 'origin/{}'.format(docs_branch)
else:
print(
'Fast DDS Python does not have either "{}" or "{}" branches'
.format(fastdds_python_branch, docs_branch))
fastdds_python_branch = 'origin/main'
fastdds_python_branch = 'origin/1.4.x'
print(f'Fast DDS Python branch is not set by env var. Using "{fastdds_python_branch}"')

# Actual checkout
print('Checking out Fast DDS Python branch "{}"'.format(
Expand Down
2 changes: 2 additions & 0 deletions docs/docutils.conf
Original file line number Diff line number Diff line change
@@ -0,0 +1,2 @@
[general]
halt_level: 2
Original file line number Diff line number Diff line change
Expand Up @@ -8,5 +8,4 @@ DomainParticipantQos
.. autoclass:: fastdds.DomainParticipantQos


.. TODO
.. autoclass:: fastdds.PARTICIPANT_QOS_DEFAULT
.. autodata:: fastdds.PARTICIPANT_QOS_DEFAULT
Original file line number Diff line number Diff line change
Expand Up @@ -7,4 +7,4 @@ DataWriterQos

.. autoclass:: fastdds.DataWriterQos

.. autoclass:: fastdds.DATAWRITER_QOS_DEFAULT
.. autodata:: fastdds.DATAWRITER_QOS_DEFAULT
Original file line number Diff line number Diff line change
Expand Up @@ -7,4 +7,4 @@ PublisherQos

.. autoclass:: fastdds.PublisherQos

.. autoclass:: fastdds.PUBLISHER_QOS_DEFAULT
.. autodata:: fastdds.PUBLISHER_QOS_DEFAULT
Original file line number Diff line number Diff line change
Expand Up @@ -7,4 +7,4 @@ DataReaderQos

.. autoclass:: fastdds.DataReaderQos

.. autoclass:: fastdds.DATAREADER_QOS_DEFAULT
.. autodata:: fastdds.DATAREADER_QOS_DEFAULT
Original file line number Diff line number Diff line change
Expand Up @@ -7,4 +7,4 @@ SubscriberQos

.. autoclass:: fastdds.SubscriberQos

.. autoclass:: fastdds.SUBSCRIBER_QOS_DEFAULT
.. autodata:: fastdds.SUBSCRIBER_QOS_DEFAULT
Original file line number Diff line number Diff line change
Expand Up @@ -7,4 +7,4 @@ TopicQos

.. autoclass:: fastdds.TopicQos

.. autoclass:: fastdds.TOPIC_QOS_DEFAULT
.. autodata:: fastdds.TOPIC_QOS_DEFAULT
1 change: 1 addition & 0 deletions docs/installation/sources/sources_mac.rst
Original file line number Diff line number Diff line change
Expand Up @@ -402,4 +402,5 @@ The :code:`Fast-DDS-Gen` folder contains the following packages:
.. _OpenSSL: https://www.openssl.org/
.. _Gtest: https://github.com/google/googletest
.. _vcstool: https://pypi.org/project/vcstool/

.. _xmlschema: https://pypi.org/project/xmlschema/

0 comments on commit 7016602

Please sign in to comment.