On 06/08/2010 05:33 AM, Jim Meyering wrote:
> new file mode 100644
> index 0000000..d33293b
> --- /dev/null
> +++ b/src/fiemap.h
> @@ -0,0 +1,102 @@
> +/* FS_IOC_FIEMAP ioctl infrastructure.
> +   Some portions copyright (C) 2007 Cluster File Systems, Inc
> +   Authors: Mark Fasheh <mfasheh@suse.com>
> +            Kalpak Shah <kalpak.shah@sun.com>
> +            Andreas Dilger <adilger@sun.com>.  */

Shouldn't we also add an FSF Copyright 2010 to this new file, to cover
our changes?

> +++ b/tests/cp/sparse-fiemap
> @@ -0,0 +1,56 @@
> +#!/bin/sh
> +# Test cp --sparse=always through fiemap copy
> +
> +# Copyright (C) 2006-2010 Free Software Foundation, Inc.

How much of this content comes from other files from 2006, vs. new
content needing only 2010?

-- 
Eric Blake   eblake@redhat.com    +1-801-349-2682
Libvirt virtualization library http://libvirt.org