Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(1464)

Issue 84520046: Remove 1.16 support code

Can't Edit
Can't Publish+Mail
Start Review
Created:
9 years, 11 months ago by axw
Modified:
9 years, 11 months ago
Reviewers:
mp+214504, wallyworld
Visibility:
Public.

Description

Remove 1.16 support code This drops most of the 1.16 support code from the client-side. There's more to remove, this is just a start. https://code.launchpad.net/~axwalk/juju-core/remove-1.16-client-support/+merge/214504 (do not edit description out of merge proposal)

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+23 lines, -669 lines) Patch
A [revision details] View 1 chunk +2 lines, -0 lines 0 comments Download
M cmd/juju/addmachine.go View 3 chunks +6 lines, -58 lines 0 comments Download
M cmd/juju/common.go View 1 chunk +0 lines, -15 lines 0 comments Download
M cmd/juju/constraints.go View 5 chunks +1 line, -43 lines 0 comments Download
M cmd/juju/deploy.go View 2 chunks +0 lines, -78 lines 0 comments Download
M cmd/juju/destroymachine.go View 3 chunks +3 lines, -28 lines 0 comments Download
M cmd/juju/environment.go View 5 chunks +2 lines, -71 lines 0 comments Download
M cmd/juju/set.go View 3 chunks +1 line, -36 lines 0 comments Download
M cmd/juju/status.go View 2 chunks +0 lines, -16 lines 0 comments Download
M cmd/juju/unset.go View 3 chunks +1 line, -35 lines 0 comments Download
cmd/juju/upgradecharm.go View 3 chunks +0 lines, -81 lines 0 comments Download
M cmd/juju/upgradejuju.go View 3 chunks +0 lines, -96 lines 0 comments Download
M environs/manual/provisioner.go View 4 chunks +7 lines, -69 lines 0 comments Download
D state/statecmd/destroymachines.go View 1 chunk +0 lines, -43 lines 0 comments Download

Messages

Total messages: 2
axw
Please take a look.
9 years, 11 months ago (2014-04-07 10:16:41 UTC) #1
wallyworld
9 years, 11 months ago (2014-04-08 03:02:47 UTC) #2
LGTM. Woohoo!
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b