all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* Re: 05/05: gnu: Add ghc-hspec-discover.
       [not found] ` <20200208144032.CCDFC21049@vcs0.savannah.gnu.org>
@ 2020-02-09  5:38   ` Timothy Sample
  2020-02-09  9:08     ` Nicolas Goaziou
  0 siblings, 1 reply; 4+ messages in thread
From: Timothy Sample @ 2020-02-09  5:38 UTC (permalink / raw)
  To: Nicolas Goaziou; +Cc: guix-devel

Hi Nicolas,

guix-commits@gnu.org writes:

> ngz pushed a commit to branch master
> in repository guix.
>
> commit a60448edcc85faa8b3cd4bd3ae8a336c842aad4d
> Author: Nicolas Goaziou <mail@nicolasgoaziou.fr>
> AuthorDate: Sat Feb 8 15:17:15 2020 +0100
>
>     gnu: Add ghc-hspec-discover.
>     
>     * gnu/packages/haskell-check.scm (ghc-hspec-discover): New variable.
> ---

We already have “hspec-discover” without the “ghc” prefix.  The two
packages look identical to me – is this an unintentional copy?


-- Tim

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: 05/05: gnu: Add ghc-hspec-discover.
  2020-02-09  5:38   ` 05/05: gnu: Add ghc-hspec-discover Timothy Sample
@ 2020-02-09  9:08     ` Nicolas Goaziou
  2020-02-09 16:25       ` Timothy Sample
  0 siblings, 1 reply; 4+ messages in thread
From: Nicolas Goaziou @ 2020-02-09  9:08 UTC (permalink / raw)
  To: Timothy Sample; +Cc: guix-devel

Hello,

Timothy Sample <samplet@ngyro.com> writes:

> We already have “hspec-discover” without the “ghc” prefix.  The two
> packages look identical to me – is this an unintentional copy?

Oh! I hadn't noticed!

Well, is there any strong reason to use "hspec-discover" over
"ghc-hspec-discover", besides that it already exists?

For the arguments in favor of using ghc-hspec-discover:
- every other "hspec" package uses the "ghc-" prefix,
- the hackage importer names it,
- the hackage importer will add "ghc-hspec-discover" as an input anyway.

So what about deprecating "hspec-discover" in favor of
"ghc-hspec-discover"?

Thanks for the heads up!

Regards,

-- 
Nicolas Goaziou

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: 05/05: gnu: Add ghc-hspec-discover.
  2020-02-09  9:08     ` Nicolas Goaziou
@ 2020-02-09 16:25       ` Timothy Sample
  2020-02-09 16:36         ` Nicolas Goaziou
  0 siblings, 1 reply; 4+ messages in thread
From: Timothy Sample @ 2020-02-09 16:25 UTC (permalink / raw)
  To: Nicolas Goaziou; +Cc: guix-devel

Hi,

Nicolas Goaziou <mail@nicolasgoaziou.fr> writes:

>
> Timothy Sample <samplet@ngyro.com> writes:
>
>> We already have “hspec-discover” without the “ghc” prefix.  The two
>> packages look identical to me – is this an unintentional copy?
>
> Oh! I hadn't noticed!
>
> Well, is there any strong reason to use "hspec-discover" over
> "ghc-hspec-discover", besides that it already exists?

The loose convention is that libraries have the prefix, and programs
don’t.  The “hspec-discover” package was added before my time, but I
guess whoever added it was following that convention.

> For the arguments in favor of using ghc-hspec-discover:
> - every other "hspec" package uses the "ghc-" prefix,
> - the hackage importer names it,
> - the hackage importer will add "ghc-hspec-discover" as an input anyway.

That is definitely a problem.  From my experience, “hspec-discover” is
often missed by the importer because it is put in “build-tool-depends”
instead of “build-depends” in the Cabal file.  Ideally, the importer
would know to add it from “build-tools-discover” and it would know what
it should be named.

> So what about deprecating "hspec-discover" in favor of
> "ghc-hspec-discover"?

I did a quick scan of the “text-conversions” source code, and it doesn’t
seem to use “hspec-discover” as a library.  It looks like it’s just a
mistake in the Cabal file.

Between the convention and the fact that “hspec-discover” is almost
always used as a program instead of a library, I would say that it’s
okay the way it is.  Beyond that, it would be quite a bit of churn to
change it with very little benefit.

> Thanks for the heads up!

No problem.  Thanks for committing packages!


-- Tim

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: 05/05: gnu: Add ghc-hspec-discover.
  2020-02-09 16:25       ` Timothy Sample
@ 2020-02-09 16:36         ` Nicolas Goaziou
  0 siblings, 0 replies; 4+ messages in thread
From: Nicolas Goaziou @ 2020-02-09 16:36 UTC (permalink / raw)
  To: Timothy Sample; +Cc: guix-devel

Timothy Sample <samplet@ngyro.com> writes:

> Between the convention and the fact that “hspec-discover” is almost
> always used as a program instead of a library, I would say that it’s
> okay the way it is.  Beyond that, it would be quite a bit of churn to
> change it with very little benefit.

Fair enough. I reverted the patch.

Thanks again.

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2020-02-09 16:36 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
     [not found] <20200208144029.3072.2814@vcs0.savannah.gnu.org>
     [not found] ` <20200208144032.CCDFC21049@vcs0.savannah.gnu.org>
2020-02-09  5:38   ` 05/05: gnu: Add ghc-hspec-discover Timothy Sample
2020-02-09  9:08     ` Nicolas Goaziou
2020-02-09 16:25       ` Timothy Sample
2020-02-09 16:36         ` Nicolas Goaziou

Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/guix.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.