Merge pull request #51 from lightstep/sarah/LS-5226

LS-5226: Update instructions for generating thrift

Author: frenchfrywpepper

Makefile

@@ -61,9 +61,9 @@ test: build
 
 # LightStep-specific: rebuilds the LightStep thrift protocol files.  Assumes
 # the command is run within the LightStep development environment (i.e. the
-# LIGHTSTEP_HOME environment variable is set).
+# MONO_REPO environment variable is set).
 thrift:
-	docker run -v "$(PWD)/lightstep:/out" -v "$(LIGHTSTEP_HOME)/go/src/crouton:/data" --rm thrift:0.10.0 \
+	docker run -v "$(PWD)/lightstep:/out" -v "$(MONO_REPO)/go/src/github.com/lightstep/common-go:/data" --rm thrift:0.10.0 \
 		thrift -r --gen py -out /out /data/crouton.thrift
-	python-modernize -w lightstep/crouton/
+	python-modernize -w $(PWD)/lightstep/crouton/
 	rm -rf lightstep/crouton/ReportingService-remote

README.md

@@ -33,6 +33,13 @@ source .tox/py27/bin/activate
 python examples/nontrivial/main.py
 ```
 
+* [Python-Modernize](https://github.com/python-modernize/python-modernize)
+
+Only required for LightStep developers
+```python
+pip install modernize
+```
+
 ## Getting Started with Tracing
 
 Please see the [example programs](examples/) for examples of how to use this library. In particular: