all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Andrew Hyatt <ahyatt@gmail.com>
To: Stefan Monnier <monnier@iro.umontreal.ca>
Cc: emacs-devel@gnu.org, Tom Gillespie <tgbugs@gmail.com>
Subject: Re: [ELPA] New package: triples
Date: Wed, 2 Nov 2022 22:32:19 -0400	[thread overview]
Message-ID: <CAM6wYYKoZcFyiBVmAKs+70m5df1uraTe+HeT6XEhQ04xohXi4w@mail.gmail.com> (raw)
In-Reply-To: <jwvv8o52slo.fsf-monnier+emacs@gnu.org>

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

On Thu, Oct 27, 2022 at 11:34 AM Stefan Monnier <monnier@iro.umontreal.ca>
wrote:

> > About the possibility of using one or the other, whichever is available,
> how
> > would that work in terms of package dependencies? I don't think it's
> > possible to have an OR dependency specified in the package-dependencies
> > line in the source, but really only one of emacsql or emacs29 would be
> > required.  If you have any insight into this, it'd be useful.
>
> Package dependencies just tell the package manager to install other
> packages to try and make it easier on users to get a working package.
> But you don't need to have a precise/exact description of the
> real dependencies in the `Package-Requires` (as long as the
> correct byte-compilation of your package actually depends on some other
> package, e.g. if your package uses a macro defined in the other package).
>
> You can just remove the `emacsql` dependency from `Package-Requires` and
> replace it with a runtime check for the presence of either `emacsql` or
> the new sqlite support that emits an error when applicable (with
> a message explaining to the user that they need to have one of those two
> thingies before they can use the package).
> [ Just make sure the error is not emitted just because the file(s) from
>   your package are loaded, but only when some code is actually called.  ]
>

This is now done, you can find the new code that can either use emacs 29
builtin sqlite, or emacsql, on a new branch, at
https://github.com/ahyatt/triples/tree/combined.



>
>
>         Stefan
>
>

[-- Attachment #2: Type: text/html, Size: 2191 bytes --]

  reply	other threads:[~2022-11-03  2:32 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-10-26  0:54 [ELPA] New package: triples Andrew Hyatt
2022-10-26  2:52 ` Stefan Monnier
2022-10-26  5:52   ` Stefan Kangas
2022-10-26 13:23   ` Andrew Hyatt
2022-10-26 13:29     ` Stefan Monnier
2022-10-27  5:15       ` Andrew Hyatt
2022-10-27 15:34         ` Stefan Monnier
2022-11-03  2:32           ` Andrew Hyatt [this message]
2022-10-27  3:36 ` Jean Louis
2022-10-27  5:19   ` Andrew Hyatt
2022-10-27 19:15     ` Jean Louis
2022-10-27 23:33     ` Jean Louis
2022-11-03  2:37       ` Andrew Hyatt
2022-11-03 16:57         ` Jean Louis

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

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

  git send-email \
    --in-reply-to=CAM6wYYKoZcFyiBVmAKs+70m5df1uraTe+HeT6XEhQ04xohXi4w@mail.gmail.com \
    --to=ahyatt@gmail.com \
    --cc=emacs-devel@gnu.org \
    --cc=monnier@iro.umontreal.ca \
    --cc=tgbugs@gmail.com \
    /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 external index

	https://git.savannah.gnu.org/cgit/emacs.git
	https://git.savannah.gnu.org/cgit/emacs/org-mode.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.