| 2016-10-12 23:31:02 |
Haw Loeung |
description |
Hi,
At present, the apt fetch.add_sources() uses 'add-apt-repository' to add defined install_sources. Unfortunately, for private PPAs, this causes issues with multiple sources added to /etc/apt/sources.list (especially on change of key):
...
| # deb http://archive.canonical.com/ubuntu xenial partner
| # deb-src http://archive.canonical.com/ubuntu xenial partner
| deb https://prodstack-cdo:1Mp...@private-ppa.launchpad.net/canonical-XXX-service/canonical-XXX-health/ubuntu xenial main
| # deb-src https://prodstack-cdo:IMpXXXXX@private-ppa.launchpad.net/canonical-is-sa/XXX-health/ubuntu xenial main
| deb https://prodstack-cdo:vgXXXXX@private-ppa.launchpad.net/canonical-is-sa/XXX-health/ubuntu xenial main
| # deb-src https://prodstack-cdo:vgXXXXX@private-ppa.launchpad.net/canonical-is-sa/XXX-health/ubuntu xenial main
| deb https://prodstack-cdo:vgXXXXX@private-ppa.launchpad.net/canonical-XXX-service/canonical-XXX-health/ubuntu xenial main
| # deb-src https://prodstack-cdo:vgXXXXX@private-ppa.launchpad.net/canonical-XXX-service/canonical-XXX-health/ubuntu xenial main
| # deb-src https://prodstack-cdo:vgXXXXX@private-ppa.launchpad.net/canonical-XXX-service/canonical-XXX-health/ubuntu xenial main
| # deb-src https://prodstack-cdo:vgXXXXX@private-ppa.launchpad.net/canonical-XXX-service/canonical-XXX-health/ubuntu xenial main
| # deb-src https://prodstack-cdo:vgXXXXX@private-ppa.launchpad.net/canonical-XXX-service/canonical-XXX-health/ubuntu xenial main
| # deb-src https://prodstack-cdo:vgXXXXX@private-ppa.launchpad.net/canonical-XXX-service/canonical-XXX-health/ubuntu xenial main
| # deb-src https://prodstack-cdo:vgXXXXX@private-ppa.launchpad.net/canonical-XXX-service/canonical-XXX-health/ubuntu xenial main
As per the add-apt-repository man page[1], it says if you use the repository string ppa:... it will add the source to /etc/apt/sources.d but unfortunately digging into the code, it doesn't support private PPAs:
software-properties-0.96.20.4/softwareproperties/ppa.py:
| class PPAShortcutHandler(object):
| def __init__(self, shortcut):
| super(PPAShortcutHandler, self).__init__()
| try:
| self.shortcut = mangle_ppa_shortcut(shortcut)
| except:
| raise ShortcutException(_("ERROR: '{shortcut}' is not a valid ppa format")
| .format(shortcut=shortcut))
| info = get_ppa_info(self.shortcut)
|
| if "private" in info and info["private"]:
| raise ShortcutException(
| _("Adding private PPAs is not supported currently"))
|
| self._info = info
Any chance we could update fetch.add_sources() to handle private PPAs properly and ship them out to /etc/apt/sources.d like we're already doing for cloud: and proposed:?
[1]http://manpages.ubuntu.com/manpages/trusty/en/man1/add-apt-repository.1.html#contenttoc3 |
Hi,
At present, the apt fetch.add_sources() uses 'add-apt-repository' to add defined install_sources. Unfortunately, for private PPAs, this causes issues with multiple sources added to /etc/apt/sources.list (especially on change of key):
...
| # deb http://archive.canonical.com/ubuntu xenial partner
| # deb-src http://archive.canonical.com/ubuntu xenial partner
|
| deb https://prodstack-cdo:1Mp...@private-ppa.launchpad.net/canonical-xxx-service/canonical-xxx-health/ubuntu xenial main
| deb https://prodstack-cdo:vgX...@private-ppa.launchpad.net/canonical-somedude/XXX-health/ubuntu xenial main
| deb https://prodstack-cdo:vgX...@private-ppa.launchpad.net/canonical-XXX-service/canonical-xxx-health/ubuntu xenial main
As per the add-apt-repository man page[1], it says if you use the repository string ppa:... it will add the source to /etc/apt/sources.d but unfortunately digging into the code, it doesn't support private PPAs:
software-properties-0.96.20.4/softwareproperties/ppa.py:
| class PPAShortcutHandler(object):
| def __init__(self, shortcut):
| super(PPAShortcutHandler, self).__init__()
| try:
| self.shortcut = mangle_ppa_shortcut(shortcut)
| except:
| raise ShortcutException(_("ERROR: '{shortcut}' is not a valid ppa format")
| .format(shortcut=shortcut))
| info = get_ppa_info(self.shortcut)
|
| if "private" in info and info["private"]:
| raise ShortcutException(
| _("Adding private PPAs is not supported currently"))
|
| self._info = info
Any chance we could update fetch.add_sources() to handle private PPAs properly and ship them out to /etc/apt/sources.d like we're already doing for cloud: and proposed:?
[1]http://manpages.ubuntu.com/manpages/trusty/en/man1/add-apt-repository.1.html#contenttoc3 |
|
