unofficial mirror of guix-devel@gnu.org 
 help / color / mirror / code / Atom feed
From: Alex Kost <alezost@gmail.com>
To: Catonano <catonano@gmail.com>
Cc: guix-devel@gnu.org
Subject: Re: [PATCH] Emacs Smartparens
Date: Wed, 18 May 2016 12:12:43 +0300	[thread overview]
Message-ID: <87futfn304.fsf@gmail.com> (raw)
In-Reply-To: <CAJ98PDyOD==L5A+JGhrJnqmrOCge8qEcG1oasV-8DdQkDEK8_Q@mail.gmail.com> (catonano@gmail.com's message of "Wed, 18 May 2016 00:44:19 +0200")

Catonano (2016-05-18 01:44 +0300) wrote:

[...]
> Smartparens uses Cask. Cask is a language specific project management
> tool.
>
> Exactly the kind of thing that Guix aims at (right ?)
>
> As far as I understand, Cask provides, among other things, some unit
> tests running facility
>
> There's a target in a makefile piloting Cask to run the test. So I
> understand. And there's a folder filled with .el files containing
> tests to be run.
>
> So, the makefile, the Cask file, Make, Cask and the tests folder are
> dependencies ONLY in development.
>
> So maybe there should be 2 outputs for this project, one vanilla
> output and one for development.

I think there is no need to do this.

> It gets worse
>
> There's a specific set of dependencies for every Smartparens submode
> (some languages have their own submode) so there should be an output
> for each of those

I think that a single output is absolutely OK: all these files (like
"smartparens-haskell.el") are a part of the whole smartparens package;
moreover there is no any "specific set of dependencies".  The only
dependency for smartparens is the "dash" library.

> And finally there's also a .travis.yml file, an images folder with
> some gif files in it, an org-mode file and some files ending with
> .feature
>
> All this seems an excess to me. A developer will be able to set up an
> environment with Make, Cask and other ad hoc dependencies and the
> files will be present in the package anyway, even if not used.
>
> It's a few kilobytes anyway.

I personally wouldn't care about cask, travis.yml and other files, and
would just create a plain package using emacs-build-system.  It will
provide the same functioning package as the one you would install with
Emacs (using "M-x list-packages").  I don't see a reason to deal with
all the additional complexity you mentioned.

-- 
Alex

  reply	other threads:[~2016-05-18  9:12 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-05-16 13:32 [PATCH] Emacs Smartparens Catonano
2016-05-16 13:58 ` Catonano
2016-05-16 14:35 ` Alex Kost
2016-05-17 22:44   ` Catonano
2016-05-18  9:12     ` Alex Kost [this message]
2016-05-18  9:24       ` Catonano
2016-05-19  8:54         ` Alex Kost

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://guix.gnu.org/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=87futfn304.fsf@gmail.com \
    --to=alezost@gmail.com \
    --cc=catonano@gmail.com \
    --cc=guix-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/guix.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).