Skip to content

Commit b4fbab4

Browse files
committed
Run pyinstaller build as normal user
... and test build on CI so we don't break it again! Fixes #503 Signed-off-by: Ben Firshman <ben@firshman.co.uk>
1 parent 6797a32 commit b4fbab4

3 files changed

Lines changed: 8 additions & 2 deletions

File tree

Dockerfile

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
11
FROM debian:wheezy
22
RUN apt-get update -qq && apt-get install -qy python python-pip python-dev git && apt-get clean
3+
RUN useradd -d /home/user -m -s /bin/bash user
34
WORKDIR /code/
45

56
ADD requirements.txt /code/
@@ -10,3 +11,5 @@ RUN pip install -r requirements-dev.txt
1011

1112
ADD . /code/
1213
RUN python setup.py install
14+
15+
RUN chown -R user /code/

script/build-linux

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3,5 +3,5 @@ set -ex
33
mkdir -p `pwd`/dist
44
chmod 777 `pwd`/dist
55
docker build -t fig .
6-
docker run -v `pwd`/dist:/code/dist fig pyinstaller -F bin/fig
7-
docker run -v `pwd`/dist:/code/dist fig dist/fig --version
6+
docker run -u user -v `pwd`/dist:/code/dist fig pyinstaller -F bin/fig
7+
docker run -u user -v `pwd`/dist:/code/dist fig dist/fig --version

wercker.yml

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -7,3 +7,6 @@ build:
77
- script:
88
name: run tests
99
code: script/test
10+
- script:
11+
name: build binary
12+
code: script/build-linux

0 commit comments

Comments
 (0)