search

robotology / yarp-matlab-bindings

SWIG-based bindings of YARP for Matlab and Octave, based on the MEX C-API.
6 stars 5 forks source link

Two variables named home and HOME #69

Closed traversaro closed 3 years ago

traversaro commented 3 years ago

yarp::conf::dirs::HOME generated HOME.m, while yarp::conf::dirs::home() generate home.m . They work fine on systems where filesystem is case sensitive, but create problem when fs is case insensitive.

traversaro commented 3 years ago

As a first glance, it seems that yarp::conf::dirs::HOME (https://github.com/robotology/yarp/blob/a17b8e762f89945c8690670fd23c7c98fdc0887d/src/libYARP_conf/src/yarp/conf/dirs.h#L35) can be ignored. While this can be ignored in the yarp.i, for the time being I can just remove it from the repo (done in https://github.com/robotology/yarp-matlab-bindings/commit/80d541e7afcb69cfb9fb8902661870a37fe2b5b6).

traversaro commented 3 years ago

After https://github.com/robotology/yarp/pull/2574 we should re-generate the bindings to avoid depending on the manual fix https://github.com/robotology/yarp-matlab-bindings/commit/80d541e7afcb69cfb9fb8902661870a37fe2b5b6 .