diff mbox series

[bug#42586] emacs-google-translate

Message ID 87mu3jd5jp.fsf@gmail.com
State Accepted
Headers show
Series [bug#42586] emacs-google-translate | expand

Checks

Context Check Description
cbaines/comparison success View comparision
cbaines/git branch success View Git branch
cbaines/applying patch success View Laminar job

Commit Message

André A. Gomes July 28, 2020, 3:26 p.m. UTC
I missed the attachment in the last message, sorry!

Comments

André A. Gomes July 28, 2020, 4:36 p.m. UTC | #1
Please ignore this thread since I forgot to actually add the patch.  Thanks.

--
André Alexandre Gomes
"Free Thought, Free World"
Brett Gilio Aug. 7, 2020, 3:45 a.m. UTC | #2
andremegafone@gmail.com (André Alexandre Gomes) writes:

> I missed the attachment in the last message, sorry!
>
> diff --git a/gnu/packages/emacs-xyz.scm b/gnu/packages/emacs-xyz.scm
> index 8618a07a21..36f8d27743 100644
> --- a/gnu/packages/emacs-xyz.scm
> +++ b/gnu/packages/emacs-xyz.scm
> @@ -78,6 +78,7 @@
>  ;;; Copyright © 2020 Eric Bavier <bavier@posteo.net>
>  ;;; Copyright © 2020 Morgan Smith <Morgan.J.Smith@outlook.com>
>  ;;; Copyright © 2020 Peng Mei Yu <i@pengmeiyu.com>
> +;;; Copyright © 2020 André A. Gomes <andremegafone@gmail.com>
>  ;;;
>  ;;; This file is part of GNU Guix.
>  ;;;
> @@ -16975,7 +16976,7 @@ the GIF result.")
>  (define-public emacs-google-translate
>    (package
>      (name "emacs-google-translate")
> -    (version "0.11.18")
> +    (version "0.12.0")
>      (source
>       (origin
>         (method git-fetch)

Hello Andre,

Thank you for your contribution. There are a few problems we need to
address before continuing. I will try to explain:

1. Each package comes with a checksum hash that needs to match to verify
the authenticity of the originating source. You modified the version,
but did not update that hash. This has can be obtained directly through
git or by running our build system on your change and letting the build
fail and it'll output the correct hash. On that note, please make sure
you are checking your changes by running the build system against them,
it would have caught this discrepency.

2. Your submission appears to be a git diff, we use git formatted
patches emailed through git send-mail to share and apply patches in our
work tree. Will you please resend your change (with the revised checksum
hash) using git format-patch and git send-mail? When doing so, please
take care to review our commit message protocol. These can be reviewed
from our contributing page.

If you need further explanations, or can not find the links to the
information I am referencing, let me know!

Thanks,
Brett Gilio
Brett Gilio Aug. 7, 2020, 4:03 a.m. UTC | #3
This package was already updated, on a second look. I hope you will
consider contributing again in the future though! :)
diff mbox series

Patch

diff --git a/gnu/packages/emacs-xyz.scm b/gnu/packages/emacs-xyz.scm
index 8618a07a21..36f8d27743 100644
--- a/gnu/packages/emacs-xyz.scm
+++ b/gnu/packages/emacs-xyz.scm
@@ -78,6 +78,7 @@ 
 ;;; Copyright © 2020 Eric Bavier <bavier@posteo.net>
 ;;; Copyright © 2020 Morgan Smith <Morgan.J.Smith@outlook.com>
 ;;; Copyright © 2020 Peng Mei Yu <i@pengmeiyu.com>
+;;; Copyright © 2020 André A. Gomes <andremegafone@gmail.com>
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -16975,7 +16976,7 @@  the GIF result.")
 (define-public emacs-google-translate
   (package
     (name "emacs-google-translate")
-    (version "0.11.18")
+    (version "0.12.0")
     (source
      (origin
        (method git-fetch)