Its's good.. However I think that rather we use ${NUPIC_CORE_REMOTE} directly in the macro, we could pass this value through a parameter to macro.. The intention of this macro is being reused for other repos too.. With these changes, this goal is dismissed..