lint: Improve check for synopses starting with package name.

* guix/scripts/lint.scm (package-name-regexp): New procedure.
  (check-synopsis-style)[check-start-with-package-name]: Use it instead
  of 'string-prefix-ci?'.
* tests/lint.scm ("synopsis: start with package name prefix"): New test.
This commit is contained in:
Ludovic Courtès 2014-11-07 14:09:19 +01:00
parent 72b030c049
commit 17854ef91d
2 changed files with 15 additions and 1 deletions

View File

@ -131,6 +131,12 @@ by two spaces; possible infraction~p at ~{~a~^, ~}"
"pkg-config should probably be a native input" "pkg-config should probably be a native input"
'inputs)))))) 'inputs))))))
(define (package-name-regexp package)
"Return a regexp that matches PACKAGE's name as a word at the beginning of a
line."
(make-regexp (string-append "^" (regexp-quote (package-name package))
"\\>")
regexp/icase))
(define (check-synopsis-style package) (define (check-synopsis-style package)
;; Emit a warning if stylistic issues are found in the synopsis of PACKAGE. ;; Emit a warning if stylistic issues are found in the synopsis of PACKAGE.
@ -168,7 +174,7 @@ by two spaces; possible infraction~p at ~{~a~^, ~}"
'synopsis))) 'synopsis)))
(define (check-start-with-package-name synopsis) (define (check-start-with-package-name synopsis)
(when (string-prefix-ci? (package-name package) synopsis) (when (regexp-exec (package-name-regexp package) synopsis)
(emit-warning package (emit-warning package
"synopsis should not start with the package name" "synopsis should not start with the package name"
'synopsis))) 'synopsis)))

View File

@ -194,6 +194,14 @@
(check-synopsis-style pkg)))) (check-synopsis-style pkg))))
"synopsis should not start with the package name"))) "synopsis should not start with the package name")))
(test-assert "synopsis: start with package name prefix"
(string-null?
(call-with-warnings
(lambda ()
(let ((pkg (dummy-package "arb"
(synopsis "Arbitrary precision"))))
(check-synopsis-style pkg))))))
(test-assert "inputs: pkg-config is probably a native input" (test-assert "inputs: pkg-config is probably a native input"
(->bool (->bool
(string-contains (string-contains