jpsamper2009

Results 5 comments of jpsamper2009

Applying the following patch and renaming `zstd_vendor-extras.cmake` to `zstd_vendor-extras.cmake.in` seems to fix the issue (but I haven't tested if it breaks other builds). ```diff diff --git a/zstd_vendor/CMakeLists.txt b/zstd_vendor/CMakeLists.txt index e3d4a22..e682a58...

@emersonknapp For sure. I just wanted to make sure it is the correct approach before going through the effort 😅

@dirk-thomas > [is] this is really a programming time decision? - I would argue yes, since one has to write one's code in such a way that it will be...

@mikaelarguedas, that was the functionality I was looking for :), but I guess it would also be nice to have it in this context too (as Dirk said) @dirk-thomas After...

Sounds good. > On Aug 21, 2017, at 9:25 AM, Dirk Thomas wrote: > > That being said, it looks like ament_package is not using the resource index for dependencies...