Skip to content

[do not merge] Prepare code-server for ARM distribution #1042

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

Closed
wants to merge 5 commits into from

Conversation

sr229
Copy link
Contributor

@sr229 sr229 commented Sep 30, 2019

Following the requests over in GH-35, we'll be doing ARM distributions by leveraging the existing CI we have.

Do not merge until coder/nbin#23 is finished.

sr229 and others added 4 commits September 21, 2019 03:37
This is the same PR but with some fixes for v2.

Co-Authored-by: nwtnsqrd <[email protected]>
Signed-off-by: Ayane Satomi <[email protected]>
they will be seperate images from now on.

Signed-off-by: Ayane Satomi <[email protected]>
@sr229 sr229 added complex enhancement Some improvement that isn't a feature labels Sep 30, 2019
@sr229 sr229 self-assigned this Sep 30, 2019
we might be doing something really bad by not putting it on the same rules as Alpine ARM

Signed-off-by: Ayane Satomi <[email protected]>
@@ -9,20 +9,27 @@ function docker-build() {
local minify="${MINIFY:-}"
if [[ "${target}" == "linux" ]] ; then
image="codercom/nbin-centos"
fi
fi
Copy link
Contributor

Choose a reason for hiding this comment

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

Useless whitespace

@@ -81,4 +92,4 @@ function main() {
fi
}

main "$@"
main "$@"
Copy link
Contributor

Choose a reason for hiding this comment

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

Missing new line

@sr229
Copy link
Contributor Author

sr229 commented Oct 21, 2019

Closing in favor of new PR

@sr229 sr229 closed this Oct 21, 2019
@sr229 sr229 deleted the sr229/arm branch October 21, 2019 15:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Some improvement that isn't a feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants