unofficial mirror of guix-patches@gnu.org 
 help / color / mirror / code / Atom feed
* [bug#46697] [PATCH] Added capstone 4.
@ 2021-02-21 22:35 Gerd Heber
  2021-02-23 13:09 ` Nicolas Goaziou
  0 siblings, 1 reply; 3+ messages in thread
From: Gerd Heber @ 2021-02-21 22:35 UTC (permalink / raw)
  To: 46697; +Cc: Gerd Heber

---
 gnu/packages/engineering.scm | 25 +++++++++++++++++++++++++
 1 file changed, 25 insertions(+)

diff --git a/gnu/packages/engineering.scm b/gnu/packages/engineering.scm
index 034dabc3f9..4987377840 100644
--- a/gnu/packages/engineering.scm
+++ b/gnu/packages/engineering.scm
@@ -1495,6 +1495,31 @@ such as x86, x86_64, arm, arm64, mips, ppc, sparc, sysz and xcore.  It provides
 bindings for Python, Java, OCaml and more.")
     (license license:bsd-3)))
 
+(define-public capstone
+  (package
+   (name "capstone")
+   (version "4.0.2")
+   (source
+    (origin
+     (method url-fetch)
+     (uri (string-append "https://github.com/aquynh/capstone"
+                         "/archive/" version ".tar.gz"))
+     (sha256
+      (base32
+       "0sjjbqps48az4map0kmai7j7dak3gy0xcq0sgx8fg09g0acdg0bw"))))
+   (build-system cmake-build-system)
+   (arguments
+    `(#:build-type "Release"
+      #:tests? #t))
+   (home-page "https://www.capstone-engine.org")
+   (synopsis "Lightweight multi-platform, multi-architecture disassembly framework")
+   (description
+    "Capstone is a lightweight multi-platform, multi-architecture disassembly
+framework. Capstone can disassemble machine code for many supported
+architectures such as x86, x86_64, arm, arm64, mips, ppc, sparc, sysz and
+xcore. It provides bindings for Python, Java, OCaml and more.")
+   (license license:bsd-3)))
+
 ;; FIXME: This package has a timestamp embedded in
 ;; lib/python3.5/site-packages/capstone/__pycache__/__iti__.cpython-35.pyc
 (define-public python-capstone
-- 
2.30.1





^ permalink raw reply related	[flat|nested] 3+ messages in thread

* [bug#46697] [PATCH] Added capstone 4.
  2021-02-21 22:35 [bug#46697] [PATCH] Added capstone 4 Gerd Heber
@ 2021-02-23 13:09 ` Nicolas Goaziou
  2023-03-18  9:32   ` bug#46697: " Liliana Marie Prikler
  0 siblings, 1 reply; 3+ messages in thread
From: Nicolas Goaziou @ 2021-02-23 13:09 UTC (permalink / raw)
  To: Gerd Heber; +Cc: 46697, Gerd Heber

Hello,

Gerd Heber <gerd.heber@gmail.com> writes:

>  gnu/packages/engineering.scm | 25 +++++++++++++++++++++++++
>  1 file changed, 25 insertions(+)

Thank you for your patch! Some comments follow.

First, there is already a capstone package in Guix, at version 3.5. Is
there a particular reason for not updating it instead of creating a new
variable?

> +     (uri (string-append "https://github.com/aquynh/capstone"
> +                         "/archive/" version ".tar.gz"))

This URI is unstable. We prefer using tagged commits instead. See for
example already defined capstone package.

> +      #:tests? #t))

This is the default, so it can be omitted.

> +   (home-page "https://www.capstone-engine.org")
> +   (synopsis "Lightweight multi-platform, multi-architecture disassembly framework")

Nitpick: I would remove multi-platform, since it makes little sense from
Guix POV.

> +   (description
> +    "Capstone is a lightweight multi-platform, multi-architecture disassembly

Ditto.

> +framework. Capstone can disassemble machine code for many supported

You need to add two spaces after full stops.

> +architectures such as x86, x86_64, arm, arm64, mips, ppc, sparc, sysz and
> +xcore. It provides bindings for Python, Java, OCaml and more.")

Also, could you provide a proper commit message, for example

    gnu: capstone: Update to 4.0.2.

    * gnu/packages/engineering.scm (capstone): Update to 4.0.2.

Could you send an updated patch?

Regards,
-- 
Nicolas Goaziou




^ permalink raw reply	[flat|nested] 3+ messages in thread

* bug#46697: [PATCH] Added capstone 4.
  2021-02-23 13:09 ` Nicolas Goaziou
@ 2023-03-18  9:32   ` Liliana Marie Prikler
  0 siblings, 0 replies; 3+ messages in thread
From: Liliana Marie Prikler @ 2023-03-18  9:32 UTC (permalink / raw)
  To: Nicolas Goaziou, Gerd Heber; +Cc: Gerd Heber, 46697-done

Am Dienstag, dem 23.02.2021 um 14:09 +0100 schrieb Nicolas Goaziou:
> Also, could you provide a proper commit message, for example
> 
>     gnu: capstone: Update to 4.0.2.
> 
>     * gnu/packages/engineering.scm (capstone): Update to 4.0.2.
> 
> Could you send an updated patch?
Given that capstone 4.0.2 currently exists in Guix, I suppose that is
no longer necessary.

Cheers




^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2023-03-18  9:33 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-02-21 22:35 [bug#46697] [PATCH] Added capstone 4 Gerd Heber
2021-02-23 13:09 ` Nicolas Goaziou
2023-03-18  9:32   ` bug#46697: " Liliana Marie Prikler

Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/guix.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).