Hi,
Greg Hogan <code@greghogan.com> skribis:
> From 9fc2c247d937d587c54b634ec6fcca2b77e22f37 Mon Sep 17 00:00:00 2001
> From: Greg Hogan <code@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
0001-gnu-Add-bloomberg-bde-tools.patch