unofficial mirror of bug-guix@gnu.org 
 help / color / mirror / code / Atom feed
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


  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).