unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
* Re: [elpa] master 785e186 3/3: Merge commit 'ec7ba4f2dbae0901724483de5868127a1cbc38e9'
       [not found] ` <E1YNCv6-0007Ru-Ib@vcs.savannah.gnu.org>
@ 2015-02-16  5:36   ` Stefan Monnier
  2015-02-16  6:24     ` Rocky Bernstein
  0 siblings, 1 reply; 5+ messages in thread
From: Stefan Monnier @ 2015-02-16  5:36 UTC (permalink / raw)
  To: emacs-devel; +Cc: rocky

Thanks for the fix.

> --- a/packages/test-simple/example/gcd.el
> +++ b/packages/test-simple/example/gcd.el
> @@ -1,11 +1,32 @@
> +;;; test-simple.el --- Simple Unit Test Framework for Emacs Lisp

Hmm... why would "gcd.el" state that it's called "test-simple.el"?
It doesn't matter to GNU ELPA, so there's no hurry to fix it.

> +;; Author: Rocky Bernstein <rocky@gnu.org>
> +;; URL: http://github.com/rocky/emacs-test-simple
> +;; Keywords: unit-test
> +;; Version: 1.0

BTW, note that these headers are normally package headers, so they only
need to be present in the package's main file (i.e. test-simple.el in
this case).  Of course, just as the previous comment, this doesn't
bother the GNU ELPA scripts, so if you like it this way, it's your call.


        Stefan



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

* Re: [elpa] master 785e186 3/3: Merge commit 'ec7ba4f2dbae0901724483de5868127a1cbc38e9'
  2015-02-16  5:36   ` [elpa] master 785e186 3/3: Merge commit 'ec7ba4f2dbae0901724483de5868127a1cbc38e9' Stefan Monnier
@ 2015-02-16  6:24     ` Rocky Bernstein
  2015-02-16  7:47       ` Stefan Monnier
  0 siblings, 1 reply; 5+ messages in thread
From: Rocky Bernstein @ 2015-02-16  6:24 UTC (permalink / raw)
  To: Stefan Monnier; +Cc: emacs-devel

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

On Mon, Feb 16, 2015 at 12:36 AM, Stefan Monnier <monnier@iro.umontreal.ca>
wrote:

> Thanks for the fix.
>
> > --- a/packages/test-simple/example/gcd.el
> > +++ b/packages/test-simple/example/gcd.el
> > @@ -1,11 +1,32 @@
> > +;;; test-simple.el --- Simple Unit Test Framework for Emacs Lisp
>
> Hmm... why would "gcd.el" state that it's called "test-simple.el"?
> It doesn't matter to GNU ELPA, so there's no hurry to fix it.
>
> > +;; Author: Rocky Bernstein <rocky@gnu.org>
> > +;; URL: http://github.com/rocky/emacs-test-simple
> > +;; Keywords: unit-test
> > +;; Version: 1.0
>
> BTW, note that these headers are normally package headers, so they only
> need to be present in the package's main file (i.e. test-simple.el in
> this case).  Of course, just as the previous comment, this doesn't
> bother the GNU ELPA scripts, so if you like it this way, it's your call.
>

Caused by mindless copy of boilerplate stuff. It would be good if this were
automated better.

I've removed some of those lines. It feels silly to me to add the full GPL
terms on test programs or a generic program used in such a test. The tests
and sample program data aren't really useful except in support of that
specific package. So the likelihood of anyone copying it and making it
proprietary is in my opinion slim; and the detriment to the general public
or programmers marginal even if this happens is negligible.  Wouldn't just
a copyright would be sufficient for such subsidiary programs?

But I'm neither a lawyer, nor do I care to spend much time dealing with
such issues. Again, this is why automating all the boilerplate would be
desirable.

[-- Attachment #2: Type: text/html, Size: 2256 bytes --]

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

* Re: [elpa] master 785e186 3/3: Merge commit 'ec7ba4f2dbae0901724483de5868127a1cbc38e9'
  2015-02-16  6:24     ` Rocky Bernstein
@ 2015-02-16  7:47       ` Stefan Monnier
  2015-02-17 13:59         ` Rocky Bernstein
  0 siblings, 1 reply; 5+ messages in thread
From: Stefan Monnier @ 2015-02-16  7:47 UTC (permalink / raw)
  To: Rocky Bernstein; +Cc: emacs-devel

> Caused by mindless copy of boilerplate stuff. It would be good if this were
> automated better.

auto-insert does automate this.

> But I'm neither a lawyer, nor do I care to spend much time dealing with
> such issues.

Same here, which is why I just add the blurb to all those files and move on.


        Stefan



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

* Re: [elpa] master 785e186 3/3: Merge commit 'ec7ba4f2dbae0901724483de5868127a1cbc38e9'
  2015-02-16  7:47       ` Stefan Monnier
@ 2015-02-17 13:59         ` Rocky Bernstein
  2015-02-18  3:54           ` Stefan Monnier
  0 siblings, 1 reply; 5+ messages in thread
From: Rocky Bernstein @ 2015-02-17 13:59 UTC (permalink / raw)
  To: Stefan Monnier; +Cc: emacs-devel

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

On Mon, Feb 16, 2015 at 2:47 AM, Stefan Monnier <monnier@iro.umontreal.ca>
wrote:

> > Caused by mindless copy of boilerplate stuff. It would be good if this
> were
> > automated better.
>
> auto-insert does automate this.
>

Tried that and it puts in too much boilerplate. Yes, I see that there is
some sort of a-list that I can customize, but I didn't want to go down that
rabbit hole.


>
> > But I'm neither a lawyer, nor do I care to spend much time dealing with
> > such issues.
>
> Same here, which is why I just add the blurb to all those files and move
> on.
>
>
>         Stefan
>

[-- Attachment #2: Type: text/html, Size: 1196 bytes --]

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

* Re: [elpa] master 785e186 3/3: Merge commit 'ec7ba4f2dbae0901724483de5868127a1cbc38e9'
  2015-02-17 13:59         ` Rocky Bernstein
@ 2015-02-18  3:54           ` Stefan Monnier
  0 siblings, 0 replies; 5+ messages in thread
From: Stefan Monnier @ 2015-02-18  3:54 UTC (permalink / raw)
  To: Rocky Bernstein; +Cc: emacs-devel

> Tried that and it puts in too much boilerplate.

Works for me, but you might like to M-x report-emacs-bug asking to
remove some parts.


        Stefan



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

end of thread, other threads:[~2015-02-18  3:54 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
     [not found] <20150216040427.28553.74719@vcs.savannah.gnu.org>
     [not found] ` <E1YNCv6-0007Ru-Ib@vcs.savannah.gnu.org>
2015-02-16  5:36   ` [elpa] master 785e186 3/3: Merge commit 'ec7ba4f2dbae0901724483de5868127a1cbc38e9' Stefan Monnier
2015-02-16  6:24     ` Rocky Bernstein
2015-02-16  7:47       ` Stefan Monnier
2015-02-17 13:59         ` Rocky Bernstein
2015-02-18  3:54           ` Stefan Monnier

Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/emacs.git

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 read-only IMAP folder(s) and NNTP newsgroup(s).