Signed-off-by: Marco A. Gutierrez <marcogg@marcogg.com>

From what I can tell, it looks like lxml has a bug
where it doesn't properly track references to objects
via the find() method.  This manifests on Windows debug
as a crash *after* we have stopped using the object, but
I believe that by that point the underlying memory has
already been freed.  Windows Debug in particular is sensitive
to this.

Fix it by doing a deepcopy of the object returned from the
find().  This code isn't performance sensitive, so it shouldn't
be a big deal to do it here, and it fixes the bug in my testing.

Signed-off-by: Chris Lalancette <clalancette@gmail.com>
(cherry picked from commit bbccb6a2)

Co-authored-by: Chris Lalancette <clalancette@gmail.com>

* [tests] use tmp_path instead of deprecated tmpdir
* cover some error cases
* [test] test cli displays help if verb missing
* add some fixme in the code
* exclude setup.py from coverage reporting
* add high lever testing for generate_artifacts verb

Signed-off-by: Mikael Arguedas <mikael.arguedas@gmail.com>
(cherry picked from commit fb226616)

Co-authored-by: Mikael Arguedas <mikael.arguedas@gmail.com>

Signed-off-by: Marco A. Gutierrez <marcogg@marcogg.com>

Signed-off-by: Marco A. Gutierrez <marcogg@marcogg.com>

Signed-off-by: Mikael Arguedas <mikael.arguedas@gmail.com>

* add `--no-daemon` and `--spin-time` explanation in readme.
* increase `--spin-time` from 3 to 4 seconds based on CI.

---------

Signed-off-by: Tomoya Fujita <Tomoya.Fujita@sony.com>
(cherry picked from commit 1a4257d5)

Co-authored-by: Tomoya Fujita <Tomoya.Fujita@sony.com>

Signed-off-by: Mikael Arguedas <mikael.arguedas@gmail.com>
(cherry picked from commit 5600fe5d)

Co-authored-by: Mikael Arguedas <mikael.arguedas@gmail.com>

* clone policies to temporary dir as subversion hack doesnt work anymore

* add get_type_description service to policies

* update MacOS similarly

* update all permissions with new topics

* dont rule out cycloneDDS

* example of enclave override

Signed-off-by: Mikael Arguedas <mikael.arguedas@gmail.com>
Co-authored-by: Chris Lalancette <clalancette@gmail.com>
(cherry picked from commit ca6bb12c)

Co-authored-by: Mikael Arguedas <mikael.arguedas@gmail.com>

Signed-off-by: Chris Lalancette <clalancette@gmail.com>

* Use modern PKCS7 to sign the certificate bytes.

Using the SSLBinding leads to a warning in newer versions
of pycryptography.  Luckily, there is a supported API
called pkcs7 that allows us to do the same thing.  Even
better, this API is supported since pycryptography 3.2,
so this should work on both Ubuntu 22.04 and Ubuntu 24.04
without warnings.

* Keep fallback path for using SSLBinding.

Windows is still using pycryptography 2.9, which doesn't
support the new pkcs7 API.

Signed-off-by: Chris Lalancette <clalancette@gmail.com>

* Fix a number of warnings on Ubuntu 24.04.

In particular:
1.  Get rid of the fallback path for argcompleter.  It
    isn't necessary anymore since all versions of argcompleter
    support these, and it was confusing mypy.
2.  Add in a proper Optional annotation.
3.  Use the newer importlib_resources.file API when it is
    available.

* Add in appropriate dependency on python3-argcomplete.

Also do a minor refactoring of the dependencies while we
are here.

Signed-off-by: Chris Lalancette <clalancette@gmail.com>

Signed-off-by: Yadunund <yadunund@openrobotics.org>

Signed-off-by: Boris Boutillier <boris.boutillier@gmail.com>

Signed-off-by: Chris Lalancette <clalancette@gmail.com>

Signed-off-by: Yadunund <yadunund@openrobotics.org>

Signed-off-by: Chris Lalancette <clalancette@openrobotics.org>

Signed-off-by: Audrow Nash <audrow@hey.com>

Signed-off-by: Chris Lalancette <clalancette@openrobotics.org>

This should help the general flakiness of this test on Windows,
and makes it match the timeouts in test_generate_policy

Signed-off-by: Chris Lalancette <clalancette@openrobotics.org>

Signed-off-by: Chris Lalancette <clalancette@openrobotics.org>

Signed-off-by: Keisuke Shima <19993104+KeisukeShima@users.noreply.github.com>

Only when daemon is in use (if not, CLI will use a different node).

Signed-off-by: Michel Hidalgo <michel@ekumenlabs.com>

Signed-off-by: Mikael Arguedas <mikael.arguedas@gmail.com>

Signed-off-by: Mikael Arguedas <mikael.arguedas@gmail.com>

Signed-off-by: Scott K Logan <logans@cottsay.net>

Signed-off-by: Chris Lalancette <clalancette@openrobotics.org>

Signed-off-by: Chris Lalancette <clalancette@openrobotics.org>

* Enable ros_discovery_info for rmw_connextdds

Signed-off-by: Andrea Sorbini <asorbini@rti.com>

Signed-off-by: Scott K Logan <logans@cottsay.net>
Co-authored-by: Mikael Arguedas <mikael.arguedas@gmail.com>