From mboxrd@z Thu Jan 1 00:00:00 1970 From: Maxim Cournoyer Subject: bug#24450: [PATCHv2] Re: pypi importer outputs strange character series in optional dependency case. Date: Sun, 16 Jun 2019 14:53:33 +0900 Message-ID: <871rzu2i02.fsf@gmail.com> References: <87pnod7ot4.fsf@gmail.com> Mime-Version: 1.0 Content-Type: text/plain Return-path: Received: from eggs.gnu.org ([2001:470:142:3::10]:34715) by lists.gnu.org with esmtp (Exim 4.86_2) (envelope-from ) id 1hcO79-0007NB-IN for bug-guix@gnu.org; Sun, 16 Jun 2019 01:54:05 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1hcO78-0006Os-CB for bug-guix@gnu.org; Sun, 16 Jun 2019 01:54:03 -0400 Received: from debbugs.gnu.org ([209.51.188.43]:54244) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1hcO78-0006Od-2J for bug-guix@gnu.org; Sun, 16 Jun 2019 01:54:02 -0400 Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1hcO77-0003OT-VB for bug-guix@gnu.org; Sun, 16 Jun 2019 01:54:02 -0400 Sender: "Debbugs-submit" Resent-Message-ID: In-Reply-To: (Ricardo Wurmus's message of "Tue, 28 May 2019 16:53:17 +0200") List-Id: Bug reports for GNU Guix List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: bug-guix-bounces+gcggb-bug-guix=m.gmane.org@gnu.org Sender: "bug-Guix" To: Ricardo Wurmus Cc: 24450@debbugs.gnu.org Ricardo Wurmus writes: > And finally: Number 9! Yay! >> From 1290f9d1f0d594fdd4723d76b94116be25da9dd5 Mon Sep 17 00:00:00 2001 >> From: Maxim Cournoyer >> Date: Sat, 30 Mar 2019 20:27:35 -0400 >> Subject: [PATCH 9/9] import: pypi: Preserve package name case when forming >> pypi-uri. >> >> Fixes issue: #33046. > > Please change this to: > > Fixes . Done! >> * guix/build-system/python.scm (pypi-uri): Update the host URI to >> "files.pythonhosted.org". >> * guix/import/pypi.scm (make-pypi-sexp): Preserve the package name case when >> the source URL calls for it. > > Is the first change to use files.pythonhosted.org required to fix this? > Or is this unrelated? > > If it is required this looks fine to me. > > Thank you! The permanent redirection was found while fixing the issue; but it's better to have the fix separate. I've separated it into its own commit. Thank you!