On 05/16/2013 02:23 AM, Eric Blake wrote:
To ensure we don't regress and cause the need for further
cleanups, add a 'make syntax-check' rule that ensures new
files have proper copyright contents.
* cfg.mk (sc_copyright_file): New rule.
Signed-off-by: Eric Blake <eblake(a)redhat.com>
---
cfg.mk | 10 ++++++++++
1 file changed, 10 insertions(+)
diff --git a/cfg.mk b/cfg.mk
index e013195..f1715e4 100644
--- a/cfg.mk
+++ b/cfg.mk
@@ -688,6 +688,13 @@ sc_copyright_address:
halt='Point to <
http://www.gnu.org/licenses/>, not an address' \
$(_sc_search_regexp)
+# We have to ship COPYING (GPL) alongside COPYING.LESSER (LGPL), but
+# we want source files to be explicit that they are LGPL.
+sc_copyright_file:
+ @prohibit='COPYING([^.]|\.LIB)' \
COPYING.asdf and similar wouldn't match, but we don't need to take care
of that.
+ halt='Refer to COPYING.LESSER for LGPL' \
+ $(_sc_search_regexp)
+
# Some functions/macros produce messages intended solely for developers
# and maintainers. Do not mark them for translation.
sc_prohibit_gettext_markup:
@@ -856,6 +863,9 @@ exclude_file_name_regexp--sc_bindtextdomain = ^(tests|examples)/
exclude_file_name_regexp--sc_copyright_address = \
^COPYING(|\.LESSER)$$
+exclude_file_name_regexp--sc_copyright_file = \
+ ^COPYING(|\.LESSER)|cfg\.mk|libvirt.spec.in$$
+
exclude_file_name_regexp--sc_flags_usage =
^(docs/|src/util/virnetdevtap\.c$$|tests/vircgroupmock\.c$$)
exclude_file_name_regexp--sc_libvirt_unmarked_diagnostics = \
ACK,
Martin