-
Notifications
You must be signed in to change notification settings - Fork 11
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
lookout-sdk binary #3
Conversation
4a7de2c
to
c4219b3
Compare
CI fails on python example due to |
_tools/install-lookout-latest.sh
Outdated
echo "Unable to extract lookout release archive" >&2 | ||
exit 2 | ||
fi | ||
|
||
if ! mv lookout_sdk_*/lookout .. ; then | ||
echo "Unable to move lookout binary file" >&2 | ||
if ! mv lookout-sdk_*/lookout-sdk ../lookout_sdk ; then |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why the rename from -sdk
to _sdk
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
because it's lookout-sdk
dir we are in, and then mv
binary with the same name to ../lookout_sdk
fails trying to overwrite it
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
added a comment with clarifications
Python packaging gong to handled in the separate PR |
Signed-off-by: Alexander Bezzubov <bzz@apache.org>
Signed-off-by: Alexander Bezzubov <bzz@apache.org>
c4219b3
to
6af4668
Compare
@@ -12,7 +12,7 @@ curl -s --connect-timeout 5 \ | |||
--retry-delay 0 \ | |||
--retry-max-time 40\ | |||
"https://api.github.com/repos/src-d/lookout/releases/latest" \ | |||
| tee -a ../lookout-install.log \ | |||
|& tee -a ../lookout-install.log \ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why &
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
allthough it's bashism, this is a nice shortcut to re-direct both, stdin and stderr together
Signed-off-by: Alexander Bezzubov <bzz@apache.org>
Conditionally disable python CI profile for now, until I fix the packaging in a separate PR. Going to merge as hotfix, as soon as CI is green. |
fix python lib packaging