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

Issue 86760043: Fix 1304767

Can't Edit
Can't Publish+Mail
Start Review
Created:
10 years ago by waigani
Modified:
10 years ago
Reviewers:
dave, mp+215368
Visibility:
Public.

Description

Fix 1304767 Testing that --upload-tools rejects building tools for ppc64 when on amd64 fails when the test is run on an actual ppc64. In other words, the test assumes a hostArch of amd64. Fix this by patching hostArch to amd64. https://code.launchpad.net/~waigani/juju-core/1304767-bootstrap-tools/+merge/215368 (do not edit description out of merge proposal)

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+7 lines, -4 lines) Patch
A [revision details] View 1 chunk +2 lines, -0 lines 0 comments Download
M cmd/juju/bootstrap_test.go View 1 chunk +5 lines, -4 lines 0 comments Download

Messages

Total messages: 2
waigani
Please take a look.
10 years ago (2014-04-11 08:33:11 UTC) #1
dave_cheney.net
10 years ago (2014-04-11 09:40:58 UTC) #2
LGTM.
Sign in to reply to this message.

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