search

robotology / wearables

Code moved to https://github.com/robotology/human-dynamics-estimation
https://github.com/robotology/human-dynamics-estimation
BSD 3-Clause "New" or "Revised" License
19 stars 11 forks source link

Fix installation of Wearable/IWear/Utils.h install #129

Closed traversaro closed 3 years ago

traversaro commented 3 years ago

PR https://github.com/robotology/wearables/pull/126 move the Wearable/IWear/Sensors/Utils.h include to Wearable/IWear/Utils.h. However, this change was not accounted for in the installation rules, neither documented in the CHANGELOG. This PR fixes the first issue to fix https://github.com/robotology/robotology-superbuild/issues/846 .

traversaro commented 3 years ago

Thanks @traversaro! Unfortunately, error was not caught by the wearables CI and myself because I have just recompiled wearables. Do you have any suggestion to spot this kind of regression directly on wearables CI?

The typical trick for checking this kind of regression is to have some kind of example to show how to use the library to users, and then compile that examples in the CI using wearables as an external project, see for example https://github.com/robotology/idyntree/blob/v4.2.0/.github/workflows/ci.yml#L301 . Another alternative is also to build a downstream project (such as HDE) in the CI.