On 06/11/2014 02:56 AM, Martin Kletzander wrote: > On Tue, Jun 10, 2014 at 01:39:03PM -0600, Eric Blake wrote: >> Roman Bogorodskiy reported a syntax-check failure when using >> FreeBSD; complaining that: >> >> prohibit_empty_first_line >> tools/libvirt_win_icon_16x16.ico:1: >> tools/libvirt_win_icon_32x32.ico:1: >> tools/libvirt_win_icon_48x48.ico:1: >> tools/libvirt_win_icon_64x64.ico:1: >> maint.mk: Prohibited empty first line >> >> In reality, the first 'line' of that file is NOT empty; but since >> it is a binary file, awk is not required to handle it gracefully. >> The simplest solution is to exempt all image files from syntax >> checks in the first place - after all, we only store them in git >> because they are inconvenient to regenerate, but they are not our >> preferred format for making modifications, and syntax check should >> only cover files that we are likely to modify. >> >> * cfg.mk (VC_LIST_ALWAYS_EXCLUDE_REGEX): Exempt images. >> (exclude_file_name_regexp--sc_prohibit_empty_lines_at_EOF): Simplify. >> (exclude_file_name_regexp--sc_trailing_blank): Likewise. >> > > ACK Pushed. -- Eric Blake eblake redhat com +1-919-301-3266 Libvirt virtualization library http://libvirt.org
Attachment:
signature.asc
Description: OpenPGP digital signature
-- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list