GNU bug report logs - #33338
[PATCH] gnu: Add emacs-webpaste

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 33338 in the body.
You can then email your comments to 33338 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: Brett Gilio <brettg <at> posteo.net>
To: guix-patches <at> gnu.org
Cc: Brett Gilio <brettg <at> posteo.net>
Subject: [PATCH] gnu: Add emacs-webpaste
Date: Sat, 10 Nov 2018 15:04:51 -0600

---
 gnu/packages/emacs.scm | 26 ++++++++++++++++++++++++++
 1 file changed, 26 insertions(+)

diff --git a/gnu/packages/emacs.scm b/gnu/packages/emacs.scm
index c6008e6a9..72112cfc0 100644
--- a/gnu/packages/emacs.scm
+++ b/gnu/packages/emacs.scm
@@ -40,6 +40,7 @@
 ;;; Copyright © 2018 Pierre-Antoine Rouby <pierre-antoine.rouby <at> inria.fr>
 ;;; Copyright © 2018 Alex Branham <alex.branham <at> gmail.com>
 ;;; Copyright © 2018 Thorsten Wilms <t_w_ <at> freenet.de>
+;;; Copyright © 2018 Brett Gilio <bgilio <at> posteo.net>
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -9837,6 +9838,31 @@ CSS, JavaScript, JSON.")
 throw a shell history.")
       (license license:gpl3+))))
 
+(define-public emacs-webpaste
+  (package
+    (name "emacs-webpaste")
+    (version "20180815.1855")
+    (source
+     (origin
+       (method url-fetch)
+       (uri (string-append
+             "https://melpa.org/packages/webpaste-"
+             version
+             ".el"))
+       (sha256
+        (base32
+         "0afbyrqs0hk9b5a7g75lar31v8bmmbgvcl64z0i52bi78vz7fdv8"))))
+    (build-system emacs-build-system)
+    (propagated-inputs
+     `(("emacs-request" ,emacs-request)))
+    (home-page "https://github.com/etu/webpaste.el")
+    (synopsis "Paste to pastebin-like services")
+    (description "This mode allows to paste whole buffers or parts of buffers to
+pastebin-like services.  It supports more than one service and will
+failover if one service fails.  More services can easily be added
+over time and prefered services can easily be configured.")
+    (license license:gpl3+)))
+
 (define-public emacs-discover-my-major
   (package
     (name "emacs-discover-my-major")
-- 
2.19.1

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

From: Brett Gilio <brettg <at> posteo.net>
To: 33338 <at> debbugs.gnu.org
Cc: Brett Gilio <brettg <at> posteo.net>
Subject: Re: bug#33338: Acknowledgement ([PATCH] gnu: Add emacs-webpaste)
Date: Sat, 10 Nov 2018 15:08:21 -0600

Small error in the patch. Please revise the email line to "brettg <at> posteo.net"

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

From: ludo <at> gnu.org (Ludovic Courtès)
To: Brett Gilio <brettg <at> posteo.net>
Cc: 33338 <at> debbugs.gnu.org
Subject: Re: [bug#33338] [PATCH] gnu: Add emacs-webpaste
Date: Mon, 19 Nov 2018 22:36:00 +0100

Hi Brett, and sorry for the delay!

Brett Gilio <brettg <at> posteo.net> skribis:

> ---
>  gnu/packages/emacs.scm | 26 ++++++++++++++++++++++++++
>  1 file changed, 26 insertions(+)

Could you add a commit log along the lines of what we usually do?  (See
‘git log’ to get an idea.)

> +  (package
> +    (name "emacs-webpaste")
> +    (version "20180815.1855")
> +    (source
> +     (origin
> +       (method url-fetch)
> +       (uri (string-append
> +             "https://melpa.org/packages/webpaste-"
> +             version
> +             ".el"))

Since files on MELPA disappear or are modified in place, it’s safer to
take them from upstream repositories.  Could you send an updated patch
that fetches code from the github.com repo?

Thank you!

Ludo’.

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

From: Brett Gilio <brettg <at> posteo.net>
To: Ludovic Courtès <ludo <at> gnu.org>
Cc: 33338 <at> debbugs.gnu.org, Brett Gilio <brettg <at> posteo.net>
Subject: Re: [bug#33338] [PATCH] gnu: Add emacs-webpaste
Date: Sat, 24 Nov 2018 15:02:38 -0600

Ludovic Courtès writes:

> Hi Brett, and sorry for the delay!
>
> Brett Gilio <brettg <at> posteo.net> skribis:
>
>> ---
>>  gnu/packages/emacs.scm | 26 ++++++++++++++++++++++++++
>>  1 file changed, 26 insertions(+)
>
> Could you add a commit log along the lines of what we usually do?  (See
> ‘git log’ to get an idea.)
>
>> +  (package
>> +    (name "emacs-webpaste")
>> +    (version "20180815.1855")
>> +    (source
>> +     (origin
>> +       (method url-fetch)
>> +       (uri (string-append
>> +             "https://melpa.org/packages/webpaste-"
>> +             version
>> +             ".el"))
>
> Since files on MELPA disappear or are modified in place, it’s safer to
> take them from upstream repositories.  Could you send an updated patch
> that fetches code from the github.com repo?
>
> Thank you!
>
> Ludo’.

Hi Ludo,

When I run git format-patch -1 I suppose it isn't adding the commit log
to the patch file? I think that is what you are trying to say. How then
should I add it, because I am making commits before I generate the
patch?

Brett Gilio

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

From: Tobias Geerinckx-Rice <somebody <at> not-sent-or-endorsed-by.tobias.gr>
To: Brett Gilio <brettg <at> posteo.net>
Cc: Ludovic Courtès <ludo <at> gnu.org>, 33338 <at> debbugs.gnu.org
Subject: Re: [bug#33338] [PATCH] gnu: Add emacs-webpaste
Date: Sun, 25 Nov 2018 11:07:32 +0100

Brett,

Brett Gilio wrote:
> When I run git format-patch -1 I suppose it isn't adding the 
> commit log
> to the patch file? I think that is what you are trying to 
> say. How then
> should I add it, because I am making commits before I generate 
> the
> patch?

That's the normal (only?) way. ‘git format-patch’es will include 
whatever commit messages you wrote during ’git commit’, and which 
are displayed by ’git log’, ‘git show’, et al.

What do the latter commands show?

Kind regards,

T G-R

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

From: Brett Gilio <brettg <at> posteo.net>
To: Tobias Geerinckx-Rice <somebody <at> not-sent-or-endorsed-by.tobias.gr>
Cc: Ludovic Courtès <ludo <at> gnu.org>, 33338 <at> debbugs.gnu.org,
 Brett Gilio <brettg <at> posteo.net>
Subject: Re: [bug#33338] [PATCH] gnu: Add emacs-webpaste
Date: Sun, 25 Nov 2018 09:52:19 -0600

Tobias Geerinckx-Rice writes:

> Brett,
>
> Brett Gilio wrote:
>> When I run git format-patch -1 I suppose it isn't adding the commit
>> log
>> to the patch file? I think that is what you are trying to say. How
>> then
>> should I add it, because I am making commits before I generate the
>> patch?
>
> That's the normal (only?) way. ‘git format-patch’es will include
> whatever commit messages you wrote during ’git commit’, and which are
> displayed by ’git log’, ‘git show’, et al.
>
> What do the latter commands show?
>
> Kind regards,
>
> T G-R

The patches are including the commit in the name of the format itself,
but it seems like ludo is wanting something contained within the patch
itself which it is not doing.

Git log is showing my commit history with the messages, git show is
also. So, I am confused as to what exactly I am not doing correctly.

Brett

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

From: ludo <at> gnu.org (Ludovic Courtès)
To: Brett Gilio <brettg <at> posteo.net>
Cc: 33338 <at> debbugs.gnu.org
Subject: Re: [bug#33338] [PATCH] gnu: Add emacs-webpaste
Date: Tue, 27 Nov 2018 11:58:19 +0100

Hello Brett,

Just a friendly reminder.  :-)

Ludo’.

ludo <at> gnu.org (Ludovic Courtès) skribis:

> Hi Brett, and sorry for the delay!
>
> Brett Gilio <brettg <at> posteo.net> skribis:
>
>> ---
>>  gnu/packages/emacs.scm | 26 ++++++++++++++++++++++++++
>>  1 file changed, 26 insertions(+)
>
> Could you add a commit log along the lines of what we usually do?  (See
> ‘git log’ to get an idea.)
>
>> +  (package
>> +    (name "emacs-webpaste")
>> +    (version "20180815.1855")
>> +    (source
>> +     (origin
>> +       (method url-fetch)
>> +       (uri (string-append
>> +             "https://melpa.org/packages/webpaste-"
>> +             version
>> +             ".el"))
>
> Since files on MELPA disappear or are modified in place, it’s safer to
> take them from upstream repositories.  Could you send an updated patch
> that fetches code from the github.com repo?
>
> Thank you!
>
> Ludo’.

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

From: Arun Isaac <arunisaac <at> systemreboot.net>
To: Ludovic Courtès <ludo <at> gnu.org>, Brett Gilio
 <brettg <at> posteo.net>
Cc: 33338-done <at> debbugs.gnu.org
Subject: Re: [bug#33338] [PATCH] gnu: Add emacs-webpaste
Date: Fri, 11 Dec 2020 13:04:46 +0530

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

Hi,

I fixed this patch, packaged the tests, and pushed to master.

Cheers! :-)

[signature.asc (application/pgp-signature, inline)]

Previous Next

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