Message ID | acba611cbd31ab29f39f46e072d8b8ae@schwi.pl |
---|---|
State | New |
Headers | show |
Series | [bug#56041,v2] gnu: Add vkmark. | expand |
Context | Check | Description |
---|---|---|
cbaines/applying patch | fail | View Laminar job |
cbaines/issue | success | View issue |
LGTM except for:
On Sun Jun 19, 2022 at 12:01 AM BST, Tomasz Jeneralczyk wrote:
> + #:use-module (gnu packages cmake)
You probably don't need this anymore.
Tomasz Jeneralczyk schreef op za 18-06-2022 om 23:01 [+0000]:
> + (license license:lgpl2.1))))
Going by, e.g.,
<https://github.com/vkmark/vkmark/blob/master/src/benchmark.cpp>, it's
not lgpl2.1 but more general (lgpl2.1+).
Greetings,
Maxime.
Tomasz Jeneralczyk schreef op za 18-06-2022 om 23:01 [+0000]:
> + (latest-release "2017.08"))
This variable referred to from a single location, so no need for this
variable.
Greetings,
Maxime.
diff --git a/gnu/packages/benchmark.scm b/gnu/packages/benchmark.scm index b167c6de6e..79e1645eba 100644 --- a/gnu/packages/benchmark.scm +++ b/gnu/packages/benchmark.scm @@ -11,6 +11,7 @@ ;;; Copyright © 2020, 2021, 2022 Maxim Cournoyer <maxim.cournoyer@gmail.com> ;;; Copyright © 2020 Greg Hogan <code@greghogan.com> ;;; Copyright © 2021 Arun Isaac <arunisaac@systemreboot.net> +;;; Copyright © 2022 Tomasz Jeneralczyk <tj@schwi.pl> ;;; ;;; This file is part of GNU Guix.