[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: First patch & hello
L p R n d n
Re: First patch & hello
Wed, 14 Nov 2018 16:51:24 +0100
Ricardo Wurmus <address@hidden> writes:
thank you for your contribution!
1 . I put the package under fontutils.scm but maybe there's a better
place for a font viewer?
This seems fine to me.
2. I found there is a glib-or-gtk-build-system, I hesitated to use as
not sure what its purpose. Can someone clarify its use?
The glib-or-gtk-build-system is an extension of the gnu-build-system
that is useful for GNOME packages and other packages that require a
little more setup after installation. Here’s what the comments in
;; This build system is an extension of the 'gnu-build-system'. It
;; accomodates the needs of applications making use of glib or gtk+
;; to be interpreted in the mathematical sense). This is achieved by
;; two phases run after the 'install' phase:
;; 'glib-or-gtk-wrap' phase:
;; a) This phase looks for GSettings schemas, GIO modules and theming
;; If any of these is found in any input package, then all programs in
;; "out/bin" are wrapped in scripts defining the nedessary environment
;; b) Looks for the existence of "libdir/gtk-3.0" directories in all
;; packages. If any is found, then the environment variable "GTK_PATH"
;; suitably set and added to the wrappers. The variable "GTK_PATH" has
;; preferred over "GTK_EXE_PREFIX" because the latter can only point to
;; single directory, while we may need to point to several ones.
;; 'glib-or-gtk-compile-schemas' phase:
;; Looks for the presence of "out/share/glib-2.0/schemas". If that
;; exists and does not include a file named "gschemas.compiled", then
;; "glib-compile-schemas" is run in that directory.
3. I tried to get a clean patch but it needed some manual work. What
your workflow to produce patches? (I use emacs)
Usually you would make a local git commit and then run
git format-patch -1
to generate a patch file from that commit. You could also directly
it to the mailing list with “git send-email”.
What follows are some comments about the patch. I’m trying to be extra
thorough; please don’t let this discourage you. Some of my comments
will just be matters of opinion :)
Thanks! It was exactly what I was looking for!
Subject: [PATCH] gnu: Add font-manager.
It seems that the commit message is missing here. In this case the
complete commit message would be:
gnu: Add font-manager.
* gnu/packages/fontutils.scm (font-manager): New variable.
Do you know why it wasn't added? I used magit format-patch.
Please remove the last sentence. Please also reindent the description
string (there should be no indentation for following lines). Also
please use two spaces after sentences. “guix lint” will remind you of
Is there a way to specify a custom location for "guix lint"? A little
bit like the "-L" option from "guix build" for example?
I suppose the right way to invoke 'guix lint' should be to use
"./pre-inst-env" in a local
guix repository, but I didn't find where I was supposed to find it. :/
Could you please send an updated patch to address@hidden
I will soon. I just tested to build the package with "rounds=2" and it
seems the package isn't deterministic. Is it acceptable or a total