unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
From: Drew Adams <drew.adams@oracle.com>
To: bruce.connor.am@gmail.com
Cc: Alexis <flexibeast@gmail.com>, emacs-devel <emacs-devel@gnu.org>
Subject: RE: Improving browsing and discoverability in the Packages Menu
Date: Mon, 20 Apr 2015 11:17:52 -0700 (PDT)	[thread overview]
Message-ID: <ff2f56fe-414e-43ad-be03-750a22abfd77@default> (raw)
In-Reply-To: <CAAdUY-KCfabYeUiye=e1rXEoKMdjwNEzK6ciq8dMXO7ifCtEag@mail.gmail.com>

(Sure wish you would send your emails as plain text, BTW.)

> > > We would just accept any keyword that doesn't already have...
> >
> > "We" is what here, exactly?  Just the use of keywords by
> > `list-packages' (or other package viewing/filtering code)?
> The idea is not to restrict or prevent anything, just recommend.
> We would extend the list of known finder.el keywords. When the byte
> compiler is compiling a package, it could issue a warning if it
> notices keywords that are not part of the known list.

I for one (perhaps the only one) would object to that.  There is no
reason to "warn" users about libraries that are not doing anything
even potentially wrong - and that includes using `Keywords' in a
file header that you might never have heard of.

Again.  File headers with keyword `Keywords' have been around for
decades.  `Keywords' is for any keywords whatsoever.  It does not
belong to package.el.

I suggested, and suggest again, that if you are going to take such
a proprietary and controlling attitude toward package.el-recognizable
keywords then package.el should add and use its own file-header
keyword, e.g., `Package Keywords' - and leave the long-existing (and
used) keyword `Keywords' alone.

> All keywords would keep working just the same (even those not on
> the list).

"Warning" users about keywords you don't recognize does not mean
that things "keep working just the same".  Forget about the warning,
if you want to pretend that things will "keep working just the same".

> This warning would be the only change here.

That's one change too much.  Have package.el do it in its own
backyard: `Package Keywords'.  Or don't have it do it at all.

> The intention was to nudge developers towards avoiding useless
> duplicates. 

Duplicates as far as package.el is concerned.  That doesn't mean
duplicates as far as every use/program is concerned.

I have no problem with package.el doing whatever it wants, in its
own territory.  But if it wants to start using existing keywords,
such as `Keywords', then it needs to play well with the others
who are already in the same sandbox.  It can't just move in and
claim the sandbox for its own.

> We wouldn't enforce anything.

The warning is uncalled for, if this is about `Keywords'.  It
can be appropriate, if it is about some new file-header keyword.

> I agree the keywords system should never become restricted, but
> I do think we need to help developers towards standardising it
> a little.

The problem is that you are referring to "the keywords system".
And you are trying to shove it into a particular use case.

File-header fields (keywords) that have been around for quite
some time should be left as is (and this should have included
`Version').  You can add whatever other fields you like, and
make their use as restrictive or helpful with as many warnings
or preventions as you like.  No problem.



  reply	other threads:[~2015-04-20 18:17 UTC|newest]

Thread overview: 37+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-02-15 22:38 Improving browsing and discoverability in the Packages Menu Artur Malabarba
2015-02-16  2:25 ` Stefan Monnier
2015-02-18 16:23 ` raman
2015-04-18 11:26   ` Ted Zlatanov
2015-04-18 13:25     ` Artur Malabarba
2015-04-18 13:25     ` Artur Malabarba
2015-04-18 21:11     ` raman
2015-04-18 23:05       ` Ted Zlatanov
2015-04-19  1:20         ` Alexis
2015-04-19  3:16           ` Stefan Monnier
2015-04-19  4:03             ` Alexis
2015-04-19 16:14             ` raman
2015-04-20  1:57               ` Stefan Monnier
2015-04-19 16:12         ` raman
2015-04-20  1:22           ` Ted Zlatanov
2015-04-20  9:02             ` Artur Malabarba
2015-04-20 15:18             ` raman
2015-04-20 15:23               ` Drew Adams
2015-04-21  0:20                 ` raman
2015-04-21  1:07                   ` Drew Adams
2015-04-19  2:59 ` Eric Abrahamsen
2015-04-19 14:58   ` Drew Adams
2015-04-20  8:38     ` Eric Abrahamsen
2015-04-20  9:26       ` Artur Malabarba
2015-04-20 10:17         ` Alexis
2015-04-20 11:30           ` Artur Malabarba
2015-04-20 14:32             ` Drew Adams
2015-04-20 17:32               ` Artur Malabarba
2015-04-20 18:17                 ` Drew Adams [this message]
2015-04-20 19:49                   ` Artur Malabarba
2015-04-20 20:30                     ` Drew Adams
2015-04-21  4:05                       ` Tassilo Horn
2015-04-21  5:25                         ` Drew Adams
2015-04-21  6:52                           ` Tassilo Horn
2015-04-21  9:04                             ` Eric Abrahamsen
2015-04-21 15:44                             ` Drew Adams
2015-04-20 14:03           ` Eric Abrahamsen

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: https://www.gnu.org/software/emacs/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=ff2f56fe-414e-43ad-be03-750a22abfd77@default \
    --to=drew.adams@oracle.com \
    --cc=bruce.connor.am@gmail.com \
    --cc=emacs-devel@gnu.org \
    --cc=flexibeast@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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).