InstallPackageNames should not return until the package has been installed

This bug blocks the UOA Empathy integration: https://bugzilla.gnome.org/show_bug.cgi?id=682802

Martin Pitt would be the most competent to fix this bug, but it's not exactly his responsibility, and it's unlikely that he'll find time to fix this soon.
So, while I'm assigning the bug to him (just so that he doesn't forget about it), anyone else is welcome to step in and reclaim the bug -- after notifying Martin about it.

The patch should fix this issue. The aptd backend of session-installer now waits until the transaction reported as finished by the progress dialog.

I think that we had an API change about the behavior AptProgressDialog.run() some very long time ago. It seems that we missed this place. Previously the run method waited until the transaction was done.

Thanks Sebastian!

This bug was fixed in the package sessioninstaller - 0.20+bzr131-0ubuntu2

---------------
sessioninstaller (0.20+bzr131-0ubuntu2) quantal; urgency=low

  * Cherry-pick upstream revision 132 to make transactions synchronous, as is
    expected (LP: #1042231)
 -- Iain Lane <email address hidden> Thu, 20 Sep 2012 16:13:18 +0100

Please help to backport this fix to precise, thanks.

Uploaded. Thanks. If someone could fill in a Test Case in the description it'd be helpful. I can't think of a good one.

$4, how did the oem run in to this problem. Do you have a test case that we can use to verify that this is fixed?

I have provided a test case in the duplicate bug #1041287.

For the record:

Ubuntu 12.04
sessioninstaller: 0.20+bzr128-0ubuntu1.1

Reproduce Steps:
1. Remove some GStreamer plugin by executing `sudo apt-get purge --yes gstreamer0.10-plugins-ugly && sudo apt-get autoremove --purge --yes`.
2. Execute the command `python /usr/share/pyshared/sessioninstaller/gstinstaller.py "gstreamer|0.10|totem|Advanced Streaming Format (ASF) demuxer|decoder-video/x-ms-asf"`.

Expect Result:
It finished after all Debian packages are installed.

Actual Result:
It shows the following messages, but there is a dialog still running the installation.

/usr/lib/python2.7/dist-packages/gobject/constants.py:24: Warning: g_boxed_type_register_static: assertion `g_type_from_name (name) == 0' failed
  import gobject._gobject
INFO:Initializing GStreamerInstaller
DEBUG:Add provide: Advanced Streaming Format (ASF) demuxer|gstreamer0.10(decoder-video/x-ms-asf)
INFO:Finished succesfully

This issue will be relative to Bug #1041032.

It will cause on_plugin_installation_done() of totem-3.0.1/src/backend/bacon-video-widget-gst-missing-plugins.c received GST_INSTALL_PLUGINS_SUCCESS, but it not actually.

Hello Guillaume, or anyone else affected,

Accepted sessioninstaller into precise-proposed. The package will build now and be available at http://launchpad.net/ubuntu/+source/sessioninstaller/0.20+bzr128-0ubuntu1.2 in a few hours, and then in the -proposed repository.

Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please change the bug tag from verification-needed to verification-done. If it does not, change the tag to verification-failed. In either case, details of your testing will help us make a better decision.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance!

Hi $4, woudl you be able to test the package in proposed so that we can ensure that the OEM is happy with it?

I have verified sessioninstaller 0.20+bzr128-0ubuntu1.2 in precise-proposed, and it works well. Thanks a lot.

The verification of this Stable Release Update has completed successfully and the package has now been released to -updates. Subsequently, the Ubuntu Stable Release Updates Team is being unsubscribed and will not receive messages about this bug report. In the event that you encounter a regression using the package from -updates please report a new bug using ubuntu-bug and tag the bug report regression-update so we can easily find any regresssions.

This bug was fixed in the package sessioninstaller - 0.20+bzr128-0ubuntu1.2

---------------
sessioninstaller (0.20+bzr128-0ubuntu1.2) precise-proposed; urgency=low

  * debian/patches/synchronous-transactions.patch: Cherry-pick patch from
    upstream (r132) to wait until transactions are done before returning (LP:
    #1042231)
 -- Iain Lane <email address hidden> Mon, 24 Sep 2012 12:48:25 +0100