While we're changing this, could you put all the variables at the top of the file please? There's no reason to have them inside a function, and it could easily cause bugs in the future (writing code in another function but assuming they are defined, but actually they're not... particularly bad in the case of ```rm -rf ${CFE_BIN_DIR}/*``` for example!)