From: Rafael Fontenelle <rafaelff@gnome.org>
To: carlo@zancanaro.id.au
Cc: 32787@debbugs.gnu.org
Subject: bug#32787: Typos in Shepperd
Date: Thu, 20 Sep 2018 20:32:36 -0300 [thread overview]
Message-ID: <CAFPa+Sn4O0tv+0+k77U_CGQroY=bqhHK8cHM7O+HSB_iM-x_+A@mail.gmail.com> (raw)
In-Reply-To: <8736u3svzk.fsf@zancanaro.id.au>
[-- Attachment #1: Type: text/plain, Size: 438 bytes --]
Hi there, Carlo
Em qui, 20 de set de 2018 às 18:31, Carlo Zancanaro
<carlo@zancanaro.id.au> escreveu:
>
> Could you send the patch through as formatted by "git
> format-patch" and with a commit message in the "ChangeLog"[1]
> format? It's probably easiest to just copy the style of commit
> messages in the repository.
No problem. See attached patch. Please check if the commit message is
as expected.
Rafael Fontenelle
[-- Attachment #2: 0001-Fix-typographic-erros.patch --]
[-- Type: text/x-patch, Size: 7715 bytes --]
From 2e8734f7e7a6560e89d3f06038d405e2655b6a47 Mon Sep 17 00:00:00 2001
From: Rafael Fontenelle <rafaelff@gnome.org>
Date: Thu, 20 Sep 2018 20:23:47 -0300
Subject: [PATCH] Fix typographic erros.
This solves some misspelled words throughout the code.
* configure.ac: procude->produce
* doc/shepherd.texi: several typo fixes
* modules/shepherd/args.scm: ambigous->ambiguous
* modules/shepherd/service.scm: childs->children
---
configure.ac | 2 +-
doc/shepherd.texi | 18 +++++++++---------
modules/shepherd/args.scm | 6 +++---
modules/shepherd/service.scm | 6 +++---
4 files changed, 16 insertions(+), 16 deletions(-)
diff --git a/configure.ac b/configure.ac
index 0039093..a270361 100644
--- a/configure.ac
+++ b/configure.ac
@@ -1,4 +1,4 @@
-dnl Process this file with autoconf to procude a configure script.
+dnl Process this file with autoconf to produce a configure script.
AC_PREREQ(2.69)
diff --git a/doc/shepherd.texi b/doc/shepherd.texi
index f54fa6f..73f2fc3 100644
--- a/doc/shepherd.texi
+++ b/doc/shepherd.texi
@@ -651,7 +651,7 @@ therefore it is desirable to specify a different one usually.
@item
@vindex stop (slot of <service>)
-@cindex Stoping a service
+@cindex Stopping a service
@cindex Service destructor
@code{stop} is, similar to @code{start}, a slot containing a
procedure. But in this case, it gets the current value of the
@@ -729,7 +729,7 @@ It tries quite hard to do this: When a service that provides a
required symbol can not be started, it will look for another service
that also provides this symbol, until starting one such service
succeeds. There is some room for theoretical improvement here, of
-course, but in pratice the current strategy already works very well.
+course, but in practice the current strategy already works very well.
This method returns the new value of the @code{running} slot
@ref{Slots of services}, which is @code{#f} if the service could not
be started.
@@ -1063,8 +1063,8 @@ message and throw an @code{assertion-failed} exception.
@end deffn
@deffn {procedure} caught-error key args
-Tell the Shepherd that a @var{key} error with @var{args} has occured. This is
-the simplest way to cause caught error result in uniformly formated
+Tell the Shepherd that a @var{key} error with @var{args} has occurred. This is
+the simplest way to cause caught error result in uniformly formatted
warning messages. The current implementation is not very good,
though.
@end deffn
@@ -1245,11 +1245,11 @@ sometimes useful, but often one also wants to operate on single
services. System V Init makes this hard: While you can start and stop
a service, @code{init} will not know about it, and use the runlevel
configuration as its source of information, opening the door for
-inconsistencies (which fortunatly are not a practical problem
+inconsistencies (which fortunately are not a practical problem
usually). In the Shepherd, this was avoided by having a central entity that is
responsible for starting and stopping the services, which therefore
knows which services are actually started (if not completely
-inproperly used, but that is a requirement which is impossible to
+improperly used, but that is a requirement which is impossible to
avoid anyway). While runlevels are not implemented yet, it is clear
that they will sit on top of the service concept, i.e. runlevels will
merely be an optional extension that the service concept does not rely
@@ -1258,7 +1258,7 @@ become necessary.
The consequence of having a daemon running that controls the services
is that we need another program as user interface which communicates
-with the daemon. Fortunatly, this makes the commands necessary for
+with the daemon. Fortunately, this makes the commands necessary for
controlling services pretty short and intuitive, and gives the
additional bonus of adding some more flexibility. For example, it is
easiely possible to grant password-protected control over certain
@@ -1269,7 +1269,7 @@ above) is that it should always know exactly what is happening,
i.e. which services are started and stopped. The alternative would
have been to not use a daemon, but to save the state on the file
system, again opening the door for inconsistencies of all sorts.
-Also, we would have to use a seperate program for respawning a service
+Also, we would have to use a separate program for respawning a service
(which just starts the services, waits until it terminates and then
starts it again). Killing the program that does the respawning (but
not the service that is supposed to be respawned) would cause horrible
@@ -1544,7 +1544,7 @@ we find which combination works? I found only one way yet: Kind of a
brute force attack: Try combinations until we find one that works.
This alone would be too slow. With 20 services we would have 2^20
-possible combinations, that is a bit more than a million. Fortunatly,
+possible combinations, that is a bit more than a million. Fortunately,
we can optimize this. First I thought we could remove all services
from the list that do not provide any symbol we need, but that is
obviously a stupid idea, as we might need them for dependencies, in
diff --git a/modules/shepherd/args.scm b/modules/shepherd/args.scm
index 3f00d11..604768f 100644
--- a/modules/shepherd/args.scm
+++ b/modules/shepherd/args.scm
@@ -27,7 +27,7 @@
;; This does mostly the same as getopt-long, except for that it is
;; able to recognize abbreviations for long options, as long as they
-;; are not ambigous. Additionally, output is done in a way that makes
+;; are not ambiguous. Additionally, output is done in a way that makes
;; localization possible.
(define-class <option> ()
@@ -102,7 +102,7 @@
;; If this returns `#f', it means no option that can be abbreviated
;; as NAME (or has exactly this name) was found. If the return
;; value is an option, it is exactly that or an abbreviation for it.
- ;; `#t' means that it is ambigous.
+ ;; `#t' means that it is ambiguous.
(define (find-long-option name)
(call/ec (lambda (return)
(let ((abbrev-for #f))
@@ -251,7 +251,7 @@ General help using GNU software: <http://www.gnu.org/gethelp/>~%")
(begin
(local-output
(if target-option
- (l10n "Option `--~a' is ambigous.")
+ (l10n "Option `--~a' is ambiguous.")
(l10n "Unknown option: `--~a'."))
name)
(local-output (l10n "Try `--help'."))
diff --git a/modules/shepherd/service.scm b/modules/shepherd/service.scm
index 8df1329..6cc52ba 100644
--- a/modules/shepherd/service.scm
+++ b/modules/shepherd/service.scm
@@ -1328,7 +1328,7 @@ removing them if needed."
(deregister-service service-name)))
(reload
"Unload all services, then load from FILE-NAME into shepherd. This
-is potentialy dangerous. You have been warned."
+is potentially dangerous. You have been warned."
(lambda (running file-name)
(and (deregister-service "all") ; unload all services
(load-config file-name)))) ; reload from FILE-NAME
@@ -1336,8 +1336,8 @@ is potentialy dangerous. You have been warned."
(daemonize
"Go into the background. Be careful, this means that a new
process will be created, so shepherd will not get SIGCHLD signals anymore
-if previously spawned childs terminate. Therefore, this action should
-usually only be used (if at all) *before* childs get spawned for which
+if previously spawned children terminate. Therefore, this action should
+usually only be used (if at all) *before* children get spawned for which
we want to receive these signals."
(lambda (running)
(case (getpid)
--
2.19.0
next prev parent reply other threads:[~2018-09-20 23:34 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-09-20 19:56 bug#32787: Typos in Shepperd Rafael Fontenelle
2018-09-20 21:24 ` Carlo Zancanaro
2018-09-20 23:32 ` Rafael Fontenelle [this message]
2018-09-21 0:46 ` Carlo Zancanaro
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
List information: https://guix.gnu.org/
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to='CAFPa+Sn4O0tv+0+k77U_CGQroY=bqhHK8cHM7O+HSB_iM-x_+A@mail.gmail.com' \
--to=rafaelff@gnome.org \
--cc=32787@debbugs.gnu.org \
--cc=carlo@zancanaro.id.au \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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).