GNU bug report logs - #44470
[PATCH] gnu: Added emacs-solidity.

Previous Next

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 44470 in the body.
You can then email your comments to 44470 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

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

From: Martin Becze <mjbecze <at> riseup.net>
To: guix-patches <at> gnu.org
Cc: Martin Becze <mjbecze <at> riseup.net>
Subject: [PATCH] gnu: Added emacs-solidity.
Date: Thu,  5 Nov 2020 09:20:52 -0600

* gnu/packages/emacs-xyz.scm (emacs-solidity): New variable.
---
 gnu/packages/emacs-xyz.scm | 25 +++++++++++++++++++++++++
 1 file changed, 25 insertions(+)

diff --git a/gnu/packages/emacs-xyz.scm b/gnu/packages/emacs-xyz.scm
index 2043ce19df..5a085cfd89 100644
--- a/gnu/packages/emacs-xyz.scm
+++ b/gnu/packages/emacs-xyz.scm
@@ -25281,3 +25281,28 @@ the TypeScript implementation.")
       (description "This package provides an Emacs client for the Rocket.chat
 service.")
       (license license:expat))))
+
+(define-public emacs-solidity
+  ;; No release.
+  (let ((commit "d166a86b83907e0cfd64c191e9dfce4b44a9843e"))
+    (package
+      (name "emacs-solidity")
+      (version (git-version "0.1.10" "1" commit))
+      (source
+       (origin
+         (method git-fetch)
+         (uri (git-reference
+               (url "https://github.com/ethereum/emacs-solidity")
+               (commit commit)))
+         (file-name (git-file-name name version))
+         (sha256
+          (base32 "19hgvsrqch2vp49ag6m76bi5qxd20v95z0ib838rib9as15b17wq"))))
+      (build-system emacs-build-system)
+      (propagated-inputs
+       `(("emacs-company" ,emacs-company)
+         ("emacs-flycheck" ,emacs-flycheck)))
+      (home-page "https://github.com/ethereum/emacs-solidity")
+      (synopsis "Major mode for writing Solidity code")
+      (description "A simple language mode for the Solidity language.  It is a
+constant work in progress as the language itself also progresses.")
+      (license license:gpl3+))))
-- 
2.29.2

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

From: Nicolas Goaziou <mail <at> nicolasgoaziou.fr>
To: Martin Becze <mjbecze <at> riseup.net>
Cc: 44470 <at> debbugs.gnu.org
Subject: Re: [bug#44470] [PATCH] gnu: Added emacs-solidity.
Date: Thu, 05 Nov 2020 18:07:44 +0100

Hello,

Martin Becze <mjbecze <at> riseup.net> writes:

> * gnu/packages/emacs-xyz.scm (emacs-solidity): New variable.

Thank you.

> +  ;; No release.
> +  (let ((commit "d166a86b83907e0cfd64c191e9dfce4b44a9843e"))

Is there any reason to use this particular instead of
e91c36cc20a2e683b930712a7f9c0ab28451b347, which is the exact "0.1.10"
release? If so, please mention it in the comment above.

> +    (package
> +      (name "emacs-solidity")
> +      (version (git-version "0.1.10" "1" commit))

If we use the commit above, `version' can simply become "0.1.10".

> +      (source
> +       (origin
> +         (method git-fetch)
> +         (uri (git-reference
> +               (url "https://github.com/ethereum/emacs-solidity")
> +               (commit commit)))
> +         (file-name (git-file-name name version))
> +         (sha256
> +          (base32 "19hgvsrqch2vp49ag6m76bi5qxd20v95z0ib838rib9as15b17wq"))))
> +      (build-system emacs-build-system)
> +      (propagated-inputs
> +       `(("emacs-company" ,emacs-company)
> +         ("emacs-flycheck" ,emacs-flycheck)))
> +      (home-page "https://github.com/ethereum/emacs-solidity")
> +      (synopsis "Major mode for writing Solidity code")
> +      (description "A simple language mode for the Solidity language.  It is a
> +constant work in progress as the language itself also progresses.")

The first sentence should be "complete", i.e., with a subject and
a verb.

Could you send an updated patch?

Regards,
-- 
Nicolas Goaziou

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

From: Martin Becze <mjbecze <at> riseup.net>
To: Nicolas Goaziou <mail <at> nicolasgoaziou.fr>
Cc: 44470 <at> debbugs.gnu.org
Subject: Re: [bug#44470] [PATCH] gnu: Added emacs-solidity.
Date: Thu, 5 Nov 2020 13:50:30 -0600

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

Thank you for the review Nocolas. I choose the latest commit because it 
fixes some spacing issues and has support for more keywords. While not 
absolutely necessary the release cycle seems to be very slow and this is 
the version that is on Melpa. Attachedd is an updated patch.

On 11/5/20 11:07 AM, Nicolas Goaziou wrote:
> Hello,
> 
> Martin Becze <mjbecze <at> riseup.net> writes:
> 
>> * gnu/packages/emacs-xyz.scm (emacs-solidity): New variable.
> 
> Thank you.
> 
>> +  ;; No release.
>> +  (let ((commit "d166a86b83907e0cfd64c191e9dfce4b44a9843e"))
> 
> Is there any reason to use this particular instead of
> e91c36cc20a2e683b930712a7f9c0ab28451b347, which is the exact "0.1.10"
> release? If so, please mention it in the comment above.
> 
>> +    (package
>> +      (name "emacs-solidity")
>> +      (version (git-version "0.1.10" "1" commit))
> 
> If we use the commit above, `version' can simply become "0.1.10".
> 
>> +      (source
>> +       (origin
>> +         (method git-fetch)
>> +         (uri (git-reference
>> +               (url "https://github.com/ethereum/emacs-solidity")
>> +               (commit commit)))
>> +         (file-name (git-file-name name version))
>> +         (sha256
>> +          (base32 "19hgvsrqch2vp49ag6m76bi5qxd20v95z0ib838rib9as15b17wq"))))
>> +      (build-system emacs-build-system)
>> +      (propagated-inputs
>> +       `(("emacs-company" ,emacs-company)
>> +         ("emacs-flycheck" ,emacs-flycheck)))
>> +      (home-page "https://github.com/ethereum/emacs-solidity")
>> +      (synopsis "Major mode for writing Solidity code")
>> +      (description "A simple language mode for the Solidity language.  It is a
>> +constant work in progress as the language itself also progresses.")
> 
> The first sentence should be "complete", i.e., with a subject and
> a verb.
> 
> Could you send an updated patch?
> 
> Regards,
> 

[v1-0001-gnu-Added-emacs-solidity.patch (text/x-patch, attachment)]

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

From: Martin Becze <mjbecze <at> riseup.net>
To: Nicolas Goaziou <mail <at> nicolasgoaziou.fr>
Cc: 44470 <at> debbugs.gnu.org
Subject: Re: [bug#44470] [PATCH] gnu: Added emacs-solidity.
Date: Thu, 5 Nov 2020 13:52:46 -0600

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

> If we use the commit above, `version' can simply become "0.1.10".

Opps sorry missed this comment. Attached is a patch that fixes that too.

On 11/5/20 11:07 AM, Nicolas Goaziou wrote:
> Hello,
> 
> Martin Becze <mjbecze <at> riseup.net> writes:
> 
>> * gnu/packages/emacs-xyz.scm (emacs-solidity): New variable.
> 
> Thank you.
> 
>> +  ;; No release.
>> +  (let ((commit "d166a86b83907e0cfd64c191e9dfce4b44a9843e"))
> 
> Is there any reason to use this particular instead of
> e91c36cc20a2e683b930712a7f9c0ab28451b347, which is the exact "0.1.10"
> release? If so, please mention it in the comment above.
> 
>> +    (package
>> +      (name "emacs-solidity")
>> +      (version (git-version "0.1.10" "1" commit))
> 
> If we use the commit above, `version' can simply become "0.1.10".
> 
>> +      (source
>> +       (origin
>> +         (method git-fetch)
>> +         (uri (git-reference
>> +               (url "https://github.com/ethereum/emacs-solidity")
>> +               (commit commit)))
>> +         (file-name (git-file-name name version))
>> +         (sha256
>> +          (base32 "19hgvsrqch2vp49ag6m76bi5qxd20v95z0ib838rib9as15b17wq"))))
>> +      (build-system emacs-build-system)
>> +      (propagated-inputs
>> +       `(("emacs-company" ,emacs-company)
>> +         ("emacs-flycheck" ,emacs-flycheck)))
>> +      (home-page "https://github.com/ethereum/emacs-solidity")
>> +      (synopsis "Major mode for writing Solidity code")
>> +      (description "A simple language mode for the Solidity language.  It is a
>> +constant work in progress as the language itself also progresses.")
> 
> The first sentence should be "complete", i.e., with a subject and
> a verb.
> 
> Could you send an updated patch?
> 
> Regards,
> 

[v3-0001-gnu-Added-emacs-solidity.patch (text/x-patch, attachment)]

Message #19 received at 44470-done <at> debbugs.gnu.org (full text, mbox):

From: Nicolas Goaziou <mail <at> nicolasgoaziou.fr>
To: Martin Becze <mjbecze <at> riseup.net>
Cc: 44470-done <at> debbugs.gnu.org
Subject: Re: [bug#44470] [PATCH] gnu: Added emacs-solidity.
Date: Thu, 05 Nov 2020 23:29:03 +0100

Hello,

Martin Becze <mjbecze <at> riseup.net> writes:

> Opps sorry missed this comment. Attached is a patch that fixes that
> too.

Actually, it is not necessary since you kept latest commit.

Patch applied. Thank you.

Regards,
-- 
Nicolas Goaziou

Previous Next

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