all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* ELPA: new package nano-agenda
@ 2021-10-12 17:01 Nicolas P. Rougier (inria)
  2021-10-12 19:19 ` Philip Kaludercic
  2021-10-19  6:39 ` Joost Kremers
  0 siblings, 2 replies; 12+ messages in thread
From: Nicolas P. Rougier (inria) @ 2021-10-12 17:01 UTC (permalink / raw)
  To: emacs-devel


Dear all,

I would like to submit a new package to ELPA which is a minimal 
non-intrusive view of org-agenda files. It displays a calendar 
view of current month (or the month corresponding to the current 
selected date) alongside a view of the daily agenda (timestamped) 
entries. The agenda can be navigated using arrows keys and killed 
using "q" or "return".

The sources are hosted at https://github.com/rougier/nano-agenda 
and the README displays what it looks like. I find it quite 
convenient for my own use but maybe it's too specific for ELPA.

Best,
Nicolas.



^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: ELPA: new package nano-agenda
  2021-10-12 17:01 ELPA: new package nano-agenda Nicolas P. Rougier (inria)
@ 2021-10-12 19:19 ` Philip Kaludercic
  2021-10-12 19:29   ` Nicolas P. Rougier (inria)
  2021-10-19  6:39 ` Joost Kremers
  1 sibling, 1 reply; 12+ messages in thread
From: Philip Kaludercic @ 2021-10-12 19:19 UTC (permalink / raw)
  To: Nicolas P. Rougier (inria); +Cc: emacs-devel

"Nicolas P. Rougier (inria)" <nicolas.rougier@inria.fr> writes:

> The sources are hosted at https://github.com/rougier/nano-agenda and
> the README displays what it looks like. I find it quite convenient for
> my own use but maybe it's too specific for ELPA.

It seems you are using ts.el, which isn't on ELPA, and additionally depends
on s.el that also (famously) isn't on ELPA.

That being said, ts seems to only have a weak dependency on s (two
usages of s-join), so maybe ts could be added to ELPA -- in one form or
another -- as it seems to have a few useful improvements over the
default time and duration functions.

Otherwise, I don't see any grave issues from quickly skimming over the
code.

-- 
	Philip Kaludercic



^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: ELPA: new package nano-agenda
  2021-10-12 19:19 ` Philip Kaludercic
@ 2021-10-12 19:29   ` Nicolas P. Rougier (inria)
  2021-10-12 19:41     ` Philip Kaludercic
  0 siblings, 1 reply; 12+ messages in thread
From: Nicolas P. Rougier (inria) @ 2021-10-12 19:29 UTC (permalink / raw)
  To: Philip Kaludercic; +Cc: emacs-devel


Philip Kaludercic <philipk@posteo.net> writes:

> It seems you are using ts.el, which isn't on ELPA, and 
> additionally depends
> on s.el that also (famously) isn't on ELPA.
>
> That being said, ts seems to only have a weak dependency on s 
> (two
> usages of s-join), so maybe ts could be added to ELPA -- in one 
> form or
> another -- as it seems to have a few useful improvements over 
> the
> default time and duration functions.

Thanks for the review, I didn't pay attention to this point 
actually. I confirm than ts would be a very nice addition to ELPA 
since it really simplifies time management for user and comes with 
a very clean API. I guess only Adam Porter can answer if he want 
to add it to ELPA or not. Should I try to remove the dependency in 
the meantime?

Nicolas




^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: ELPA: new package nano-agenda
  2021-10-12 19:29   ` Nicolas P. Rougier (inria)
@ 2021-10-12 19:41     ` Philip Kaludercic
  2021-10-18 22:50       ` Adam Porter
  0 siblings, 1 reply; 12+ messages in thread
From: Philip Kaludercic @ 2021-10-12 19:41 UTC (permalink / raw)
  To: Nicolas P. Rougier (inria); +Cc: adam, emacs-devel

"Nicolas P. Rougier (inria)" <nicolas.rougier@inria.fr> writes:

> Philip Kaludercic <philipk@posteo.net> writes:
>
>> It seems you are using ts.el, which isn't on ELPA, and additionally
>> depends
>> on s.el that also (famously) isn't on ELPA.
>>
>> That being said, ts seems to only have a weak dependency on s (two
>> usages of s-join), so maybe ts could be added to ELPA -- in one form
>> or
>> another -- as it seems to have a few useful improvements over the
>> default time and duration functions.
>
> Thanks for the review, I didn't pay attention to this point
> actually. I confirm than ts would be a very nice addition to ELPA
> since it really simplifies time management for user and comes with a
> very clean API. I guess only Adam Porter can answer if he want to add
> it to ELPA or not. 

As he is the only major contributor, this might be worth considering.
I've Cc'ed Adam to see what he thinks.

What might be an issue is if the "ts-" namespace is considered to be
"too valuable", as was the issue with "s-" and "f-". The maintainers
would have to decide on that, I can only speculate.

> Should I try to remove the dependency in the meantime?

Personally, I'd wait to see.

-- 
	Philip Kaludercic



^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: ELPA: new package nano-agenda
  2021-10-12 19:41     ` Philip Kaludercic
@ 2021-10-18 22:50       ` Adam Porter
  2021-10-18 23:05         ` Stefan Monnier
  0 siblings, 1 reply; 12+ messages in thread
From: Adam Porter @ 2021-10-18 22:50 UTC (permalink / raw)
  To: Philip Kaludercic; +Cc: Nicolas P. Rougier (inria), emacs-devel

Hi Philip, Nicolas, et al,

On Tue, Oct 12, 2021 at 2:41 PM Philip Kaludercic <philipk@posteo.net> wrote:
>
> "Nicolas P. Rougier (inria)" <nicolas.rougier@inria.fr> writes:
>
> > Philip Kaludercic <philipk@posteo.net> writes:
> >
> >> It seems you are using ts.el, which isn't on ELPA, and additionally
> >> depends
> >> on s.el that also (famously) isn't on ELPA.
> >>
> >> That being said, ts seems to only have a weak dependency on s (two
> >> usages of s-join), so maybe ts could be added to ELPA -- in one form
> >> or
> >> another -- as it seems to have a few useful improvements over the
> >> default time and duration functions.
> >
> > Thanks for the review, I didn't pay attention to this point
> > actually. I confirm than ts would be a very nice addition to ELPA
> > since it really simplifies time management for user and comes with a
> > very clean API. I guess only Adam Porter can answer if he want to add
> > it to ELPA or not.
>
> As he is the only major contributor, this might be worth considering.
> I've Cc'ed Adam to see what he thinks.

Yes, I'd be glad to have ts.el in GNU ELPA.

And I'd be glad to remove the dependency on s.el to facilitate that.

The only potential issue that I can see is that it currently emits
extra warnings at load time (or is it compile time?  I forget...) on
Emacs 28 due to more strict checking in the byte compiler.  The
warnings are harmless, and everything works correctly in spite of
them.  The last time I tried to find the cause of them, I wasn't
successful despite some effort.  I don't know if that would or should
keep it out of GNU ELPA; I would guess that it wouldn't, and that the
warnings can always be fixed or worked around later.

> What might be an issue is if the "ts-" namespace is considered to be
> "too valuable", as was the issue with "s-" and "f-". The maintainers
> would have to decide on that, I can only speculate.

I hope that wouldn't be the case.  In fact, I've already "defended"
the "ts-" prefix against other packages.  (That sounds aggressive or
confrontational, which I don't intend, but I'm not sure how else to
put it.  Actually, the maintainer of the elisp-tree-sitter repo,
Tuấn-Anh Nguyễn (@ubolonton), was very kind to rename his package's
prefix to `tsc-` to avoid conflicts when I asked him last year.  See
https://github.com/emacs-tree-sitter/elisp-tree-sitter/issues/35 )
And it's already used as a dependency in several other Elisp packages.

-- 
Thanks,
Adam



^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: ELPA: new package nano-agenda
  2021-10-18 22:50       ` Adam Porter
@ 2021-10-18 23:05         ` Stefan Monnier
  2021-10-19 11:55           ` Philip Kaludercic
  0 siblings, 1 reply; 12+ messages in thread
From: Stefan Monnier @ 2021-10-18 23:05 UTC (permalink / raw)
  To: Adam Porter; +Cc: Philip Kaludercic, Nicolas P. Rougier (inria), emacs-devel

> Yes, I'd be glad to have ts.el in GNU ELPA.

That'd be nice.

> And I'd be glad to remove the dependency on s.el to facilitate that.

Yes, that'd be necessary before we can add it to GNU ELPA.

> The only potential issue that I can see is that it currently emits
> extra warnings at load time (or is it compile time?  I forget...) on
> Emacs 28 due to more strict checking in the byte compiler.

That's fine (code doesn't have to be bug-free, luckily, otherwise not
only GNU ELPA would be empty, but Emacs wouldn't exist either).

>> What might be an issue is if the "ts-" namespace is considered to be
>> "too valuable", as was the issue with "s-" and "f-". The maintainers
>> would have to decide on that, I can only speculate.

two-letter (well, 3 if you count the dash) prefixes are fine, thanks
(we have loads of them).


        Stefan




^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: ELPA: new package nano-agenda
  2021-10-12 17:01 ELPA: new package nano-agenda Nicolas P. Rougier (inria)
  2021-10-12 19:19 ` Philip Kaludercic
@ 2021-10-19  6:39 ` Joost Kremers
  2021-10-19 10:19   ` Nicolas P. Rougier (inria)
  1 sibling, 1 reply; 12+ messages in thread
From: Joost Kremers @ 2021-10-19  6:39 UTC (permalink / raw)
  To: Nicolas P. Rougier (inria); +Cc: emacs-devel


On Tue, Oct 12 2021, Nicolas P. Rougier (inria) wrote:
> I would like to submit a new package to ELPA which is a minimal non-intrusive
> view of org-agenda files. It displays a calendar view of current month (or the
> month corresponding to the current selected date) alongside a view of the daily
> agenda (timestamped) entries. The agenda can be navigated using arrows keys and
> killed using "q" or "return".

I would suggest adding an autoload cookie to the main entry point `nano-agenda`.
That way, after installing the package, you don't have to `require` it first,
you can just do `M-x nano-agenda`.

-- 
Joost Kremers
Life has its moments



^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: ELPA: new package nano-agenda
  2021-10-19  6:39 ` Joost Kremers
@ 2021-10-19 10:19   ` Nicolas P. Rougier (inria)
  0 siblings, 0 replies; 12+ messages in thread
From: Nicolas P. Rougier (inria) @ 2021-10-19 10:19 UTC (permalink / raw)
  To: Joost Kremers; +Cc: emacs-devel


Thanks, I've added it.

Nicolas


Joost Kremers <joostkremers@fastmail.fm> writes:

> On Tue, Oct 12 2021, Nicolas P. Rougier (inria) wrote:
>> I would like to submit a new package to ELPA which is a minimal 
>> non-intrusive
>> view of org-agenda files. It displays a calendar view of 
>> current month (or the
>> month corresponding to the current selected date) alongside a 
>> view of the daily
>> agenda (timestamped) entries. The agenda can be navigated using 
>> arrows keys and
>> killed using "q" or "return".
>
> I would suggest adding an autoload cookie to the main entry 
> point `nano-agenda`.
> That way, after installing the package, you don't have to 
> `require` it first,
> you can just do `M-x nano-agenda`.




^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: ELPA: new package nano-agenda
  2021-10-18 23:05         ` Stefan Monnier
@ 2021-10-19 11:55           ` Philip Kaludercic
  2021-10-19 12:07             ` Stefan Kangas
  0 siblings, 1 reply; 12+ messages in thread
From: Philip Kaludercic @ 2021-10-19 11:55 UTC (permalink / raw)
  To: Stefan Monnier; +Cc: Adam Porter, Nicolas P. Rougier (inria), emacs-devel

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

>>> What might be an issue is if the "ts-" namespace is considered to be
>>> "too valuable", as was the issue with "s-" and "f-". The maintainers
>>> would have to decide on that, I can only speculate.
>
> two-letter (well, 3 if you count the dash) prefixes are fine, thanks
> (we have loads of them).

Really? When I run 

        (seq-count
         (lambda (ent)
           (<= (length (symbol-name (car ent))) 2))
         package-archive-contents)

all I get is one result (xr).

-- 
	Philip Kaludercic



^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: ELPA: new package nano-agenda
  2021-10-19 11:55           ` Philip Kaludercic
@ 2021-10-19 12:07             ` Stefan Kangas
  2021-10-19 12:24               ` Philip Kaludercic
  0 siblings, 1 reply; 12+ messages in thread
From: Stefan Kangas @ 2021-10-19 12:07 UTC (permalink / raw)
  To: Philip Kaludercic
  Cc: Adam Porter, emacs-devel, Stefan Monnier,
	Nicolas P. Rougier (inria)

Philip Kaludercic <philipk@posteo.net> writes:

> all I get is one result (xr).

~/wip/emacs $ find -name '??.el' | wc -l
      28



^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: ELPA: new package nano-agenda
  2021-10-19 12:07             ` Stefan Kangas
@ 2021-10-19 12:24               ` Philip Kaludercic
  2021-10-21 14:28                 ` Philip Kaludercic
  0 siblings, 1 reply; 12+ messages in thread
From: Philip Kaludercic @ 2021-10-19 12:24 UTC (permalink / raw)
  To: Stefan Kangas
  Cc: Adam Porter, emacs-devel, Stefan Monnier,
	Nicolas P. Rougier (inria)

Stefan Kangas <stefan@marxist.se> writes:

> Philip Kaludercic <philipk@posteo.net> writes:
>
>> all I get is one result (xr).
>
> ~/wip/emacs $ find -name '??.el' | wc -l
>       28

Ah, the core has more, but even then a number are obsolete (cl, gs, vi),
some are autogenerated (PY, QJ, SW) and a number are part of specific
packages (org has ob, oc, ol, ox; cedet as db, fw, ia, sb, el). The only
real two packages names are bs, js and vc (if you include Elisp
libraries, then gv, pp and rx would be included too).

The other question with ts is should the missing functionality be added
directly to Emacs itself?



^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: ELPA: new package nano-agenda
  2021-10-19 12:24               ` Philip Kaludercic
@ 2021-10-21 14:28                 ` Philip Kaludercic
  0 siblings, 0 replies; 12+ messages in thread
From: Philip Kaludercic @ 2021-10-21 14:28 UTC (permalink / raw)
  To: Stefan Kangas
  Cc: Adam Porter, Nicolas P. Rougier (inria), Stefan Monnier,
	emacs-devel

Philip Kaludercic <philipk@posteo.net> writes:

> Stefan Kangas <stefan@marxist.se> writes:
>
>> Philip Kaludercic <philipk@posteo.net> writes:
>>
>>> all I get is one result (xr).
>>
>> ~/wip/emacs $ find -name '??.el' | wc -l
>>       28
>
> Ah, the core has more, but even then a number are obsolete (cl, gs, vi),
> some are autogenerated (PY, QJ, SW) and a number are part of specific
> packages (org has ob, oc, ol, ox; cedet as db, fw, ia, sb, el). The only
> real two packages names are bs, js and vc (if you include Elisp
> libraries, then gv, pp and rx would be included too).
>
> The other question with ts is should the missing functionality be added
> directly to Emacs itself?

Either way, something should be done. ts.el should be added or
nano-agenda  should remove the (silent) dependency, because in it's
current state it breaks for anyone who doesn't install ts manually.

-- 
	Philip Kaludercic



^ permalink raw reply	[flat|nested] 12+ messages in thread

end of thread, other threads:[~2021-10-21 14:28 UTC | newest]

Thread overview: 12+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2021-10-12 17:01 ELPA: new package nano-agenda Nicolas P. Rougier (inria)
2021-10-12 19:19 ` Philip Kaludercic
2021-10-12 19:29   ` Nicolas P. Rougier (inria)
2021-10-12 19:41     ` Philip Kaludercic
2021-10-18 22:50       ` Adam Porter
2021-10-18 23:05         ` Stefan Monnier
2021-10-19 11:55           ` Philip Kaludercic
2021-10-19 12:07             ` Stefan Kangas
2021-10-19 12:24               ` Philip Kaludercic
2021-10-21 14:28                 ` Philip Kaludercic
2021-10-19  6:39 ` Joost Kremers
2021-10-19 10:19   ` Nicolas P. Rougier (inria)

Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/emacs.git
	https://git.savannah.gnu.org/cgit/emacs/org-mode.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.