A line-blinking library in ELPA?

A line-blinking library in ELPA?

[Clément Pit--Claudel]

[-- Attachment #1: Type: text/plain, Size: 887 bytes --]

Hi all,

I'm writing a package that targets Emacsen before 22, so I can't use
xref. Still, I do have a very simplistic jump to source feature, and xref's
blinking would be nice to have.

I could:
* make my package depend on beacon (written by Artur), and use beacon-blink.
* make my package depend on hl-line+ (written by Drew), and use flash-line-highlight.
* make my package depend on crosshairs (written by Drew), and use crosshairs-flash.
* copy xref-pulse-momentarily (written by Dmitry), and use it.
* write my own line blinking

Did I miss other options? Given the number of re-implementations of that
feature, maybe this functionality would be a good candidate for an ELPA package?
I imagine it might be hard to do for xref-pulse-momentarily since core needs it
(and we don't have core ELPA yet), so maybe beacon-blink? Artur, what do you
think?

Clément.


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

Re: A line-blinking library in ELPA?

[Clément Pit--Claudel]

[-- Attachment #1: Type: text/plain, Size: 190 bytes --]

On 01/09/2016 07:51 PM, Clément Pit--Claudel wrote:
> Did I miss other options? 

Ah, it seems pulse.el has been part of CEDET for a while, actually! That's the option I was missing.


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

Re: A line-blinking library in ELPA?

[Stefan Monnier]

> I imagine it might be hard to do for xref-pulse-momentarily since core
> needs it (and we don't have core ELPA yet),

We don't have "core ELPA" yet, but we do have "ELPA core" instead
(i.e. the ability to export core packages via ELPA).  This is currently
used for python.el, ntlm, soap-client, and let-alist.


        Stefan

Re: A line-blinking library in ELPA?

[Clément Pit--Claudel]

[-- Attachment #1: Type: text/plain, Size: 412 bytes --]

On 01/09/2016 11:11 PM, Stefan Monnier wrote:
>> I imagine it might be hard to do for xref-pulse-momentarily since core
>> needs it (and we don't have core ELPA yet),
> 
> We don't have "core ELPA" yet, but we do have "ELPA core" instead
> (i.e. the ability to export core packages via ELPA).  This is currently
> used for python.el, ntlm, soap-client, and let-alist.

Neat! I didn't know about this.


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

Re: A line-blinking library in ELPA?

[John Wiegley]

[-- Attachment #1: Type: text/plain, Size: 687 bytes --]

>>>>> Stefan Monnier <monnier@iro.umontreal.ca> writes:

> We don't have "core ELPA" yet, but we do have "ELPA core" instead (i.e. the
> ability to export core packages via ELPA). This is currently used for
> python.el, ntlm, soap-client, and let-alist.

I would like "core ELPA" (and "tarball ELPA") to become a reality in 25.2 or
26.1 (depending on how impactful the changes will be), so we should start
thinking about what is necessary on master to make this happen, and what is
necessary procedurally in other places.

-- 
John Wiegley                  GPG fingerprint = 4710 CF98 AF9B 327B B80F
http://newartisans.com                          60E1 46C4 BD1A 7AC1 4BA2

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 629 bytes --]

Re: A line-blinking library in ELPA?

[Artur Malabarba]

[-- Attachment #1: Type: text/plain, Size: 813 bytes --]

On Jan 9, 2016 10:51 PM, "Clément Pit--Claudel" <clement.pit@gmail.com>
wrote:
> I could:
> * make my package depend on beacon (written by Artur), and use
beacon-blink.

Note that beacon depends on Emacs 24, so I don't know if it would help you.

> Given the number of re-implementations of that
> feature, maybe this functionality would be a good candidate for an ELPA
package?
> I imagine it might be hard to do for xref-pulse-momentarily since core
needs it
> (and we don't have core ELPA yet), so maybe beacon-blink? Artur, what do
you
> think?

I'm not sure what you're asking. Beacon is on GNU Elpa, isn't it?

As for `xref-pulse-momentarily'... We could make pulse.el available on
Gelpa (if it isn't already) without moving it out of core. Gelpa supports
that now.

Cheers,
Artur

[-- Attachment #2: Type: text/html, Size: 1042 bytes --]

Re: A line-blinking library in ELPA?

[Clément Pit--Claudel]

[-- Attachment #1: Type: text/plain, Size: 1168 bytes --]

On 01/09/2016 11:20 PM, Artur Malabarba wrote:
> On Jan 9, 2016 10:51 PM, "Clément Pit--Claudel" <clement.pit@gmail.com <mailto:clement.pit@gmail.com>> wrote:
>> I could:
>> * make my package depend on beacon (written by Artur), and use beacon-blink.
> 
> Note that beacon depends on Emacs 24, so I don't know if it would help you.

It would be fine :)

>> Given the number of re-implementations of that
>> feature, maybe this functionality would be a good candidate for an ELPA package?
>> I imagine it might be hard to do for xref-pulse-momentarily since core needs it
>> (and we don't have core ELPA yet), so maybe beacon-blink? Artur, what do you
>> think?
> 
> I'm not sure what you're asking. Beacon is on GNU Elpa, isn't it?

Yes, it is. I just wasn't whether it was a good practice to add teh whole beacon as a dependency if I was just going to use the blinking.

> As for `xref-pulse-momentarily'... We could make pulse.el available on Gelpa (if it isn't already) without moving it out of core. Gelpa supports that now.

I target 24 as well, so pulse.el works very nicely! I just didn't find it initially.

Thanks Artur!
Clément.


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

Re: A line-blinking library in ELPA?

[Artur Malabarba]

[-- Attachment #1: Type: text/plain, Size: 250 bytes --]

On Jan 10, 2016 2:32 AM, "Clément Pit--Claudel" <clement.pit@gmail.com>
wrote:
> Yes, it is. I just wasn't whether it was a good practice to add teh whole
beacon as a dependency if I was just going to use the blinking.

I'd say it's fine. :-)

[-- Attachment #2: Type: text/html, Size: 358 bytes --]