juju-release-tools

Merge lp:~nskaggs/juju-release-tools/fix-godeps-removal into lp:juju-release-tools

  1. fix-godeps-removal
  2. Merge into trunk
Proposed by Nicholas Skaggs
Status: Merged
Merged at revision: 329
Proposed branch: lp:~nskaggs/juju-release-tools/fix-godeps-removal
Merge into: lp:juju-release-tools
Diff against target: 12 lines (+1/-1)
1 file modified
make-release-tarball.bash (+1/-1)
To merge this branch: bzr merge lp:~nskaggs/juju-release-tools/fix-godeps-removal
Reviewer Review Type Date Requested Status
Christopher Lee (community) Approve
Review via email: mp+302748@code.launchpad.net

Description of the change

Fix the removal of godeps

To post a comment you must log in.
Revision history for this message
Christopher Lee (veebers) wrote :

LGTM

review: Approve
lp:~nskaggs/juju-release-tools/fix-godeps-removal updated
330. By Nicholas Skaggs

really get it right

Revision history for this message
Christopher Lee (veebers) wrote :

Ah right. LGTM still :-)

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'make-release-tarball.bash'
2--- make-release-tarball.bash 2016-08-11 12:56:37 +0000
3+++ make-release-tarball.bash 2016-08-12 03:20:57 +0000
4@@ -93,7 +93,7 @@
5 GOPATH=$WORK $GODEPS -u "$WORKPACKAGE/dependencies.tsv"
6
7 # Remove godeps, and non-free data
8-rm -r $WORK/src/launchpad.net/godeps
9+rm -rf $WORK/src/github.com/rogpeppe/godeps
10 rm -rf $WORK/src/github.com/kisielk
11 rm -rf $WORK/src/code.google.com/p/go.net/html/charset/testdata/
12 rm -f $WORK/src/code.google.com/p/go.net/html/charset/*test.go

Subscribers

People subscribed via source and target branches