search

epics-base / ci-scripts

Continuous Integration Scripts for EPICS Modules
Other
8 stars 18 forks source link

Fix sticky `recurse="--recursive"` #24

Closed tboegi closed 4 years ago

tboegi commented 4 years ago

By default, modules are cloned recursive. (This is especially important for EPICS base 7).

However, when a module defines e.g. a module does should not be cloned recursive, e.g. MOTOR_RECURSIVE=NO then the shell variable "recurse" stays set to "--recurse" running git clone

Make the DEP_RECURSIVE work and reset it to an empty string if needed.

ralphlange commented 4 years ago

Thanks! I tend to forget that in bash all variables are global by default.

Eventually I used a slightly different fix (making that variable local as I had expected it to be) and added a regression test.