GNU bug report logs -
#65938
[PATCH] gnu: Add edbrowse.
Previous Next
Reported by: "B. Wilson" <elaexuotee <at> wilsonb.com>
Date: Thu, 14 Sep 2023 06:44:02 UTC
Severity: normal
Tags: patch
Done: Christopher Baines <mail <at> cbaines.net>
Bug is archived. No further changes may be made.
Full log
Message #13 received at 65938-done <at> debbugs.gnu.org (full text, mbox):
[Message part 1 (text/plain, inline)]
Christina O'Donnell <cdo <at> mutix.org> writes:
> From: "Christina O'Donnell" <cdo <at> mutix.org>
>
> * gnu/packages/web-browsers.scm (edbrowse): New variable.
> ---
> gnu/packages/web-browsers.scm | 59 +++++++++++++++++++++++++++++++++++
> 1 file changed, 59 insertions(+)
>
> Dear B. Wilson,
>
> Thank you for this patch that you submitted in September. Sorry it's
> been a while since you sent this in, Guix is a bit short of developers
> and reviewers at the moment.
>
> I'm a new contributor trying to help with the backlog of patches, so
> you can take my points with a grain of salt.
>
> This package is good. It applies, builds and runs. The formatting is
> more or less right.
>
> I have no idea how to use or test edbrowse but it runs at least and
> passes `guix lint`.
>
> I've made the following changes:
>
> - I've reformatted the code to the guix style.
> - I've updated from 3.8.7 to 3.8.9.
> - I've removed the EBDEBUG make flag as this is a release build.
> - I've changed QUICKJS_DIR to QUICKJS_LIB as this was changed
> between 3.8.7 and 3.8.9.
> - I've opted for passing '#:tests #f' instead of deleting 'check.
> - I've set the doc path to /share/doc/ instead of /doc/.
>
> Aside from these points, I think this patch is ready to be pushed.
Thanks both, I've pushed this now to master as
869fa14b786e81f5eb832bc7fd8250248b12651b.
I made a few more changes:
- Using a gexp for the arguments (this is the newer style).
- Reset the author and added Christina as a co-author in the commit
message
- Moved the patching to a phase rather than a snippet, as I think this
is simpler
Thanks again,
Chris
[signature.asc (application/pgp-signature, inline)]
This bug report was last modified 1 year and 97 days ago.
Previous Next
GNU bug tracking system
Copyright (C) 1999 Darren O. Benham,
1997,2003 nCipher Corporation Ltd,
1994-97 Ian Jackson.