[Merge] lp:~khurshid-alam/indicator-sound/fix-build-impish into lp:indicator-sound
Khurshid Alam
mp+409379 at code.launchpad.net
Wed Sep 29 16:34:31 UTC 2021
Khurshid Alam has proposed merging lp:~khurshid-alam/indicator-sound/fix-build-impish into lp:indicator-sound.
Commit message:
* Fix build against vala > 48 & vala >= 50
* Add symbolic icons to the tests
* Avoid possible null value in Greeterlist and GreeterBroadcast object.
See https://github.com/AyatanaIndicators/ayatana-indicator-sound/commit/8df9168f5587cfd31e0bf6a4170a4c4f9784dbb7
* Fix build against GLib > 2.64: Replace HashTable with GenericSet
* Fix static member access
* Drop deprecated DBusProxy.create_for_bus
* Add libdbustest-1 include path
* Unit Tests: Unset environment variable and session bus in teardown
* debian/control: Remove qt5-default & add libgmock-dev in build depends. See https://bugs.launchpad.net/bugs/1921781
* Unit Tests: The desktop pulseaudio daemon also needs the stream-restore module these days
* Unit Tests: Disable notification tests that can not be fixed now. Most of them are phone related and depends on liburl-dispatcher library which was dropped from code since it is no longer in the universe repository. See https://bazaar.launchpad.net/~indicator-applet-developers/indicator-sound/trunk.16.10/revision/564
* Unit Tests: Disable indicator tests that can not be fixed now. These tests needs to be re-enabled later
- tests/indicator-test.cc:PhoneMenu
- tests/indicator-test.cc:DesktopMenu
- tests/indicator-test.cc:BaseActions
- tests/integration/test-indicator.cpp:All tests
- tests/volume-control-test.cc:BasicObject
* debian/rules: Re enable live tests logs
Requested reviews:
Indicator Applet Developers (indicator-applet-developers)
For more details, see:
https://code.launchpad.net/~khurshid-alam/indicator-sound/fix-build-impish/+merge/409379
Regarding tests, most only works on unity-8/phone environment and some are failing due to bugs libdbustest and gmock. As per my testing this doesn't hamper functionality on desktop, indicator-sound works as expected. These tests can be enabled for desktop later if someone able to fix those.
--
Your team Indicator Applet Developers is requested to review the proposed merge of lp:~khurshid-alam/indicator-sound/fix-build-impish into lp:indicator-sound.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: review-diff.txt
Type: text/x-diff
Size: 43004 bytes
Desc: not available
URL: <https://lists.ubuntu.com/archives/ubuntu-reviews/attachments/20210929/837bdb74/attachment-0001.diff>
More information about the Ubuntu-reviews
mailing list