Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add script to fetch netplugin release or build #275

Merged
merged 2 commits into from Oct 24, 2017

Conversation

chrisplo
Copy link
Contributor

Adding a script that can be used by the Makefile to either maintain
existing behavior by default (download an existing release), or by
adding environment variables, build netplugin based on a requested
branch.

Signed-off-by: Chris Plock chrisplo@cisco.com

Adding a script that can be used by the Makefile to either maintain
existing behavior by default (download an existing release), or by
adding environment variables, build netplugin based on a requested
branch.

Signed-off-by: Chris Plock <chrisplo@cisco.com>
${CONTIV_ARTIFACT_STAGING}/
# create a link so other scripts can find the file without knowing the SHA
cd ${CONTIV_ARTIFACT_STAGING}
ln -sf netplugin-${GIT_COMMIT}.tar.bz2 $CONTIV_NETPLUGIN_TARBALL_NAME
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could we check that $CONTIV_NETPLUGIN_TARBALL_NAME is set at the top of the script instead of relying on -u to raise an error?

@dseevr dseevr requested a review from a team October 19, 2017 23:47
@dseevr dseevr merged commit f06d901 into contiv:master Oct 24, 2017
@chrisplo chrisplo deleted the add_netplugin_build_script branch October 26, 2017 04:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants