all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Ted Zlatanov <tzz@lifelogs.com>
To: emacs-devel@gnu.org
Subject: Re: [PATCH] package.el: check tarball signature
Date: Wed, 02 Oct 2013 09:53:54 -0400	[thread overview]
Message-ID: <87d2nnvkwd.fsf@flea.lifelogs.com> (raw)
In-Reply-To: 877gdvamle.fsf-ueno@gnu.org

On Wed, 02 Oct 2013 21:22:53 +0900 Daiki Ueno <ueno@gnu.org> wrote: 

DU> Ted Zlatanov <tzz@lifelogs.com> writes:
DU> For what purpose would you need signature generation?
>> 
>> So the maintainer can create a signature from Emacs instead of
>> externally.  The signer is intended to be a maintainer after review, not
>> a package creator.

DU> I'm fine with signing with dput for Debian and gnupload for GNU, who
DU> else of you really wants that feature.  Reference?

I want it.

If we move to a branch-pull request-merge model, this will be much less
important since the signing will happen at the time of the merge on the
server; the reviewer never needs to manually sign anything.  But at
least for now we need interactive tools to automate that process and
gnupload would certainly fill that need.  So please don't dwell on this.

>> It's something you would run on the ELPA server, not at upload time.

DU> I'd rather use other scripting language to do such a batch job.

OK, I think there's room for both views.  Let's assume I will implement
it if I need it, and it shouldn't stop you.  Note I didn't mention it in
my "wishlist" for your v2 patch, so I don't consider it essential like
per-archive signing.

>> package.el is not just an installer UI, it's a full package manager.

DU> Why the uploading part is separated into package-x.el then?

Good point, I think you're right.  Thanks for the digging.  If I add
signing from Emacs I'll put it in package-x.el.

DU> I'm sorry, I couldn't find anything I can reuse in your patch.  It even
DU> succeeds signature verification when GPG reports bad signatures.
>> 
>> That's one of the EPG-related pieces I mentioned need fixing.  But at
>> this point your v2 patch has done the work so there's no point in arguing.

DU> Thanks for understanding.  I should have been involved in this earlier.
DU> What I'm really surprised is no progress on this for almost one
DU> year.

Yes, I know.  I was part of the problem: extremely busy with work and
"almost done" all the time.  Let's make an effort together and get it
done now.  I think it's an important part of Emacs' future.

DU> Also, why did you choose ".gpgsig" extension rather than ".sig",
DU> which has already been used on ftp.gnu.org for a decade?
>> 
>> I think the extension name is not that important, but here specifically
>> I wanted to indicate it's generated by GPG.  .sig will obviously work
>> exactly the same way.

DU> It's important, if we would like to use common tools like gnupload too.

OK with me, please consider me in favor of .sig.

Ted




  reply	other threads:[~2013-10-02 13:53 UTC|newest]

Thread overview: 32+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-09-30 19:48 [PATCH] package.el: check tarball signature Daiki Ueno
2013-09-30 19:58 ` Eli Zaretskii
2013-10-02  6:20   ` [PATCHv2] " Daiki Ueno
2013-10-02 10:43     ` Ted Zlatanov
2013-09-30 21:54 ` [PATCH] " Ted Zlatanov
2013-09-30 22:56   ` Stefan Monnier
2013-10-02 11:17     ` Ted Zlatanov
2013-10-02  7:16   ` Daiki Ueno
2013-10-02 10:41     ` Ted Zlatanov
2013-10-02 12:22       ` Daiki Ueno
2013-10-02 13:53         ` Ted Zlatanov [this message]
2013-10-03  3:51           ` Stefan Monnier
2013-10-02 13:15     ` Thien-Thi Nguyen
2013-10-03  3:45       ` Stefan Monnier
2013-10-03  3:52     ` Stefan Monnier
2013-10-03  7:18       ` Daiki Ueno
2013-10-03 14:19         ` Ted Zlatanov
2013-10-03 15:01           ` Stefan Monnier
2013-10-04 19:23             ` Eli Zaretskii
2013-10-04 21:14               ` Ted Zlatanov
2013-10-05  0:34                 ` Daiki Ueno
2013-10-05  5:40                   ` Stephen J. Turnbull
2013-10-05 10:03                     ` Ted Zlatanov
2013-10-05 15:07                       ` Stephen J. Turnbull
2013-10-05 21:51                         ` Ted Zlatanov
2013-10-05  9:57                   ` Ted Zlatanov
2013-10-05  7:09                 ` Eli Zaretskii
2013-10-05 10:11                   ` Ted Zlatanov
2013-10-05 12:37                     ` Eli Zaretskii
2013-10-05 13:53                       ` Stefan Monnier
2013-10-04  2:46           ` Daiki Ueno
2013-10-04 16:19             ` Ted Zlatanov

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

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

  git send-email \
    --in-reply-to=87d2nnvkwd.fsf@flea.lifelogs.com \
    --to=tzz@lifelogs.com \
    --cc=emacs-devel@gnu.org \
    /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 external index

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

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.