Merge lp:~mir-team/platform-api/expose-mir-connection into lp:platform-api
| Status: | Merged |
|---|---|
| Approved by: | Gerry Boland on 2015-02-05 |
| Approved revision: | 300 |
| Merged at revision: | 296 |
| Proposed branch: | lp:~mir-team/platform-api/expose-mir-connection |
| Merge into: | lp:platform-api |
| Diff against target: |
619 lines (+166/-32) 23 files modified
CMakeLists.txt (+5/-2) debian/changelog (+7/-0) debian/control (+1/-0) debian/libubuntu-application-api2.symbols (+2/-1) include/ubuntu/application/instance.h (+15/-0) include/ubuntu/application/ui/clipboard.h (+7/-0) include/ubuntu/application/ui/form_factor.h (+5/-0) include/ubuntu/application/ui/options.h (+9/-2) include/ubuntu/application/ui/session.h (+12/-3) include/ubuntu/application/ui/stage.h (+5/-0) include/ubuntu/application/ui/window.h (+22/-4) include/ubuntu/application/ui/window_orientation.h (+5/-0) include/ubuntu/application/ui/window_parent.h (+19/-7) include/ubuntu/application/ui/window_properties.h (+19/-5) include/ubuntu/application/ui/window_state.h (+5/-0) include/ubuntu/application/ui/window_type.h (+5/-0) src/ubuntu/application/base_module.h (+2/-2) src/ubuntu/application/common/mirclient/ubuntu_application_api_mirclient.cpp (+7/-0) src/ubuntu/application/common/mirserver/ubuntu_application_api_mirserver.cpp (+7/-0) src/ubuntu/application/desktop/module_version.h (+2/-2) src/ubuntu/application/testbackend/module_version.h (+2/-2) src/ubuntu/application/touch/module_version.h (+2/-2) src/ubuntu/application/ubuntu_application_api.cpp (+1/-0) |
| To merge this branch: | bzr merge lp:~mir-team/platform-api/expose-mir-connection |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Gerry Boland | 2014-12-17 | Approve on 2015-02-05 | |
| PS Jenkins bot | continuous-integration | Approve on 2015-02-04 | |
| Alberto Aguirre | Approve on 2015-02-04 | ||
| Albert Astals Cid (community) | Abstain on 2015-02-02 | ||
|
Review via email:
|
|||
Commit Message
Expose MirConnection form UApplicationIntance and deprecate the ua_ui namespace.
Description of the Change
Hi! This is the first step of a transition plan to move QtUbuntu to the Mir Client API and remove the usage of platform API as an intermediate layer insulating MirClient. This has been discussed with the Mir team and tvoss and I won't recount the entire discussion here. In short, significant Mir API changes/additions are required for desktop functionality and replicating them all in PAPI has been deemed costly/unnecessary.
This branch exposes MirConnection from u_application_
I dont think there are any downstream users of the ua_ui namespace besides QtUbuntu but I'm not sure.
QtUbuntu branch incoming before holidays...
| Gerry Boland (gerboland) wrote : | # |
Typo in commit message.
I've no problem with the API addition, but we need a more concrete deprecation strategy. Just adding comment to the relevant header files is not enough IMO.
- 291. By Robert Carr on 2015-01-22
-
Use deprecated attribute
| Robert Carr (robertcarr) wrote : | # |
Hi I've updated things to use the actual deprecated attribute...I didn't do this initially out of fear of breaking downstream but forgot we can release this at the same time as the qtubuntu port easily and nothing else in the silos using platform-api.
Out of consideration for potential unknown consumers I've left the methods in with deprecation notice though. They should be removed after one release. It's not as if they are particularly high burden (look at bzr log to see how much they change ;)).
Thanks,
Robert
| PS Jenkins bot (ps-jenkins) wrote : | # |
PASSED: Continuous integration, rev:291
http://
Executed test runs:
SUCCESS: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
Click here to trigger a rebuild:
http://
| Gerry Boland (gerboland) wrote : | # |
Shouldn't libubuntu-
- 292. By Robert Carr on 2015-01-26
-
Update minor version, package version and add mirclient dependency to devpackage
- 293. By Robert Carr on 2015-01-26
-
Merge trunk
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:293
http://
Executed test runs:
FAILURE: http://
FAILURE: http://
FAILURE: http://
Click here to trigger a rebuild:
http://
- 294. By Robert Carr on 2015-01-26
-
Correct version bump
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:294
http://
Executed test runs:
FAILURE: http://
FAILURE: http://
FAILURE: http://
Click here to trigger a rebuild:
http://
- 295. By Robert Carr on 2015-01-26
-
Again
| PS Jenkins bot (ps-jenkins) wrote : | # |
PASSED: Continuous integration, rev:295
http://
Executed test runs:
SUCCESS: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
Click here to trigger a rebuild:
http://
| Gerry Boland (gerboland) wrote : | # |
I think with the train, you just need to specify the major package version "2.8.0" - not the whole "2.8.0+
Looks ok otherwise
- 296. By Robert Carr on 2015-01-28
-
Fix changelog version
| PS Jenkins bot (ps-jenkins) wrote : | # |
PASSED: Continuous integration, rev:296
http://
Executed test runs:
SUCCESS: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
Click here to trigger a rebuild:
http://
| Albert Astals Cid (aacid) wrote : | # |
The debian/changelog file seems to have a broken merge?
| Robert Carr (robertcarr) wrote : | # |
Should be ok now?
| PS Jenkins bot (ps-jenkins) wrote : | # |
PASSED: Continuous integration, rev:297
http://
Executed test runs:
SUCCESS: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
Click here to trigger a rebuild:
http://
| Gerry Boland (gerboland) wrote : | # |
dpkg-gensymbols: warning: some new symbols appeared in the symbols file: see diff output below
dpkg-gensymbols: warning: debian/
Please update debian/
- 298. By Alberto Aguirre on 2015-02-04
-
update symbols
- 299. By Alberto Aguirre on 2015-02-04
-
merge lp:platform-api, fix changelog conflict
- 300. By Alberto Aguirre on 2015-02-04
-
update versions
| Alberto Aguirre (albaguirre) wrote : | # |
> dpkg-gensymbols: warning: some new symbols appeared in the symbols file: see
> diff output below
> dpkg-gensymbols: warning: debian/
> doesn't match completely debian/
>
> Please update debian/
Fixed.
| Robert Carr (robertcarr) wrote : | # |
:) Thanks
| PS Jenkins bot (ps-jenkins) wrote : | # |
PASSED: Continuous integration, rev:298
http://
Executed test runs:
SUCCESS: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
Click here to trigger a rebuild:
http://
| PS Jenkins bot (ps-jenkins) wrote : | # |
PASSED: Continuous integration, rev:300
http://
Executed test runs:
SUCCESS: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
Click here to trigger a rebuild:
http://
- 301. By Michał Sawicz on 2015-03-18
-
Merge lp:platform-api
| Oliver Grawert (ogra) wrote : | # |
if you want this acked by a core dev, pretty please mention the package dependency change in the changelog or commit message.

PASSED: Continuous integration, rev:290 jenkins. qa.ubuntu. com/job/ platform- api-ci/ 400/ jenkins. qa.ubuntu. com/job/ platform- api-vivid- amd64-ci/ 13 jenkins. qa.ubuntu. com/job/ platform- api-vivid- armhf-ci/ 13 jenkins. qa.ubuntu. com/job/ platform- api-vivid- armhf-ci/ 13/artifact/ work/output/ *zip*/output. zip jenkins. qa.ubuntu. com/job/ platform- api-vivid- i386-ci/ 13
http://
Executed test runs:
SUCCESS: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
Click here to trigger a rebuild: s-jenkins. ubuntu- ci:8080/ job/platform- api-ci/ 400/rebuild
http://