diff mbox series

[bug#35661] doc: Fix typo.

Message ID 87imujpbph.fsf@GlaDOS.home
State Accepted
Headers show
Series [bug#35661] doc: Fix typo. | expand

Checks

Context Check Description
cbaines/applying patch fail Apply failed

Commit Message

Diego Nicola Barbato May 9, 2019, 9:19 p.m. UTC
Hi Guix,

The documentation of 'shepherd-service' erroneously calls the
'requirement' field 'requirements'.  The attached patch fixes that.

I have also noticed that the 'auto-start?' field is not mentioned at
all.

Regards,

Diego

Comments

Arun Isaac May 11, 2019, 7:44 a.m. UTC | #1
> The documentation of 'shepherd-service' erroneously calls the
> 'requirement' field 'requirements'.  The attached patch fixes that.

Thanks! I added a copyright header for you and pushed the patch.

> I have also noticed that the 'auto-start?' field is not mentioned at
> all.

Could you submit a patch fixing this?
Diego Nicola Barbato May 11, 2019, 3:45 p.m. UTC | #2
Arun Isaac <arunisaac@systemreboot.net> writes:

>> The documentation of 'shepherd-service' erroneously calls the
>> 'requirement' field 'requirements'.  The attached patch fixes that.
>
> Thanks! I added a copyright header for you and pushed the patch.

Thanks!  I was not aware that removing a single character would warrant
the addition of a copyright header.  I have made less trivial changes in
the past and I have never added a copyright line.  Should I do something
about that?
Arun Isaac May 11, 2019, 10:20 p.m. UTC | #3
> I was not aware that removing a single character would warrant the
> addition of a copyright header.

I am not aware of any policy about this in the Guix project. But, I have
seen people do it both ways -- both adding and not adding copyright
headers for trivial changes. So, I generally add copyright headers for
all changes no matter how trivial.

> I have made less trivial changes in the past and I have never added a
> copyright line.  Should I do something about that?

I think you can ignore those instances. Even I have made trivial changes
before and not added a copyright header.
diff mbox series

Patch

From 635fa669c8d52afd2363c99bfa03b2315af29e74 Mon Sep 17 00:00:00 2001
From: Diego Nicola Barbato <dnbarbato@posteo.de>
Date: Thu, 9 May 2019 22:44:45 +0200
Subject: [PATCH] doc: Fix typo.

* doc/guix.texi (shepherd-service): 'requirement' instead of 'requirements'.
---
 doc/guix.texi | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/doc/guix.texi b/doc/guix.texi
index df7208229c..f88fa827d4 100644
--- a/doc/guix.texi
+++ b/doc/guix.texi
@@ -25279,7 +25279,7 @@  These are the names that may be passed to @command{herd start},
 shepherd, The GNU Shepherd Manual}).  @xref{Slots of services, the
 @code{provides} slot,, shepherd, The GNU Shepherd Manual}, for details.
 
-@item @code{requirements} (default: @code{'()})
+@item @code{requirement} (default: @code{'()})
 List of symbols denoting the Shepherd services this one depends on.
 
 @cindex one-shot services, for the Shepherd
-- 
2.21.0