Re: [PATCH] gnu: Add p7zip.

2016-09-04 Thread Ricardo Wurmus
Hi Kei, > Ricardo Wurmus writes: > >> Kei Kebreau writes: >> >>> Ricardo Wurmus writes: >>> Kei Kebreau writes: > Whoops, last patch was a bit messy and stacked on the previous one. This > patch should be better! I just wanted to push a slightly modified version of

Re: [PATCH] gnu: Add p7zip.

2016-09-02 Thread Kei Kebreau
Ricardo Wurmus writes: > Efraim Flashner writes: > >> On Thu, Sep 01, 2016 at 03:05:34PM -0400, Kei Kebreau wrote: >>> Ricardo Wurmus writes: >>> >>> > Kei Kebreau writes: >>> > >>> >> Ricardo Wurmus writes: >>> >> >>> >>> Kei Kebreau writes: >>> >>> >>> Whoops, last patch was a bit me

Re: [PATCH] gnu: Add p7zip.

2016-09-02 Thread Ricardo Wurmus
Efraim Flashner writes: > On September 2, 2016 2:51:03 PM GMT+03:00, Ricardo Wurmus > wrote: >> >>Efraim Flashner writes: >> >>> On Thu, Sep 01, 2016 at 03:05:34PM -0400, Kei Kebreau wrote: Ricardo Wurmus writes: > Kei Kebreau writes: > >> Ricardo Wurmus writes: >

Re: [PATCH] gnu: Add p7zip.

2016-09-02 Thread Efraim Flashner
On September 2, 2016 2:51:03 PM GMT+03:00, Ricardo Wurmus wrote: > >Efraim Flashner writes: > >> On Thu, Sep 01, 2016 at 03:05:34PM -0400, Kei Kebreau wrote: >>> Ricardo Wurmus writes: >>> >>> > Kei Kebreau writes: >>> > >>> >> Ricardo Wurmus writes: >>> >> >>> >>> Kei Kebreau writes: >>>

Re: [PATCH] gnu: Add p7zip.

2016-09-02 Thread Ricardo Wurmus
Efraim Flashner writes: > On Thu, Sep 01, 2016 at 03:05:34PM -0400, Kei Kebreau wrote: >> Ricardo Wurmus writes: >> >> > Kei Kebreau writes: >> > >> >> Ricardo Wurmus writes: >> >> >> >>> Kei Kebreau writes: >> >>> >> Whoops, last patch was a bit messy and stacked on the previous one.

Re: [PATCH] gnu: Add p7zip.

2016-09-02 Thread Efraim Flashner
On Thu, Sep 01, 2016 at 03:05:34PM -0400, Kei Kebreau wrote: > Ricardo Wurmus writes: > > > Kei Kebreau writes: > > > >> Ricardo Wurmus writes: > >> > >>> Kei Kebreau writes: > >>> > Whoops, last patch was a bit messy and stacked on the previous one. This > patch should be better! >

Re: [PATCH] gnu: Add p7zip.

2016-09-01 Thread Kei Kebreau
ty can try to build from the patch? > > The problem might be with the inline patch in the email. Don’t know. > > ~~ Ricardo I'll try sending it again, for lack of a better plan. If all else fails I can post it on paste.lisp.org. From 1359649b23f1d7f2455797ac71549f00892abe10 Mo

Re: [PATCH] gnu: Add p7zip.

2016-09-01 Thread Ricardo Wurmus
Kei Kebreau writes: > Ricardo Wurmus writes: > >> Kei Kebreau writes: >> >>> Whoops, last patch was a bit messy and stacked on the previous one. This >>> patch should be better! >> >> I just wanted to push a slightly modified version of this (attached) but >> I cannot actually build the packag

Re: [PATCH] gnu: Add p7zip.

2016-09-01 Thread Kei Kebreau
m a clean Guix tree. I don't exactly know how to proceed From here. Perhaps a third party can try to build from the patch? > From 709859c8c8d7d05a523c715f3163e02cec37131b Mon Sep 17 00:00:00 2001 > From: Kei Kebreau > Date: Sat, 27 Aug 2016 06:33:26 -0400 > Subject: [PATCH] gnu: Ad

Re: [PATCH] gnu: Add p7zip.

2016-08-31 Thread Ricardo Wurmus
due to different line endings. Could you please take a look at this again and make sure that the patch to the sources applies? ~~ Ricardo >From 709859c8c8d7d05a523c715f3163e02cec37131b Mon Sep 17 00:00:00 2001 From: Kei Kebreau Date: Sat, 27 Aug 2016 06:33:26 -0400 Subject: [PATCH] gnu: Add p7zi

Re: [PATCH] gnu: Add p7zip.

2016-08-27 Thread Kei Kebreau
t; (or (%current-target-system) >>> + (%current-system))) >>> +`(("nasm" ,nasm))) >>> + (else '() >>> +(home-page "http://p7zip.sourceforge.net/&q

Re: [PATCH] gnu: Add p7zip.

2016-08-22 Thread Kei Kebreau
;nasm" ,nasm))) >> + (else '() >> +(home-page "http://p7zip.sourceforge.net/";) >> + (synopsis "Command-line file archiver with high compression ratio") >> +(description "p7zip is the Unix command-line port of 7-Zip, a fi

Re: [PATCH] gnu: Add p7zip.

2016-08-22 Thread Ricardo Wurmus
Kei Kebreau writes: > Here's the corresponding patch. Maybe you or someone else can double (or > triple?) > check and make sure there are no proprietary files the source after running > "./pre-inst-env guix build -S p7zip." The patch looks good. It’s fine to delete the non-free files in a sni

Re: [PATCH] gnu: Add p7zip.

2016-08-12 Thread Kei Kebreau
f I’m not mistaken this >>> means that deleting the files in a snippet and applying this patch on >>> top of it would be okay. >>> >>> What do others think? >>> >>> ~~ Ricardo >> >> Just to clarify, I'm interpreting what you are

Re: [PATCH] gnu: Add p7zip.

2016-08-09 Thread Ricardo Wurmus
Kei Kebreau writes: > Ricardo Wurmus writes: > >> k...@openmailbox.org writes: >> >>> On 2016-08-09 04:05, Ricardo Wurmus wrote: Leo Famulari writes: >> > I see that you took care of removing non-free parts in a patch and a >> > build phase. Since “guix build -S p7zip” does

Re: [PATCH] gnu: Add p7zip.

2016-08-09 Thread Kei Kebreau
Ricardo Wurmus writes: > k...@openmailbox.org writes: > >> On 2016-08-09 04:05, Ricardo Wurmus wrote: >>> Leo Famulari writes: >>> > > I see that you took care of removing non-free parts in a patch and a > > build phase. Since “guix build -S p7zip” doesn’t run the build phase > > u

Re: [PATCH] gnu: Add p7zip.

2016-08-09 Thread Ricardo Wurmus
k...@openmailbox.org writes: > On 2016-08-09 04:05, Ricardo Wurmus wrote: >> Leo Famulari writes: >> > I see that you took care of removing non-free parts in a patch and a > build phase. Since “guix build -S p7zip” doesn’t run the build phase > users would still end up with a so

Re: [PATCH] gnu: Add p7zip.

2016-08-09 Thread kei
On 2016-08-09 04:05, Ricardo Wurmus wrote: Leo Famulari writes: > I see that you took care of removing non-free parts in a patch and a > build phase. Since “guix build -S p7zip” doesn’t run the build phase > users would still end up with a source archive containing the non-free > parts. The

Re: [PATCH] gnu: Add p7zip.

2016-08-09 Thread Ricardo Wurmus
Leo Famulari writes: >> > I see that you took care of removing non-free parts in a patch and a >> > build phase. Since “guix build -S p7zip” doesn’t run the build phase >> > users would still end up with a source archive containing the non-free >> > parts. The best way is thus to remove things

Re: [PATCH] gnu: Add p7zip.

2016-08-08 Thread Leo Famulari
On Tue, Aug 02, 2016 at 01:38:17PM -0500, Eric Bavier wrote: > On 2016-08-02 02:06, Ricardo Wurmus wrote: > > Hi Kei, > > > > > This patch should install p7zip correctly on every architecture > > > supported by Guix. Ideally, this should be tested it on each arch for > > > the sake of completeness

Re: [PATCH] gnu: Add p7zip.

2016-08-03 Thread Kei Kebreau
Kei Kebreau writes: > Ricardo Wurmus writes: > >> Hi Kei, >> >>> This patch should install p7zip correctly on every architecture >>> supported by Guix. Ideally, this should be tested it on each arch for >>> the sake of completeness and safety. I've got x86_64 covered so far. :-) >> >> thank you

Re: [PATCH] gnu: Add p7zip.

2016-08-02 Thread Eric Bavier
On 2016-08-02 02:06, Ricardo Wurmus wrote: Hi Kei, This patch should install p7zip correctly on every architecture supported by Guix. Ideally, this should be tested it on each arch for the sake of completeness and safety. I've got x86_64 covered so far. :-) thank you for this patch! I see t

Re: [PATCH] gnu: Add p7zip.

2016-08-02 Thread Kei Kebreau
Ricardo Wurmus writes: > Hi Kei, > >> This patch should install p7zip correctly on every architecture >> supported by Guix. Ideally, this should be tested it on each arch for >> the sake of completeness and safety. I've got x86_64 covered so far. :-) > > thank you for this patch! > > I see that y

Re: [PATCH] gnu: Add p7zip.

2016-08-02 Thread Ricardo Wurmus
Hi Kei, > This patch should install p7zip correctly on every architecture > supported by Guix. Ideally, this should be tested it on each arch for > the sake of completeness and safety. I've got x86_64 covered so far. :-) thank you for this patch! I see that you took care of removing non-free pa

[PATCH] gnu: Add p7zip.

2016-08-01 Thread Kei Kebreau
breau Date: Mon, 1 Aug 2016 20:42:59 -0400 Subject: [PATCH] gnu: Add p7zip. * gnu/packages/compression.scm (p7zip): New variable. * gnu/packages/patches/remove-nonfree-p7zip-code.patch: New patch. * gnu/local.mk (dist_patch_DATA): Add it. --- gnu/local.mk