-
Notifications
You must be signed in to change notification settings - Fork 6k
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
WIP - Redo CI and Include Automated Docker Deployment #1586
Closed
Closed
Changes from all commits
Commits
Show all changes
24 commits
Select commit
Hold shift + click to select a range
d360093
see if os x actually builds
95928c5
try this?
3afb008
try this
2d23420
try this and see if deps install
81c2b33
see if adding compiler field speeds up the builds...probably won't
5c3c627
see if multiple compilers work in a matrix. Try taking away compiler …
3493518
try to begin docker support and try to reduce script usage in origina…
37c78f5
try not rerunning the tests 3 times...has this been fixed
fa4a717
added notes to travis.yml, added notes for our brew installation of d…
94d9366
add travis specific dependency installation script, update travis.yml…
bea7b4b
forgot to give execution permission here
9c8daca
make brew install of cpp-eth verbose so as to prevent stalling of travis
00d2a40
add docker to all services...see if this works on os x
18745c8
got the dockerfile building for tests, need a way to connect the ipc …
3d84391
added dockerfiles for arch, jessie, fedora, and xenial. Begin reorgan…
93e12f5
get rid of extra docker files. Modify dockerfile to include another b…
623d257
forgot a bit in the travis deps installation script
d556a73
turn release off for docker
774810e
make non ipc tests executable
9c0b190
just use make instead of trying to build solc and soltest individually
8dda9fb
fix some things in scripts
be9ca20
create ipc test script and edit travis.yml to correct
143f3bb
remove non used docker scripts
c7f625e
add the executables to the bin
File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,12 +1,21 @@ | ||
FROM alpine | ||
MAINTAINER chriseth <[email protected]> | ||
#Official solidity docker image | ||
|
||
RUN \ | ||
apk --no-cache --update add build-base cmake boost-dev git && \ | ||
sed -i -E -e 's/include <sys\/poll.h>/include <poll.h>/' /usr/include/boost/asio/detail/socket_types.hpp && \ | ||
git clone --depth 1 --recursive -b develop https://github.com/ethereum/solidity && \ | ||
cd /solidity && cmake -DCMAKE_BUILD_TYPE=Release -DTESTS=0 -DSTATIC_LINKING=1 && \ | ||
cd /solidity && make solc && install -s solc/solc /usr/bin && \ | ||
cd / && rm -rf solidity && \ | ||
apk del sed build-base git make cmake gcc g++ musl-dev curl-dev boost-dev && \ | ||
rm -rf /var/cache/apk/* | ||
#Establish working directory as soliditiy | ||
WORKDIR /solidity | ||
#Determine whether or not we are in the testing suite | ||
ARG TEST=Off | ||
#Determine what build type we are using | ||
ARG BUILD=Release | ||
#Copy working directory on travis to the image | ||
COPY / $WORKDIR | ||
|
||
#Install dependencies, eliminate annoying warnings, test if we are in a test environment, if so, | ||
#add in soltest to the binary. If not, assume this is basic solidity image, build it and ship it out. | ||
RUN ./scripts/install_deps.sh && sed -i -E -e 's/include <sys\/poll.h>/include <poll.h>/' /usr/include/boost/asio/detail/socket_types.hpp | ||
RUN test $TEST == On && cmake -DCMAKE_BUILD_TYPE=$BUILD -DSTATIC_LINKING=1 || cmake -DCMAKE_BUILD_TYPE=$BUILD -DTESTS=0 -DSTATIC_LINKING=1 | ||
RUN test $TEST == On && (make && install -s solc/solc /usr/bin && install -s test/soltest /usr/bin) || (make solc && install -s solc/solc /usr/bin) | ||
RUN cd / && rm -rf solidity | ||
RUN apk del sed build-base git make cmake gcc g++ musl-dev curl-dev boost-dev | ||
RUN rm -rf /var/cache/apk/* |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,13 @@ | ||
#!/usr/bin/env bash | ||
|
||
if [ -z "$1" ]; then | ||
BUILD_TYPE=Release | ||
else | ||
BUILD_TYPE="$1" | ||
fi | ||
|
||
mkdir -p build | ||
cd build | ||
cmake .. -DCMAKE_BUILD_TYPE="$BUILD_TYPE" | ||
make -j2 && install -s solc/solc /usr/bin && install -s test/soltest | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. remove -s |
||
cd .. |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,16 @@ | ||
#!/usr/bin/env sh | ||
|
||
#mount volume so that host can access the tmp directory and the geth.ipc process | ||
docker run -di \ | ||
--volume /tmp:/tmp \ | ||
--name testEth \ | ||
ethereum/client-cpp \ | ||
--test -d /tmp/testeth | ||
|
||
echo "--> Running tests without optimizer..." | ||
soltest --run_test=SolidityOptimizer,GasMeterTests,SolidityEndToEndTest -- --ipcpath=/tmp/testeth/geth.ipc && \ | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. change source for suites to add test label for "require IPC" |
||
echo "--> Running tests WITH optimizer..." && \ | ||
soltest --run_test=SolidityOptimizer,GasMeterTests,SolidityEndToEndTest -- --optimize --ipcpath=/tmp/testeth/geth.ipc | ||
ERROR_CODE=$? | ||
docker kill testEth | ||
exit $ERROR_CODE |
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
put into one run (makes image smaller)