diff mbox series

[6/30] semihosting: Fix Lesser GPL version number

Message ID 20201016144243.26817-1-chetan4windows@gmail.com
State Accepted
Commit 07e6f009a0a6a38238820f3d198dbe36f2bbb74f
Headers show
Series None | expand

Commit Message

Chetan Oct. 16, 2020, 2:42 p.m. UTC
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.

Signed-off-by: Chetan Pant <chetan4windows@gmail.com>
---
 include/hw/semihosting/semihost.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Alex Bennée Oct. 16, 2020, 4:06 p.m. UTC | #1
Chetan Pant <chetan4windows@gmail.com> writes:

> 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.

>

> Signed-off-by: Chetan Pant <chetan4windows@gmail.com>


Acked-by: Alex Bennée <alex.bennee@linaro.org>


-- 
Alex Bennée
Thomas Huth Oct. 19, 2020, 10:37 a.m. UTC | #2
On 16/10/2020 16.42, 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.

> 

> Signed-off-by: Chetan Pant <chetan4windows@gmail.com>

> ---

>  include/hw/semihosting/semihost.h | 2 +-

>  1 file changed, 1 insertion(+), 1 deletion(-)

> 

> diff --git a/include/hw/semihosting/semihost.h b/include/hw/semihosting/semihost.h

> index b8ce511..0c55ade 100644

> --- a/include/hw/semihosting/semihost.h

> +++ b/include/hw/semihosting/semihost.h

> @@ -6,7 +6,7 @@

>   * This library is free software; you can redistribute it and/or

>   * modify it under the terms of the GNU Lesser General Public

>   * License as published by the Free Software Foundation; either

> - * version 2 of the License, or (at your option) any later version.

> + * version 2.1 of the License, or (at your option) any later version.

>   *

>   * This library is distributed in the hope that it will be useful,

>   * but WITHOUT ANY WARRANTY; without even the implied warranty of

> 


Reviewed-by: Thomas Huth <thuth@redhat.com>
Eric Blake Oct. 19, 2020, 8:35 p.m. UTC | #3
On 10/16/20 9:42 AM, Chetan Pant wrote:
> There is no "version 2" of the "Lesser" General Public License.

Meta-comment: Looking in the archives, I see some odd threading going on:

https://lists.gnu.org/archive/html/qemu-devel/2020-10/threads.html#04020

You sent 1/30 and 2/30 as independent threads, then deeply threaded 
3-8/30, but I don't see a 0/30 cover letter, nor 9-30/30.

When you use deep threading, it gets awkward to read replies in mailers 
that group by thread depth before date:

+ 1/30
  \+ re: 1/30
+ 2/30
  \+ re: 1/30
+ 3/30
| + 4/30
| | + 5/30
| | | + 6/30
| | | | + 7/30
| | | | | + 8/30
| | | | |  \+ re: 8/30
| | | |  \+ re: 7/30
| | |  \+ re: 6/30
| |  \+ re: 5/30
|  \+ re: 4/30
  \+ re: 3/30

instead of the more typical

+ 0/30
| + 1/30
|  \+ re: 1/30
| + 2/30
|  \+ re: 2/30
| + 3/30
|  \+ re: 3/30
| + 4/30
|  \+ re: 4/30
...
| + 30/30
|  \+ re: 30/30
  \+ re: 0/30

Figuring out how to make your mailer thread properly will make it easier 
to interact with your future patches.
diff mbox series

Patch

diff --git a/include/hw/semihosting/semihost.h b/include/hw/semihosting/semihost.h
index b8ce511..0c55ade 100644
--- a/include/hw/semihosting/semihost.h
+++ b/include/hw/semihosting/semihost.h
@@ -6,7 +6,7 @@ 
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Lesser General Public
  * License as published by the Free Software Foundation; either
- * version 2 of the License, or (at your option) any later version.
+ * version 2.1 of the License, or (at your option) any later version.
  *
  * This library is distributed in the hope that it will be useful,
  * but WITHOUT ANY WARRANTY; without even the implied warranty of