Merge lp:~alan-griffiths/mir/bump-protocol-version into lp:mir
Proposed by
Alan Griffiths
Status: | Merged |
---|---|
Approved by: | Gerry Boland |
Approved revision: | no longer in the source branch. |
Merged at revision: | 4203 |
Proposed branch: | lp:~alan-griffiths/mir/bump-protocol-version |
Merge into: | lp:mir |
Diff against target: |
18 lines (+2/-2) 1 file modified
src/include/common/mir/protobuf/protocol_version.h (+2/-2) |
To merge this branch: | bzr merge lp:~alan-griffiths/mir/bump-protocol-version |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Mir CI Bot | continuous-integration | Approve | |
Gerry Boland (community) | Approve | ||
Alan Griffiths | Pending | ||
Review via email: mp+326583@code.launchpad.net |
This proposal supersedes a proposal from 2017-06-29.
Commit message
The protocol version needs to change we dropped message RaiseRequest (and subsumed it into RequestOperation)
To post a comment you must log in.
While this is technically correct, it might be better to retain compatibility with 0.26 clients. For that to happen:
1. the server needs to support RaiseRequest (deprecated)
2. the protocol needs bumping to 4.1
The advantage being that (for example) a 0.27 based server snap could interwork with a 0.26 based client snap. (Of course, if the client snap relied on mir-libs this wouldn't be necessary.)
Opinions?