Merge lp:~mikemc/ubuntuone-client/darwin-platform-platform into lp:ubuntuone-client
| Status: | Merged | ||||
|---|---|---|---|---|---|
| Approved by: | dobey on 2012-09-26 | ||||
| Approved revision: | 1322 | ||||
| Merged at revision: | 1323 | ||||
| Proposed branch: | lp:~mikemc/ubuntuone-client/darwin-platform-platform | ||||
| Merge into: | lp:ubuntuone-client | ||||
| Diff against target: |
18 lines (+4/-3) 1 file modified
ubuntuone/platform/__init__.py (+4/-3) |
||||
| To merge this branch: | bzr merge lp:~mikemc/ubuntuone-client/darwin-platform-platform | ||||
| Related bugs: |
|
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Roberto Alsina (community) | 2012-09-25 | Approve on 2012-09-26 | |
|
Review via email:
|
|||
Commit Message
- Send 'darwin' to server as platform name in auth. (LP: #1037435)
Description of the Change
- Send 'darwin' to server as platform name in auth. (LP: #1037435)
per verterok on irc, I can make this string whatever I want and the server metrics will do the right thing:
mmcc: verterok: okay, great. just wanted to check that I won't break anything on that end that expects only those two strings
mmcc: I'll make it send 'darwin'.
verterok: mmcc: we don't use sys.platform because of the linux/linux2 difference
verterok: mmcc: yes, it shouldn't break anything
verterok: mmcc: the only code that uses that info is this: https:/
To test on darwin, run-mac-tests on tests/platform. the rest of the whole test suite is still kind of broken.

