Merge lp:~willcooke/gdm/ubuntu into lp:~ubuntu-desktop/gdm/ubuntu

[Will Cooke]

* Added workaround to disable Bluetooth in gdm so that A2DP high quality
  BT audio is available in the users session (LP: #1703415):
  + debian/default.pa:
    - Unloads Bluetooth modules so that they are unavailble in GDM
  + debian/rules
    - Creates /var/lib/gdm3/.config/pulse and copies above script in

Unloads Bluetooth audio modules from gdm to free up the A2DP profile in the users session. This allows high quality BT audio to work, but stops a11y tools in gdm from being able to use BT audio devices.
This is not a long term fix, and will need to be revisited for 18.04.

[lp:~willcooke/gdm/ubuntu]

415. By Ken VanDine on 2017-07-11

* debian/gdm3.config
  - Handle the transition from lightdm in gdm3 rather than ubuntu-session
* debian/control.in
  - Dropped depends from ubuntu-session

416. By Ken VanDine on 2017-07-11

Fixed ubuntu-session version check

417. By Ken VanDine on 2017-07-11

Dropped depends for ubuntu-session

418. By Ken VanDine on 2017-07-11

releasing package gdm3 version 3.24.2-1ubuntu5

[Daniel van Vugt (vanvugt)]

Still needs merging (when I tried it)?

+N debian/default.pa
 M debian/changelog
 M debian/gdm3.install
 M debian/gdm3.postinst
Text conflict in debian/changelog
1 conflicts encountered.

review: Needs Information

[Daniel van Vugt (vanvugt)]

OK,

1. There is still an invisible conflict per above (when I tried merging this myself).

2. I can't seem to test it for reasons unrelated to this branch (bug 1703747).

review: Needs Fixing

[Daniel van Vugt (vanvugt)]

OK, revision 416 seems to have solved my Bluetooth issue (using today's new PulseAudio too).

Only had to grab the proposed fix for bug 1703746 to build it.

review: Approve

[lp:~willcooke/gdm/ubuntu]

419. By Didier Roche on 2017-07-12

debian/patches/ubuntu_prefer_x11_session.patch:
Prefer X11 session over wayland. As we are not ready yet for
switching to wayland yet, let's prefer X11 session when no specific
choice was made. (LP: #1703601)
This will be reverted once we switch to wayland by default.

420. By Didier Roche on 2017-07-12

releasing package gdm3 version 3.24.2-1ubuntu6

421. By Iain Lane on 2017-07-12

debian/gdm3.config: Check the version of gdm3 instead of ubuntu-session.

And some other minor improvements.

422. By Iain Lane on 2017-07-12

Compare with a ~ version, so that any backports of this exact version aren't re-migrated

423. By Iain Lane on 2017-07-12

releasing package gdm3 version 3.24.2-1ubuntu7

[Jeremy Bicha (jbicha)]

Please fix up your debian/changelog entries. It should not have things like:
+>>>>>>> MERGE-SOURCE

3.24.2-1ubuntu5~will1

review: Needs Fixing

[lp:~willcooke/gdm/ubuntu]

424. By Daniel van Vugt on 2017-07-13

First attempt

425. By Daniel van Vugt on 2017-07-13

Remember the changelog

[Daniel van Vugt (vanvugt)]

Sorry, more conflicts:
https://code.launchpad.net/~vanvugt/gdm/fix-1704050/+merge/327360

[lp:~willcooke/gdm/ubuntu]

426. By Will Cooke on 2017-07-13

* debian/default.pa
  - Added new file to disable Bluetooth audio devices in PulseAudio from
    gdm3. (LP: #1703415)
* debian/gd3.install
  - Added details of the default.pa file
* debian/gd3.postinst
  - Added installation of default.pa and creation of dir if it doesn't
    exist.

[Will Cooke (willcooke)]

Refreshed from the latest and fixed the changelog.
We shouldn't merge this yet though.

[Daniel van Vugt (vanvugt)]

I think this branch is the best option for the time being. The Arch wiki recommends a similar workaround, and upstream PulseAudio acknowledges it could do better:
  https://bugs.freedesktop.org/show_bug.cgi?id=98144
But fixing that will take a lot more time and Pulse internals expertise.

As for the a11y concern, while valid I think that fails to consider the fact that the bug being fixed here is a much bigger than a11y. All Bluetooth audio users won't have any audio. So that is logically a bigger target audience than a11y users who rely on Bluetooth.

Only one minor issue I think: There is no uninstall/postrm step to wind back /var/lib/gdm3/.config/pulse/default.pa

review: Approve

[Will Cooke (willcooke)]

Should that rm be as part of purge, or for everything? i.e. do I put it in
the "if [ $1 = "purge" ]" section, or just at the end of the script?

On 13 July 2017 at 12:42, Daniel van Vugt <email address hidden>
wrote:

> Review: Approve
>
> I think this branch is the best option for the time being. The Arch wiki
> recommends a similar workaround, and upstream PulseAudio acknowledges it
> could do better:
> https://bugs.freedesktop.org/show_bug.cgi?id=98144
> But fixing that will take a lot more time and Pulse internals expertise.
>
> As for the a11y concern, while valid I think that fails to consider the
> fact that the bug being fixed here is a much bigger than a11y. All
> Bluetooth audio users won't have any audio. So that is logically a bigger
> target audience than a11y users who rely on Bluetooth.
>
> Only one minor issue I think: There is no uninstall/postrm step to wind
> back /var/lib/gdm3/.config/pulse/default.pa
> --
> https://code.launchpad.net/~willcooke/gdm/ubuntu/+merge/327165
> You are the owner of lp:~willcooke/gdm/ubuntu.
>

[Will Cooke (willcooke)]

Spoke to Didier about the postrm.

Line 26 has:

        rm -rf /var/lib/gdm3

So that should take care of it.

[Jeremy Bicha (jbicha)]

Thank you!

Ubuntu GNOME has had complaints of this issue (LP: #1489651) back to 16.04 LTS so we should consider SRUing this.

review: Approve

=== modified file 'debian/changelog'
--- debian/changelog	2017-07-13 08:30:44 +0000
+++ debian/changelog	2017-07-13 11:29:06 +0000
@@ -1,3 +1,16 @@
+gdm3 (3.24.2-1ubuntu9) artful; urgency=medium
+
+  * debian/default.pa
+    - Added new file to disable Bluetooth audio devices in PulseAudio from
+      gdm3. (LP: #1703415)
+  * debian/gd3.install
+    - Added details of the default.pa file
+  * debian/gd3.postinst
+    - Added installation of default.pa and creation of dir if it doesn't
+      exist.
+
+ -- Will Cooke <[e-mail]>  Thu, 13 Jul 2017 12:14:34 +0100
+
 gdm3 (3.24.2-1ubuntu8) artful; urgency=medium
 
   * Avoid double-NULL terminated array that breaks append operations

=== added file 'debian/default.pa'
--- debian/default.pa	1970-01-01 00:00:00 +0000
+++ debian/default.pa	2017-07-13 11:29:06 +0000
@@ -0,0 +1,17 @@
+#!/usr/bin/pulseaudio -nF
+#
+# Work around for PA not allowing access to A2DP profiles in the user session
+# because GDM already has it open.
+# LP: #1703415
+
+# load system wide configuration
+.include /etc/pulse/default.pa
+
+### unload driver modules for Bluetooth hardware
+.ifexists module-bluetooth-policy.so
+  unload-module module-bluetooth-policy
+.endif
+
+.ifexists module-bluetooth-discover.so
+  unload-module module-bluetooth-discover
+.endif

=== modified file 'debian/gdm3.install'
--- debian/gdm3.install	2017-06-29 14:20:51 +0000
+++ debian/gdm3.install	2017-07-13 11:29:06 +0000
@@ -24,3 +24,6 @@
 debian/generate-config			usr/share/gdm
 debian/insserv.conf.d			etc
 data/dconf/defaults/*			usr/share/gdm/dconf
+# Add the default.pa file to disable Bluetooth audio in GDM. LP: #1703415
+debian/default.pa                       usr/share/gdm
+

=== modified file 'debian/gdm3.postinst'
--- debian/gdm3.postinst	2017-05-21 03:04:06 +0000
+++ debian/gdm3.postinst	2017-07-13 11:29:06 +0000
@@ -88,6 +88,13 @@
   ln -sf $UNIT $USERUNITDIR/sockets.target.wants
 fi
 
+# Install PA workaround for LP:#1703415
+# Stops GDM using Bluetooth audio so that it's available in the users session
+if ! [ -f /var/lib/gdm3/.config/pulse/default.pa ]; then
+  mkdir -p /var/lib/gdm3/.config/pulse
+  cp /usr/share/gdm/default.pa /var/lib/gdm3/.config/pulse/default.pa
+fi
+
 chown -R gdm:gdm /var/lib/gdm3
 
 if [ -L /etc/pam.d/gdm-welcome ]; then