Skip to content

Enable builds on CircleCI #68

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

Merged
merged 1 commit into from
Nov 15, 2018
Merged

Enable builds on CircleCI #68

merged 1 commit into from
Nov 15, 2018

Conversation

kmagiera
Copy link
Member

CircleCI builds will make it much easier in the future to test PRs but also automate deployments, and will help verify that the binaries we publish comes from reproductible builds. Currently there is a CI running at wix which can only be accessed by a few people. Having builds run on a public CI service will make the whole process more transparent and also easier to troubleshoot in case os issues.

It took me a while to restore builds to work on CircleCI and the root cause of all the problems seemed to be docker environment. While running inside docker the builds will often crash (segfault) during random commands (clang or when running python). We also observed that strange behavior when running on our own hosts inside docker. In order for it to work I had to switch circleCI config from using docker to "machine" which runs some other notion of virtualization.

Copy link
Member

@gengjiawen gengjiawen left a comment

Choose a reason for hiding this comment

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

can you post a link for current build.

@kmagiera
Copy link
Member Author

@kmagiera kmagiera merged commit bfb932e into master Nov 15, 2018
@kmagiera kmagiera deleted the circlecibuilds branch November 15, 2018 18:29
@gengjiawen
Copy link
Member

https://circleci.com/gh/gengjiawen/jsc-android-buildscripts/15, here is the previous build on docker.
I still fond of docker for it's easy for others to build their own jsc.

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