unofficial mirror of bug-guix@gnu.org 
 help / color / mirror / code / Atom feed
* bug#67038: Inconsistent use of space before ellipsis
@ 2023-11-10  9:12 Ricardo Wurmus
  2023-11-16  9:43 ` Simon Tournier
  2023-11-19 16:55 ` bug#67038: [PATCH] guix: status: Fix ellipsis inconsistencies Adriel Dumas--Jondeau via Bug reports for GNU Guix
  0 siblings, 2 replies; 5+ messages in thread
From: Ricardo Wurmus @ 2023-11-10  9:12 UTC (permalink / raw)
  To: 67038

When running “guix shell python python-numpy” we see output like this:

--8<---------------cut here---------------start------------->8---
applying 3 grafts for python-pytest-bootstrap-7.1.3 ...
applying 9 grafts for libxslt-1.1.37 ...
applying 6 grafts for python-lxml-4.9.1 ...
waiting for locks or build slots...
applying 4 grafts for python-xmlschema-1.2.5 ...
applying 7 grafts for python-pytest-7.1.3 ...
applying 11 grafts for python-numpy-1.23.2 ...
building CA certificate bundle...
listing Emacs sub-directories...
building fonts directory...
building directory of Info manuals...
building profile with 2 packages...
--8<---------------cut here---------------end--------------->8---

Note the inconsistency in whether we separate the ellipses from the
preceding text with a space or not.

-- 
Ricardo




^ permalink raw reply	[flat|nested] 5+ messages in thread

* bug#67038: Inconsistent use of space before ellipsis
  2023-11-10  9:12 bug#67038: Inconsistent use of space before ellipsis Ricardo Wurmus
@ 2023-11-16  9:43 ` Simon Tournier
  2023-11-16 14:09   ` pelzflorian (Florian Pelz)
  2023-11-19 16:55 ` bug#67038: [PATCH] guix: status: Fix ellipsis inconsistencies Adriel Dumas--Jondeau via Bug reports for GNU Guix
  1 sibling, 1 reply; 5+ messages in thread
From: Simon Tournier @ 2023-11-16  9:43 UTC (permalink / raw)
  To: Ricardo Wurmus, 67038

[-- Attachment #1: Type: text/plain, Size: 1098 bytes --]

Hi,

On Fri, 10 Nov 2023 at 10:12, Ricardo Wurmus <rekado@elephly.net> wrote:

> --8<---------------cut here---------------start------------->8---
> applying 3 grafts for python-pytest-bootstrap-7.1.3 ...
> applying 9 grafts for libxslt-1.1.37 ...
> applying 6 grafts for python-lxml-4.9.1 ...
> waiting for locks or build slots...
> applying 4 grafts for python-xmlschema-1.2.5 ...
> applying 7 grafts for python-pytest-7.1.3 ...
> applying 11 grafts for python-numpy-1.23.2 ...
> building CA certificate bundle...
> listing Emacs sub-directories...
> building fonts directory...
> building directory of Info manuals...
> building profile with 2 packages...
> --8<---------------cut here---------------end--------------->8---
>
> Note the inconsistency in whether we separate the ellipses from the
> preceding text with a space or not.

Well, to my knowledge, the typographical rule is not clear depending on
some style; without going to USA vs UK vs ….

For consistency, I suggest the rule: ’word...’ and not ’word ...’

Something like the patch attached.


[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: p.patch --]
[-- Type: text/x-diff, Size: 776 bytes --]

diff --git a/guix/status.scm b/guix/status.scm
index fd89ba9dd3..0ed6d3318a 100644
--- a/guix/status.scm
+++ b/guix/status.scm
@@ -494,8 +494,8 @@ (define* (print-build-event event old-status status
            (let ((count (match (assq-ref properties 'graft)
                           (#f  0)
                           (lst (or (assq-ref lst 'count) 0)))))
-             (format port (info (N_ "applying ~a graft for ~a ..."
-                                    "applying ~a grafts for ~a ..."
+             (format port (info (N_ "applying ~a graft for ~a..."
+                                    "applying ~a grafts for ~a..."
                                     count))
                      count
                      (string-drop-right (store-path-package-name drv)

[-- Attachment #3: Type: text/plain, Size: 22 bytes --]


WDYT?

Cheers,
simon

^ permalink raw reply related	[flat|nested] 5+ messages in thread

* bug#67038: Inconsistent use of space before ellipsis
  2023-11-16  9:43 ` Simon Tournier
@ 2023-11-16 14:09   ` pelzflorian (Florian Pelz)
  2023-11-17 15:47     ` Ludovic Courtès
  0 siblings, 1 reply; 5+ messages in thread
From: pelzflorian (Florian Pelz) @ 2023-11-16 14:09 UTC (permalink / raw)
  To: Simon Tournier; +Cc: Ricardo Wurmus, 67038

Simon Tournier <zimon.toutoune@gmail.com> writes:
> For consistency, I suggest the rule: ’word...’ and not ’word ...’
>
> Something like the patch attached.

+1

This is more consistent.  Looking at the translation files in po/guix,
only guix/status.scm:568 uses the space before ellipsis.  The message
"applying ~a graft" does not occur in the manual, so it does not need
changing.

(match event
  […]
  (('download-started item uri _ ...)
   (when print-urls?
     (erase-current-line*)
     (format port (info (G_ "downloading from ~a ...")) uri)
     (newline port)))

Should we keep this space?  Perhaps keep it; it is clearer where the
URI ends.

Regards,
Florian




^ permalink raw reply	[flat|nested] 5+ messages in thread

* bug#67038: Inconsistent use of space before ellipsis
  2023-11-16 14:09   ` pelzflorian (Florian Pelz)
@ 2023-11-17 15:47     ` Ludovic Courtès
  0 siblings, 0 replies; 5+ messages in thread
From: Ludovic Courtès @ 2023-11-17 15:47 UTC (permalink / raw)
  To: pelzflorian (Florian Pelz); +Cc: Ricardo Wurmus, 67038, Simon Tournier

Hi,

"pelzflorian (Florian Pelz)" <pelzflorian@pelzflorian.de> skribis:

> Simon Tournier <zimon.toutoune@gmail.com> writes:
>> For consistency, I suggest the rule: ’word...’ and not ’word ...’
>>
>> Something like the patch attached.
>
> +1

+1!

> (match event
>   […]
>   (('download-started item uri _ ...)
>    (when print-urls?
>      (erase-current-line*)
>      (format port (info (G_ "downloading from ~a ...")) uri)
>      (newline port)))
>
> Should we keep this space?  Perhaps keep it; it is clearer where the
> URI ends.

Yes, we should keep this one: somebody added it on the grounds that it
allowed them to click on URLs in their terminal and get the correct URL,
without the period.

Ludo’.




^ permalink raw reply	[flat|nested] 5+ messages in thread

* bug#67038: [PATCH] guix: status: Fix ellipsis inconsistencies.
  2023-11-10  9:12 bug#67038: Inconsistent use of space before ellipsis Ricardo Wurmus
  2023-11-16  9:43 ` Simon Tournier
@ 2023-11-19 16:55 ` Adriel Dumas--Jondeau via Bug reports for GNU Guix
  1 sibling, 0 replies; 5+ messages in thread
From: Adriel Dumas--Jondeau via Bug reports for GNU Guix @ 2023-11-19 16:55 UTC (permalink / raw)
  To: 67038
  Cc: Christopher Baines, Josselin Poiret, Ludovic Courtès,
	Mathieu Othacehe, Ricardo Wurmus, Simon Tournier,
	Tobias Geerinckx-Rice

Fixes <https://issues.guix.gnu.org/67038>.
Reported by Ricardo Wurmus.

* guix/status.scm (print-build-event): Fix ellipsis inconsistencies.
---
 guix/status.scm | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

We just made this patch after Capitole du Libre at Toulouse as an example for
my first patch \o/

diff --git a/guix/status.scm b/guix/status.scm
index fd89ba9dd3..0ed6d3318a 100644
--- a/guix/status.scm
+++ b/guix/status.scm
@@ -494,8 +494,8 @@ (define* (print-build-event event old-status status
            (let ((count (match (assq-ref properties 'graft)
                           (#f  0)
                           (lst (or (assq-ref lst 'count) 0)))))
-             (format port (info (N_ "applying ~a graft for ~a ..."
-                                    "applying ~a grafts for ~a ..."
+             (format port (info (N_ "applying ~a graft for ~a..."
+                                    "applying ~a grafts for ~a..."
                                     count))
                      count
                      (string-drop-right (store-path-package-name drv)

base-commit: 761810d32a0550912b5bb36a0c9f60141bcb7d20
-- 
2.41.0





^ permalink raw reply related	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2023-11-22 17:51 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-11-10  9:12 bug#67038: Inconsistent use of space before ellipsis Ricardo Wurmus
2023-11-16  9:43 ` Simon Tournier
2023-11-16 14:09   ` pelzflorian (Florian Pelz)
2023-11-17 15:47     ` Ludovic Courtès
2023-11-19 16:55 ` bug#67038: [PATCH] guix: status: Fix ellipsis inconsistencies Adriel Dumas--Jondeau via Bug reports for GNU Guix

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