unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
From: Ted Zlatanov <tzz@lifelogs.com>
To: emacs-devel@gnu.org
Subject: Re: cl-defstruct-based package.el, now with ert tests!
Date: Thu, 14 Mar 2013 08:11:43 -0400	[thread overview]
Message-ID: <87hakecha8.fsf@lifelogs.com> (raw)
In-Reply-To: loom.20130314T051456-867@post.gmane.org

On Thu, 14 Mar 2013 04:36:36 +0000 (UTC) Daniel Hackney <dan@haxney.org> wrote: 

DH> Ted Zlatanov <tzz <at> lifelogs.com> writes:
>> Agreed, maybe Daniel can explain why using `tar-mode' was questionable.
>> Performance?

DH> The answer is "it seemed yucky." `tar-mode' was clearly not intended to act as
DH> an API for other programs looking to read tar files, so using it as such was
DH> kind of hacky. Using the external "tar" program seems like the cleaner
DH> alternative. Since package.el already uses the external tar program for
DH> `package-tar-file-info', I figured depending on tar for more substantial stuff
DH> was fine. If people really don't want to depend on the external program, I'll
DH> remove it and just use `tar-mode'.

I'm OK with the change if it doesn't break any existing W32 support.
Otherwise we're leaving many Emacs users unable to use package.el.

`tar-mode' seems pretty OK as far as the data extraction.  So maybe we
just need to pull the data functions out into a utility library?

Either way, I'll wait for this to happen before working on the ELPA
security (signing packages).  So if I can help, let me know.

DH> Is there any way to gunzip the data in a buffer without using an external
DH> program? It would be nice to be able to use tar.gz files.

Sorry, I don't know, but `ldd emacs' says libz is already available at
least on GNU/Linux.  So the C functions should be there.

Ted




  parent reply	other threads:[~2013-03-14 12:11 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-03-12  1:03 cl-defstruct-based package.el, now with ert tests! Daniel Hackney
2013-03-12  2:01 ` Tom Tromey
2013-03-12 12:03   ` Dmitry Gutov
2013-03-12 18:13     ` Ted Zlatanov
2013-03-14  4:36       ` Daniel Hackney
2013-03-14 10:53         ` Stephen J. Turnbull
2013-03-14 12:11         ` Ted Zlatanov [this message]
2013-03-14 13:49         ` Stefan Monnier
2013-03-14 17:53           ` chad
2013-03-15  4:00           ` Daniel Hackney
2013-03-15 12:56             ` Stefan Monnier
2013-03-15 13:46               ` Eli Zaretskii
2013-03-20 18:01             ` Ted Zlatanov
2013-03-12 16:25   ` Daniel Hackney

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=87hakecha8.fsf@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 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).