On Mon, Dec 05, 2011 at 01:24:13PM -0500, Matt Wagner wrote:
On Mon, Dec 05, 2011 at 06:25:56PM +0100, Francesco Vollero wrote:
>
> Signed-off-by: Francesco Vollero <fvollero(a)redhat.com>
> ---
> src/lib/tasks/apache_license.rake | 16 ++++++++++++++++
> 1 files changed, 16 insertions(+), 0 deletions(-)
> create mode 100644 src/lib/tasks/apache_license.rake
>
> diff --git a/src/lib/tasks/apache_license.rake b/src/lib/tasks/apache_license.rake
> new file mode 100644
> index 0000000..4e444bd
> --- /dev/null
> +++ b/src/lib/tasks/apache_license.rake
> @@ -0,0 +1,16 @@
> +namespace :license do
> + desc 'Verify that every file contains apache license'
> +
> + task :verify do |t|
> +
> + puts "Files that contain GPL License: %d" %
> + (%x[find ./ -type f -iname *.rb -exec grep -qE
"http://www.gnu.org/copyleft/gpl.html" {} \\; -print |wc -l])
> + puts "Files that does not have GPL nor ASL: %d" %
Nit: I would rephrase this to "Files that have neither GPL nor ASL"
Thanks, i've updated that :)
> + (%x[find ./ -type f -iname *.rb \\! -exec grep -qE
"Copyright|copyleft" {} \\; -print|wc -l])
> +
> + puts "Files that contain Apache license: %d" %
> + (%x[find ./ -type f -iname *.rb -exec grep -qE "Apache License"
{} \\; -print|wc -l])
> + end
> +
> + #namespace end
> +end
> --
> 1.7.4.4
ACK, this works fine, with a nit inline above.
I wonder if we might want to list the actual files that don't include
licenses, rather than just giving a count? Though I suppose that might
not be needed all that often.
Yes i can "categorize" the namespace with: files without license, or gpl.
Waiting for a feedback on that since in my version it is in this way but i think someone
thought was unuseful with large files.
Cheers,
Francesco
-- Matt