lint: Check non-translated package descriptions.

* guix/ui.scm (texi->plain-text): Export.
* guix/scripts/lint.scm (check-description-style): Use it instead of
  'package-description-string'.
This commit is contained in:
Mathieu Lirzin 2015-09-20 12:27:23 +02:00
parent a7a4fd9a7c
commit 2748ee3bcf
2 changed files with 12 additions and 7 deletions

View file

@ -143,12 +143,14 @@ (define (check-not-empty description)
(_ "description should not be empty")
'description)))
(define (check-texinfo-markup package)
"Check that PACKAGE description can be parsed as a Texinfo fragment."
(define (check-texinfo-markup description)
"Check that DESCRIPTION can be parsed as a Texinfo fragment. If the
markup is valid return a plain-text version of DESCRIPTION, otherwise #f."
(catch 'parser-error
(lambda () (package-description-string package))
(lambda () (texi->plain-text description))
(lambda (keys . args)
(emit-warning package (_ "Texinfo markup in description is invalid")))))
(emit-warning package (_ "Texinfo markup in description is invalid"))
#f)))
(define (check-proper-start description)
(unless (or (properly-starts-sentence? description)
@ -179,9 +181,11 @@ (define (check-end-of-sentence-space description)
(let ((description (package-description package)))
(when (string? description)
(check-not-empty description)
(check-texinfo-markup package)
(check-proper-start description)
(check-end-of-sentence-space description))))
;; Use raw description for this because Texinfo rendering automatically
;; fixes end of sentence space.
(check-end-of-sentence-space description)
(and=> (check-texinfo-markup description)
check-proper-start))))
(define (check-inputs-should-be-native package)
;; Emit a warning if some inputs of PACKAGE are likely to belong to its

View file

@ -75,6 +75,7 @@ (define-module (guix ui)
switch-symlinks
config-directory
fill-paragraph
texi->plain-text
package-description-string
string->recutils
package->recutils