Re: [PATCH] add emacsclient support to open with file:linum syntax

[Jorge Alberto Garcia]

On Wed, Mar 30, 2016 at 8:21 AM, Kaushal Modi <address@hidden> wrote:
>> On 03/29/2016 03:35 PM, Kaushal Modi wrote:
>> I myself, use an awk script to rearrange the arguments to work with the
>> current +LINE FILE format accepted by emacsclient ( Ref:
>> https://lists.gnu.org/archive/html/emacs-devel/2016-01/msg00541.html ).
>> So the existing emacsclient implementation already works for you.
>>
>> They are not workarounds, because grep output cannot be used unchanged.
>> Even with the proposed option syntax, users would need to employ a script
>> such as yours to pick out the part of the grep output that they need, and
>> use that information to formulate arguments for emacsclient.
>> The longstanding +LINE option syntax is compatible with other popular free
>> editors such as 'vim' and 'nano', and there is some value in encouraging its
>> usage rather than reinventing the wheel.
>>
>>
>> Having this patch installed will prevent the need for such workarounds
>
>
>
> Jorge,
>
> Based on the counter arguments provided by Paul, I now tend to agree to what
> he says.
>
> 1. The current method (+LINE FILE and using awk to arrange stuff) has been
> working well for me for quite few years.
> 2. It makes sense to have users stick to one convention (+LINE FILE) instead
> of having users now split between "+LINE FILE" and "FILE:LINE".
>
> I do not use copy/paste manually but instead make a shell alias using that
> awk snippet do things. So with either "+LINE FILE" or "FILE:LINE", I will
> need that awk script.
>
> That said, I appreciate the effort you put in in creating this patch and
> this reply, by no means, is intended to discourage you from your future work
> (especially in C).

Thanks Modi,
Folks

I agree with Paul, mx-grep is a better way but is not always possible
to use it, but
use case are different, here you already have a location specified.

Emacs users are very diverse, MX-grep/bash not always is present on
the same system
is a shame but not always is possible/confortable to have a workflow
running within emacs.

I will not push anymore for this patch but please consider this
argument before rejection.
-Jorge