Re: [PATCH 2/2] gnu: Add emacs-org-trello.

2016-10-28 Thread Roel Janssen
Ricardo Wurmus writes: > Roel Janssen writes: > >> So, the only proper way to do it, is to manually add a test phase in >> each package definition. But then we still have to add a lot of >> dependencies that are only needed to run the tests, and even then the >> tests don't say much.. In the c

Re: [PATCH 2/2] gnu: Add emacs-org-trello.

2016-10-28 Thread Ricardo Wurmus
Roel Janssen writes: > So, the only proper way to do it, is to manually add a test phase in > each package definition. But then we still have to add a lot of > dependencies that are only needed to run the tests, and even then the > tests don't say much.. In the case of this package, the test f

Re: [PATCH 2/2] gnu: Add emacs-org-trello.

2016-10-28 Thread Federico Beffa
On Fri, Oct 28, 2016 at 11:30 AM, Alex Kost wrote: > Ricardo Wurmus (2016-10-27 20:48 +0200) wrote: > >> Alex Kost writes: >> @Alex: could you please confirm this? Should these inputs be mentioned at all? If so, is it sufficient to add them to native-inputs? >>> >>> As for me, I don't

Re: [PATCH 2/2] gnu: Add emacs-org-trello.

2016-10-28 Thread Roel Janssen
Alex Kost writes: > Ricardo Wurmus (2016-10-27 20:48 +0200) wrote: > >> Alex Kost writes: >> @Alex: could you please confirm this? Should these inputs be mentioned at all? If so, is it sufficient to add them to native-inputs? >>> >>> As for me, I don't see a reason to add these addit

Re: [PATCH 2/2] gnu: Add emacs-org-trello.

2016-10-28 Thread Alex Kost
Ricardo Wurmus (2016-10-27 20:48 +0200) wrote: > Alex Kost writes: > >>> @Alex: could you please confirm this? Should these inputs be mentioned >>> at all? If so, is it sufficient to add them to native-inputs? >> >> As for me, I don't see a reason to add these additional inputs, >> emacs-build-

Re: [PATCH 2/2] gnu: Add emacs-org-trello.

2016-10-27 Thread Alex Kost
Ricardo Wurmus (2016-10-27 16:09 +0200) wrote: > Roel Janssen writes: [...] >> +(define-public emacs-org-trello >> + (package >> +(name "emacs-org-trello") >> +(version "0.7.9") >> +(source (origin >> + (method url-fetch) >> + (uri (string-append >> +

Re: [PATCH 2/2] gnu: Add emacs-org-trello.

2016-10-27 Thread Ricardo Wurmus
Alex Kost writes: >> @Alex: could you please confirm this? Should these inputs be mentioned >> at all? If so, is it sufficient to add them to native-inputs? > > As for me, I don't see a reason to add these additional inputs, > emacs-build-system will not run tests anyway. Thank you for clarif

Re: [PATCH 2/2] gnu: Add emacs-org-trello.

2016-10-27 Thread Ricardo Wurmus
Roel Janssen writes: >> The cask file says: >> >> (development >> (depends-on "dash-functional") >> (depends-on "dash") > > @code{dash} provides @code{dash-functional}. > >> (depends-on "undercover") > > This one is needed to run the tests. > >> (depends-on "ert-runner") >> (depends-on "ert

Re: [PATCH 2/2] gnu: Add emacs-org-trello.

2016-10-27 Thread Roel Janssen
Ricardo Wurmus writes: > Hi Roel, > > funny, this has also been on my pile of unsubmitted, bit-rotting patches > (because I must use Trello at work and I much rather use org-mode). I > remember that this wasn’t straight-forward to package and required a lot > more (build-time) inputs. We seem t

Re: [PATCH 2/2] gnu: Add emacs-org-trello.

2016-10-25 Thread Ricardo Wurmus
Hi Roel, funny, this has also been on my pile of unsubmitted, bit-rotting patches (because I must use Trello at work and I much rather use org-mode). I remember that this wasn’t straight-forward to package and required a lot more (build-time) inputs. The cask file says: (development (depends-