/just/in/case:-)

The attachment "DistUpgrade.utils.py.diff" seems to be a patch. If it isn't, please remove the "patch" flag from the attachment, remove the "patch" tag, and if you are a member of the ~ubuntu-reviewers, unsubscribe the team.

[This is an automated message performed by a Launchpad user owned by ~brian-murray, for any issues please contact him.]

It is not clear to me how switching from partition to rpartition is functionally different. Could you explain the need to switch from one to the other?

In case there’s another right parenthesis from the process name. str.partition would get the first parenthesis while rpartition would get the last one, which is guaranteed to be the right one (no pun intended :−). Unlikely but why not play it safe when possible. ps does the same.

See for yourself:

ln -s /usr/bin/python xyzzy:-\)plugh
./xyzzy:-\)plugh <<E
statline = open("/proc/self/stat").read()
print(statline)
print("Parent PID is not " + statline.partition(")")[2].split()[1])
print("But rather " + statline.rpartition(")")[2].split()[1])
E

utils.py actually comes from update-manager, so I've switched the package.

This bug was fixed in the package update-manager - 1:15.04.2

---------------
update-manager (1:15.04.2) vivid; urgency=medium

  * UpdateManager/Core/utils.py: use rpartition in case the process name
    contains the right parenthesis character. Thanks to Roman Odaisky for the
    patch. (LP: #1399916)
  * UpdateManager/Dialogs.py: set the focus to "Restart Later" instead of
    "Restart Now", so people don't accidentally reboot. (LP: #1421044)
 -- Brian Murray <email address hidden> Wed, 18 Feb 2015 11:38:56 -0800