unofficial mirror of guix-patches@gnu.org 
 help / color / mirror / code / Atom feed
From: Pierre Neidhardt <mail@ambrevar.xyz>
To: Guillaume Le Vaillant <glv@posteo.net>
Cc: 37817@debbugs.gnu.org
Subject: [bug#37817] [PATCH 1/7] gnu: Add cl-heap., [bug#37817] [PATCH 2/7] gnu: Add curry-compose-reader-macros., [bug#37817] [PATCH 3/7] gnu: Add yason., [bug#37817] [PATCH 4/7] gnu: sbcl-iterate: Add missing native input., [bug#37817] [PATCH 5/7] gnu: Add ecl-iterate., [bug#37817] [PATCH 6/7] gnu: Add stefil., [bug#37817] [PATCH 7/7] gnu: Add graph.
Date: Mon, 21 Oct 2019 12:16:30 +0200	[thread overview]
Message-ID: <8736fmbdz5.fsf@ambrevar.xyz> (raw)
In-Reply-To: <875zkibehy.fsf@yamatai>

[-- Attachment #1: Type: text/plain, Size: 725 bytes --]

Guillaume Le Vaillant <glv@posteo.net> writes:

> Also, I saw that there is a 'graph.scm' file with definitions of graph
> related packages. Should I put 'sbcl-graph', 'sbcl-graph-dot' and
> 'sbcl-graph-json' in it, or should I keep them in 'lisp.scm'?

In my understanding, file separation works as follows:

1. Make sure the file compilation graph is as simple as possible.

2. If 1. does not apply (e.g. same complexity is both cases), place the
package where it's most relevant.

3. If neither 1 nor 2 apply, that is, if a package is relevant in both
files, well, do what suits you best :)

Here my intuition would be to leave them in lisp.scm.

Cheers!

-- 
Pierre Neidhardt
https://ambrevar.xyz/

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 487 bytes --]

  reply	other threads:[~2019-10-21 10:17 UTC|newest]

Thread overview: 21+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-10-18 20:38 [bug#37817] [PATCH 0/7] Add graph Common Lisp library and dependencies Guillaume Le Vaillant
2019-10-18 20:42 ` [bug#37817] [PATCH 1/7] gnu: Add cl-heap Guillaume Le Vaillant
2019-10-18 20:42   ` [bug#37817] [PATCH 2/7] gnu: Add curry-compose-reader-macros Guillaume Le Vaillant
2019-10-18 20:42   ` [bug#37817] [PATCH 3/7] gnu: Add yason Guillaume Le Vaillant
2019-10-18 20:42   ` [bug#37817] [PATCH 4/7] gnu: sbcl-iterate: Add missing native input Guillaume Le Vaillant
2019-10-18 20:42   ` [bug#37817] [PATCH 5/7] gnu: Add ecl-iterate Guillaume Le Vaillant
2019-10-18 20:42   ` [bug#37817] [PATCH 6/7] gnu: Add stefil Guillaume Le Vaillant
2019-10-18 20:42   ` [bug#37817] [PATCH 7/7] gnu: Add graph Guillaume Le Vaillant
2019-10-21  9:16   ` [bug#37817] [PATCH 1/7] gnu: Add cl-heap., [bug#37817] [PATCH 2/7] gnu: Add curry-compose-reader-macros., [bug#37817] [PATCH 3/7] gnu: Add yason., [bug#37817] [PATCH 4/7] gnu: sbcl-iterate: Add missing native input., [bug#37817] [PATCH 5/7] gnu: Add ecl-iterate., [bug#37817] [PATCH 6/7] gnu: Add stefil., " Pierre Neidhardt
2019-10-21 10:05     ` Guillaume Le Vaillant
2019-10-21 10:16       ` Pierre Neidhardt [this message]
2019-10-21 12:02         ` [bug#37817] [PATCH v2 1/9] gnu: Add cl-heap Guillaume Le Vaillant
2019-10-21 12:02           ` [bug#37817] [PATCH v2 2/9] gnu: Add curry-compose-reader-macros Guillaume Le Vaillant
2019-10-21 12:02           ` [bug#37817] [PATCH v2 3/9] gnu: Add yason Guillaume Le Vaillant
2019-10-21 12:02           ` [bug#37817] [PATCH v2 4/9] gnu: sbcl-iterate: Add missing native input Guillaume Le Vaillant
2019-10-21 12:02           ` [bug#37817] [PATCH v2 5/9] gnu: Add ecl-iterate Guillaume Le Vaillant
2019-10-21 12:03           ` [bug#37817] [PATCH v2 6/9] gnu: Add stefil Guillaume Le Vaillant
2019-10-21 12:03           ` [bug#37817] [PATCH v2 7/9] gnu: Add graph Guillaume Le Vaillant
2019-10-21 12:03           ` [bug#37817] [PATCH v2 8/9] gnu: Add sbcl-graph-dot Guillaume Le Vaillant
2019-10-21 12:03           ` [bug#37817] [PATCH v2 9/9] gnu: Add sbcl-graph-json Guillaume Le Vaillant
2019-10-22  8:50         ` [bug#37817] [PATCH 1/7] gnu: Add cl-heap., [bug#37817] [PATCH 2/7] gnu: Add curry-compose-reader-macros., [bug#37817] [PATCH 3/7] gnu: Add yason., [bug#37817] [PATCH 4/7] gnu: sbcl-iterate: Add missing native input., [bug#37817] [PATCH 5/7] gnu: Add ecl-iterate., [bug#37817] [PATCH 6/7] gnu: Add stefil., [bug#37817] [PATCH 7/7] gnu: Add graph Pierre Neidhardt

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=8736fmbdz5.fsf@ambrevar.xyz \
    --to=mail@ambrevar.xyz \
    --cc=37817@debbugs.gnu.org \
    --cc=glv@posteo.net \
    /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).