[Merge] lp:~manishsinha/software-properties/fix-887249-handle-404-error into lp:software-properties

Manish Sinha (मनीष सिन्हा) manishsinha at ubuntu.com
Mon Nov 7 19:24:35 UTC 2011


Manish Sinha (मनीष सिन्हा) has proposed merging lp:~manishsinha/software-properties/fix-887249-handle-404-error into lp:software-properties.

Requested reviews:
  Ubuntu Core Development Team (ubuntu-core-dev)
Related bugs:
  Bug #887249 in Software Properties: "Providing wrong ppa format in add-apt-repository throws a 404 error which is not handled appropriately"
  https://bugs.launchpad.net/software-properties/+bug/887249

For more details, see:
https://code.launchpad.net/~manishsinha/software-properties/fix-887249-handle-404-error/+merge/81488

In case the user provides a wrong username or ppa name, then the exception is handled

* The exception handler checks that if the username starts with a ~, in this case 
  it suggests does the user mean the username without a ~. e.g. if the user does
  sudo apt-add-repository ppa:~gnome3team/stable, then the script will ask the user
  if he means ppa:gnome3team/stable
* Even better friendlier messages can be provided, but for that it will introduce 
  lauchpadlib as a dependency

-- 
https://code.launchpad.net/~manishsinha/software-properties/fix-887249-handle-404-error/+merge/81488
Your team Ubuntu Core Development Team is requested to review the proposed merge of lp:~manishsinha/software-properties/fix-887249-handle-404-error into lp:software-properties.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: review-diff.txt
Type: text/x-diff
Size: 1030 bytes
Desc: not available
URL: <https://lists.ubuntu.com/archives/ubuntu-reviews/attachments/20111107/781283c6/attachment.diff>


More information about the Ubuntu-reviews mailing list