GNU bug report logs - #51592
[PATCH 0/2] gnu: Add bloomberg-bde.

Previous Next

Full log

Message #11 received at 51592 <at> debbugs.gnu.org (full text, mbox):

From: Greg Hogan <code <at> greghogan.com>
To: Ludovic Courtès <ludo <at> gnu.org>
Cc: 51592 <at> debbugs.gnu.org
Subject: Re: bug#51592: [PATCH 0/2] gnu: Add bloomberg-bde.
Date: Mon, 20 Dec 2021 20:26:17 -0500

[Message part 1 (text/plain, inline)]

On Wed, Dec 1, 2021 at 10:50 AM Ludovic Courtès <ludo <at> gnu.org> wrote:

> Hi,
>
> Greg Hogan <code <at> greghogan.com> skribis:
>
> > From 9fc2c247d937d587c54b634ec6fcca2b77e22f37 Mon Sep 17 00:00:00 2001
> > From: Greg Hogan <code <at> greghogan.com>
> > Date: Wed, 3 Nov 2021 15:22:19 +0000
> > Subject: [PATCH 2/2] gnu: Add bloomberg-bde.
> >
> > * gnu/packages/cpp.scm (bloomberg-bde): New variable.
>
> [...]
>
> > +      (name "bloomberg-bde")
> > +      ; Recent releases are not tagged so commit must be used for
> checkout.
> > +      (version "3.93.0.0")
> > +      (source (origin
> > +                (method git-fetch)
> > +                (uri (git-reference
> > +                      (url "https://github.com/bloomberg/bde")
> > +                      (commit commit)))
>
> There are several bundled code bases, including pcre2, which is already
> packaged.
>
> Could you look into unbundling at least pcre2, and ideally all of these?
> You can remove ‘thirdparty/’ subdirectories from the snippet.
>
> If some of them are too hard to unbundle, we can leave a FIXME and
> address them later.
>

I added a FIXME. It is noted that the third-party sources are modified.


> Nitpick: Please use two semicolons to introduce comments that are on a
> line of their own.  :-)
>

Fixed. Thanks for the reminder.


> Thanks in advance, and apologies for the delay!
>

In addition to rebasing against master and updating the inputs formatting,
I was able to switch the native-input dependency from python2 to python
since my simple patch was accepted upstream. Always nice to see that.


> Ludo’.
>

Greg

[Message part 2 (text/html, inline)]

[0001-gnu-Add-bloomberg-bde-tools.patch (application/octet-stream, attachment)]

[0002-gnu-Add-bloomberg-bde.patch (application/octet-stream, attachment)]

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.