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

Unified Diff: go/README.md

Issue 1375553003: go: Roll Goopfile, remove two no longer used packages. (Closed) Base URL: https://chromium.googlesource.com/infra/infra.git@master
Patch Set: Created 5 years, 3 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
« no previous file with comments | « go/Goopfile.lock ('k') | go/clean_goop.py » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: go/README.md
diff --git a/go/README.md b/go/README.md
index 7079e4d2922ba9ec7693cac7838763c753b6493e..0e3d5f2f5e5b9e6b0b3e4f83964fc5bb2097e780 100644
--- a/go/README.md
+++ b/go/README.md
@@ -163,6 +163,26 @@ Use roll_goop.py script. The full workflow:
git cl upload
git cl try -m tryserver.infra -b "Infra Win Tester"
+If you suspect that some packages are no longer needed, you can run
+clean_goop.py script. It will detect potentially unused packages and print them.
+It's not fully trusted to work yet, and thus it's not used automatically.
+
+ ./env.py python clean_goop.py
+ <remove what it said to remove>
+
+ ./env.py goop update
+ git diff Goopfile.lock
+ <ensure packages are gone from Goopfile.lock>
+
+ # Test that our code still works.
+ ./env.py go test github.com/luci/gae/...
+ ./env.py go test github.com/luci/luci-go/...
+ ./env.py go test infra/...
+
+Note: both roll_goop.py and clean_goop.py have lists of exceptional packages
+hardcoded in them. If you encounter some stubborn package that is handled badly
+by these scripts, consider adding it to the exceptions.
+
## Git mirrors for dependencies
« no previous file with comments | « go/Goopfile.lock ('k') | go/clean_goop.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698