Thanks for the patch Nick!

While your use of self.sourceslist.add() seems good, I don't like how it
is using similar logic that is in ppa.py in SoftwareProperties.py's
add_source_from_line(). Wouldn't it be better to extract out the parts
of new_deb_entry and new_debsrc_entry and give those parts to
self.sourceslist.add()?

NAK pending comments from submitter. Please resubscribe ubuntu-sponsors
once you have commented and/or upddated your patch. Thanks again!

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/854841

Title:
  add-apt-repository will write duplicate entries to
  /etc/apt/sources.list

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/software-properties/+bug/854841/+subscriptions

-- 
ubuntu-bugs mailing list
[email protected]
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

Reply via email to