Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1375)

Unified Diff: vpython/venv/test_data/README.md

Issue 2699063004: vpython: Add VirtualEnv creation package. (Closed)
Patch Set: remake binaries Created 3 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: vpython/venv/test_data/README.md
diff --git a/vpython/venv/test_data/README.md b/vpython/venv/test_data/README.md
new file mode 100644
index 0000000000000000000000000000000000000000..a870b7d55275d18e69a4f54f01f91e25d1dbaf97
--- /dev/null
+++ b/vpython/venv/test_data/README.md
@@ -0,0 +1,29 @@
+## Test Data?
+
+This test data directory is used by VirtualEnv tests to simulate a full
+environment setup.
+
+A fake CIPD client in `venv_test.go` will load this, treating each intermediate
+file as a CIPD archive:
+
+* If the file is a directory, it will be recursively copied when installed.
+* If the file is a ZIP archive (ends with `.zip`), it will be unzipped when
+ installed.
+
+It sucks that we're committing some binaries to the repository, but it's the
+best option to have a cross-platform self-contained unit test that really
+exercises the interesting parts of the setup. The `virtualenv` binary is the
+only large binary, though. It's downloaded directly from the Internet.
iannucci 2017/02/23 00:54:21 It looks like it's committed to the repo as part o
dnj 2017/02/23 20:38:49 ATM, this package is not bound to CIPD or our infr
+
+The other binaries are wheels, generated from source that is also checked into
+the `test_data` directory. These wheels have equivalent `.src` components that
+were used to generate them.
+
+To build a wheel from source, `cd` into a source directory and run:
+
+ $ python setup.py bdist_wheel
+
+The wheel will be created in `/dist/`.
+
+However, note that it is unlikely any of the wheels will actually need to be
+regenerated, since they are simple artifacts.

Powered by Google App Engine
This is Rietveld 408576698