Re: [PATCH 5/5] Fixing Lesser GPL version number [5/5]

["Daniel P. Berrangé" <berrange@redhat.com>]

On Fri, Oct 09, 2020 at 06:44:10AM +0000, Chetan Pant wrote:
> There is no "version 2" of the "Lesser" General Public License. It is
> either "GPL version 2.0" or "Lesser GPL version 2.1". This patch replaces all
> occurrences of "Lesser GPL version 2" with "Lesser GPL version 2.1" in comment section.
> Also, It came to notice that some of the files that were edited for the change
> were not following latest comment rules. For example using "//" to mark comment
> instead of "/*". That is also fixed in this patch.
> 
> This patch is divided in 5 parts, directory wise, in order to make reviewing process easier.
> Below listed are the parts of the patch, where asterisk denotes the part you are currently viewing.
> 
> 	[ ] Files in authz/backends/block/linux-user/tests/migration directory (82 Files)
> 	[ ] Files in hw/include/disas (100 files)
> 	[ ] Files inside target/ 'alpha,arm,cris,hppa,i386' (96 files)
> 	[ ] Files inside target/ 'lm32,microblaze,mips,ppc,rx,sparc,tilegx,tricore,xtensa' (63 files)
> 	[*] Files in ui/util/include/scripts and QEMU root directory (76 Files)
> 
> Below is how the license version was corrected:
> 
> 1. To find the number of file having "Lesser GPL version 2 ":
> 	grep -l Lesser $(grep -rl "version 2 " * ) > result.dat
>    Total of 417 files were found (After manually exluding the files like COPYING and COPYING.LIB from the result)
> 
> 2. To find the number of occurences of "version 2 " in the resulted files:
> 	egrep -c "version 2 " $(cat result.dat)
>    410 files had "version 2" occurence 1 time (name of those files was saved in one_timers.dat)
>    and in 7 files "version 2" occurences were multiple times.
> 
> 3. Files having occurence exactly 1 time were corrected using below command:
> 	sed -i "s/version 2 /version 2.1 /g" $(cat one_timers.dat)
>    For rest of 7 files, correction was done manually.
> 
> Signed-off-by: Chetan Pant <chetan4windows@gmail.com>
> ---
>  cpus-common.c                         | 2 +-
>  exec-vary.c                           | 2 +-
>  exec.c                                | 2 +-
>  include/authz/base.h                  | 2 +-
>  include/authz/list.h                  | 2 +-
>  include/authz/listfile.h              | 2 +-
>  include/authz/pamacct.h               | 2 +-
>  include/authz/simple.h                | 2 +-

Can you include these with the "authz" pieces from patch 1 and I'll
queue them


>  include/io/channel-buffer.h           | 2 +-
>  include/io/channel-command.h          | 2 +-
>  include/io/channel-file.h             | 2 +-
>  include/io/channel-socket.h           | 2 +-
>  include/io/channel-tls.h              | 2 +-
>  include/io/channel-util.h             | 2 +-
>  include/io/channel-watch.h            | 2 +-
>  include/io/channel-websock.h          | 2 +-
>  include/io/channel.h                  | 2 +-
>  include/io/dns-resolver.h             | 2 +-
>  include/io/task.h                     | 2 +-

Combine with the "io" pieces from patch 1.


Regards,
Daniel
-- 
|: https://berrange.com      -o-    https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org         -o-            https://fstop138.berrange.com :|
|: https://entangle-photo.org    -o-    https://www.instagram.com/dberrange :|