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

[3.1] pinned_build.sh - prints out warning only if the platorm is not Ubuntu and makes build directory relative to where the build starts #487

Merged
merged 1 commit into from
Jun 21, 2022

Conversation

linh2931
Copy link
Member

#363 reports pinned_build.sh always prints "Currently only supporting Ubuntu based builds. Proceed at your own risk.", even on Ubuntu platform.

#364 reports the build directory is made relative to the dependency directory; it should have been relative to the directory where the script starts.

This PR fixes those two issues.

Tested in two different directories:

scripts/pinned_build.sh ~/dep build 16
./pinned_build.sh ~/dep build 16

…rectory relative to where the build starts - 3.1
@linh2931 linh2931 merged commit 92d3547 into release/3.1.x Jun 21, 2022
@linh2931 linh2931 deleted the fix_pinned_build_sh_3_1 branch June 21, 2022 23:10
@linh2931 linh2931 restored the fix_pinned_build_sh_3_1 branch June 22, 2022 15:28
@linh2931 linh2931 deleted the fix_pinned_build_sh_3_1 branch June 22, 2022 15:33
@arhag arhag changed the title Print out warning only if the platorm is not Ubuntu and make build directory relative to where the build starts - 3.1 [3.1] Print out warning only if the platorm is not Ubuntu and make build directory relative to where the build starts Jul 1, 2022
@linh2931 linh2931 changed the title [3.1] Print out warning only if the platorm is not Ubuntu and make build directory relative to where the build starts [3.1] pinned_build.sh - prints out warning only if the platorm is not Ubuntu and makes build directory relative to where the build starts Jul 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants