GNU bug report logs - #38272
[PATCH] gnu: Add emacs-openwith.

Previous Next

Package: guix-patches;

Reported by: "David Wilson" <david <at> daviwil.com>

Date: Tue, 19 Nov 2019 15:19:03 UTC

Severity: normal

Tags: patch

Done: Marius Bakke <mbakke <at> fastmail.com>

Bug is archived. No further changes may be made.

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

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

View this report as an mbox folder, status mbox, maintainer mbox


Report forwarded to guix-patches <at> gnu.org:
bug#38272; Package guix-patches. (Tue, 19 Nov 2019 15:19:04 GMT) Full text and rfc822 format available.

Acknowledgement sent to "David Wilson" <david <at> daviwil.com>:
New bug report received and forwarded. Copy sent to guix-patches <at> gnu.org. (Tue, 19 Nov 2019 15:19:04 GMT) Full text and rfc822 format available.

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

From: "David Wilson" <david <at> daviwil.com>
To: guix-patches <at> gnu.org
Subject: [PATCH] gnu: Add emacs-openwith.
Date: Tue, 19 Nov 2019 07:17:59 -0800
[Message part 1 (text/plain, inline)]
Hi Guix!

This patch adds the "openwith" package for Emacs to make it easier to launch external applications for configured file types.  I've tested the package and run it through "guix lint", all looks good.

Thanks!

David
[0001-gnu-Add-emacs-openwith.patch (text/x-patch, attachment)]

Information forwarded to guix-patches <at> gnu.org:
bug#38272; Package guix-patches. (Tue, 19 Nov 2019 20:22:01 GMT) Full text and rfc822 format available.

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

From: Marius Bakke <mbakke <at> fastmail.com>
To: David Wilson <david <at> daviwil.com>, 38272 <at> debbugs.gnu.org
Subject: Re: [bug#38272] [PATCH] gnu: Add emacs-openwith.
Date: Tue, 19 Nov 2019 21:21:01 +0100
[Message part 1 (text/plain, inline)]
"David Wilson" <david <at> daviwil.com> writes:

> Hi Guix!
>
> This patch adds the "openwith" package for Emacs to make it easier to launch external applications for configured file types.  I've tested the package and run it through "guix lint", all looks good.

Thanks!

[...]

> +(define-public emacs-openwith
> +  (let ((commit "1dc89670822966fab6e656f6519fdd7f01e8301a")
> +        (revision "0"))
> +    (package
> +      (name "emacs-openwith")
> +      (home-page "https://github.com/emacsmirror/openwith")

The actual home page is here according to the README:

https://bitbucket.org/jpkotta/openwith

[...]

> +      (synopsis "Emacs")

This synopsis is a bit short.  :-)

> +      (description
> +       "This package enables external applications to be launched for
> +configured file types when you use commands like @code{find-file}, etc.")

Maybe "This package lets you associate file name patterns with
external applications that are automatically invoked when you use
commands like @code{find-file}, etc".

Dunno, YMMV :-) It just wasn't clear to me how it worked (and how to
interpret the "etc") without the extra context.

Can you send an updated patch?
[signature.asc (application/pgp-signature, inline)]

Information forwarded to guix-patches <at> gnu.org:
bug#38272; Package guix-patches. (Wed, 20 Nov 2019 13:54:02 GMT) Full text and rfc822 format available.

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

From: "David Wilson" <david <at> daviwil.com>
To: "Marius Bakke" <mbakke <at> fastmail.com>, 38272 <at> debbugs.gnu.org
Subject: Re: [bug#38272] [PATCH] gnu: Add emacs-openwith.
Date: Wed, 20 Nov 2019 05:53:25 -0800
[Message part 1 (text/plain, inline)]
Hi Marius, thanks for the reply!

On Tue, Nov 19, 2019, at 12:21 PM, Marius Bakke wrote:
> 
> The actual home page is here according to the README:
> 
> https://bitbucket.org/jpkotta/openwith
> 

Thanks!  Fixed.

> 
> This synopsis is a bit short.  :-)
> 

I swear I put something in for that, sorry I missed it!

> 
> Maybe "This package lets you associate file name patterns with
> external applications that are automatically invoked when you use
> commands like @code{find-file}, etc".
> 

I've taken your suggestion and improved on it like so:

"This package enables you to associate file name patterns with external
applications that are automatically invoked when you use commands like
@code{find-file}.  For example, you can have it open @code{png} files with
@code{feh} and @code{mp4} files with @code{mpv}.  This is especially useful
when browsing files with Dired."

Updated patch is attached, thanks again!

David
[0001-gnu-Add-emacs-openwith.patch (text/x-patch, attachment)]

Reply sent to Marius Bakke <mbakke <at> fastmail.com>:
You have taken responsibility. (Thu, 21 Nov 2019 18:37:02 GMT) Full text and rfc822 format available.

Notification sent to "David Wilson" <david <at> daviwil.com>:
bug acknowledged by developer. (Thu, 21 Nov 2019 18:37:03 GMT) Full text and rfc822 format available.

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

From: Marius Bakke <mbakke <at> fastmail.com>
To: David Wilson <david <at> daviwil.com>, 38272-done <at> debbugs.gnu.org
Subject: Re: [bug#38272] [PATCH] gnu: Add emacs-openwith.
Date: Thu, 21 Nov 2019 19:36:49 +0100
[Message part 1 (text/plain, inline)]
"David Wilson" <david <at> daviwil.com> writes:

>> 
>> Maybe "This package lets you associate file name patterns with
>> external applications that are automatically invoked when you use
>> commands like @code{find-file}, etc".
>> 
>
> I've taken your suggestion and improved on it like so:
>
> "This package enables you to associate file name patterns with external
> applications that are automatically invoked when you use commands like
> @code{find-file}.  For example, you can have it open @code{png} files with
> @code{feh} and @code{mp4} files with @code{mpv}.  This is especially useful
> when browsing files with Dired."

Thank you!  That description is much better.  :-)

> Updated patch is attached, thanks again!

I found that 'git-fetch' failed, so I changed the patch to use
'hg-fetch'.  It still produced the same hash.

But, I read in another message that BitBucket is shutting down Mercurial
support in 2020, so I'm not sure what to do about it.  Thoughts?

In any case the patch is applied!
[signature.asc (application/pgp-signature, inline)]

Information forwarded to guix-patches <at> gnu.org:
bug#38272; Package guix-patches. (Fri, 22 Nov 2019 14:06:02 GMT) Full text and rfc822 format available.

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

From: zimoun <zimon.toutoune <at> gmail.com>
To: 38272 <at> debbugs.gnu.org, Marius Bakke <mbakke <at> fastmail.com>, 
 David Wilson <david <at> daviwil.com>
Subject: Re: bug#38272: [PATCH] gnu: Add emacs-openwith.
Date: Fri, 22 Nov 2019 15:05:35 +0100
Hi Marius,

On Thu, 21 Nov 2019 at 19:37, Marius Bakke <mbakke <at> fastmail.com> wrote:

> But, I read in another message that BitBucket is shutting down Mercurial
> support in 2020, so I'm not sure what to do about it.  Thoughts?

The timeline of the shutdown is here [1] so we have still some time. :-)

The best seems to first list all the packages offended via some scheme
code -- I have no idea how many packages will be impacted.
Then contact each upstream to know what is their plan, i.e., where the
new location of the source will be: git conversion and still on
BitBucket; still hg and move to elsewhere; etc.

I do not know if it is better to open only one bug report to track the
progress (maybe my preference), or to open one bug report per package.

What do you think?


All the best,
simon

[1] https://bitbucket.org/blog/sunsetting-mercurial-support-in-bitbucket




Information forwarded to guix-patches <at> gnu.org:
bug#38272; Package guix-patches. (Mon, 25 Nov 2019 12:12:01 GMT) Full text and rfc822 format available.

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

From: Marius Bakke <mbakke <at> fastmail.com>
To: zimoun <zimon.toutoune <at> gmail.com>, 38272 <at> debbugs.gnu.org,
 David Wilson <david <at> daviwil.com>
Subject: Re: bug#38272: [PATCH] gnu: Add emacs-openwith.
Date: Sun, 24 Nov 2019 23:28:07 +0100
[Message part 1 (text/plain, inline)]
zimoun <zimon.toutoune <at> gmail.com> writes:

> Hi Marius,
>
> On Thu, 21 Nov 2019 at 19:37, Marius Bakke <mbakke <at> fastmail.com> wrote:
>
>> But, I read in another message that BitBucket is shutting down Mercurial
>> support in 2020, so I'm not sure what to do about it.  Thoughts?
>
> The timeline of the shutdown is here [1] so we have still some time. :-)
>
> [1] https://bitbucket.org/blog/sunsetting-mercurial-support-in-bitbucket

Thanks for the link.  I hope the affected archives will be available
through the Software Heritage API for the time travellers among us.

> The best seems to first list all the packages offended via some scheme
> code -- I have no idea how many packages will be impacted.
> Then contact each upstream to know what is their plan, i.e., where the
> new location of the source will be: git conversion and still on
> BitBucket; still hg and move to elsewhere; etc.

For this package in particular, I think it's okay to go with the
original emacsmirror source David used, as the code had not changed
since 2012.  :-)

More active packages probably already have a migration plan, so I
suppose we'll have to deal with it on a case by case basis.

> I do not know if it is better to open only one bug report to track the
> progress (maybe my preference), or to open one bug report per package.
>
> What do you think?

I think one bug to track it is sufficient, but no strong opinion.
[signature.asc (application/pgp-signature, inline)]

bug archived. Request was from Debbugs Internal Request <help-debbugs <at> gnu.org> to internal_control <at> debbugs.gnu.org. (Mon, 23 Dec 2019 12:24:04 GMT) Full text and rfc822 format available.

This bug report was last modified 4 years and 120 days ago.

Previous Next


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