On 05/12/2014 04:03 AM, Vratislav Podzimek wrote:
On Thu, 2014-05-08 at 12:07 -0400, David Shea wrote:
> Instead of the nfs:[<options>:]<server>:/<path> syntax used by the
boot
> options, the kickstart repo command expects NFS URLs of the form
> nfs://<server>:/<path>. Correct the handling of these URLs within yum.
> ---
> pyanaconda/packaging/yumpayload.py | 7 ++++---
> 1 file changed, 4 insertions(+), 3 deletions(-)
>
> diff --git a/pyanaconda/packaging/yumpayload.py b/pyanaconda/packaging/yumpayload.py
> index 58b2309..ba33774 100644
> --- a/pyanaconda/packaging/yumpayload.py
> +++ b/pyanaconda/packaging/yumpayload.py
> @@ -633,10 +633,11 @@ reposdir=%s
> def _configureAddOnRepo(self, repo):
> """ Configure a single ksdata repo. """
> url = repo.baseurl
> - if url and url.startswith("nfs:"):
> - (opts, server, path) = iutil.parseNfsUrl(url)
> + if url and url.startswith("nfs://"):
> + # Let the assignment throw ValueError for bad NFS urls from kickstart
> + (server, path) = url.strip("nfs://").split(":", 2)
something.split(":", 2) may return 3 items. The second arg is the number
of splits not parts.
Oops. Thanks.