提交 f131dacf 编写于 作者: A Andrea Bolognani

cfg.mk: Simplify backslash alignment check

The use of [[:blank:]] was intended to cover tabs as well, but
it couldn't possibly work in its current form, so the regex was
tweaked in d09429ab.

With the original reason for using [[:blank:]] now gone, we can
replace its usage with plain spaces. A comment about the purpose
of the check is added as well.
Signed-off-by: NAndrea Bolognani <abologna@redhat.com>
上级 16a9a281
...@@ -1037,8 +1037,10 @@ sc_prohibit_http_urls: ...@@ -1037,8 +1037,10 @@ sc_prohibit_http_urls:
halt='Links must use https:// protocol' \ halt='Links must use https:// protocol' \
$(_sc_search_regexp) $(_sc_search_regexp)
# Alignment is usually achieved through spaces (at least two of them)
# or tabs (at least one of them) right before the trailing backslash
sc_prohibit_backslash_alignment: sc_prohibit_backslash_alignment:
@prohibit='([[:blank:]][[:blank:]]| )\\$$' \ @prohibit='( | )\\$$' \
in_vc_files='*\.([chx]|am|mk)$$' \ in_vc_files='*\.([chx]|am|mk)$$' \
halt='Do not attempt to right-align backslashes' \ halt='Do not attempt to right-align backslashes' \
$(_sc_search_regexp) $(_sc_search_regexp)
......
Markdown is supported
0% .
You are about to add 0 people to the discussion. Proceed with caution.
先完成此消息的编辑!
想要评论请 注册