unofficial mirror of bug-guix@gnu.org 
 help / color / mirror / code / Atom feed
* bug#65774: python updater clears inputs, leaves propagated-inputs empty
@ 2023-09-06  5:15 Maxim Cournoyer
  2023-09-06 17:18 ` Sergio Pastor Pérez
  0 siblings, 1 reply; 3+ messages in thread
From: Maxim Cournoyer @ 2023-09-06  5:15 UTC (permalink / raw)
  To: 65774

Hi,

I've had the Python updater produce this when attempting to update
fontmake:

gnu/packages/fontutils.scm:780:2: warning: fontmake: 'propagated-inputs'
field not found; leaving it unchanged

gnu/packages/fontutils.scm:780:2: warning: fontmake: expected
'propagated-inputs' value: (python-attrs python-fontmath
python-fonttools python-glyphslib python-ufo2ft python-ufolib2)

--8<---------------cut here---------------start------------->8---
modified   gnu/packages/fontutils.scm
@@ -779,16 +779,16 @@ (define-public psautohint-font-data
 (define-public fontmake
   (package
     (name "fontmake")
-    (version "3.4.0")
+    (version "3.7.1")
     (source (origin
               (method url-fetch)
               (uri (pypi-uri "fontmake" version ".zip"))
               (sha256
                (base32
-                "0fc5c9csjpy1aa4c03p7nvjgls5wjplhmmf42n0cmvrlh6cm7wl3"))))
+                "0ib7fvwgwazm7qfj4a3rkqkb40xfbj40rnvsmkvl2isg2ky3vg9m"))))
     (build-system python-build-system)
-    (inputs (list python-fontmath python-glyphslib))
-    (native-inputs (list unzip python-setuptools-scm))
+    (inputs (list))
+    (native-inputs (list zip))
     (home-page "https://github.com/googlefonts/fontmake")
     (synopsis
      "Compile fonts from sources (UFO, Glyphs) to binary (OpenType, TrueType)")
--8<---------------cut here---------------end--------------->8---

The choice of using inputs here was conscious, as it is a command, not a
library.  Perhaps it could check if the name starts with 'python-' or
not?  It's a bit 'magic', but it would help.

-- 
Thanks,
Maxim




^ permalink raw reply	[flat|nested] 3+ messages in thread

* bug#65774: python updater clears inputs, leaves propagated-inputs empty
  2023-09-06  5:15 bug#65774: python updater clears inputs, leaves propagated-inputs empty Maxim Cournoyer
@ 2023-09-06 17:18 ` Sergio Pastor Pérez
  2023-09-06 18:38   ` Maxim Cournoyer
  0 siblings, 1 reply; 3+ messages in thread
From: Sergio Pastor Pérez @ 2023-09-06 17:18 UTC (permalink / raw)
  To: Maxim Cournoyer; +Cc: 65774

Hi,

I'm afraid you lost me here. Why is the `inputs' field empty? I don't
understand what you meand with the checking of 'python-'.

Keep in mind that I have very little experince with python packaging so
I'm may not understand you due to my lack of background.


Maxim Cournoyer <maxim.cournoyer@gmail.com> writes:

> Hi,
>
> I've had the Python updater produce this when attempting to update
> fontmake:
>
> gnu/packages/fontutils.scm:780:2: warning: fontmake: 'propagated-inputs'
> field not found; leaving it unchanged
>
> gnu/packages/fontutils.scm:780:2: warning: fontmake: expected
> 'propagated-inputs' value: (python-attrs python-fontmath
> python-fonttools python-glyphslib python-ufo2ft python-ufolib2)
>
> --8<---------------cut here---------------start------------->8---
> modified   gnu/packages/fontutils.scm
> @@ -779,16 +779,16 @@ (define-public psautohint-font-data
>  (define-public fontmake
>    (package
>      (name "fontmake")
> -    (version "3.4.0")
> +    (version "3.7.1")
>      (source (origin
>                (method url-fetch)
>                (uri (pypi-uri "fontmake" version ".zip"))
>                (sha256
>                 (base32
> -                "0fc5c9csjpy1aa4c03p7nvjgls5wjplhmmf42n0cmvrlh6cm7wl3"))))
> +                "0ib7fvwgwazm7qfj4a3rkqkb40xfbj40rnvsmkvl2isg2ky3vg9m"))))
>      (build-system python-build-system)
> -    (inputs (list python-fontmath python-glyphslib))
> -    (native-inputs (list unzip python-setuptools-scm))
> +    (inputs (list))
> +    (native-inputs (list zip))
>      (home-page "https://github.com/googlefonts/fontmake")
>      (synopsis
>       "Compile fonts from sources (UFO, Glyphs) to binary (OpenType, TrueType)")
> --8<---------------cut here---------------end--------------->8---
>
> The choice of using inputs here was conscious, as it is a command, not a
> library.  Perhaps it could check if the name starts with 'python-' or
> not?  It's a bit 'magic', but it would help.





^ permalink raw reply	[flat|nested] 3+ messages in thread

* bug#65774: python updater clears inputs, leaves propagated-inputs empty
  2023-09-06 17:18 ` Sergio Pastor Pérez
@ 2023-09-06 18:38   ` Maxim Cournoyer
  0 siblings, 0 replies; 3+ messages in thread
From: Maxim Cournoyer @ 2023-09-06 18:38 UTC (permalink / raw)
  To: Sergio Pastor Pérez; +Cc: 65774

Hi,

Sergio Pastor Pérez <sergio.pastorperez@outlook.es> writes:

> Hi,
>
> I'm afraid you lost me here. Why is the `inputs' field empty?

I don't know!  That's the current behavior of the Python updater, at
least when I ran it on fontmake (in our patches tracker at
https://issues.guix.gnu.org/64957).

> understand what you meand with the checking of 'python-'.

I meant looking whether the package variable (usually also its name) is
prefixed by "python-" to differentiate between a library or a command.
It's not a very good heuristic/test, I'm afraid.

-- 
Thanks,
Maxim




^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2023-09-12  1:53 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-09-06  5:15 bug#65774: python updater clears inputs, leaves propagated-inputs empty Maxim Cournoyer
2023-09-06 17:18 ` Sergio Pastor Pérez
2023-09-06 18:38   ` Maxim Cournoyer

Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/guix.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).