Re: [SUMMARY] [[bbb:OrgMeetup]] on Wed, Aug 9, 19:00 UTC+3

2023-08-13 Thread Esteban Ordóñez
Thank you for the great summary, Ihor Radchenko!

Re: Htmlize support, maintenance, and Org mode (was: [MAINTENANCE] Org orphanage?)

2023-08-13 Thread Samuel Wales
[fyi that is probably not related: i use htmlize.el for functions it has that allow you to copy a region omitting invisible parts. e.g. partly folded magit. i haven't found other code that workd for that and myb rain could not construct any.] On 8/13/23, Ihor Radchenko wrote: > Jonas

Re: [POLL] Should we accept breaking changes to get rid of Org libraries that perform side effects when loading?

2023-08-13 Thread Samuel Wales
unable to do much of a search atm. but i recall 3-4 org vars that used to say so in their docstrings but didn't seem to need to be to me at the time. perhaps they have been fixed or i was mistaken. regexp components docstring in bugfix still say reload or restart. biut mayube that is obsolete.

Re: [PATCH] Re: what is the purpose of "This link has already been stored"?

2023-08-13 Thread Samuel Wales
i currently want to copy a link location and then paste that link loadtion. i don't recall theis breaking before. it does now. On 8/13/23, Ihor Radchenko wrote: > Bastien Guerry writes: > >> Here is another suggestion: >> >> 1) Remove the option and make adding the dup link on top the

Re: Passing table to Ruby session

2023-08-13 Thread Mike Gauland
On 11/08/23 19:47, Ihor Radchenko wrote: On Org side, the best we might do is splitting the long command into multiline (if ruby REPL supports line continuation like \ this or similar). Of course, it will be a workaround. I've redefined org-babel-ruby-var-to-ruby in my init.el to add a

Re: org-bibtex-yank failing with error Wrong type argument: stringp, nil

2023-08-13 Thread Ihor Radchenko
Jeremias Gonzalez writes: > On Sun, Aug 13, 2023 at 8:44 AM Ihor Radchenko wrote: > >> Does the attached patch fix the problem? >> > > The patch works for me, thank you very much! Thanks for testing! Applied, onto bugfix.

Re: org-bibtex-yank failing with error Wrong type argument: stringp, nil

2023-08-13 Thread Jeremias Gonzalez
On Sun, Aug 13, 2023 at 8:44 AM Ihor Radchenko wrote: > Does the attached patch fix the problem? > The patch works for me, thank you very much!

Re: org-bibtex-yank failing with error Wrong type argument: stringp, nil

2023-08-13 Thread Ihor Radchenko
Jeremias Gonzalez writes: >> Do I understand correctly that you attempted to run org-bibtex-read from >> a non-bibtex buffer? >> > > At a top level, I was using org-bibtex-yank with an org buffer (and in my > ideal case with no bibtex buffers whatsoever) as the documentation for that > function

Htmlize support, maintenance, and Org mode (was: [MAINTENANCE] Org orphanage?)

2023-08-13 Thread Ihor Radchenko
Jonas Bernoulli writes: > `htmlize' is currently maintained at https://github.com/hniksic/emacs-htmlize > but its maintainer hasn't been responding to any issues and pull-requests > for quite some time now and seems to be inactive on Github altogether. Hmm... Org has built-in htmlize support

Re: org-bibtex-yank failing with error Wrong type argument: stringp, nil

2023-08-13 Thread Jeremias Gonzalez
On Sun, Aug 13, 2023 at 12:57 AM Ihor Radchenko wrote: > "J. G." writes: > > > Adding the following two lines to my init fixes the error in my case: > > (require 'bibtex)(bibtex-set-dialect 'biblatex nil) > > Do I understand correctly that you attempted to run org-bibtex-read from > a

Re: [PATCH] org.el: Respect org-extend-today-until in timestamps with ++

2023-08-13 Thread Ihor Radchenko
"Valentin G. J. Herrmann" via "General discussions about Org-mode." writes: > In the attachments is a better version with tests that also addresses > your latest critique. Thanks! > + (setq time > +(save-match-data > +

Re: [PATCH] org.el: Respect org-extend-today-until in timestamps with ++

2023-08-13 Thread Valentin G. J. Herrmann
In the attachments is a better version with tests that also addresses your latest critique. On 13/08/2023 14.44, Ihor Radchenko wrote: > Ihor Radchenko writes: > >>> (while (or (= nshift 0) >>> - (not (time-less-p nil time))) >>> +

Re: [patch] Fix inner smart quotes in French

2023-08-13 Thread Ihor Radchenko
Bastien Guerry writes: >> Are you referring to `org-export-smart-quotes-alist'? It is a defconst. > > Ah, indeed. I'd say using a defcustom here would be useful. Is changing defconst to defcustom ok for bugfix? -- Ihor Radchenko // yantar92, Org mode contributor, Learn more about Org mode at

Re: [PATCH] org.el: Respect org-extend-today-until in timestamps with ++

2023-08-13 Thread Ihor Radchenko
Ihor Radchenko writes: >> (while (or (= nshift 0) >> - (not (time-less-p nil time))) >> + (not (time-less-p nil (time-add >> time-to-extend time > > And this compares "now" with year 1970. Will always

Re: [PATCH] org.el: Respect org-extend-today-until in timestamps with ++

2023-08-13 Thread Ihor Radchenko
Valentin Herrmann via "General discussions about Org-mode." writes: > * org.el (org-auto-repeat-maybe): Changed org-auto-repeat-maybe, so that > switching a repeating todo with a timestamp of the form <… ++…> respects > `org-extend-today-until'. Thanks, this would make sense. However, I have

Re: [PATCH] Re: what is the purpose of "This link has already been stored"?

2023-08-13 Thread Ihor Radchenko
Bastien Guerry writes: > Here is another suggestion: > > 1) Remove the option and make adding the dup link on top the default. > > 2) Also remove the current C-u C-u C-u arg and make it the default >when a region is active. > > (1) is because removing this option would be a breaking change,

Re: [DISCUSSION] The future of org-mouse.el and org-inlinetask.el

2023-08-13 Thread Ihor Radchenko
Bastien Guerry writes: >> *> TODO inlinetask >> Inlinetask contents >> *> END > > -1 on this one -- I'd rather get rid of the > > * TODO ... > * END > > construct altogether. I also don't like it, but have no better idea about a good markup to identify inlinetask body. Might be *> TODO

Re: [patch] Fix inner smart quotes in French

2023-08-13 Thread Bastien Guerry
Ihor Radchenko writes: >> It's difficult to say what's more common but since we cannot expect >> the « « inner » » style to be correctly implemented in, say, HTML, and >> since, even in LaTeX, babel-french defaults to the « “inner” » style, >> I suggest we stick to the « “inner” » style (this is

Re: [PATCH] Re: what is the purpose of "This link has already been stored"?

2023-08-13 Thread Bastien Guerry
Hi Ihor, Thanks for bearing with me while I discuss something is done, required time and work, and isn't probably a priority. Ihor Radchenko writes: > Bastien Guerry writes: > >> (1) by always store the latest link on top and remove old dups. >> >> (2) always store latest link at the top and

Re: [DISCUSSION] The future of org-mouse.el and org-inlinetask.el

2023-08-13 Thread Bastien Guerry
Ihor Radchenko writes: > I'd say that we should not remove org-mouse.el, but instead load it by > default. Maybe even fully integrating org-mouse.el into other libraries > like org-keys.el and org.el. +1 on that. > For example, we can define inlinetasks as > > *> TODO inlinetask > ***> TODO

Re: [patch] Fix inner smart quotes in French

2023-08-13 Thread Ihor Radchenko
Bastien Guerry writes: >> What I >> don't know is why babel-french defaults to the « “inner” » style. Is it >> the most used currently in France? If the « « inner » » style is more >> canonical, I don't mind having my patch 'fix' reverted. > > It's difficult to say what's more common but since

Re: [patch] Fix inner smart quotes in French

2023-08-13 Thread Bastien Guerry
Hi Juan, Juan Manuel Macías writes: > In any case (apart from LaTeX), it is clear that I was wrong when I said > that the inner quotes that my patch 'corrects' were 'incorrect'. Indeed, and thanks for the discussion, interesting. > What I > don't know is why babel-french defaults to the «

[PATCH] org.el: Respect org-extend-today-until in timestamps with ++

2023-08-13 Thread General discussions about Org-mode.
* org.el (org-auto-repeat-maybe): Changed org-auto-repeat-maybe, so that switching a repeating todo with a timestamp of the form <… ++…> respects `org-extend-today-until'. --- lisp/org.el | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/lisp/org.el b/lisp/org.el index

Re: [POLL] Should we accept breaking changes to get rid of Org libraries that perform side effects when loading?

2023-08-13 Thread Ihor Radchenko
Samuel Wales writes: > 3. > > istr loading org-id is or was what enables org-ids? i'd rather have > org-id work by default. OR maybe require activating. org-id is mostly fine, except that it (1) adds a new link type. (2) adds a hook that saves ids before exiting Emacs. In general, it is not

[DISCUSSION] The future of org-mouse.el and org-inlinetask.el (was: [POLL] Should we accept breaking changes to get rid of Org libraries that perform side effects when loading?)

2023-08-13 Thread Ihor Radchenko
Bastien Guerry writes: > Also, I'm not sure org-mouse.el has its place in Org's core nowadays. org-mouse implements mouse bindings + context menus. Context menu support is something that major modes generally expect to provide: 24.2.1 Major Mode Conventions • Consider adding mode-specific

Re: [PATCH] Testing: Ensure 'org-id-locations-file' is set before updating

2023-08-13 Thread Ihor Radchenko
Max Nikulin writes: >> In the absence of clear understanding what went wrong, I cannot accept >> this patch, unfortunately. > > mkdir ~/hide > mv -i ~/.emacs* ~/hide > make test-dirty BTEST_RE=test-org-link > > Finding ID locations (26/26 files): >

Re: how can I get the output of a =#+begin_src org= in the current buffer?

2023-08-13 Thread Ihor Radchenko
Edgar Lux writes: > Regarding the typo, it does not show here. But in the info buffer, the =(see > *note...= becomes (see see ...). > > See (:P) attachment This is some info rendering bug in Emacs 28. Nothing to do with Org. Also, it no longer occurs in Emacs 29. -- Ihor Radchenko //

Re: [BUG] make cursor visible in calendar [9.6.7 ( @ /Users/eugenekim/.emacs.d/elpa/org-9.6.7/)]

2023-08-13 Thread Ihor Radchenko
eugene kim writes: > I'd like to see the cursor when using keyboard to select a date from the > calendar when I use > org's schedule or deadline. > > It can be done with keyboard, but the cursor is not visible. May you please provide steps to reproduce the problem? For me, the selected date is

Re: [patch] ox-latex.el: fix blank lines behavior in verse block

2023-08-13 Thread Ihor Radchenko
Juan Manuel Macías writes: >>> + (concat "\\(" >>> + (regexp-quote org-latex-line-break-safe) >>> + "\n\\)" >>> + "\\(^[ \t]*" >>> + (regexp-quote org-latex-line-break-safe) >>> +

Re: [BUG] Warning (org-element-cache)

2023-08-13 Thread Ihor Radchenko
Carmen Shea writes: > The following Warning message appeared while I was entering text into an > Org-Roam file. > >  [BUG] Warning (org-element-cache): org-elemt--cache: Org parser error > in 2023073121272-student_reports_august_2023.org::7669. Resetting. The > error was:

Re: org-bibtex-yank failing with error Wrong type argument: stringp, nil

2023-08-13 Thread Ihor Radchenko
"J. G." writes: > Adding the following two lines to my init fixes the error in my case: > (require 'bibtex)(bibtex-set-dialect 'biblatex nil) Do I understand correctly that you attempted to run org-bibtex-read from a non-bibtex buffer? -- Ihor Radchenko // yantar92, Org mode contributor,

Re: [BUG][SECURITY] ob-sqlite header args allows execution of arbitrary shell commands

2023-08-13 Thread Ihor Radchenko
Max Nikulin writes: > 8< > #+begin_src elisp :results none >(require 'ob-sqlite) > #+end_src > > #+begin_src sqlite :db /tmp/ob.sqlite$(date >/tmp/ob-sqlite-vuln.log) >select 1 > #+end_src > >8 > > Executing of the sqlite code block causes creation of the >