This fails in case USER_CMSSW_REPO is not defined. [ ! "X$USER_CMSSW_REPO" = X ] is probably what we want, no?