[PATCH] gnu: pypi importer: Remove python-setuptools and hint on how to re-add it.

[PATCH] gnu: pypi importer: Remove python-setuptools and hint on how to re-add it.

[Danny Milosavljevic]

[-- Attachment #1: Type: text/plain, Size: 220 bytes --]

gnu: pypi importer: Remove python-setuptools and hint on how to re-add it.

* guix/import/pypi.scm (make-pypi-sexp): Modified.
---
 guix/import/pypi.scm | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)


[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: 0001-gnu-pypi-importer-Remove-python-setuptools-and-hint-.patch --]
[-- Type: text/x-patch; name="0001-gnu-pypi-importer-Remove-python-setuptools-and-hint-.patch", Size: 1152 bytes --]

diff --git a/guix/import/pypi.scm b/guix/import/pypi.scm
index 343445a..d177d33 100644
--- a/guix/import/pypi.scm
+++ b/guix/import/pypi.scm
@@ -233,7 +233,7 @@ name/variable pairs describing the required inputs of this package."
   (sort
     (map (lambda (input)
            (list input (list 'unquote (string->symbol input))))
-         (append '("python-setuptools")
+         (append ;'("python-setuptools")
                  ;; Argparse has been part of Python since 2.7.
                  (remove (cut string=? "python-argparse" <>)
                          (guess-requirements source-url wheel-url tarball))))
@@ -271,7 +271,8 @@ VERSION, SOURCE-URL, HOME-PAGE, SYNOPSIS, DESCRIPTION, and LICENSE."
              (home-page ,home-page)
              (synopsis ,synopsis)
              (description ,description)
-             (license ,(license->symbol license)))))))
+             (license ,(license->symbol license))
+             (properties `((python2-variant . ,(delay python2-foobar)))))))))
 
 (define (pypi->guix-package package-name)
   "Fetch the metadata for PACKAGE-NAME from pypi.python.org, and return the

Re: [PATCH] gnu: pypi importer: Remove python-setuptools and hint on how to re-add it.

[ng0]

Hi,

Danny Milosavljevic <dannym@scratchpost.org> writes:

> gnu: pypi importer: Remove python-setuptools and hint on how to re-add it.

Can you be more specific why it is removed? I don't understand the harm
it does or doesn't do when it's included.
Would this not be bad for packages which require setuptools?

> * guix/import/pypi.scm (make-pypi-sexp): Modified.
> ---
>  guix/import/pypi.scm | 5 +++--
>  1 file changed, 3 insertions(+), 2 deletions(-)
>
> diff --git a/guix/import/pypi.scm b/guix/import/pypi.scm
> index 343445a..d177d33 100644
> --- a/guix/import/pypi.scm
> +++ b/guix/import/pypi.scm
> @@ -233,7 +233,7 @@ name/variable pairs describing the required inputs of this package."
>    (sort
>      (map (lambda (input)
>             (list input (list 'unquote (string->symbol input))))
> -         (append '("python-setuptools")
> +         (append ;'("python-setuptools")
>                   ;; Argparse has been part of Python since 2.7.
>                   (remove (cut string=? "python-argparse" <>)
>                           (guess-requirements source-url wheel-url tarball))))
> @@ -271,7 +271,8 @@ VERSION, SOURCE-URL, HOME-PAGE, SYNOPSIS, DESCRIPTION, and LICENSE."
>               (home-page ,home-page)
>               (synopsis ,synopsis)
>               (description ,description)
> -             (license ,(license->symbol license)))))))
> +             (license ,(license->symbol license))
> +             (properties `((python2-variant . ,(delay python2-foobar)))))))))
>  
>  (define (pypi->guix-package package-name)
>    "Fetch the metadata for PACKAGE-NAME from pypi.python.org, and return the

-- 
♥Ⓐ  ng0
For non-prism friendly talk find me on http://www.psyced.org

Re: [PATCH] gnu: pypi importer: Remove python-setuptools and hint on how to re-add it.

[Danny Milosavljevic]

Hi,

> Can you be more specific why it is removed? I don't understand the harm
> it does or doesn't do when it's included.
> Would this not be bad for packages which require setuptools?

Python 3.4 integrated setuptools into the main distribution. It doesn't require an external python-setuptools. I don't understand why Guix even still has a "python-setuptools" package. (python2-setuptools is OK - but the other one: why?)

New packages are usually Python 3 compatible. Also, the python-build-system defaults to Python 3.

That's why I assume that the package S-Expression printed is for Python 3 (and so do many other Guix parts). There, it doesn't make sense to include setuptools anymore.

Also, it's better if the package expression doesn't include python-setuptools by default because a lazy packager (i.e. me) will just copy & paste it and call it a day. Then the new package depends on python-setuptools even though it doesn't need it at all.

I'm not sure what to do about the Python 2 package expression. I think this function only can return one expression.

If possible, I'd like it to automatically print the Python 2 package expression, for example

(define-public python2-lockfile
  (let ((base (package-with-python2 (strip-python2-variant python-lockfile))))
    (package
      (inherit base)
      (native-inputs `(("python2-setuptools" ,python2-setuptools)
                       ,@(package-native-inputs base))))))

, as well.

Re: [PATCH] gnu: pypi importer: Remove python-setuptools and hint on how to re-add it.

[Pjotr Prins]

On Tue, Aug 16, 2016 at 09:39:52PM +0200, Danny Milosavljevic wrote:
> Hi,
> 
> > Can you be more specific why it is removed? I don't understand the harm
> > it does or doesn't do when it's included.
> > Would this not be bad for packages which require setuptools?

Not all python packages require setuptools.

Pj.

Re: [PATCH] gnu: pypi importer: Remove python-setuptools and hint on how to re-add it.

[Leo Famulari]

On Tue, Aug 16, 2016 at 08:13:10PM +0200, Danny Milosavljevic wrote:
> gnu: pypi importer: Remove python-setuptools and hint on how to re-add it.

I'm in favor of not automatically adding setuptools to packages created
with the PyPi importer. For the packages that require setuptools,
packagers can add it themselves. I reply to too many patches with
"Please remove setuptools".

I don't understand the importer well enough to comment on this
particular patch.