On 06/10/2014 12:18 PM, Roman Bogorodskiy wrote:
> +sc_prohibit_empty_first_line:
> + @awk 'BEGIN { fail=0; } \
> + FNR == 1 { if ($$0 == "") { print FILENAME ":1:"; fail=1; }
} \
> + END { if (fail == 1) { \
> + print "$(ME): Prohibited empty first line" >
"/dev/stderr"; \
> + } exit fail; }' $$($(VC_LIST_EXCEPT));
> +
> # We don't use this feature of maint.mk.
> prev_version_file = /dev/null
>
> @@ -1115,3 +1122,6 @@ exclude_file_name_regexp--sc_avoid_attribute_unused_in_header =
\
>
> exclude_file_name_regexp--sc_prohibit_mixed_case_abbreviations = \
> ^src/(vbox/vbox_CAPI.*.h|esx/esx_vi.(c|h)|esx/esx_storage_backend_iscsi.c)$$
> +
> +exclude_file_name_regexp--sc_prohibit_empty_first_line = \
> +
^(README|daemon/THREADS\.txt|src/esx/README|docs/library.xen|tests/vmwareverdata/fusion-5.0.3.txt)$$
It started to fail for me with this change:
prohibit_empty_first_line
tools/libvirt_win_icon_16x16.ico:1:
Hmm, it might be simpler just to globally exclude all image formats,
rather than repeating lists of image files in affected rules.
I was able to fix it with:
exclude_file_name_regexp--sc_prohibit_empty_first_line = \
-
^(README|daemon/THREADS\.txt|src/esx/README|docs/library.xen|tests/vmwareverdata/fusion-5.0.3.txt)$$
+
^(README|daemon/THREADS\.txt|src/esx/README|docs/library.xen|tests/vmwareverdata/fusion-5.0.3.txt|tools/.*\.ico)$$
Not quite consistent with other places where we ignore image files; for
example:
exclude_file_name_regexp--sc_trailing_blank = \
(/qemuhelpdata/|\.(fig|gif|ico|png)$$)
but my thoughts are that we should really ditch those one-off exception
lists for graphic binary files, and instead modify this list:
VC_LIST_ALWAYS_EXCLUDE_REGEX = \
(^(HACKING|docs/(news\.html\.in|.*\.patch))|\.po)$$
--
Eric Blake eblake redhat com +1-919-301-3266
Libvirt virtualization library
http://libvirt.org