[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Chicken-hackers] [PATCH] install-extension fix for when the second
From: |
Peter Bex |
Subject: |
Re: [Chicken-hackers] [PATCH] install-extension fix for when the second arg contains pairs (SOURCE DEST) |
Date: |
Mon, 19 Nov 2012 21:41:19 +0100 |
User-agent: |
Mutt/1.4.2.3i |
On Mon, Oct 15, 2012 at 06:56:42PM -0400, Mario Domenech Goulart wrote:
> Hi,
>
> The attached patch fixes install-extension for cases when the second
> argument (file list) contains pairs. Example:
>
> (install-extension 'foo '(("orig" "dest")))
>
> Here's a real error:
> http://tests.call-cc.org/master/linux/x86/2012/10/15/salmonella-report/install/crunch.html
>
> Thanks to Peter Bex for identifying the cause and indicating a fix.
Felix didn't look at it anymore (and nobody else did, either) so I've
double-checked that it fixes the crunch issue and went ahead and pushed
this even though I was involved in creating the patch.
If it introduces new problems, feel free to revert.
Cheers,
Peter
--
http://sjamaan.ath.cx
--
"The process of preparing programs for a digital computer
is especially attractive, not only because it can be economically
and scientifically rewarding, but also because it can be an aesthetic
experience much like composing poetry or music."
-- Donald Knuth
- Re: [Chicken-hackers] [PATCH] install-extension fix for when the second arg contains pairs (SOURCE DEST),
Peter Bex <=