unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
From: "Stephen J. Turnbull" <stephen@xemacs.org>
To: Daniel Hackney <dan@haxney.org>
Cc: emacs-devel@gnu.org
Subject: Re: cl-defstruct-based package.el, now with ert tests!
Date: Thu, 14 Mar 2013 19:53:24 +0900	[thread overview]
Message-ID: <8738vyfe1n.fsf@uwakimon.sk.tsukuba.ac.jp> (raw)
In-Reply-To: <loom.20130314T051456-867@post.gmane.org>

Daniel Hackney writes:
 > Ted Zlatanov <tzz <at> lifelogs.com> writes:
 > > Agreed, maybe Daniel can explain why using `tar-mode' was questionable.
 > > Performance?
 > 
 > The answer is "it seemed yucky." `tar-mode' was clearly not
 > intended to act as an API for other programs looking to read tar
 > files, so using it as such was kind of hacky. Using the external
 > "tar" program seems like the cleaner alternative.

FWIW, nobody has ever complained about XEmacs's assumption that users
have tar and gzip available to install packages.  (In fact, use of an
external ftp program to perform FTP for distribution has caused a lot
of problems over the years due to firewalls and issues with
configuration of passive FTP.  But no problems with tar and gzip, and
not with ftp the program that I can recall.)

 > Since package.el already uses the external tar program for
 > `package-tar-file-info', I figured depending on tar for more
 > substantial stuff was fine. If people really don't want to depend
 > on the external program, I'll remove it and just use `tar-mode'.
 > 
 > Is there any way to gunzip the data in a buffer without using an external
 > program? It would be nice to be able to use tar.gz files.

Is that the right question to ask?  It would not be hard to link in
zlib to do this, but it creates an additional *dependency*.  So I
think the question is "without external dependencies".  I suspect an
elisp implementation of gunzip would be pretty slow and possibly hard
to get right.



  reply	other threads:[~2013-03-14 10:53 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 [this message]
2013-03-14 12:11         ` Ted Zlatanov
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=8738vyfe1n.fsf@uwakimon.sk.tsukuba.ac.jp \
    --to=stephen@xemacs.org \
    --cc=dan@haxney.org \
    --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).