Re: [PATCH 3/5] build: Add 'emacs-build-system'

2015-07-15 Thread Ludovic Courtès
Alex Kost skribis: > So I suggest to add a phase for deleting non-".el[c]" files from the > ".guix.d/package" directory. Agreed; what about putting READMEs etc. elsewhere, like under ‘share/doc/PACKAGE’ if we want to preserve them? > And just in case: I have nothing against GNU ELPA repository

Re: [PATCH 3/5] build: Add 'emacs-build-system'

2015-07-10 Thread Federico Beffa
On Fri, Jul 10, 2015 at 8:47 AM, Alex Kost wrote: > Federico Beffa (2015-07-09 23:41 +0300) wrote: > >> On Thu, Jul 9, 2015 at 10:51 AM, Alex Kost wrote: >>> Federico Beffa (2015-07-08 23:22 +0300) wrote: >>> On Tue, Jul 7, 2015 at 6:58 PM, Alex Kost wrote: > A side note: I think genera

Re: [PATCH 3/5] build: Add 'emacs-build-system'

2015-07-09 Thread Alex Kost
Federico Beffa (2015-07-09 23:41 +0300) wrote: > On Thu, Jul 9, 2015 at 10:51 AM, Alex Kost wrote: >> Federico Beffa (2015-07-08 23:22 +0300) wrote: >> >>> On Tue, Jul 7, 2015 at 6:58 PM, Alex Kost wrote: A side note: I think generally it would be preferable to use an upstream release

Re: [PATCH 3/5] build: Add 'emacs-build-system'

2015-07-09 Thread Federico Beffa
On Thu, Jul 9, 2015 at 10:51 AM, Alex Kost wrote: > Federico Beffa (2015-07-08 23:22 +0300) wrote: > >> On Tue, Jul 7, 2015 at 6:58 PM, Alex Kost wrote: >>> A side note: I think generally it would be preferable to use an upstream >>> release in the package recipe rather than to use a melpa(-stabl

Re: [PATCH 3/5] build: Add 'emacs-build-system'

2015-07-09 Thread Alex Kost
Federico Beffa (2015-07-08 23:22 +0300) wrote: > On Tue, Jul 7, 2015 at 6:58 PM, Alex Kost wrote: >> A side note: I think generally it would be preferable to use an upstream >> release in the package recipe rather than to use a melpa(-stable) URL, >> i.e.: >> >> http://foo-upstream.org/foo-0.1.

Re: [PATCH 3/5] build: Add 'emacs-build-system'

2015-07-08 Thread Federico Beffa
On Tue, Jul 7, 2015 at 6:58 PM, Alex Kost wrote: > A side note: I think generally it would be preferable to use an upstream > release in the package recipe rather than to use a melpa(-stable) URL, > i.e.: > > http://foo-upstream.org/foo-0.1.tar.gz instead of > http://stable.melpa.org/packages

Re: [PATCH 3/5] build: Add 'emacs-build-system'

2015-07-07 Thread Alex Kost
Federico Beffa (2015-07-07 10:21 +0300) wrote: > Alex Kost writes: > >> Wouldn't it be better to name it "elpa-build-system" as that's what it >> is in my opinion, since it does not retrieve the source directly from >> upstream but from ELPA/MELPA/... "repositories" instead. > > The build system

Re: [PATCH 3/5] build: Add 'emacs-build-system'

2015-07-07 Thread Federico Beffa
Alex Kost writes: > Wouldn't it be better to name it "elpa-build-system" as that's what it > is in my opinion, since it does not retrieve the source directly from > upstream but from ELPA/MELPA/... "repositories" instead. The build system doesn't assume anything about the source location. > > I

Re: [PATCH 3/5] build: Add 'emacs-build-system'

2015-07-06 Thread Alex Kost
Wouldn't it be better to name it "elpa-build-system" as that's what it is in my opinion, since it does not retrieve the source directly from upstream but from ELPA/MELPA/... "repositories" instead. I imagine there may appear another build system for simple emacs packages (that don't provide Makefi

Re: [PATCH 3/5] build: Add 'emacs-build-system'

2015-06-27 Thread Ludovic Courtès
'match-lambda' appears not to be documented in Guile.) Oops. > From e24d0b11280f4fcd106f371b98b7481f7c044eb0 Mon Sep 17 00:00:00 2001 > From: Federico Beffa > Date: Sun, 21 Jun 2015 10:10:05 +0200 > Subject: [PATCH 3/5] build: Add 'emacs-build-system'. > > *

Re: [PATCH 3/5] build: Add 'emacs-build-system'

2015-06-25 Thread Federico Beffa
On Thu, Jun 25, 2015 at 1:57 PM, Ludovic Courtès wrote: > Federico Beffa skribis: > >> On Mon, Jun 22, 2015 at 9:40 PM, Thompson, David >> wrote: >>> On Mon, Jun 22, 2015 at 3:33 PM, Federico Beffa wrote: But, my question was NOT: how can I see white spaces. Rather: is there a Gu

Re: [PATCH 3/5] build: Add 'emacs-build-system'

2015-06-25 Thread Federico Beffa
> > (match-lambda > ((label . directory) >(emacs-package? (package-name+version directory > > (Which means the ‘first’ above should have been ‘second’?) I'm not sure I understand your comment: 'package-name->name+version' takes a package name, therefor

Re: [PATCH 3/5] build: Add 'emacs-build-system'

2015-06-25 Thread Ludovic Courtès
Federico Beffa skribis: > From 725d42eb3e5c44bcec1bd81988d1f952e6be02a4 Mon Sep 17 00:00:00 2001 > From: Federico Beffa > Date: Sun, 21 Jun 2015 10:10:05 +0200 > Subject: [PATCH 3/5] build: Add 'emacs-build-system'. > > * Makefile.am (MODULES): Add 'guix/buil

Re: [PATCH 3/5] build: Add 'emacs-build-system'

2015-06-25 Thread Ludovic Courtès
Federico Beffa skribis: > On Mon, Jun 22, 2015 at 9:40 PM, Thompson, David > wrote: >> On Mon, Jun 22, 2015 at 3:33 PM, Federico Beffa wrote: >>> >>> But, my question was NOT: how can I see white spaces. Rather: is there >>> a Guix coding style "rule" which states that white spaces there are >>

Re: [PATCH 3/5] build: Add 'emacs-build-system'

2015-06-24 Thread Federico Beffa
ward-paragraph' (if that's what is > bound to M-up/down for you). > > -- > Alex From 725d42eb3e5c44bcec1bd81988d1f952e6be02a4 Mon Sep 17 00:00:00 2001 From: Federico Beffa Date: Sun, 21 Jun 2015 10:10:05 +0200 Subject: [PATCH 3/5] build: Add 'emacs-build-syste

Re: [PATCH 3/5] build: Add 'emacs-build-system'

2015-06-23 Thread Alex Kost
Federico Beffa (2015-06-22 22:33 +0300) wrote: > On Mon, Jun 22, 2015 at 7:59 PM, Alex Kost wrote: > >> Well, these trailing spaces are artifacts of (inaccurate) coding. Emacs >> can't read user's mind to decide if the spaces are redundant or >> intended. > > No need to read the mind... you just

Re: [PATCH 3/5] build: Add 'emacs-build-system'

2015-06-22 Thread Federico Beffa
On Mon, Jun 22, 2015 at 9:40 PM, Thompson, David wrote: > On Mon, Jun 22, 2015 at 3:33 PM, Federico Beffa wrote: >> >> But, my question was NOT: how can I see white spaces. Rather: is there >> a Guix coding style "rule" which states that white spaces there are >> undesired. >> >> I personally pre

Re: [PATCH 3/5] build: Add 'emacs-build-system'

2015-06-22 Thread Thompson, David
On Mon, Jun 22, 2015 at 3:33 PM, Federico Beffa wrote: > > But, my question was NOT: how can I see white spaces. Rather: is there > a Guix coding style "rule" which states that white spaces there are > undesired. > > I personally prefer to have them, because then, if I use M-up/down, I > move to t

Re: [PATCH 3/5] build: Add 'emacs-build-system'

2015-06-22 Thread Federico Beffa
On Mon, Jun 22, 2015 at 7:59 PM, Alex Kost wrote: > Federico Beffa (2015-06-22 11:51 +0300) wrote: > >> On Sun, Jun 21, 2015 at 10:40 PM, Alex Kost wrote: new file mode 100644 index 000..da2b594 --- /dev/null +++ b/guix/build-system/emacs.scm >>> [...] +

Re: [PATCH 3/5] build: Add 'emacs-build-system'

2015-06-22 Thread Alex Kost
Federico Beffa (2015-06-22 11:51 +0300) wrote: > On Sun, Jun 21, 2015 at 10:40 PM, Alex Kost wrote: >>> new file mode 100644 >>> index 000..da2b594 >>> --- /dev/null >>> +++ b/guix/build-system/emacs.scm >> [...] >>> +#:tests? ,tests? >>> +#:phases ,pha

Re: [PATCH 3/5] build: Add 'emacs-build-system'

2015-06-22 Thread Mathieu Lirzin
Federico Beffa writes: > I'm following the Emacs behavior. Is there a convention to suppress > all spaces (in spite of what Emacs does)? 'M-x delete-trailing-whitespace' ? -- Mathieu Lirzin

Re: [PATCH 3/5] build: Add 'emacs-build-system'

2015-06-22 Thread Federico Beffa
On Sun, Jun 21, 2015 at 10:40 PM, Alex Kost wrote: >> new file mode 100644 >> index 000..da2b594 >> --- /dev/null >> +++ b/guix/build-system/emacs.scm > [...] >> +#:tests? ,tests? >> +#:phases ,phases >> +#:outputs %outputs >> +

Re: [PATCH 3/5] build: Add 'emacs-build-system'

2015-06-21 Thread Alex Kost
Federico Beffa (2015-06-21 11:31 +0300) wrote: [...] > diff --git a/doc/guix.texi b/doc/guix.texi > index 3ca105a..00fe5bb 100644 > --- a/doc/guix.texi > +++ b/doc/guix.texi > @@ -2404,6 +2404,17 @@ Which Haskell compiler is used can be specified with > the @code{#:haskell} > parameter which def

[PATCH 3/5] build: Add 'emacs-build-system'

2015-06-21 Thread Federico Beffa
A new 'emacs-buld-system'. Fede From 9c090ddd7de011b369208cdeb28eb3eaa5dc6011 Mon Sep 17 00:00:00 2001 From: Federico Beffa Date: Sun, 21 Jun 2015 10:10:05 +0200 Subject: [PATCH 3/5] build: Add 'emacs-build-system'. * Makefile.am (MODULES): Add 'guix/build-system/ema