all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Maxime Devos <maximedevos@telenet.be>
To: Roman Scherer <roman.scherer@burningswell.com>
Cc: 54581@debbugs.gnu.org
Subject: [bug#54581] Add emacs-sqlite3-api package
Date: Sun, 27 Mar 2022 21:34:23 +0200	[thread overview]
Message-ID: <b93d9994ea7e26499e316c43009623734adfeee6.camel@telenet.be> (raw)
In-Reply-To: <87zglbs1hp.fsf@burningswell.com>

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

Roman Scherer schreef op zo 27-03-2022 om 19:55 [+0200]:
> But I also wonder if generating the consts.c file is actually better
> than just using the checked in consts.c file. The command to generate
> the file fetches a web page from the Sqlite website and builds the
> consts.c file with this information. And I think this is kind of a
> brittle approach. How would this approach would even work with
> Software
> heritage?

As-is, it won't even build, because the build container does not have
network access.  Something you could try, is adding an origin to the
'inputs':

(package
  (name "emacs-sqlite...")
  (inputs `(("web-page" ,(origin [...]
https://sqlite.org/c3ref/constlist.html [...])))))

and do some appropriate substitutions.  Problem is that this page is
not stable ...

To resolve this, you could modify gen-consts.py to not use the web page
to filter things out, basically, delete:

useful_codes = []
with open(sys.argv[1]) as f:
  for l in f.readlines():
    useful_codes.append(l.rstrip())

and

  #print("{0}".format(fields[1]))
  if not fields[1] in useful_codes:
    #print("{0} excluded".format(fields[1]))
    continue

and strip out the curl stuff from gen-consts.sh.

If you do this, don't forget to remove the non-deterministic NOW=$(date
...).

Greetings,
Maxime.

[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 260 bytes --]

  parent reply	other threads:[~2022-03-27 19:35 UTC|newest]

Thread overview: 20+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-03-26 14:09 [bug#54581] Add emacs-sqlite3-api package Roman Scherer
2022-03-26 21:17 ` Maxime Devos
2022-03-26 21:20 ` Maxime Devos
2022-03-26 21:21 ` Maxime Devos
2022-03-26 21:23 ` Maxime Devos
2022-03-26 21:25 ` Maxime Devos
2022-03-27 17:55   ` Roman Scherer
2022-03-27 19:19     ` Maxime Devos
2022-03-27 19:20     ` Maxime Devos
2022-03-27 19:34     ` Maxime Devos [this message]
2022-03-27 19:45     ` Maxime Devos
2022-04-01  9:07       ` Roman Scherer
2022-04-01  9:57         ` Maxime Devos
2022-04-01 10:11           ` Roman Scherer
2022-05-05  7:46             ` Roman Scherer
2022-05-05 10:51               ` Maxime Devos
2022-09-18 12:44             ` bug#54581: " Nicolas Goaziou
2022-09-19  6:46               ` [bug#54581] " Roman Scherer
2022-04-01  9:59         ` Maxime Devos
2022-04-01 10:13           ` Roman Scherer

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=b93d9994ea7e26499e316c43009623734adfeee6.camel@telenet.be \
    --to=maximedevos@telenet.be \
    --cc=54581@debbugs.gnu.org \
    --cc=roman.scherer@burningswell.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/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.