* manual: stale bindings for org-clock-report and org-columns-insert-dblock
@ 2019-01-11 2:19 Kyle Meyer
2019-01-11 11:37 ` stardiviner
2019-03-03 15:44 ` Nikolay Kudryavtsev
0 siblings, 2 replies; 12+ messages in thread
From: Kyle Meyer @ 2019-01-11 2:19 UTC (permalink / raw)
To: stardiviner, emacs-orgmode
Hello,
34b71a0ca (Add a dispatcher command for inserting dynamic blocks,
2018-12-23) removed the key bindings of org-clock-report and
org-columns-insert-dblock, but these bindings are still listed in
org-manual.org. stardiviner, could you please have a look at updating
the manual to fully capture the changes from 34b71a0ca? I think it'd
also be good to mention these removed key bindings in ORG-NEWS.
Thanks.
--
Kyle
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: manual: stale bindings for org-clock-report and org-columns-insert-dblock
2019-01-11 2:19 manual: stale bindings for org-clock-report and org-columns-insert-dblock Kyle Meyer
@ 2019-01-11 11:37 ` stardiviner
2019-03-03 15:44 ` Nikolay Kudryavtsev
1 sibling, 0 replies; 12+ messages in thread
From: stardiviner @ 2019-01-11 11:37 UTC (permalink / raw)
To: Kyle Meyer; +Cc: emacs-orgmode
Kyle Meyer <kyle@kyleam.com> writes:
> Hello,
>
> 34b71a0ca (Add a dispatcher command for inserting dynamic blocks,
> 2018-12-23) removed the key bindings of org-clock-report and
> org-columns-insert-dblock, but these bindings are still listed in
> org-manual.org. stardiviner, could you please have a look at updating
> the manual to fully capture the changes from 34b71a0ca? I think it'd
> also be good to mention these removed key bindings in ORG-NEWS.
>
> Thanks.
You're right, I have not noticed them. I will add a patch to submit
soon. Thanks for your mention, Kyle.
--
[ stardiviner ]
I try to make every word tell the meaning what I want to express.
Blog: https://stardiviner.github.io/
IRC(freenode): stardiviner, Matrix: stardiviner
GPG: F09F650D7D674819892591401B5DF1C95AE89AC3
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: manual: stale bindings for org-clock-report and org-columns-insert-dblock
2019-01-11 2:19 manual: stale bindings for org-clock-report and org-columns-insert-dblock Kyle Meyer
2019-01-11 11:37 ` stardiviner
@ 2019-03-03 15:44 ` Nikolay Kudryavtsev
2019-03-03 17:14 ` Kyle Meyer
2019-03-04 13:26 ` stardiviner
1 sibling, 2 replies; 12+ messages in thread
From: Nikolay Kudryavtsev @ 2019-03-03 15:44 UTC (permalink / raw)
To: Kyle Meyer, stardiviner, emacs-orgmode
Am I reading it correctly that C-c C-x C-r is no longer a valid
keybinding? Is this the reason why it now behaves so stupidly, creating
a block with :scope file and asking for that file? Can someone point me
to a discussion of that change? Manual is still not updated about it and
nothing in ORG-NEWS.
Quickly using that key was quite useful to pop a clock report for a task
and I really don't see why would anyone want to remove that.
--
Best Regards,
Nikolay Kudryavtsev
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: manual: stale bindings for org-clock-report and org-columns-insert-dblock
2019-03-03 15:44 ` Nikolay Kudryavtsev
@ 2019-03-03 17:14 ` Kyle Meyer
2019-03-04 15:16 ` Nikolay Kudryavtsev
2019-03-04 13:26 ` stardiviner
1 sibling, 1 reply; 12+ messages in thread
From: Kyle Meyer @ 2019-03-03 17:14 UTC (permalink / raw)
To: Nikolay Kudryavtsev, stardiviner, emacs-orgmode
Nikolay Kudryavtsev <nikolay.kudryavtsev@gmail.com> writes:
> Am I reading it correctly that C-c C-x C-r is no longer a valid
> keybinding?
Yes, the binding was removed in 34b71a0ca (Add a dispatcher command for
inserting dynamic blocks, 2018-12-23).
> Is this the reason why it now behaves so stupidly, creating
> a block with :scope file and asking for that file?
[You could have phrased that less abrasively.]
I don't know. I don't use org-clock-report, haven't looked closely at
34b71a0ca, and didn't follow the discussion around the change, but ...
> Can someone point me to a discussion of that change?
I think the relevant discussion started with 871s7lvn4n.fsf@gmail.com
* https://lists.gnu.org/archive/html/emacs-orgmode/2018-11/msg00183.html
* https://lists.gnu.org/archive/html/emacs-orgmode/2018-12/msg00026.html
--
Kyle
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: manual: stale bindings for org-clock-report and org-columns-insert-dblock
2019-03-03 15:44 ` Nikolay Kudryavtsev
2019-03-03 17:14 ` Kyle Meyer
@ 2019-03-04 13:26 ` stardiviner
1 sibling, 0 replies; 12+ messages in thread
From: stardiviner @ 2019-03-04 13:26 UTC (permalink / raw)
To: Nikolay Kudryavtsev; +Cc: emacs-orgmode
Nikolay Kudryavtsev <nikolay.kudryavtsev@gmail.com> writes:
> Am I reading it correctly that C-c C-x C-r is no longer a valid keybinding? Is
> this the reason why it now behaves so stupidly, creating a block with :scope
> file and asking for that file? Can someone point me to a discussion of that
> change? Manual is still not updated about it and nothing in ORG-NEWS.
>
> Quickly using that key was quite useful to pop a clock report for a task and I
> really don't see why would anyone want to remove that.
I only added dispatcher for commands, but not modify command functions itself.
If I remember correctly, that commit indeed added changes in ORG-NEWS. So...
That's all I can say. I suggestion you check out Git log history with blame, to
figure what change that command behavior.
And don't be rude to people.
--
[ stardiviner ]
I try to make every word tell the meaning what I want to express.
Blog: https://stardiviner.github.io/
IRC(freenode): stardiviner, Matrix: stardiviner
GPG: F09F650D7D674819892591401B5DF1C95AE89AC3
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: manual: stale bindings for org-clock-report and org-columns-insert-dblock
2019-03-03 17:14 ` Kyle Meyer
@ 2019-03-04 15:16 ` Nikolay Kudryavtsev
2019-03-04 16:11 ` Nicolas Goaziou
2019-03-04 18:32 ` Nick Dokos
0 siblings, 2 replies; 12+ messages in thread
From: Nikolay Kudryavtsev @ 2019-03-04 15:16 UTC (permalink / raw)
To: Kyle Meyer, stardiviner, emacs-orgmode
Thanks for the tip.
After reading that discussion and a few hours of research, there are
actually three different problems I have with org-clock-report.
The first is that drop in clock reports now use :scope file by default,
instead of :scope subtree.
The second is that the clock report function is set up in some weird way
that it gets broken if you use icicles. Gonna look into it further, not
sure whether org or icicles is the bad actor here. The behavior I'm
seeing is as follows:
The command spawns empty table with :scope file, asks for a file, when I
select that file, it asks for a function and I have no idea which
function should I choose here.
For those two problems I've bisected the offending commit to 1a5151c51.
The third is the removal of C-c C-x C-r. I've read the discussion there,
but it seems like nobody really wanted it removed, this is just a
consequence of refactoring. So, can we get C-c C-x C-r org-clock-report
back, if only as a closure over org-dynamic-block-insert-dblock?
Also, ever since that Stardiviner's commit the dynamic reloading of org
seems to be broken on master. When I try to I get:
Symbol’s function definition is void: org-dynamic-block-define
Another thing, since org is moving towards unifying dynamic blocks,
maybe the next step is to add begin_src blocks into the mix in
org-dynamic-block-insert-dblock? It seems that they historically have
different syntax, but are in practice the same thing. That's actually
one place where I could use a wizard.
And why the function is called org-dynamic-block-insert-dblock and not
org-insert-dblock? Seems redundant to me.
Also can someone explain how do I get from org elpa version to the git
commit it's based on? E. g., in org-version.el I have
"9.2.1-33-g029cf6-elpaplus" but g029cf6 does not seem like a real commit
in the git repo.
--
Best Regards,
Nikolay Kudryavtsev
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: manual: stale bindings for org-clock-report and org-columns-insert-dblock
2019-03-04 15:16 ` Nikolay Kudryavtsev
@ 2019-03-04 16:11 ` Nicolas Goaziou
2019-03-05 1:51 ` Nikolay Kudryavtsev
2019-03-09 20:09 ` Nikolay Kudryavtsev
2019-03-04 18:32 ` Nick Dokos
1 sibling, 2 replies; 12+ messages in thread
From: Nicolas Goaziou @ 2019-03-04 16:11 UTC (permalink / raw)
To: Nikolay Kudryavtsev; +Cc: emacs-orgmode
Hello,
Nikolay Kudryavtsev <nikolay.kudryavtsev@gmail.com> writes:
> The first is that drop in clock reports now use :scope file by
> default, instead of :scope subtree.
Indeed. This is because clocktable default properties include :scope
property. I removed it. Thank you.
> The second is that the clock report function is set up in some weird
> way that it gets broken if you use icicles. Gonna look into it
> further, not sure whether org or icicles is the bad actor here. The
> behavior I'm seeing is as follows:
>
> The command spawns empty table with :scope file, asks for a file, when
> I select that file, it asks for a function and I have no idea which
> function should I choose here.
I don't use Icicles, but I cannot reproduce it. What are the exact steps
to do so?
> The third is the removal of C-c C-x C-r. I've read the discussion
> there, but it seems like nobody really wanted it removed, this is just
> a consequence of refactoring. So, can we get C-c C-x C-r
> org-clock-report back, if only as a closure over
> org-dynamic-block-insert-dblock?
I don't see the need for that. You can, however, bind `C-c C-x C-r' to
`org-clock-report' in your config.
> Also, ever since that Stardiviner's commit the dynamic reloading of
> org seems to be broken on master. When I try to I get:
>
> Symbol’s function definition is void: org-dynamic-block-define
You seem to have a mixed installation. This function only exists in
development branch, i.e., Org 9.3, not ELPA.
> Another thing, since org is moving towards unifying dynamic blocks,
> maybe the next step is to add begin_src blocks into the mix in
> org-dynamic-block-insert-dblock? It seems that they historically have
> different syntax, but are in practice the same thing. That's actually
> one place where I could use a wizard.
My practice differs. I don't see them as the same thing. Their syntax is
vaguely related, but that's all.
Note you can insert source blocks with 'C-c C-,`
> And why the function is called org-dynamic-block-insert-dblock and not
> org-insert-dblock? Seems redundant to me.
Hysterical raisins, I assume.
> Also can someone explain how do I get from org elpa version to the git
> commit it's based on? E. g., in org-version.el I have
> "9.2.1-33-g029cf6-elpaplus" but g029cf6 does not seem like a real
> commit in the git repo.
Drop the "g". The relative commit is "029cf6".
Regards,
--
Nicolas Goaziou
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: manual: stale bindings for org-clock-report and org-columns-insert-dblock
2019-03-04 15:16 ` Nikolay Kudryavtsev
2019-03-04 16:11 ` Nicolas Goaziou
@ 2019-03-04 18:32 ` Nick Dokos
1 sibling, 0 replies; 12+ messages in thread
From: Nick Dokos @ 2019-03-04 18:32 UTC (permalink / raw)
To: emacs-orgmode
Nikolay Kudryavtsev <nikolay.kudryavtsev@gmail.com> writes:
> ...
> Also can someone explain how do I get from org elpa version to the git
> commit it's based on? E. g., in org-version.el I have
> "9.2.1-33-g029cf6-elpaplus" but g029cf6 does not seem like a real
> commit in the git repo.
Try it without the 'g': that's just a prefix. The commit is 029cf6.
--
Nick
"There are only two hard problems in computer science: cache
invalidation, naming things, and off-by-one errors." -Martin Fowler
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: manual: stale bindings for org-clock-report and org-columns-insert-dblock
2019-03-04 16:11 ` Nicolas Goaziou
@ 2019-03-05 1:51 ` Nikolay Kudryavtsev
2019-03-09 20:09 ` Nikolay Kudryavtsev
1 sibling, 0 replies; 12+ messages in thread
From: Nikolay Kudryavtsev @ 2019-03-05 1:51 UTC (permalink / raw)
To: emacs-orgmode
[-- Attachment #1: Type: text/plain, Size: 1060 bytes --]
> I don't use Icicles, but I cannot reproduce it. What are the exact steps
> to do so?
Sorry, it's not clear whether you use Icicles. At the moment the recipe is:
1. Install icicles as described here
<https://www.emacswiki.org/emacs/Icicles_-_Libraries>. Enable it by
doing icy-mode.
2. Do org-clock-report.
I'll try to see whether there's some other way this odd behavior shows
itself, that can be seen without using Icicles. At the moment, do you
see anything in that commit that made this command, let's say abnormal?
> I don't see the need for that. You can, however, bind `C-c C-x C-r' to
> `org-clock-report' in your config.
Minor thing, but I still see this as a regression, without a good
reason, but it's your call...
> You seem to have a mixed installation. This function only exists in
> development branch, i.e., Org 9.3, not ELPA.
I'll double check and report.
> Note you can insert source blocks with 'C-c C-,`
Didn't knew that, thanks.
> Drop the "g". The relative commit is "029cf6".
Thanks.
--
Best Regards,
Nikolay Kudryavtsev
[-- Attachment #2: Type: text/html, Size: 1952 bytes --]
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: manual: stale bindings for org-clock-report and org-columns-insert-dblock
2019-03-04 16:11 ` Nicolas Goaziou
2019-03-05 1:51 ` Nikolay Kudryavtsev
@ 2019-03-09 20:09 ` Nikolay Kudryavtsev
2019-03-10 20:52 ` Nicolas Goaziou
1 sibling, 1 reply; 12+ messages in thread
From: Nikolay Kudryavtsev @ 2019-03-09 20:09 UTC (permalink / raw)
To: emacs-orgmode
I was able to debug the Icicles interaction bug:
In org-dblock-write:clocktable there's a check ((pred functionp)
(funcall scope)) which checks whether the scope is a function. Since the
default scope is currently "file", it gets funcalled. Icicles provides a
function called file. There it can be disabled by setting (setq
icicle-define-alias-commands-flag nil), but getting to that would
generally mean that the user ran into some problems and did the same
debugging I had to do.
I'm not 100% sure but I'd rather prefer org not calling any functions
with generic names like "file" or "subtree", so maybe prepending some
predicate on a call is warranted, so instead of "file", something like
"org-scope-file" would get checked, but I don't really understand the
intended use of this feature.
As for dynamic reloading, I retested and can confirm that it's broken
since 69c300bbf. To repeat, start with elpa org, add master org to the
load path, do org-reload.
--
Best Regards,
Nikolay Kudryavtsev
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: manual: stale bindings for org-clock-report and org-columns-insert-dblock
2019-03-09 20:09 ` Nikolay Kudryavtsev
@ 2019-03-10 20:52 ` Nicolas Goaziou
2019-05-07 15:40 ` Nikolay Kudryavtsev
0 siblings, 1 reply; 12+ messages in thread
From: Nicolas Goaziou @ 2019-03-10 20:52 UTC (permalink / raw)
To: Nikolay Kudryavtsev; +Cc: emacs-orgmode
Hello,
Nikolay Kudryavtsev <nikolay.kudryavtsev@gmail.com> writes:
> I was able to debug the Icicles interaction bug:
>
> In org-dblock-write:clocktable there's a check ((pred functionp)
> (funcall scope)) which checks whether the scope is a function. Since
> the default scope is currently "file", it gets funcalled. Icicles
> provides a function called file. There it can be disabled by setting
> (setq icicle-define-alias-commands-flag nil), but getting to that
> would generally mean that the user ran into some problems and did the
> same debugging I had to do.
Icicles is doing it wrong. Packages are supposed to provide only
prefixed functions, since there is no namespace in Emacs.
> I'm not 100% sure but I'd rather prefer org not calling any functions
> with generic names like "file" or "subtree", so maybe prepending some
> predicate on a call is warranted, so instead of "file", something like
> "org-scope-file" would get checked, but I don't really understand the
> intended use of this feature.
I pushed a different change. Tell me if it fixes your issue.
> As for dynamic reloading, I retested and can confirm that it's broken
> since 69c300bbf. To repeat, start with elpa org, add master org to the
> load path, do org-reload.
So, you are doing a mixed installation on purpose. No wonder you have
troubles. Don't reload a different installation on top of a loaded one.
Regards,
--
Nicolas Goaziou
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: manual: stale bindings for org-clock-report and org-columns-insert-dblock
2019-03-10 20:52 ` Nicolas Goaziou
@ 2019-05-07 15:40 ` Nikolay Kudryavtsev
0 siblings, 0 replies; 12+ messages in thread
From: Nikolay Kudryavtsev @ 2019-05-07 15:40 UTC (permalink / raw)
To: emacs-orgmode
Thanks, everything's fine now. Sorry for late reply.
> Icicles is doing it wrong. Packages are supposed to provide only
> prefixed functions, since there is no namespace in Emacs.
As far as I could grasp, those are shorthand alias functions created for
easier access. Luckily that behavior can be disabled by a variable. And
they're probably enabled by default because nobody complained yet.
> So, you are doing a mixed installation on purpose. No wonder you have
> troubles. Don't reload a different installation on top of a loaded one.
Yes, wishful thinking on my part. Been doing that for ages and the
reason I didn't ran into many issues is that I was usually reloading
versions that are pretty close, while bisecting for bugs. In my defense
I assumed that org-reload does some powerful unload magic. And it's
definitely a kind of magic that I would be scared to even look at, so it
never crossed my mind to look into it's sources.;-)
--
Best Regards,
Nikolay Kudryavtsev
^ permalink raw reply [flat|nested] 12+ messages in thread
end of thread, other threads:[~2019-05-07 15:40 UTC | newest]
Thread overview: 12+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-01-11 2:19 manual: stale bindings for org-clock-report and org-columns-insert-dblock Kyle Meyer
2019-01-11 11:37 ` stardiviner
2019-03-03 15:44 ` Nikolay Kudryavtsev
2019-03-03 17:14 ` Kyle Meyer
2019-03-04 15:16 ` Nikolay Kudryavtsev
2019-03-04 16:11 ` Nicolas Goaziou
2019-03-05 1:51 ` Nikolay Kudryavtsev
2019-03-09 20:09 ` Nikolay Kudryavtsev
2019-03-10 20:52 ` Nicolas Goaziou
2019-05-07 15:40 ` Nikolay Kudryavtsev
2019-03-04 18:32 ` Nick Dokos
2019-03-04 13:26 ` stardiviner
Code repositories for project(s) associated with this public inbox
https://git.savannah.gnu.org/cgit/emacs/org-mode.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).