diff mbox series

[bug#39258,3/4] ui: Do not translate package synopsis a second time.

Message ID 20200601000030.7443-4-arunisaac@systemreboot.net
State Work in progress
Headers show
Series Optimize guix search | expand

Checks

Context Check Description
cbaines/comparison success View comparision
cbaines/git branch success View Git branch
cbaines/applying patch success View Laminar job
cbaines/comparison success View comparision
cbaines/git branch success View Git branch
cbaines/applying patch success View Laminar job

Commit Message

Arun Isaac June 1, 2020, midnight UTC
* guix/ui.scm (package->recutils): package-synopsis-string already returns a
translated string. Do not attempt to translate it again.
---
 guix/ui.scm | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

Comments

Ludovic Courtès June 9, 2020, 8:33 a.m. UTC | #1
Arun Isaac <arunisaac@systemreboot.net> skribis:

> * guix/ui.scm (package->recutils): package-synopsis-string already returns a
> translated string. Do not attempt to translate it again.

LGTM!
diff mbox series

Patch

diff --git a/guix/ui.scm b/guix/ui.scm
index 56754dba83..744758d1f3 100644
--- a/guix/ui.scm
+++ b/guix/ui.scm
@@ -1463,8 +1463,7 @@  HYPERLINKS? is true, emit hyperlink escape sequences when appropriate."
           (string-map (match-lambda
                         (#\newline #\space)
                         (chr       chr))
-                      (or (and=> (package-synopsis-string p) P_)
-                          "")))
+                      (or (package-synopsis-string p) "")))
   (format port "~a~%"
           (string->recutils
            (string-trim-right