Skip to content

Commit

Permalink
fix yelp make targets
Browse files Browse the repository at this point in the history
  • Loading branch information
Rob-Johnson committed Nov 10, 2015
1 parent 29ba4eb commit 8fd34bb
Show file tree
Hide file tree
Showing 7 changed files with 8 additions and 56 deletions.
3 changes: 1 addition & 2 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@ itest: test
tox -e paasta_itests

# See the makefile in yelp_package/Makefile for packaging stuff
itest_%:
itest_%_yelp:
make -C yelp_package $@

# Steps to release
Expand All @@ -43,4 +43,3 @@ clean:
find . -name '*.pyc' -delete
find . -name '__pycache__' -delete
rm -rf .tox

2 changes: 1 addition & 1 deletion debian/rules
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ override_dh_auto_build:
true

override_dh_installdocs:
tox -e manpages
tox -i http://pypi.yelpcorp.com -e manpages

# do not call `make test` as part of packaging
override_dh_auto_test:
Expand Down
2 changes: 1 addition & 1 deletion requirements.txt
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@ lockfile==0.9.1
-e git://github.com/yelp/marathon-python.git@e7e2116159528c45afdbb8b01c03fbd0333e0108#egg=marathon-python
mccabe==0.3.1
mesos.cli==0.1.3
mesos.interface==0.22.1.2
mesos.interface==0.22.1
ordereddict==1.1
path.py==8.1
ply==3.4
Expand Down
4 changes: 2 additions & 2 deletions yelp_package/Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@ build_lucid_docker:
[ -d ../dist ] || mkdir ../dist
cd dockerfiles/lucid/ && docker build -t "paastatools_lucid_container" .
package_lucid_yelp: build_lucid_docker
cp setup.py ../; cp requirements.txt ../
cat requirements-extras.txt >> ../requirements.txt
$(DOCKER_RUN_LUCID) /bin/bash -c "dpkg-buildpackage -d && mv ../*.deb dist/"
$(DOCKER_RUN_LUCID) chown -R $(UID):$(GID) /work
itest_lucid_yelp: package_lucid_yelp
Expand All @@ -37,7 +37,7 @@ build_trusty_docker:
[ -d ../dist ] || mkdir ../dist
cd dockerfiles/trusty/ && docker build -t "paastatools_trusty_container" .
package_trusty_yelp: build_trusty_docker
cp setup.py ../; cp requirements.txt ../
cat requirements-extras.txt >> ../requirements.txt
$(DOCKER_RUN_TRUSTY) /bin/bash -c "dpkg-buildpackage -d && mv ../*.deb dist/"
$(DOCKER_RUN_TRUSTY) chown -R $(UID):$(GID) /work
itest_trusty_yelp: package_trusty_yelp
Expand Down
2 changes: 1 addition & 1 deletion yelp_package/dockerfiles/lucid/Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@ MAINTAINER Kyle Anderson <[email protected]>
# Make sure we get a package suitable for building this package correctly.
# Per dnephin we need https://github.com/spotify/dh-virtualenv/pull/20
# Which at this time is in this package
RUN apt-get update && apt-get -y install dpkg-dev python-tox python-setuptools python-dev debhelper dh-virtualenv=0.6-yelp2 python-yaml python-pytest pyflakes python2.7 python2.7-dev help2man zsh
RUN apt-get update && apt-get -y install dpkg-dev python-tox python-setuptools python-dev debhelper dh-virtualenv=0.6-yelp2 python-yaml python-pytest pyflakes python2.7 python2.7-dev help2man zsh git

ENV HOME /work
ENV PWD /work
Expand Down
2 changes: 2 additions & 0 deletions yelp_package/requirements-extras.txt
Original file line number Diff line number Diff line change
@@ -0,0 +1,2 @@
yelp-cgeom==1.3.1
yelp-logging==1.0.37
49 changes: 0 additions & 49 deletions yelp_package/requirements.txt

This file was deleted.

0 comments on commit 8fd34bb

Please sign in to comment.