ci: Add derivation inputs.
[jackhill/guix/guix.git] / doc / contributing.texi
CommitLineData
8c01b9d0
ML
1@node Contributing
2@chapter Contributing
3
4This project is a cooperative effort, and we need your help to make it
5grow! Please get in touch with us on @email{guix-devel@@gnu.org} and
c0087d33 6@code{#guix} on the Libera Chat IRC network. We welcome ideas, bug
8c01b9d0
ML
7reports, patches, and anything that may be helpful to the project. We
8particularly welcome help on packaging (@pxref{Packaging Guidelines}).
9
e15fcdd1
LC
10@cindex code of conduct, of contributors
11@cindex contributor covenant
dcb7119a
AS
12We want to provide a warm, friendly, and harassment-free environment, so
13that anyone can contribute to the best of their abilities. To this end
14our project uses a ``Contributor Covenant'', which was adapted from
478af028 15@url{https://contributor-covenant.org/}. You can find a local version in
dcb7119a 16the @file{CODE-OF-CONDUCT} file in the source tree.
e15fcdd1 17
dfcdd9c2
LC
18Contributors are not required to use their legal name in patches and
19on-line communication; they can use any name or pseudonym of their
20choice.
21
8c01b9d0
ML
22@menu
23* Building from Git:: The latest and greatest.
24* Running Guix Before It Is Installed:: Hacker tricks.
25* The Perfect Setup:: The right tools.
afe7408e 26* Packaging Guidelines:: Growing the distribution.
8c01b9d0
ML
27* Coding Style:: Hygiene of the contributor.
28* Submitting Patches:: Share your work.
a7bde77d 29* Tracking Bugs and Patches:: Using Debbugs.
2d315cd4 30* Commit Access:: Pushing to the official repository.
5800d2aa 31* Updating the Guix Package:: Updating the Guix package definition.
1897a6ef 32* Translating Guix:: Make Guix speak your native language.
8c01b9d0
ML
33@end menu
34
35@node Building from Git
36@section Building from Git
37
38If you want to hack Guix itself, it is recommended to use the latest
dbfcadfc
LC
39version from the Git repository:
40
41@example
42git clone https://git.savannah.gnu.org/git/guix.git
43@end example
44
b3011dbb
LC
45@cindex authentication, of a Guix checkout
46How do you ensure that you obtained a genuine copy of the repository?
3e9c6ee1 47To do that, run @command{guix git authenticate}, passing it the commit
63c799c1
LC
48and OpenPGP fingerprint of the @dfn{channel introduction}
49(@pxref{Invoking guix git authenticate}):
b3011dbb 50
63c799c1
LC
51@c The commit and fingerprint below must match those of the channel
52@c introduction in '%default-channels'.
b3011dbb 53@example
3e9c6ee1 54git fetch origin keyring:keyring
63c799c1
LC
55guix git authenticate 9edb3f66fd807b096b48283debdcddccfea34bad \
56 "BBB0 2DDF 2CEA F6A8 0D1D E643 A2A0 6DF2 A33A 54FA"
b3011dbb
LC
57@end example
58
59@noindent
63c799c1
LC
60This command completes with exit code zero on success; it prints an
61error message and exits with a non-zero code otherwise.
62
63As you can see, there is a chicken-and-egg problem: you first need to
64have Guix installed. Typically you would install Guix System
65(@pxref{System Installation}) or Guix on top of another distro
66(@pxref{Binary Installation}); in either case, you would verify the
67OpenPGP signature on the installation medium. This ``bootstraps'' the
68trust chain.
b3011dbb 69
5fb95cc5
LC
70The easiest way to set up a development environment for Guix is, of
71course, by using Guix! The following command starts a new shell where
72all the dependencies and appropriate environment variables are set up to
73hack on Guix:
8c01b9d0 74
5fb95cc5 75@example
43ec98ef 76guix environment guix --pure
5fb95cc5
LC
77@end example
78
79@xref{Invoking guix environment}, for more information on that command.
0636742b
FP
80
81If you are unable to use Guix when building Guix from a checkout, the
82following are the required packages in addition to those mentioned in the
83installation instructions (@pxref{Requirements}).
84
85@itemize
21656ffa
VS
86@item @url{https://gnu.org/software/autoconf/, GNU Autoconf};
87@item @url{https://gnu.org/software/automake/, GNU Automake};
88@item @url{https://gnu.org/software/gettext/, GNU Gettext};
89@item @url{https://gnu.org/software/texinfo/, GNU Texinfo};
90@item @url{https://www.graphviz.org/, Graphviz};
91@item @url{https://www.gnu.org/software/help2man/, GNU Help2man (optional)}.
0636742b
FP
92@end itemize
93
94On Guix, extra dependencies can be added by instead running @command{guix
95environment} with @option{--ad-hoc}:
5fb95cc5
LC
96
97@example
43ec98ef 98guix environment guix --pure --ad-hoc help2man git strace
5fb95cc5
LC
99@end example
100
101Run @command{./bootstrap} to generate the build system infrastructure
102using Autoconf and Automake. If you get an error like this one:
8c01b9d0
ML
103
104@example
105configure.ac:46: error: possibly undefined macro: PKG_CHECK_MODULES
106@end example
107
5fb95cc5 108@noindent
8c01b9d0 109it probably means that Autoconf couldn’t find @file{pkg.m4}, which is
5fb95cc5
LC
110provided by pkg-config. Make sure that @file{pkg.m4} is available. The
111same holds for the @file{guile.m4} set of macros provided by Guile. For
112instance, if you installed Automake in @file{/usr/local}, it wouldn’t
113look for @file{.m4} files in @file{/usr/share}. In that case, you have
114to invoke the following command:
8c01b9d0
ML
115
116@example
117export ACLOCAL_PATH=/usr/share/aclocal
118@end example
119
aabe6d38 120@xref{Macro Search Path,,, automake, The GNU Automake Manual}, for
8c01b9d0
ML
121more information.
122
3a25c631
LC
123Then, run @command{./configure} as usual. Make sure to pass
124@code{--localstatedir=@var{directory}} where @var{directory} is the
125@code{localstatedir} value used by your current installation (@pxref{The
6f406e5d
LC
126Store}, for information about this), usually @file{/var}. Note that you
127will probably not run @command{make install} at the end (you don't have
128to) but it's still important to pass the right @code{localstatedir}.
8c01b9d0 129
5b97804e
LF
130Finally, you have to invoke @code{make && make check} to build Guix and
131run the tests (@pxref{Running the Test Suite}). If anything fails, take
132a look at installation instructions (@pxref{Installation}) or send a
133message to the @email{guix-devel@@gnu.org, mailing list}.
8c01b9d0 134
33391ee5
LC
135From there on, you can authenticate all the commits included in your
136checkout by running:
137
138@example
139make authenticate
140@end example
141
142The first run takes a couple of minutes, but subsequent runs are faster.
143
ef6596a2
MÁAV
144Or, when your configuration for your local Git repository doesn't match
145the default one, you can provide the reference for the @code{keyring}
146branch through the variable @code{GUIX_GIT_KEYRING}. The following
147example assumes that you have a Git remote called @samp{myremote}
148pointing to the official repository:
149
150@example
151make authenticate GUIX_GIT_KEYRING=myremote/keyring
152@end example
153
33391ee5
LC
154@quotation Note
155You are advised to run @command{make authenticate} after every
156@command{git pull} invocation. This ensures you keep receiving valid
157changes to the repository.
158@end quotation
159
8c01b9d0
ML
160
161@node Running Guix Before It Is Installed
162@section Running Guix Before It Is Installed
163
164In order to keep a sane working environment, you will find it useful to
165test the changes made in your local source tree checkout without
166actually installing them. So that you can distinguish between your
167``end-user'' hat and your ``motley'' costume.
168
169To that end, all the command-line tools can be used even if you have not
ad911c83
RL
170run @code{make install}. To do that, you first need to have an
171environment with all the dependencies available (@pxref{Building from
172Git}), and then simply prefix each command with @command{./pre-inst-env}
173(the @file{pre-inst-env} script lives in the top build tree of Guix; it
174is generated by running @command{./bootstrap} followed by
175@command{./configure}). As an example, here is how you would build the
176@code{hello} package as defined in your working tree (this assumes
177@command{guix-daemon} is already running on your system; it's OK if it's
178a different version):
8c01b9d0
ML
179
180@example
8c01b9d0
ML
181$ ./pre-inst-env guix build hello
182@end example
183
184@noindent
0636742b 185Similarly, an example for a Guile session using the Guix modules:
8c01b9d0
ML
186
187@example
188$ ./pre-inst-env guile -c '(use-modules (guix utils)) (pk (%current-system))'
96856613
LC
189
190;;; ("x86_64-linux")
191@end example
192
193@noindent
194@cindex REPL
195@cindex read-eval-print loop
196@dots{} and for a REPL (@pxref{Using Guile Interactively,,, guile, Guile
197Reference Manual}):
198
199@example
200$ ./pre-inst-env guile
201scheme@@(guile-user)> ,use(guix)
202scheme@@(guile-user)> ,use(gnu)
203scheme@@(guile-user)> (define snakes
204 (fold-packages
205 (lambda (package lst)
206 (if (string-prefix? "python"
207 (package-name package))
208 (cons package lst)
209 lst))
210 '()))
211scheme@@(guile-user)> (length snakes)
212$1 = 361
8c01b9d0
ML
213@end example
214
9022861d
LC
215If you are hacking on the daemon and its supporting code or if
216@command{guix-daemon} is not already running on your system, you can
217launch it straight from the build tree@footnote{The @option{-E} flag to
218@command{sudo} guarantees that @code{GUILE_LOAD_PATH} is correctly set
219such that @command{guix-daemon} and the tools it uses can find the Guile
220modules they need.}:
221
222@example
223$ sudo -E ./pre-inst-env guix-daemon --build-users-group=guixbuild
224@end example
225
8c01b9d0
ML
226The @command{pre-inst-env} script sets up all the environment variables
227necessary to support this, including @env{PATH} and @env{GUILE_LOAD_PATH}.
228
ef54b61d 229Note that @command{./pre-inst-env guix pull} does @emph{not} upgrade the
75e24d7b 230local source tree; it simply updates the @file{~/.config/guix/current}
ef54b61d 231symlink (@pxref{Invoking guix pull}). Run @command{git pull} instead if
75e24d7b 232you want to upgrade your local source tree.
ef54b61d 233
8c01b9d0
ML
234
235@node The Perfect Setup
236@section The Perfect Setup
237
238The Perfect Setup to hack on Guix is basically the perfect setup used
239for Guile hacking (@pxref{Using Guile in Emacs,,, guile, Guile Reference
240Manual}). First, you need more than an editor, you need
21656ffa
VS
241@url{https://www.gnu.org/software/emacs, Emacs}, empowered by the
242wonderful @url{https://nongnu.org/geiser/, Geiser}. To set that up, run:
f73ab814
LC
243
244@example
bcdb8e98 245guix package -i emacs guile emacs-geiser emacs-geiser-guile
f73ab814 246@end example
8c01b9d0
ML
247
248Geiser allows for interactive and incremental development from within
249Emacs: code compilation and evaluation from within buffers, access to
250on-line documentation (docstrings), context-sensitive completion,
251@kbd{M-.} to jump to an object definition, a REPL to try out your code,
252and more (@pxref{Introduction,,, geiser, Geiser User Manual}). For
253convenient Guix development, make sure to augment Guile’s load path so
254that it finds source files from your checkout:
255
256@lisp
257;; @r{Assuming the Guix checkout is in ~/src/guix.}
bb38ece4
AK
258(with-eval-after-load 'geiser-guile
259 (add-to-list 'geiser-guile-load-path "~/src/guix"))
8c01b9d0
ML
260@end lisp
261
262To actually edit the code, Emacs already has a neat Scheme mode. But in
263addition to that, you must not miss
c2c73f58 264@url{https://www.emacswiki.org/emacs/ParEdit, Paredit}. It provides
8c01b9d0
ML
265facilities to directly operate on the syntax tree, such as raising an
266s-expression or wrapping it, swallowing or rejecting the following
267s-expression, etc.
268
42cdcdff
RW
269@cindex code snippets
270@cindex templates
271@cindex reducing boilerplate
272We also provide templates for common git commit messages and package
273definitions in the @file{etc/snippets} directory. These templates can
21656ffa 274be used with @url{https://joaotavora.github.io/yasnippet/, YASnippet} to
42cdcdff
RW
275expand short trigger strings to interactive text snippets. You may want
276to add the snippets directory to the @var{yas-snippet-dirs} variable in
277Emacs.
278
279@lisp
280;; @r{Assuming the Guix checkout is in ~/src/guix.}
281(with-eval-after-load 'yasnippet
282 (add-to-list 'yas-snippet-dirs "~/src/guix/etc/snippets"))
283@end lisp
284
285The commit message snippets depend on @url{https://magit.vc/, Magit} to
286display staged files. When editing a commit message type @code{add}
287followed by @kbd{TAB} to insert a commit message template for adding a
288package; type @code{update} followed by @kbd{TAB} to insert a template
9286c295
AI
289for updating a package; type @code{https} followed by @kbd{TAB} to
290insert a template for changing the home page URI of a package to HTTPS.
42cdcdff
RW
291
292The main snippet for @code{scheme-mode} is triggered by typing
293@code{package...} followed by @kbd{TAB}. This snippet also inserts the
294trigger string @code{origin...}, which can be expanded further. The
295@code{origin} snippet in turn may insert other trigger strings ending on
296@code{...}, which also can be expanded further.
297
9a397114
OP
298@cindex insert or update copyright
299@cindex @code{M-x guix-copyright}
300@cindex @code{M-x copyright-update}
4f4fb215 301We additionally provide insertion and automatic update of a copyright in
9a397114
OP
302@file{etc/copyright.el}. You may want to set your full name, mail, and
303load a file.
304
305@lisp
306(setq user-full-name "Alice Doe")
307(setq user-mail-address "alice@@mail.org")
308;; @r{Assuming the Guix checkout is in ~/src/guix.}
309(load-file "~/src/guix/etc/copyright.el")
310@end lisp
311
312To insert a copyright at the current line invoke @code{M-x guix-copyright}.
313
314To update a copyright you need to specify a @code{copyright-names-regexp}.
315
316@lisp
317(setq copyright-names-regexp
318 (format "%s <%s>" user-full-name user-mail-address))
319@end lisp
320
321You can check if your copyright is up to date by evaluating @code{M-x
322copyright-update}. If you want to do it automatically after each buffer
323save then add @code{(add-hook 'after-save-hook 'copyright-update)} in
324Emacs.
8c01b9d0 325
afe7408e
LC
326@node Packaging Guidelines
327@section Packaging Guidelines
328
329@cindex packages, creating
330The GNU distribution is nascent and may well lack some of your favorite
331packages. This section describes how you can help make the distribution
332grow.
333
334Free software packages are usually distributed in the form of
335@dfn{source code tarballs}---typically @file{tar.gz} files that contain
336all the source files. Adding a package to the distribution means
337essentially two things: adding a @dfn{recipe} that describes how to
338build the package, including a list of other packages required to build
339it, and adding @dfn{package metadata} along with that recipe, such as a
340description and licensing information.
341
342In Guix all this information is embodied in @dfn{package definitions}.
343Package definitions provide a high-level view of the package. They are
344written using the syntax of the Scheme programming language; in fact,
345for each package we define a variable bound to the package definition,
346and export that variable from a module (@pxref{Package Modules}).
347However, in-depth Scheme knowledge is @emph{not} a prerequisite for
348creating packages. For more information on package definitions,
349@pxref{Defining Packages}.
350
351Once a package definition is in place, stored in a file in the Guix
352source tree, it can be tested using the @command{guix build} command
353(@pxref{Invoking guix build}). For example, assuming the new package is
354called @code{gnew}, you may run this command from the Guix build tree
355(@pxref{Running Guix Before It Is Installed}):
356
357@example
358./pre-inst-env guix build gnew --keep-failed
359@end example
360
361Using @code{--keep-failed} makes it easier to debug build failures since
362it provides access to the failed build tree. Another useful
363command-line option when debugging is @code{--log-file}, to access the
364build log.
365
366If the package is unknown to the @command{guix} command, it may be that
367the source file contains a syntax error, or lacks a @code{define-public}
368clause to export the package variable. To figure it out, you may load
369the module from Guile to get more information about the actual error:
370
371@example
372./pre-inst-env guile -c '(use-modules (gnu packages gnew))'
373@end example
374
375Once your package builds correctly, please send us a patch
376(@pxref{Submitting Patches}). Well, if you need help, we will be happy to
377help you too. Once the patch is committed in the Guix repository, the
378new package automatically gets built on the supported platforms by
aa9ba865 379@url{@value{SUBSTITUTE-URL}, our continuous integration system}.
afe7408e
LC
380
381@cindex substituter
382Users can obtain the new package definition simply by running
383@command{guix pull} (@pxref{Invoking guix pull}). When
384@code{@value{SUBSTITUTE-SERVER}} is done building the package, installing the
385package automatically downloads binaries from there
386(@pxref{Substitutes}). The only place where human intervention is
387needed is to review and apply the patch.
388
389
390@menu
391* Software Freedom:: What may go into the distribution.
392* Package Naming:: What's in a name?
393* Version Numbers:: When the name is not enough.
394* Synopses and Descriptions:: Helping users find the right package.
71e746cc 395* Snippets versus Phases:: Whether to use a snippet, or a build phase.
15fba3b1 396* Emacs Packages:: Your Elisp fix.
afe7408e
LC
397* Python Modules:: A touch of British comedy.
398* Perl Modules:: Little pearls.
399* Java Packages:: Coffee break.
8d1b22b2 400* Rust Crates:: Beware of oxidation.
afe7408e
LC
401* Fonts:: Fond of fonts.
402@end menu
403
404@node Software Freedom
405@subsection Software Freedom
406
407@c Adapted from http://www.gnu.org/philosophy/philosophy.html.
408@cindex free software
409The GNU operating system has been developed so that users can have
410freedom in their computing. GNU is @dfn{free software}, meaning that
21656ffa 411users have the @url{https://www.gnu.org/philosophy/free-sw.html,four
afe7408e
LC
412essential freedoms}: to run the program, to study and change the program
413in source code form, to redistribute exact copies, and to distribute
414modified versions. Packages found in the GNU distribution provide only
415software that conveys these four freedoms.
416
417In addition, the GNU distribution follow the
21656ffa 418@url{https://www.gnu.org/distros/free-system-distribution-guidelines.html,free
afe7408e
LC
419software distribution guidelines}. Among other things, these guidelines
420reject non-free firmware, recommendations of non-free software, and
421discuss ways to deal with trademarks and patents.
422
423Some otherwise free upstream package sources contain a small and optional
424subset that violates the above guidelines, for instance because this subset
425is itself non-free code. When that happens, the offending items are removed
426with appropriate patches or code snippets in the @code{origin} form of the
427package (@pxref{Defining Packages}). This way, @code{guix
428build --source} returns the ``freed'' source rather than the unmodified
429upstream source.
430
431
432@node Package Naming
433@subsection Package Naming
434
435@cindex package name
1dfc08f7 436A package actually has two names associated with it.
afe7408e
LC
437First, there is the name of the @emph{Scheme variable}, the one following
438@code{define-public}. By this name, the package can be made known in the
439Scheme code, for instance as input to another package. Second, there is
440the string in the @code{name} field of a package definition. This name
441is used by package management commands such as
442@command{guix package} and @command{guix build}.
443
444Both are usually the same and correspond to the lowercase conversion of
445the project name chosen upstream, with underscores replaced with
446hyphens. For instance, GNUnet is available as @code{gnunet}, and
447SDL_net as @code{sdl-net}.
448
1b1a61f8
TGRBGG
449A noteworthy exception to this rule is when the project name is only a
450single character, or if an older maintained project with the same name
451already exists---regardless of whether it has already been packaged for
452Guix. Use common sense to make such names unambiguous and meaningful.
453For example, Guix's package for the shell called ``s'' upstream is
454@code{s-shell} and @emph{not} @code{s}. Feel free to ask your fellow
455hackers for inspiration.
456
afe7408e
LC
457We do not add @code{lib} prefixes for library packages, unless these are
458already part of the official project name. But @pxref{Python
459Modules} and @ref{Perl Modules} for special rules concerning modules for
460the Python and Perl languages.
461
462Font package names are handled differently, @pxref{Fonts}.
463
464
465@node Version Numbers
466@subsection Version Numbers
467
468@cindex package version
469We usually package only the latest version of a given free software
470project. But sometimes, for instance for incompatible library versions,
471two (or more) versions of the same package are needed. These require
472different Scheme variable names. We use the name as defined
473in @ref{Package Naming}
474for the most recent version; previous versions use the same name, suffixed
475by @code{-} and the smallest prefix of the version number that may
476distinguish the two versions.
477
478The name inside the package definition is the same for all versions of a
479package and does not contain any version number.
480
481For instance, the versions 2.24.20 and 3.9.12 of GTK+ may be packaged as follows:
482
8ba31e8b 483@lisp
afe7408e
LC
484(define-public gtk+
485 (package
486 (name "gtk+")
487 (version "3.9.12")
488 ...))
489(define-public gtk+-2
490 (package
491 (name "gtk+")
492 (version "2.24.20")
493 ...))
8ba31e8b 494@end lisp
afe7408e 495If we also wanted GTK+ 3.8.2, this would be packaged as
8ba31e8b 496@lisp
afe7408e
LC
497(define-public gtk+-3.8
498 (package
499 (name "gtk+")
500 (version "3.8.2")
501 ...))
8ba31e8b 502@end lisp
afe7408e
LC
503
504@c See <https://lists.gnu.org/archive/html/guix-devel/2016-01/msg00425.html>,
505@c for a discussion of what follows.
506@cindex version number, for VCS snapshots
507Occasionally, we package snapshots of upstream's version control system
508(VCS) instead of formal releases. This should remain exceptional,
509because it is up to upstream developers to clarify what the stable
510release is. Yet, it is sometimes necessary. So, what should we put in
511the @code{version} field?
512
513Clearly, we need to make the commit identifier of the VCS snapshot
514visible in the version string, but we also need to make sure that the
515version string is monotonically increasing so that @command{guix package
516--upgrade} can determine which version is newer. Since commit
517identifiers, notably with Git, are not monotonically increasing, we add
518a revision number that we increase each time we upgrade to a newer
519snapshot. The resulting version string looks like this:
520
521@example
5222.0.11-3.cabba9e
523 ^ ^ ^
524 | | `-- upstream commit ID
525 | |
526 | `--- Guix package revision
527 |
528latest upstream version
529@end example
530
531It is a good idea to strip commit identifiers in the @code{version}
532field to, say, 7 digits. It avoids an aesthetic annoyance (assuming
533aesthetics have a role to play here) as well as problems related to OS
534limits such as the maximum shebang length (127 bytes for the Linux
ba2104aa 535kernel). It is best to use the full commit identifiers in
afe7408e
LC
536@code{origin}s, though, to avoid ambiguities. A typical package
537definition may look like this:
538
8ba31e8b 539@lisp
afe7408e
LC
540(define my-package
541 (let ((commit "c3f29bc928d5900971f65965feaae59e1272a3f7")
542 (revision "1")) ;Guix package revision
543 (package
544 (version (git-version "0.9" revision commit))
545 (source (origin
546 (method git-fetch)
547 (uri (git-reference
548 (url "git://example.org/my-package.git")
549 (commit commit)))
550 (sha256 (base32 "1mbikn@dots{}"))
551 (file-name (git-file-name name version))))
552 ;; @dots{}
553 )))
8ba31e8b 554@end lisp
afe7408e
LC
555
556@node Synopses and Descriptions
557@subsection Synopses and Descriptions
558
559@cindex package description
560@cindex package synopsis
561As we have seen before, each package in GNU@tie{}Guix includes a
562synopsis and a description (@pxref{Defining Packages}). Synopses and
563descriptions are important: They are what @command{guix package
564--search} searches, and a crucial piece of information to help users
565determine whether a given package suits their needs. Consequently,
566packagers should pay attention to what goes into them.
567
568Synopses must start with a capital letter and must not end with a
569period. They must not start with ``a'' or ``the'', which usually does
570not bring anything; for instance, prefer ``File-frobbing tool'' over ``A
571tool that frobs files''. The synopsis should say what the package
572is---e.g., ``Core GNU utilities (file, text, shell)''---or what it is
573used for---e.g., the synopsis for GNU@tie{}grep is ``Print lines
574matching a pattern''.
575
576Keep in mind that the synopsis must be meaningful for a very wide
577audience. For example, ``Manipulate alignments in the SAM format''
578might make sense for a seasoned bioinformatics researcher, but might be
579fairly unhelpful or even misleading to a non-specialized audience. It
580is a good idea to come up with a synopsis that gives an idea of the
581application domain of the package. In this example, this might give
582something like ``Manipulate nucleotide sequence alignments'', which
583hopefully gives the user a better idea of whether this is what they are
584looking for.
585
586Descriptions should take between five and ten lines. Use full
587sentences, and avoid using acronyms without first introducing them.
588Please avoid marketing phrases such as ``world-leading'',
589``industrial-strength'', and ``next-generation'', and avoid superlatives
590like ``the most advanced''---they are not helpful to users looking for a
591package and may even sound suspicious. Instead, try to be factual,
592mentioning use cases and features.
593
594@cindex Texinfo markup, in package descriptions
595Descriptions can include Texinfo markup, which is useful to introduce
596ornaments such as @code{@@code} or @code{@@dfn}, bullet lists, or
597hyperlinks (@pxref{Overview,,, texinfo, GNU Texinfo}). However you
598should be careful when using some characters for example @samp{@@} and
599curly braces which are the basic special characters in Texinfo
600(@pxref{Special Characters,,, texinfo, GNU Texinfo}). User interfaces
601such as @command{guix package --show} take care of rendering it
602appropriately.
603
604Synopses and descriptions are translated by volunteers
2a9784ff
JL
605@uref{https://translate.fedoraproject.org/projects/guix/packages, at
606Weblate} so that as many users as possible can read them in
afe7408e
LC
607their native language. User interfaces search them and display them in
608the language specified by the current locale.
609
610To allow @command{xgettext} to extract them as translatable strings,
611synopses and descriptions @emph{must be literal strings}. This means
612that you cannot use @code{string-append} or @code{format} to construct
613these strings:
614
615@lisp
616(package
617 ;; @dots{}
618 (synopsis "This is translatable")
619 (description (string-append "This is " "*not*" " translatable.")))
620@end lisp
621
622Translation is a lot of work so, as a packager, please pay even more
623attention to your synopses and descriptions as every change may entail
624additional work for translators. In order to help them, it is possible
625to make recommendations or instructions visible to them by inserting
626special comments like this (@pxref{xgettext Invocation,,, gettext, GNU
627Gettext}):
628
93c25181 629@lisp
afe7408e
LC
630;; TRANSLATORS: "X11 resize-and-rotate" should not be translated.
631(description "ARandR is designed to provide a simple visual front end
632for the X11 resize-and-rotate (RandR) extension. @dots{}")
93c25181 633@end lisp
afe7408e 634
71e746cc
MC
635@node Snippets versus Phases
636@subsection Snippets versus Phases
637
638@cindex snippets, when to use
639The boundary between using an origin snippet versus a build phase to
640modify the sources of a package can be elusive. Origin snippets are
641typically used to remove unwanted files such as bundled libraries,
642nonfree sources, or to apply simple substitutions. The source derived
643from an origin should produce a source that can be used to build the
644package on any system that the upstream package supports (i.e., act as
645the corresponding source). In particular, origin snippets must not
646embed store items in the sources; such patching should rather be done
647using build phases. Refer to the @code{origin} record documentation for
648more information (@pxref{origin Reference}).
afe7408e 649
15fba3b1
MC
650@node Emacs Packages
651@subsection Emacs Packages
652
653@cindex emacs, packaging
654@cindex elisp, packaging
655Emacs packages should preferably use the Emacs build system
656(@pxref{emacs-build-system}), for uniformity and the benefits provided
657by its build phases, such as the auto-generation of the autoloads file
eea7cc31 658and the byte compilation of the sources. Because there is no
15fba3b1
MC
659standardized way to run a test suite for Emacs packages, tests are
660disabled by default. When a test suite is available, it should be
eea7cc31 661enabled by setting the @code{#:tests?} argument to @code{#true}. By
15fba3b1
MC
662default, the command to run the test is @command{make check}, but any
663command can be specified via the @code{#:test-command} argument. The
664@code{#:test-command} argument expects a list containing a command and
eea7cc31 665its arguments, to be invoked during the @code{check} phase.
15fba3b1
MC
666
667The Elisp dependencies of Emacs packages are typically provided as
668@code{propagated-inputs} when required at run time. As for other
669packages, build or test dependencies should be specified as
670@code{native-inputs}.
671
672Emacs packages sometimes depend on resources directories that should be
673installed along the Elisp files. The @code{#:include} argument can be
674used for that purpose, by specifying a list of regexps to match. The
675best practice when using the @code{#:include} argument is to extend
676rather than override its default value (accessible via the
677@code{%default-include} variable). As an example, a yasnippet extension
678package typically include a @file{snippets} directory, which could be
679copied to the installation directory using:
680
681@lisp
682#:include (cons "^snippets/" %default-include))
683@end lisp
684
685When encountering problems, it is wise to check for the presence of the
686@code{Package-Requires} extension header in the package main source
687file, and whether any dependencies and their versions listed therein are
688satisfied.
689
afe7408e
LC
690@node Python Modules
691@subsection Python Modules
692
693@cindex python
694We currently package Python 2 and Python 3, under the Scheme variable names
695@code{python-2} and @code{python} as explained in @ref{Version Numbers}.
696To avoid confusion and naming clashes with other programming languages, it
697seems desirable that the name of a package for a Python module contains
698the word @code{python}.
699
39356057
LF
700Some modules are compatible with only one version of Python, others with
701both. If the package Foo is compiled with Python 3, we name it
702@code{python-foo}. If it is compiled with Python 2, we name it
703@code{python2-foo}. Packages should be added when they are necessary;
704we don't add Python 2 variants of the package unless we are going to use
705them.
afe7408e
LC
706
707If a project already contains the word @code{python}, we drop this;
708for instance, the module python-dateutil is packaged under the names
709@code{python-dateutil} and @code{python2-dateutil}. If the project name
710starts with @code{py} (e.g.@: @code{pytz}), we keep it and prefix it as
711described above.
712
713@subsubsection Specifying Dependencies
714@cindex inputs, for Python packages
715
716Dependency information for Python packages is usually available in the
717package source tree, with varying degrees of accuracy: in the
718@file{setup.py} file, in @file{requirements.txt}, or in @file{tox.ini}.
719
720Your mission, when writing a recipe for a Python package, is to map
721these dependencies to the appropriate type of ``input'' (@pxref{package
722Reference, inputs}). Although the @code{pypi} importer normally does a
723good job (@pxref{Invoking guix import}), you may want to check the
724following check list to determine which dependency goes where.
725
726@itemize
727
728@item
729We currently package Python 2 with @code{setuptools} and @code{pip}
730installed like Python 3.4 has per default. Thus you don't need to
731specify either of these as an input. @command{guix lint} will warn you
732if you do.
733
734@item
735Python dependencies required at run time go into
736@code{propagated-inputs}. They are typically defined with the
737@code{install_requires} keyword in @file{setup.py}, or in the
738@file{requirements.txt} file.
739
740@item
741Python packages required only at build time---e.g., those listed with
742the @code{setup_requires} keyword in @file{setup.py}---or only for
743testing---e.g., those in @code{tests_require}---go into
744@code{native-inputs}. The rationale is that (1) they do not need to be
745propagated because they are not needed at run time, and (2) in a
746cross-compilation context, it's the ``native'' input that we'd want.
747
748Examples are the @code{pytest}, @code{mock}, and @code{nose} test
749frameworks. Of course if any of these packages is also required at
750run-time, it needs to go to @code{propagated-inputs}.
751
752@item
753Anything that does not fall in the previous categories goes to
754@code{inputs}, for example programs or C libraries required for building
755Python packages containing C extensions.
756
757@item
758If a Python package has optional dependencies (@code{extras_require}),
759it is up to you to decide whether to add them or not, based on their
760usefulness/overhead ratio (@pxref{Submitting Patches, @command{guix
761size}}).
762
763@end itemize
764
765
766@node Perl Modules
767@subsection Perl Modules
768
769@cindex perl
770Perl programs standing for themselves are named as any other package,
771using the lowercase upstream name.
772For Perl packages containing a single class, we use the lowercase class name,
773replace all occurrences of @code{::} by dashes and prepend the prefix
774@code{perl-}.
775So the class @code{XML::Parser} becomes @code{perl-xml-parser}.
776Modules containing several classes keep their lowercase upstream name and
777are also prepended by @code{perl-}. Such modules tend to have the word
778@code{perl} somewhere in their name, which gets dropped in favor of the
779prefix. For instance, @code{libwww-perl} becomes @code{perl-libwww}.
780
781
782@node Java Packages
783@subsection Java Packages
784
785@cindex java
786Java programs standing for themselves are named as any other package,
787using the lowercase upstream name.
788
789To avoid confusion and naming clashes with other programming languages,
790it is desirable that the name of a package for a Java package is
791prefixed with @code{java-}. If a project already contains the word
792@code{java}, we drop this; for instance, the package @code{ngsjava} is
793packaged under the name @code{java-ngs}.
794
795For Java packages containing a single class or a small class hierarchy,
796we use the lowercase class name, replace all occurrences of @code{.} by
797dashes and prepend the prefix @code{java-}. So the class
798@code{apache.commons.cli} becomes package
799@code{java-apache-commons-cli}.
800
801
8d1b22b2
EF
802@node Rust Crates
803@subsection Rust Crates
804
805@cindex rust
806Rust programs standing for themselves are named as any other package, using the
807lowercase upstream name.
808
809To prevent namespace collisions we prefix all other Rust packages with the
810@code{rust-} prefix. The name should be changed to lowercase as appropriate and
811dashes should remain in place.
812
813In the rust ecosystem it is common for multiple incompatible versions of a
784048c2
HG
814package to be used at any given time, so all package definitions should have a
815versioned suffix. The versioned suffix is the left-most non-zero digit (and
816any leading zeros, of course). This follows the ``caret'' version scheme
817intended by Cargo. Examples@: @code{rust-clap-2}, @code{rust-rand-0.6}.
8d1b22b2
EF
818
819Because of the difficulty in reusing rust packages as pre-compiled inputs for
820other packages the Cargo build system (@pxref{Build Systems,
821@code{cargo-build-system}}) presents the @code{#:cargo-inputs} and
822@code{cargo-development-inputs} keywords as build system arguments. It would be
823helpful to think of these as similar to @code{propagated-inputs} and
824@code{native-inputs}. Rust @code{dependencies} and @code{build-dependencies}
825should go in @code{#:cargo-inputs}, and @code{dev-dependencies} should go in
826@code{#:cargo-development-inputs}. If a Rust package links to other libraries
827then the standard placement in @code{inputs} and the like should be used.
828
829Care should be taken to ensure the correct version of dependencies are used; to
830this end we try to refrain from skipping the tests or using @code{#:skip-build?}
831when possible. Of course this is not always possible, as the package may be
832developed for a different Operating System, depend on features from the Nightly
833Rust compiler, or the test suite may have atrophied since it was released.
834
835
afe7408e
LC
836@node Fonts
837@subsection Fonts
838
839@cindex fonts
840For fonts that are in general not installed by a user for typesetting
841purposes, or that are distributed as part of a larger software package,
842we rely on the general packaging rules for software; for instance, this
843applies to the fonts delivered as part of the X.Org system or fonts that
844are part of TeX Live.
845
846To make it easier for a user to search for fonts, names for other packages
847containing only fonts are constructed as follows, independently of the
848upstream package name.
849
850The name of a package containing only one font family starts with
851@code{font-}; it is followed by the foundry name and a dash @code{-}
852if the foundry is known, and the font family name, in which spaces are
853replaced by dashes (and as usual, all upper case letters are transformed
854to lower case).
855For example, the Gentium font family by SIL is packaged under the name
856@code{font-sil-gentium}.
857
858For a package containing several font families, the name of the collection
859is used in the place of the font family name.
860For instance, the Liberation fonts consist of three families,
861Liberation Sans, Liberation Serif and Liberation Mono.
862These could be packaged separately under the names
863@code{font-liberation-sans} and so on; but as they are distributed together
864under a common name, we prefer to package them together as
865@code{font-liberation}.
866
867In the case where several formats of the same font family or font collection
868are packaged separately, a short form of the format, prepended by a dash,
869is added to the package name. We use @code{-ttf} for TrueType fonts,
870@code{-otf} for OpenType fonts and @code{-type1} for PostScript Type 1
871fonts.
872
873
8c01b9d0
ML
874@node Coding Style
875@section Coding Style
876
877In general our code follows the GNU Coding Standards (@pxref{Top,,,
878standards, GNU Coding Standards}). However, they do not say much about
879Scheme, so here are some additional rules.
880
881@menu
882* Programming Paradigm:: How to compose your elements.
883* Modules:: Where to store your code?
884* Data Types and Pattern Matching:: Implementing data structures.
885* Formatting Code:: Writing conventions.
886@end menu
887
888@node Programming Paradigm
889@subsection Programming Paradigm
890
891Scheme code in Guix is written in a purely functional style. One
892exception is code that involves input/output, and procedures that
893implement low-level concepts, such as the @code{memoize} procedure.
894
895@node Modules
896@subsection Modules
897
898Guile modules that are meant to be used on the builder side must live in
899the @code{(guix build @dots{})} name space. They must not refer to
900other Guix or GNU modules. However, it is OK for a ``host-side'' module
901to use a build-side module.
902
903Modules that deal with the broader GNU system should be in the
904@code{(gnu @dots{})} name space rather than @code{(guix @dots{})}.
905
906@node Data Types and Pattern Matching
907@subsection Data Types and Pattern Matching
908
909The tendency in classical Lisp is to use lists to represent everything,
910and then to browse them ``by hand'' using @code{car}, @code{cdr},
911@code{cadr}, and co. There are several problems with that style,
912notably the fact that it is hard to read, error-prone, and a hindrance
913to proper type error reports.
914
915Guix code should define appropriate data types (for instance, using
916@code{define-record-type*}) rather than abuse lists. In addition, it
917should use pattern matching, via Guile’s @code{(ice-9 match)} module,
f06f569a
LC
918especially when matching lists (@pxref{Pattern Matching,,, guile, GNU
919Guile Reference Manual}).
8c01b9d0
ML
920
921@node Formatting Code
922@subsection Formatting Code
923
7bb2b10c
LC
924@cindex formatting code
925@cindex coding style
8c01b9d0
ML
926When writing Scheme code, we follow common wisdom among Scheme
927programmers. In general, we follow the
21656ffa 928@url{https://mumble.net/~campbell/scheme/style.txt, Riastradh's Lisp
8c01b9d0
ML
929Style Rules}. This document happens to describe the conventions mostly
930used in Guile’s code too. It is very thoughtful and well written, so
931please do read it.
932
933Some special forms introduced in Guix, such as the @code{substitute*}
934macro, have special indentation rules. These are defined in the
8ca0c88a
AK
935@file{.dir-locals.el} file, which Emacs automatically uses. Also note
936that Emacs-Guix provides @code{guix-devel-mode} mode that indents and
937highlights Guix code properly (@pxref{Development,,, emacs-guix, The
938Emacs-Guix Reference Manual}).
7bb2b10c
LC
939
940@cindex indentation, of code
941@cindex formatting, of code
942If you do not use Emacs, please make sure to let your editor knows these
943rules. To automatically indent a package definition, you can also run:
944
945@example
557d9c8d 946./etc/indent-code.el gnu/packages/@var{file}.scm @var{package}
7bb2b10c
LC
947@end example
948
949@noindent
950This automatically indents the definition of @var{package} in
557d9c8d
LC
951@file{gnu/packages/@var{file}.scm} by running Emacs in batch mode. To
952indent a whole file, omit the second argument:
953
954@example
955./etc/indent-code.el gnu/services/@var{file}.scm
956@end example
8c01b9d0 957
60912a88
LC
958@cindex Vim, Scheme code editing
959If you are editing code with Vim, we recommend that you run @code{:set
960autoindent} so that your code is automatically indented as you type.
961Additionally,
962@uref{https://www.vim.org/scripts/script.php?script_id=3998,
963@code{paredit.vim}} may help you deal with all these parentheses.
964
8c01b9d0
ML
965We require all top-level procedures to carry a docstring. This
966requirement can be relaxed for simple private procedures in the
967@code{(guix build @dots{})} name space, though.
968
969Procedures should not have more than four positional parameters. Use
970keyword parameters for procedures that take more than four parameters.
971
972
973@node Submitting Patches
974@section Submitting Patches
975
976Development is done using the Git distributed version control system.
977Thus, access to the repository is not strictly necessary. We welcome
978contributions in the form of patches as produced by @code{git
230efa87 979format-patch} sent to the @email{guix-patches@@gnu.org} mailing list.
2d315cd4
LC
980Seasoned Guix developers may also want to look at the section on commit
981access (@pxref{Commit Access}).
230efa87 982
a7bde77d
LC
983This mailing list is backed by a Debbugs instance, which allows us to
984keep track of submissions (@pxref{Tracking Bugs and Patches}). Each
985message sent to that mailing list gets a new tracking number assigned;
986people can then follow up on the submission by sending email to
987@code{@var{NNN}@@debbugs.gnu.org}, where @var{NNN} is the tracking
988number (@pxref{Sending a Patch Series}).
230efa87 989
8c01b9d0
ML
990Please write commit logs in the ChangeLog format (@pxref{Change Logs,,,
991standards, GNU Coding Standards}); you can check the commit history for
992examples.
993
994Before submitting a patch that adds or modifies a package definition,
fcc58db6
LC
995please run through this check list:
996
997@enumerate
308c08d3
RW
998@item
999If the authors of the packaged software provide a cryptographic
1000signature for the release tarball, make an effort to verify the
1001authenticity of the archive. For a detached GPG signature file this
1002would be done with the @code{gpg --verify} command.
1003
cbd02397
LC
1004@item
1005Take some time to provide an adequate synopsis and description for the
1006package. @xref{Synopses and Descriptions}, for some guidelines.
1007
fcc58db6
LC
1008@item
1009Run @code{guix lint @var{package}}, where @var{package} is the
8c01b9d0 1010name of the new or modified package, and fix any errors it reports
fcc58db6
LC
1011(@pxref{Invoking guix lint}).
1012
1013@item
1014Make sure the package builds on your platform, using @code{guix build
1015@var{package}}.
1016
89339a35
DM
1017@item
1018We recommend you also try building the package on other supported
1019platforms. As you may not have access to actual hardware platforms, we
1020recommend using the @code{qemu-binfmt-service-type} to emulate them. In
1021order to enable it, add the following service to the list of services in
1022your @code{operating-system} configuration:
1023
8ba31e8b 1024@lisp
89339a35
DM
1025(service qemu-binfmt-service-type
1026 (qemu-binfmt-configuration
03e64049 1027 (platforms (lookup-qemu-platforms "arm" "aarch64"))
8ba31e8b 1028@end lisp
89339a35
DM
1029
1030Then reconfigure your system.
1031
1032You can then build packages for different platforms by specifying the
1033@code{--system} option. For example, to build the "hello" package for
b28e4e3c
EF
1034the armhf, aarch64, or mips64 architectures, you would run the following
1035commands, respectively:
89339a35
DM
1036@example
1037guix build --system=armhf-linux --rounds=2 hello
1038guix build --system=aarch64-linux --rounds=2 hello
89339a35
DM
1039@end example
1040
d222522e
LC
1041@item
1042@cindex bundling
1043Make sure the package does not use bundled copies of software already
1044available as separate packages.
1045
1046Sometimes, packages include copies of the source code of their
1047dependencies as a convenience for users. However, as a distribution, we
1048want to make sure that such packages end up using the copy we already
1049have in the distribution, if there is one. This improves resource usage
1050(the dependency is built and stored only once), and allows the
1051distribution to make transverse changes such as applying security
1052updates for a given software package in a single place and have them
1053affect the whole system---something that bundled copies prevent.
1054
fcc58db6
LC
1055@item
1056Take a look at the profile reported by @command{guix size}
1057(@pxref{Invoking guix size}). This will allow you to notice references
1058to other packages unwillingly retained. It may also help determine
1059whether to split the package (@pxref{Packages with Multiple Outputs}),
024e358c 1060and which optional dependencies should be used. In particular, avoid adding
0afeb746 1061@code{texlive} as a dependency: because of its extreme size, use
48cccf06 1062the @code{texlive-tiny} package or @code{texlive-union} procedure instead.
fcc58db6
LC
1063
1064@item
1065For important changes, check that dependent package (if applicable) are
1066not affected by the change; @code{guix refresh --list-dependent
8c01b9d0
ML
1067@var{package}} will help you do that (@pxref{Invoking guix refresh}).
1068
916b5eba
LC
1069@c See <https://lists.gnu.org/archive/html/guix-devel/2016-10/msg00933.html>.
1070@cindex branching strategy
1071@cindex rebuild scheduling strategy
1072Depending on the number of dependent packages and thus the amount of
1073rebuilding induced, commits go to different branches, along these lines:
1074
1075@table @asis
1076@item 300 dependent packages or less
1077@code{master} branch (non-disruptive changes).
1078
bb9a99e6 1079@item between 300 and 1,800 dependent packages
916b5eba 1080@code{staging} branch (non-disruptive changes). This branch is intended
bb9a99e6 1081to be merged in @code{master} every 6 weeks or so. Topical changes
916b5eba 1082(e.g., an update of the GNOME stack) can instead go to a specific branch
4de68873
LF
1083(say, @code{gnome-updates}). This branch is not expected to be
1084buildable or usable until late in its development process.
916b5eba 1085
bb9a99e6 1086@item more than 1,800 dependent packages
916b5eba
LC
1087@code{core-updates} branch (may include major and potentially disruptive
1088changes). This branch is intended to be merged in @code{master} every
4de68873
LF
10896 months or so. This branch is not expected to be buildable or usable
1090until late in its development process.
916b5eba
LC
1091@end table
1092
aa9ba865 1093All these branches are @uref{@value{SUBSTITUTE-URL},
38ab778f 1094tracked by our build farm} and merged into @code{master} once
189b1543
LC
1095everything has been successfully built. This allows us to fix issues
1096before they hit users, and to reduce the window during which pre-built
1097binaries are not available.
1098
175bea0c 1099When we decide to start building the @code{staging} or
58853df8
LF
1100@code{core-updates} branches, they will be forked and renamed with the
1101suffix @code{-frozen}, at which time only bug fixes may be pushed to the
1102frozen branches. The @code{core-updates} and @code{staging} branches
1103will remain open to accept patches for the next cycle. Please ask on
1104the mailing list or IRC if unsure where to place a patch.
38ab778f
MB
1105@c TODO: It would be good with badges on the website that tracks these
1106@c branches. Or maybe even a status page.
1107
d23c20f1 1108@item
5b74fe06
LC
1109@cindex determinism, of build processes
1110@cindex reproducible builds, checking
d23c20f1
LC
1111Check whether the package's build process is deterministic. This
1112typically means checking whether an independent build of the package
1113yields the exact same result that you obtained, bit for bit.
1114
5b74fe06
LC
1115A simple way to do that is by building the same package several times in
1116a row on your machine (@pxref{Invoking guix build}):
1117
1118@example
1119guix build --rounds=2 my-package
1120@end example
1121
1122This is enough to catch a class of common non-determinism issues, such
1123as timestamps or randomly-generated output in the build result.
1124
1125Another option is to use @command{guix challenge} (@pxref{Invoking guix
1126challenge}). You may run it once the package has been committed and
0bc02bec 1127built by @code{@value{SUBSTITUTE-SERVER}} to check whether it obtains the same
5b74fe06
LC
1128result as you did. Better yet: Find another machine that can build it
1129and run @command{guix publish}. Since the remote build machine is
1130likely different from yours, this can catch non-determinism issues
1131related to the hardware---e.g., use of different instruction set
1132extensions---or to the operating system kernel---e.g., reliance on
1133@code{uname} or @file{/proc} files.
d23c20f1 1134
3c2d03a2
LC
1135@item
1136When writing documentation, please use gender-neutral wording when
1137referring to people, such as
1138@uref{https://en.wikipedia.org/wiki/Singular_they, singular
1139``they''@comma{} ``their''@comma{} ``them''}, and so forth.
1140
3a78fab8 1141@item
1142Verify that your patch contains only one set of related changes.
1143Bundling unrelated changes together makes reviewing harder and slower.
1144
1145Examples of unrelated changes include the addition of several packages,
1146or a package update along with fixes to that package.
1147
7bb2b10c
LC
1148@item
1149Please follow our code formatting rules, possibly running the
557d9c8d 1150@command{etc/indent-code.el} script to do that automatically for you
7bb2b10c
LC
1151(@pxref{Formatting Code}).
1152
4feb589b
PN
1153@item
1154When possible, use mirrors in the source URL (@pxref{Invoking guix download}).
1155Use reliable URLs, not generated ones. For instance, GitHub archives are not
1156necessarily identical from one generation to the next, so in this case it's
1157often better to clone the repository. Don't use the @command{name} field in
1158the URL: it is not very useful and if the name changes, the URL will probably
1159be wrong.
1160
bf99d7e0 1161@item
da31e7d9
PN
1162Check if Guix builds (@pxref{Building from Git}) and address the
1163warnings, especially those about use of undefined symbols.
bf99d7e0 1164
9fcf2820
PN
1165@item
1166Make sure your changes do not break Guix and simulate a @code{guix pull} with:
1167@example
1168guix pull --url=/path/to/your/checkout --profile=/tmp/guix.master
1169@end example
1170
fcc58db6
LC
1171@end enumerate
1172
a40424bd 1173When posting a patch to the mailing list, use @samp{[PATCH] @dots{}} as
a1891cbf
BW
1174a subject, if your patch is to be applied on a branch other than
1175@code{master}, say @code{core-updates}, specify it in the subject like
1176@samp{[PATCH core-updates] @dots{}}. You may use your email client or
1177the @command{git send-email} command (@pxref{Sending a Patch Series}).
1178We prefer to get patches in plain text messages, either inline or as
1179MIME attachments. You are advised to pay attention if your email client
1180changes anything like line breaks or indentation which could potentially
1181break the patches.
5a183a1e 1182
c8d6fa77
FL
1183Expect some delay when you submit your very first patch to
1184@email{guix-patches@@gnu.org}. You have to wait until you get an
4b08aad5 1185acknowledgement with the assigned tracking number. Future acknowledgements
c8d6fa77
FL
1186should not be delayed.
1187
4619b59c
JN
1188When a bug is resolved, please close the thread by sending an email to
1189@email{@var{NNN}-done@@debbugs.gnu.org}.
1190
5a183a1e
JN
1191@unnumberedsubsec Sending a Patch Series
1192@anchor{Sending a Patch Series}
1193@cindex patch series
1194@cindex @code{git send-email}
1195@cindex @code{git-send-email}
1196
1197When sending a patch series (e.g., using @code{git send-email}), please
1198first send one message to @email{guix-patches@@gnu.org}, and then send
1199subsequent patches to @email{@var{NNN}@@debbugs.gnu.org} to make sure
1200they are kept together. See
1201@uref{https://debbugs.gnu.org/Advanced.html, the Debbugs documentation}
fb8b99a5
AB
1202for more information. You can install @command{git send-email} with
1203@command{guix install git:send-email}.
5a183a1e 1204@c Debbugs bug: https://debbugs.gnu.org/db/15/15361.html
a7bde77d
LC
1205
1206@node Tracking Bugs and Patches
1207@section Tracking Bugs and Patches
1208
1209@cindex bug reports, tracking
1210@cindex patch submissions, tracking
1211@cindex issue tracking
1212@cindex Debbugs, issue tracking system
1213Bug reports and patch submissions are currently tracked using the
1214Debbugs instance at @uref{https://bugs.gnu.org}. Bug reports are filed
1215against the @code{guix} ``package'' (in Debbugs parlance), by sending
1216email to @email{bug-guix@@gnu.org}, while patch submissions are filed
1217against the @code{guix-patches} package by sending email to
1218@email{guix-patches@@gnu.org} (@pxref{Submitting Patches}).
1219
1220A web interface (actually @emph{two} web interfaces!) are available to
1221browse issues:
1222
1223@itemize
1224@item
cdea5265
LC
1225@url{https://issues.guix.gnu.org} provides a pleasant
1226interface@footnote{The web interface at
1227@url{https://issues.guix.gnu.org} is powered by Mumi, a nice piece of
1228software written in Guile, and you can help! See
1229@url{https://git.elephly.net/gitweb.cgi?p=software/mumi.git}.} to browse
1230bug reports and patches, and to participate in discussions;
1231@item
a7bde77d
LC
1232@url{https://bugs.gnu.org/guix} lists bug reports;
1233@item
1234@url{https://bugs.gnu.org/guix-patches} lists patch submissions.
1235@end itemize
1236
cdea5265
LC
1237To view discussions related to issue number @var{n}, go to
1238@indicateurl{https://issues.guix.gnu.org/@var{n}} or
a7bde77d
LC
1239@indicateurl{https://bugs.gnu.org/@var{n}}.
1240
1241If you use Emacs, you may find it more convenient to interact with
1242issues using @file{debbugs.el}, which you can install with:
1243
1244@example
1245guix install emacs-debbugs
1246@end example
1247
1248For example, to list all open issues on @code{guix-patches}, hit:
1249
1250@example
1251@kbd{C-u} @kbd{M-x} debbugs-gnu @kbd{RET} @kbd{RET} guix-patches @kbd{RET} n y
1252@end example
1253
1254@xref{Top,,, debbugs-ug, Debbugs User Guide}, for more information on
1255this nifty tool!
2d315cd4
LC
1256
1257@node Commit Access
1258@section Commit Access
1259
1260@cindex commit access, for developers
1261For frequent contributors, having write access to the repository is
ef09cb86
LC
1262convenient. When you deem it necessary, consider applying for commit
1263access by following these steps:
1264
1265@enumerate
1266@item
1267Find three committers who would vouch for you. You can view the list of
1268committers at
1269@url{https://savannah.gnu.org/project/memberlist.php?group=guix}. Each
1270of them should email a statement to @email{guix-maintainers@@gnu.org} (a
1271private alias for the collective of maintainers), signed with their
1272OpenPGP key.
1273
1274Committers are expected to have had some interactions with you as a
1275contributor and to be able to judge whether you are sufficiently
1276familiar with the project's practices. It is @emph{not} a judgment on
1277the value of your work, so a refusal should rather be interpreted as
1278``let's try again later''.
1279
1280@item
1281Send @email{guix-maintainers@@gnu.org} a message stating your intent,
1282listing the three committers who support your application, signed with
1283the OpenPGP key you will use to sign commits, and giving its fingerprint
1284(see below). See @uref{https://emailselfdefense.fsf.org/en/}, for an
1285introduction to public-key cryptography with GnuPG.
1286
4a84deda
LC
1287@c See <https://sha-mbles.github.io/>.
1288Set up GnuPG such that it never uses the SHA1 hash algorithm for digital
1289signatures, which is known to be unsafe since 2019, for instance by
1290adding the following line to @file{~/.gnupg/gpg.conf} (@pxref{GPG
1291Esoteric Options,,, gnupg, The GNU Privacy Guard Manual}):
1292
1293@example
1294digest-algo sha512
1295@end example
1296
ef09cb86
LC
1297@item
1298Maintainers ultimately decide whether to grant you commit access,
1299usually following your referrals' recommendation.
1300
1301@item
84133320 1302@cindex OpenPGP, signed commits
ef09cb86
LC
1303If and once you've been given access, please send a message to
1304@email{guix-devel@@gnu.org} to say so, again signed with the OpenPGP key
1305you will use to sign commits (do that before pushing your first commit).
1306That way, everyone can notice and ensure you control that OpenPGP key.
1307
84133320
LC
1308@quotation Important
1309Before you can push for the first time, maintainers must:
1310
1311@enumerate
1312@item
1313add your OpenPGP key to the @code{keyring} branch;
1314@item
1315add your OpenPGP fingerprint to the @file{.guix-authorizations} file of
1316the branch(es) you will commit to.
1317@end enumerate
1318@end quotation
ef09cb86
LC
1319
1320@item
1321Make sure to read the rest of this section and... profit!
1322@end enumerate
1323
1324@quotation Note
1325Maintainers are happy to give commit access to people who have been
1326contributing for some time and have a track record---don't be shy and
1327don't underestimate your work!
1328
1329However, note that the project is working towards a more automated patch
1330review and merging system, which, as a consequence, may lead us to have
1331fewer people with commit access to the main repository. Stay tuned!
1332@end quotation
1333
1334If you get commit access, please make sure to follow
2d315cd4
LC
1335the policy below (discussions of the policy can take place on
1336@email{guix-devel@@gnu.org}).
1337
1338Non-trivial patches should always be posted to
1339@email{guix-patches@@gnu.org} (trivial patches include fixing typos,
1340etc.). This mailing list fills the patch-tracking database
1341(@pxref{Tracking Bugs and Patches}).
1342
1343For patches that just add a new package, and a simple one, it's OK to
1344commit, if you're confident (which means you successfully built it in a
1345chroot setup, and have done a reasonable copyright and license
1346auditing). Likewise for package upgrades, except upgrades that trigger
1347a lot of rebuilds (for example, upgrading GnuTLS or GLib). We have a
1348mailing list for commit notifications (@email{guix-commits@@gnu.org}),
1349so people can notice. Before pushing your changes, make sure to run
1350@code{git pull --rebase}.
1351
1352All commits that are pushed to the central repository on Savannah must
1353be signed with an OpenPGP key, and the public key should be uploaded to
1354your user account on Savannah and to public key servers, such as
1355@code{keys.openpgp.org}. To configure Git to automatically sign
1356commits, run:
1357
1358@example
1359git config commit.gpgsign true
1360git config user.signingkey CABBA6EA1DC0FF33
1361@end example
1362
1363You can prevent yourself from accidentally pushing unsigned commits to
1364Savannah by using the pre-push Git hook called located at
1365@file{etc/git/pre-push}:
1366
1367@example
1368cp etc/git/pre-push .git/hooks/pre-push
1369@end example
1370
1371When pushing a commit on behalf of somebody else, please add a
1372@code{Signed-off-by} line at the end of the commit log message---e.g.,
1373with @command{git am --signoff}. This improves tracking of who did
1374what.
1375
1ad5209d
LC
1376When adding channel news entries (@pxref{Channels, Writing Channel
1377News}), make sure they are well-formed by running the following command
1378right before pushing:
1379
1380@example
1381make check-channel-news
1382@end example
1383
2d315cd4
LC
1384For anything else, please post to @email{guix-patches@@gnu.org} and
1385leave time for a review, without committing anything (@pxref{Submitting
1386Patches}). If you didn’t receive any reply after two weeks, and if
1387you're confident, it's OK to commit.
1388
1389That last part is subject to being adjusted, allowing individuals to commit
1390directly on non-controversial changes on parts they’re familiar with.
98ebcf1c 1391
9ade2b72
LF
1392In order to reduce the possibility of mistakes, committers will have
1393their Savannah account removed from the Guix Savannah project and their
1394key removed from @file{.guix-authorizations} after 12 months of
1395inactivity; they can ask to regain commit access by emailing the
1396maintainers, without going through the vouching process.
1397
98ebcf1c
LC
1398One last thing: the project keeps moving forward because committers not
1399only push their own awesome changes, but also offer some of their time
1400@emph{reviewing} and pushing other people's changes. As a committer,
1401you're welcome to use your expertise and commit rights to help other
1402contributors, too!
5800d2aa
MC
1403
1404@node Updating the Guix Package
1405@section Updating the Guix Package
1406
1407@cindex update-guix-package, updating the guix package
1408It is sometimes desirable to update the @code{guix} package itself (the
1409package defined in @code{(gnu packages package-management)}), for
1410example to make new daemon features available for use by the
1411@code{guix-service-type} service type. In order to simplify this task,
1412the following command can be used:
1413
1414@example
1415make update-guix-package
1416@end example
1417
1418The @code{update-guix-package} make target will use the last known
1419@emph{commit} corresponding to @code{HEAD} in your Guix checkout,
1420compute the hash of the Guix sources corresponding to that commit and
1421update the @code{commit}, @code{revision} and hash of the @code{guix}
1422package definition.
1423
1424To validate that the updated @code{guix} package hashes are correct and
1425that it can be built successfully, the following command can be run from
1426the directory of your Guix checkout:
1427
1428@example
1429./pre-inst-env guix build guix
1430@end example
1431
1432To guard against accidentally updating the @code{guix} package to a
1433commit that others can't refer to, a check is made that the commit used
1434has already been pushed to the Savannah-hosted Guix git repository.
1435
1436This check can be disabled, @emph{at your own peril}, by setting the
3de898b4
MC
1437@code{GUIX_ALLOW_ME_TO_USE_PRIVATE_COMMIT} environment variable. When
1438this variable is set, the updated package source is also added to the
1439store. This is used as part of the release process of Guix.
1897a6ef
JL
1440
1441@cindex translation
1442@cindex l10n
1443@cindex i18n
1444@cindex native language support
1445@node Translating Guix
1446@section Translating Guix
1447
1448Writing code and packages is not the only way to provide a meaningful
1449contribution to Guix. Translating to a language you speak is another
1450example of a valuable contribution you can make. This section is designed
1451to describe the translation process. It gives you advice on how you can
1452get involved, what can be translated, what mistakes you should avoid and
1453what we can do to help you!
1454
1455Guix is a big project that has multiple components that can be translated.
1456We coordinate the translation effort on a
1457@uref{https://translate.fedoraproject.org/projects/guix/,Weblate instance}
1458hosted by our friends at Fedora. You will need an account to submit
1459translations.
1460
1461Some of the software packaged in Guix also contain translations. We do not
1462host a translation platform for them. If you want to translate a package
ef7275cd 1463provided by Guix, you should contact their developers or find the information
1897a6ef
JL
1464on their website. As an example, you can find the homepage of the
1465@code{hello} package by typing @code{guix show hello}. On the ``homepage''
1466line, you will see @url{https://www.gnu.org/software/hello/} as the homepage.
1467
1468Many GNU and non-GNU packages can be translated on the
1469@uref{https://translationproject.org,Translation Project}. Some projects
1470with multiple components have their own platform. For instance, GNOME has
1471its own platform, @uref{https://l10n.gnome.org/,Damned Lies}.
1472
1473Guix has five components hosted on Weblate.
1474
1475@itemize
1476@item @code{guix} contains all the strings from the Guix software (the
1477 guided system installer, the package manager, etc), excluding packages.
1478@item @code{packages} contains the synopsis (single-sentence description
1479 of a package) and description (longer description) of packages in Guix.
1480@item @code{website} contains the official Guix website, except for
1481 blog posts and multimedia content.
1482@item @code{documentation-manual} corresponds to this manual.
1483@item @code{documentation-cookbook} is the component for the cookbook.
1484@end itemize
1485
1486@subsubheading General Directions
1487
1488Once you get an account, you should be able to select a component from
1489@uref{https://translate.fedoraproject.org/projects/guix/,the guix project},
1490and select a language. If your language does not appear in the list, go
1491to the bottom and click on the ``Start new translation'' button. Select
1492the language you want to translate to from the list, to start your new
1493translation.
1494
1495Like lots of other free software packages, Guix uses
1496@uref{https://www.gnu.org/software/gettext,GNU Gettext} for its translations,
1497with which translatable strings are extracted from the source code to so-called
1498PO files.
1499
1500Even though PO files are text files, changes should not be made with a text
1501editor but with PO editing software. Weblate integrates PO editing
1502functionality. Alternatively, translators can use any of various
1503free-software tools for filling in translations, of which
1504@uref{https://poedit.net/,Poedit} is one example, and (after logging in)
1505@uref{https://docs.weblate.org/en/latest/user/files.html,upload} the changed
1506file. There is also a special
1507@uref{https://www.emacswiki.org/emacs/PoMode,PO editing mode} for users of GNU
1508Emacs. Over time translators find out what software they are happy with and
1509what features they need.
1510
1511On Weblate, you will find various links to the editor, that will show various
1512subsets (or all) of the strings. Have a look around and at the
1513@uref{https://docs.weblate.org/en/latest/,documentation} to familiarize
1514yourself with the platform.
1515
1516@subsubheading Translation Components
1517
1518In this section, we provide more detailed guidance on the translation
1519process, as well as details on what you should or should not do. When in
1520doubt, please contact us, we will be happy to help!
1521
1522@table @asis
1523@item guix
1524Guix is written in the Guile programming language, and some strings contain
1525special formating that is interpreted by Guile. These special formating
1526should be highlighted by Weblate. They start with @code{~} followed by one
1527or more characters.
1528
1529When printing the string, Guile replaces the special formating symbols with
1530actual values. For instance, the string @samp{ambiguous package specification
1531`~a'} would be substituted to contain said package specification instead of
1532@code{~a}. To properly translate this string, you must keep the formating
1533code in your translation, although you can place it where it makes sense in
1534your language. For instance, the French translation says @samp{spécification
1535du paquet « ~a » ambiguë} because the adjective needs to be placed in the
1536end of the sentence.
1537
1538If there are multiple formating symbols, make sure to respect the order.
1539Guile does not know in which order you intended the string to be read, so it
1540will substitute the symbols in the same order as the English sentence.
1541
1542As an example, you cannot translate @samp{package '~a' has been superseded by
1543'~a'} by @samp{'~a' superseeds package '~a'}, because the meaning would be
ef7275cd 1544reversed. If @var{foo} is superseded by @var{bar}, the translation would read
1897a6ef
JL
1545@samp{'foo' superseeds package 'bar'}. To work around this problem, it
1546is possible to use more advanced formating to select a given piece of data,
1547instead of following the default English order. @xref{Formatted Output,,,
1548guile, GNU Guile Reference Manual}, for more information on formating in Guile.
1549
1550@item packages
1551
1552Package descriptions occasionally contain Texinfo markup (@pxref{Synopses
1553and Descriptions}). Texinfo markup looks like @samp{@@code@{rm -rf@}},
1554@samp{@@emph@{important@}}, etc. When translating, please leave markup as is.
1555
1556The characters after ``@@'' form the name of the markup, and the text between
1557``@{'' and ``@}'' is its content. In general, you should not translate the
1558content of markup like @code{@@code}, as it contains literal code that do not
1559change with language. You can translate the content of formating markup such
1560as @code{@@emph}, @code{@@i}, @code{@@itemize}, @code{@@item}. However, do
1561not translate the name of the markup, or it will not be recognized. Do
1562not translate the word after @code{@@end}, it is the name of the markup that
1563is closed at this position (e.g.@: @code{@@itemize ... @@end itemize}).
1564
1565@item documentation-manual and documentation-cookbook
1566
1567The first step to ensure a successful translation of the manual is to find
1568and translate the following strings @emph{first}:
1569
1570@itemize
1571@item @code{version.texi}: Translate this string as @code{version-xx.texi},
1572 where @code{xx} is your language code (the one shown in the URL on
1573 weblate).
1574@item @code{contributing.texi}: Translate this string as
1575 @code{contributing.xx.texi}, where @code{xx} is the same language code.
1576@item @code{Top}: Do not translate this string, it is important for Texinfo.
1577 If you translate it, the document will be empty (missing a Top node).
1578 Please look for it, and register @code{Top} as its translation.
1579@end itemize
1580
1581Translating these strings first ensure we can include your translation in
1582the guix repository without breaking the make process or the
1583@command{guix pull} machinery.
1584
1585The manual and the cookbook both use Texinfo. As for @code{packages}, please
1586keep Texinfo markup as is. There are more possible markup types in the manual
1587than in the package descriptions. In general, do not translate the content
1588of @code{@@code}, @code{@@file}, @code{@@var}, @code{@@value}, etc. You
1589should translate the content of formating markup such as @code{@@emph},
1590@code{@@i}, etc.
1591
1592The manual contains sections that can be refered to by name by @code{@@ref},
1593@code{@@xref} and @code{@@pxref}. We have a mechanism in place so you do
1594not have to translate their content. If you keep the English title, we will
1595automatically replace it with your translation of that title. This ensures
1596that Texinfo will always be able to find the node. If you decide to change
1597the translation of the title, the references will automatically be updated
1598and you will not have to update them all yourself.
1599
1600When translating references from the cookbook to the manual, you need to
1601replace the name of the manual and the name of the section. For instance,
1602to translate @code{@@pxref@{Defining Packages,,, guix, GNU Guix Reference
1603Manual@}}, you would replace @code{Defining Packages} with the title of that
1604section in the translated manual @emph{only} if that title is translated.
1605If the title is not translated in your language yet, do not translate it here,
1606or the link will be broken. Replace @code{guix} with @code{guix.xx} where
1607@code{xx} is your language code. @code{GNU Guix Reference Manual} is the
1608text of the link. You can translate it however you wish.
1609
1610@item website
1611
1612The website pages are written using SXML, an s-expression version of HTML,
1613the basic language of the web. We have a process to extract translatable
1614strings from the source, and replace complex s-expressions with a more familiar
1615XML markup, where each markup is numbered. Translators can arbitrarily change
1616the ordering, as in the following example.
1617
1618@example
1619#. TRANSLATORS: Defining Packages is a section name
1620#. in the English (en) manual.
1621#: apps/base/templates/about.scm:64
1622msgid "Packages are <1>defined<1.1>en</1.1><1.2>Defining-Packages.html</1.2></1> as native <2>Guile</2> modules."
1623msgstr "Pakete werden als reine <2>Guile</2>-Module <1>definiert<1.1>de</1.1><1.2>Pakete-definieren.html</1.2></1>."
1624@end example
1625
1626Note that you need to include the same markups. You cannot skip any.
1627@end table
1628
1629In case you make a mistake, the component might fail to build properly with your
1630language, or even make guix pull fail. To prevent that, we have a process
1631in place to check the content of the files before pushing to our repository.
1632We will not be able to update the translation for your language in Guix, so
1633we will notify you (through weblate and/or by email) so you get a chance to
1634fix the issue.
1635
1636@subsubheading Outside of Weblate
1637
1638Currently, some parts of Guix cannot be translated on Weblate, help wanted!
1639
1640@itemize
1641@item @command{guix pull} news can be translated in @file{news.scm}, but is not
1642 available from Weblate. If you want to provide a translation, you
1643 can prepare a patch as described above, or simply send us your
1644 translation with the name of the news entry you translated and your
1645 language. @xref{Writing Channel News}, for more information about
1646 channel news.
1647@item Guix blog posts cannot currently be translated.
1648@item The installer script (for foreign distributions) is entirely in English.
1649@item Some of the libraries Guix uses cannot be translated or are translated
1650 outside of the Guix project. Guile itself is not internationalized.
1651@item Other manuals linked from this manual or the cookbook might not be
1652 translated.
1653@end itemize
1654
1655@subsubheading Translation Infrastructure
1656
1657Weblate is backed by a git repository from which it discovers new strings to
1658translate and pushes new and updated translations. Normally, it would be
1659enough to give it commit access to our repositories. However, we decided
1660to use a separate repository for two reasons. First, we would have to give
1661Weblate commit access and authorize its signing key, but we do not trust it
ef7275cd 1662in the same way we trust guix developers, especially since we do not manage
1897a6ef
JL
1663the instance ourselves. Second, if translators mess something up, it can
1664break the generation of the website and/or guix pull for all our users,
1665independently of their language.
1666
1667For these reasons, we use a dedicated repository to host translations, and we
1668synchronize it with our guix and artworks repositories after checking no issue
1669was introduced in the translation.
1670
ef7275cd 1671Developers can download the latest PO files from weblate in the Guix
1897a6ef
JL
1672repository by runnig the @command{make download-po} target. It will
1673automatically download the latest files from weblate, reformat them to a
1674canonical form, and check they do not contain issues. The manual needs to be
1675built again to check for additional issues that might crash Texinfo.
1676
ef7275cd 1677Before pushing new translation files, developers should add them to the
1897a6ef
JL
1678make machinery so the translations are actually available. The process
1679differs for the various components.
1680
1681@itemize
1682@item New po files for the @code{guix} and @code{packages} components must
1683 be registered by adding the new language to @file{po/guix/LINGUAS} or
1684 @file{po/packages/LINGUAS}.
1685@item New po files for the @code{documentation-manual} component must be
1686 registered by adding the file name to @code{DOC_PO_FILES} in
1687 @file{po/doc/local.mk}, the generated @file{%D%/guix.xx.texi} manual to
1688 @code{info_TEXINFOS} in @file{doc/local.mk} and the generated
1689 @file{%D%/guix.xx.texi} and @file{%D%/contributing.xx.texi} to
1690 @code{TRANSLATED_INFO} also in @file{doc/local.mk}.
1691@item New po files for the @code{documentation-cookbook} component must be
1692 registered by adding the file name to @code{DOC_COOKBOOK_PO_FILES} in
1693 @file{po/doc/local.mk}, the generated @file{%D%/guix-cookbook.xx.texi}
1694 manual to @code{info_TEXINFOS} in @file{doc/local.mk} and the generated
1695 @file{%D%/guix-cookbook.xx.texi} to @code{TRANSLATED_INFO} also
1696 in @file{doc/local.mk}.
1697@item New po files for the @code{website} component must be added to the
1698 @code{guix-artwork} repository, in @file{website/po/}.
1699 @file{website/po/LINGUAS} and @file{website/po/ietf-tags.scm} must
1700 be updated accordingly (see @file{website/i18n-howto.txt} for more
1701 information on the process).
1702@end itemize