linux-mm.kvack.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] mm: replace FSF address with web source in license notices
@ 2017-11-14  9:44 Martin Kepplinger
  2017-11-14  9:49 ` Michal Hocko
  2017-11-22 22:20 ` Pavel Machek
  0 siblings, 2 replies; 6+ messages in thread
From: Martin Kepplinger @ 2017-11-14  9:44 UTC (permalink / raw)
  To: catalin.marinas; +Cc: linux-mm, linux-kernel, Martin Kepplinger

A few years ago the FSF moved and "59 Temple Place" is wrong. Having this
still in our source files feels old and unmaintained.

Let's take the license statement serious and not confuse users.

As https://www.gnu.org/licenses/gpl-howto.html suggests, we replace the
postal address with "<http://www.gnu.org/licenses/>" in the mm directory.

Signed-off-by: Martin Kepplinger <martink@posteo.de>
---
 mm/kmemleak-test.c | 3 +--
 mm/kmemleak.c      | 3 +--
 2 files changed, 2 insertions(+), 4 deletions(-)

diff --git a/mm/kmemleak-test.c b/mm/kmemleak-test.c
index dd3c23a801b1..9a13ad2c0cca 100644
--- a/mm/kmemleak-test.c
+++ b/mm/kmemleak-test.c
@@ -14,8 +14,7 @@
  * GNU General Public License for more details.
  *
  * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA
+ * along with this program.  If not, see <http://www.gnu.org/licenses/>.
  */
 
 #define pr_fmt(fmt) "kmemleak: " fmt
diff --git a/mm/kmemleak.c b/mm/kmemleak.c
index e4738d5e9b8c..e6d6d3c9f543 100644
--- a/mm/kmemleak.c
+++ b/mm/kmemleak.c
@@ -14,8 +14,7 @@
  * GNU General Public License for more details.
  *
  * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA
+ * along with this program.  If not, see <http://www.gnu.org/licenses/>.
  *
  *
  * For more information on the algorithm and kmemleak usage, please see
-- 
2.11.0

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>

^ permalink raw reply related	[flat|nested] 6+ messages in thread

* Re: [PATCH] mm: replace FSF address with web source in license notices
  2017-11-14  9:44 [PATCH] mm: replace FSF address with web source in license notices Martin Kepplinger
@ 2017-11-14  9:49 ` Michal Hocko
  2017-11-14  9:55   ` Martin Kepplinger
  2017-11-22 22:20 ` Pavel Machek
  1 sibling, 1 reply; 6+ messages in thread
From: Michal Hocko @ 2017-11-14  9:49 UTC (permalink / raw)
  To: Martin Kepplinger; +Cc: catalin.marinas, linux-mm, linux-kernel

On Tue 14-11-17 10:44:38, Martin Kepplinger wrote:
> A few years ago the FSF moved and "59 Temple Place" is wrong. Having this
> still in our source files feels old and unmaintained.
> 
> Let's take the license statement serious and not confuse users.
> 
> As https://www.gnu.org/licenses/gpl-howto.html suggests, we replace the
> postal address with "<http://www.gnu.org/licenses/>" in the mm directory.

Why to change this now? Isn't there a general plan to move to SPDX?
-- 
Michal Hocko
SUSE Labs

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH] mm: replace FSF address with web source in license notices
  2017-11-14  9:49 ` Michal Hocko
@ 2017-11-14  9:55   ` Martin Kepplinger
  2017-11-14 10:02     ` Michal Hocko
  0 siblings, 1 reply; 6+ messages in thread
From: Martin Kepplinger @ 2017-11-14  9:55 UTC (permalink / raw)
  To: Michal Hocko; +Cc: catalin.marinas, linux-mm, linux-kernel

Am 14.11.2017 10:49 schrieb Michal Hocko:
> On Tue 14-11-17 10:44:38, Martin Kepplinger wrote:
>> A few years ago the FSF moved and "59 Temple Place" is wrong. Having 
>> this
>> still in our source files feels old and unmaintained.
>> 
>> Let's take the license statement serious and not confuse users.
>> 
>> As https://www.gnu.org/licenses/gpl-howto.html suggests, we replace 
>> the
>> postal address with "<http://www.gnu.org/licenses/>" in the mm 
>> directory.
> 
> Why to change this now? Isn't there a general plan to move to SPDX?

Shouldn't a move to SPDX only be additions to what we currently have? 
That's
at least what the "reuse" project suggests, see 
https://reuse.software/practices/
with "Don’t remove existing headers, but only add to them."

thanks

                                        martin

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH] mm: replace FSF address with web source in license notices
  2017-11-14  9:55   ` Martin Kepplinger
@ 2017-11-14 10:02     ` Michal Hocko
  2017-11-14 10:21       ` Martin Kepplinger
  0 siblings, 1 reply; 6+ messages in thread
From: Michal Hocko @ 2017-11-14 10:02 UTC (permalink / raw)
  To: Martin Kepplinger; +Cc: catalin.marinas, linux-mm, linux-kernel

On Tue 14-11-17 10:55:35, Martin Kepplinger wrote:
> Am 14.11.2017 10:49 schrieb Michal Hocko:
> > On Tue 14-11-17 10:44:38, Martin Kepplinger wrote:
> > > A few years ago the FSF moved and "59 Temple Place" is wrong. Having
> > > this
> > > still in our source files feels old and unmaintained.
> > > 
> > > Let's take the license statement serious and not confuse users.
> > > 
> > > As https://www.gnu.org/licenses/gpl-howto.html suggests, we replace
> > > the
> > > postal address with "<http://www.gnu.org/licenses/>" in the mm
> > > directory.
> > 
> > Why to change this now? Isn't there a general plan to move to SPDX?
> 
> Shouldn't a move to SPDX only be additions to what we currently have? That's
> at least what the "reuse" project suggests, see
> https://reuse.software/practices/
> with "Dona??t remove existing headers, but only add to them."

I thought the primary motivation was to unify _all_ headers and get rid
of all the duplication. (aside from files which do not have any license
which is under discussion elsewhere).
-- 
Michal Hocko
SUSE Labs

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH] mm: replace FSF address with web source in license notices
  2017-11-14 10:02     ` Michal Hocko
@ 2017-11-14 10:21       ` Martin Kepplinger
  0 siblings, 0 replies; 6+ messages in thread
From: Martin Kepplinger @ 2017-11-14 10:21 UTC (permalink / raw)
  To: Michal Hocko; +Cc: catalin.marinas, linux-mm, linux-kernel

Am 14.11.2017 11:02 schrieb Michal Hocko:
> On Tue 14-11-17 10:55:35, Martin Kepplinger wrote:
>> Am 14.11.2017 10:49 schrieb Michal Hocko:
>> > On Tue 14-11-17 10:44:38, Martin Kepplinger wrote:
>> > > A few years ago the FSF moved and "59 Temple Place" is wrong. Having
>> > > this
>> > > still in our source files feels old and unmaintained.
>> > >
>> > > Let's take the license statement serious and not confuse users.
>> > >
>> > > As https://www.gnu.org/licenses/gpl-howto.html suggests, we replace
>> > > the
>> > > postal address with "<http://www.gnu.org/licenses/>" in the mm
>> > > directory.
>> >
>> > Why to change this now? Isn't there a general plan to move to SPDX?
>> 
>> Shouldn't a move to SPDX only be additions to what we currently have? 
>> That's
>> at least what the "reuse" project suggests, see
>> https://reuse.software/practices/
>> with "Don’t remove existing headers, but only add to them."
> 
> I thought the primary motivation was to unify _all_ headers and get rid
> of all the duplication. (aside from files which do not have any license
> which is under discussion elsewhere).

I doubt that this can be fully accieved in the long run :) It'd be nice 
of course in
some way.

But I also doubt that it'd be so easy to remove the permission 
statements.
The FSF who's license we use suggest to have them, but others do too.
And as mentioned, "using SPDX" doesn't imply "not having permission
statements".

But I think that's off-topic actually. Moving to SPDX could still be 
done in
any way whatsoever after this. This change fixes a *mistake* and can 
reduce
confusion or even support license compliance, who knows :)

thanks
                                         martin

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH] mm: replace FSF address with web source in license notices
  2017-11-14  9:44 [PATCH] mm: replace FSF address with web source in license notices Martin Kepplinger
  2017-11-14  9:49 ` Michal Hocko
@ 2017-11-22 22:20 ` Pavel Machek
  1 sibling, 0 replies; 6+ messages in thread
From: Pavel Machek @ 2017-11-22 22:20 UTC (permalink / raw)
  To: Martin Kepplinger; +Cc: catalin.marinas, linux-mm, linux-kernel

[-- Attachment #1: Type: text/plain, Size: 1483 bytes --]

On Tue 2017-11-14 10:44:38, Martin Kepplinger wrote:
> A few years ago the FSF moved and "59 Temple Place" is wrong. Having this
> still in our source files feels old and unmaintained.
> 
> Let's take the license statement serious and not confuse users.
> 
> As https://www.gnu.org/licenses/gpl-howto.html suggests, we replace the
> postal address with "<http://www.gnu.org/licenses/>" in the mm directory.
> 
> Signed-off-by: Martin Kepplinger <martink@posteo.de>
> ---
>  mm/kmemleak-test.c | 3 +--
>  mm/kmemleak.c      | 3 +--
>  2 files changed, 2 insertions(+), 4 deletions(-)
> 
> diff --git a/mm/kmemleak-test.c b/mm/kmemleak-test.c
> index dd3c23a801b1..9a13ad2c0cca 100644
> --- a/mm/kmemleak-test.c
> +++ b/mm/kmemleak-test.c
> @@ -14,8 +14,7 @@
>   * GNU General Public License for more details.
>   *
>   * You should have received a copy of the GNU General Public License
> - * along with this program; if not, write to the Free Software
> - * Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA
> + * along with this program.  If not, see <http://www.gnu.org/licenses/>.
>   */
>  

With all the SPDX work, I don't think this is useful. Talk to Greg?

We do ship copy of GPL, so perhaps the paragraph can be just deleted?

If not, maybe it should be http_s_?
									Pavel
-- 
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 181 bytes --]

^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2017-11-22 22:20 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-11-14  9:44 [PATCH] mm: replace FSF address with web source in license notices Martin Kepplinger
2017-11-14  9:49 ` Michal Hocko
2017-11-14  9:55   ` Martin Kepplinger
2017-11-14 10:02     ` Michal Hocko
2017-11-14 10:21       ` Martin Kepplinger
2017-11-22 22:20 ` Pavel Machek

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).