Re: Saving some kitten, plus some questions along the way

2024-05-21 Thread Max Nikulin
On 20/05/2024 04:24, Stefan Monnier wrote: Subject: [PATCH 3/4] testing/org-test.el (): Remove dead code (featurep 'org) is always non-nil here since we have a (require 'org) further up. I suspect other `require`s nearby could be removed or moved to toplevel. Just a guess. Perhaps earlier

Re: Using a custom environment for a src language

2024-05-21 Thread Max Nikulin
On 20/05/2024 08:57, Rustom Mody wrote: I have my own Python environment called PYT. #+BEGIN_PYT contents #+END_PYT See `org-latex-custom-lang-environments' for #+begin_src python.

Re: Adding custom providers for thingatpt.el

2024-05-21 Thread Max Nikulin
On 20/05/2024 09:33, Jim Porter wrote: +++ b/lisp/org.el @@ -81,6 +81,7 @@ (require 'calendar) (require 'find-func) (require 'format-spec) +(require 'thingatpt) So it becomes hard dependency. However it seems thingatpt is anyway loaded through some indirect dependency.

Re: org-babel shell in windows not finishing command

2024-05-21 Thread Max Nikulin
On 21/05/2024 01:37, Ihor Radchenko wrote: echo 'echo foo' | cmdproxy.exe yields : Microsoft Windows [Version 10.0.19045.2251] : (c) Microsoft Corporation. All rights reserved. : : c:\Users\johndoe\Org>echo foo : foo Out of curiosity, does the /q option help to suppress the banner?

Re: [DISCUSSION] The meaning of :cmdline header argument across babel backends

2024-05-21 Thread Max Nikulin
On 21/05/2024 01:01, Matt wrote: I'm okay with these. I can start on a patch for :script-args and :program-args within ob-shell. Frankly speaking your plan is not clear for me. My special concern is DWIM behavior https://list.orgmode.org/874jbkcmyg.fsf@localhost (Ihor Radchenko Mon, 29 Apr

Re: [PATCH] Add support for shortdoc link type

2024-05-16 Thread Max Nikulin
On 13/05/2024 20:04, Bruno Cardoso wrote: On 2024-05-13, 18:14 +0700, Max Nikulin wrote: Sorry, but did you manually edit the patch? I didn't. I generated it again and applied without errors. Thank you. Now your patch can be applied on my side as well. Please, proceed with the copyright

Re: [POLL] Dealing with +1m/y repeaters when jumping to impossible date (should 05-31 +1m be 07-01 or 06-30?)

2024-05-16 Thread Max Nikulin
On 14/05/2024 19:56, Ihor Radchenko wrote: Max Nikulin writes: +(defun org-time-inc (unit value time) Is there a chance that support of intervals like 1h20m will be required later? Not sure again. I simply used ts-inc function signature from Adam's ts.el as reference. ts.el has `ts-adjust

Re: [PATCH] oc-csl: New custom option `org-cite-csl-sentence-case-bibtex-titles'

2024-05-15 Thread Max Nikulin
On 11/05/2024 22:33, András Simonyi wrote: since bibtex and biblatex requires title fields to be in title case but CSL assumes that they are in sentence-case, citeproc-el converts title fields in bib(la)tex bibliography databases into sentence-case before processing them except for entries with

Re: [POLL] Dealing with +1m/y repeaters when jumping to impossible date (should 05-31 +1m be 07-01 or 06-30?)

2024-05-15 Thread Max Nikulin
On 14/05/2024 19:56, Ihor Radchenko wrote: Max Nikulin writes: On 13/05/2024 17:07, Ihor Radchenko wrote: <2025-01-31 Fri +1m> <2025-02-28 Fri +1m> <2025-03-28 Fri +1m> Instead of using timestamp obtained on previous step, use original timestamp and multip

Re: [POLL] Dealing with +1m/y repeaters when jumping to impossible date (should 05-31 +1m be 07-01 or 06-30?)

2024-05-14 Thread Max Nikulin
On 13/05/2024 17:07, Ihor Radchenko wrote: However, there are still some issues remaining. When updating timestamps repeating monthly across months with 30, 31, and 28 days we get <2025-01-31 Fri +1m> <2025-02-28 Fri +1m> <2025-03-28 Fri +1m> ... <2026-01-28 Wed +1m> Instead of using

Re: [PATCH] Add support for shortdoc link type

2024-05-13 Thread Max Nikulin
On 11/05/2024 23:58, Bruno Cardoso wrote: On 2024-05-10, 18:09 +0700, Max Nikulin wrote: <https://orgmode.org/worg/org-contribute.html#copyright> I haven't yet. Will be glad to. I hope, next patch revision will be the final one, so it can be notification that the procedure is com

Re: [PATCH] Add support for shortdoc link type

2024-05-10 Thread Max Nikulin
On 09/05/2024 04:26, Bruno Cardoso wrote: On 2024-05-08, 18:20 +0700, Max Nikulin wrote: I have not managed to get all known groups with `shortdoc-display-group' in Emacs-28. Is it a feature of `shortdoc' in Emacs-29? In Emacs-29 `shortdoc' is an alias for `shortdoc-display-group

Re: [PATCH] Fix regex for determining image width from attribute

2024-05-09 Thread Max Nikulin
On 08/05/2024 17:54, Ihor Radchenko wrote: Max Nikulin writes: #+attr_html: :alt Image width test #+attr_beamer: :width \linewidth #+attr_latex: :width +.5\textwidth #+attr_md: :width 75% [[file:babelfish.png]] - I do not mind that just "\linewidth" is ignored. - The case of &q

Re: [PATCH] Add support for shortdoc link type

2024-05-08 Thread Max Nikulin
On 08/05/2024 08:11, Bruno Cardoso wrote: The following now work as expected: - [[shortdoc:string]] - [[shortdoc:string::#try-completion]] - [[shortdoc:string::identical contents]] Thanks. Bruno, the patch should have proper commit message, see

Re: [PATCH] Fix regex for determining image width from attribute

2024-05-08 Thread Max Nikulin
On 02/05/2024 23:28, Ihor Radchenko wrote: 1. #+attr_org is prioritised I ma afraid, the code is a bit fragile. Consider #+attr_html: :alt Image width test #+attr_beamer: :width \linewidth #+attr_latex: :width +.5\textwidth #+attr_md: :width 75% [[file:babelfish.png]] - I do not mind that

Re: [DISCUSSION] Sorting strings in Org mode vs. system locale

2024-05-07 Thread Max Nikulin
On 07/05/2024 20:09, Ihor Radchenko wrote: Max Nikulin writes: I consider the following as a kind of graceful degradation (defun org-sort-function-fallback-downcase (a b LOCALE IGNORE-CASE) (if ignore-case (string-collate-lessp (downcase a) (downcase b) locale ignore-case

Re: Bug: Final zero after decimal point is stripped when inline code evaluated

2024-05-07 Thread Max Nikulin
On 07/05/2024 21:25, Charles Millar wrote: On 5/7/24 7:49 AM, Max Nikulin wrote: On 07/05/2024 03:05, Charles Millar wrote: (setq toconvert 5.000) (number-to-string toconvert) evaluates to "5". I get "5.0" (Linux), so I suspect some mistake. I am using Debian T

Re: Bug: Final zero after decimal point is stripped when inline code evaluated

2024-05-07 Thread Max Nikulin
On 07/05/2024 03:05, Charles Millar wrote: I appreciate that mathematically a trailing zero or zeros may be non-significant; however, in my use case, i.e. correct format in a text, they are necessary. Another example, in addition to my Dollars and cents scenario, may be a table that that has

Re: [DISCUSSION] Sorting strings in Org mode vs. system locale

2024-05-07 Thread Max Nikulin
On 05/05/2024 18:59, Ihor Radchenko wrote: Ihor Radchenko writes: If a user access same files from Linux and macOS then it may be really annoying to get different order of entries in agenda. For most of Linux users it is better to use more smart `string-collate-lessp'. Some care is required to

Re: Bug: Final zero after decimal point is stripped when inline code evaluated

2024-05-06 Thread Max Nikulin
On 06/05/2024 19:44, Ihor Radchenko wrote: Charles Millar writes: #+NAME: TABLE2 |22578.60| src_latex[:var printthis=TABLE2[-1,-1] :eval yes :results replace :exports results]{\num{printthis}} {{{results(@@latex:\num{22578.6}@@)}}} This is expected. When you assign :var, Org mode reads the

[PATCH] test-org-macro.el: Add test for CVE-2024-30202 (was: Re: [ANN] Emergency bugfix release: Org mode 9.6.23)

2024-05-06 Thread Max Nikulin
af8cddb44f5ee01fb1c7c9cf664ddbc83c63ac56 Mon Sep 17 00:00:00 2001 From: Max Nikulin Date: Mon, 6 May 2024 19:04:17 +0700 Subject: [PATCH] test-org-macro.el: Add test for CVE-2024-30202 * testing/lisp/test-org-macro.el (test-org-macro/initialize-templates): A new test that no code is evaluated when an Org file

Re: [PATCH] Add support for shortdoc link type

2024-05-06 Thread Max Nikulin
On 06/05/2024 04:35, Bruno Cardoso wrote: Fixed. I have tried the patch. It mostly works, so I do not insist on further polishing. However it should be resent with proper description. I had dropped the search code because I wasn't considering the search option properly. If I understand

Re: [PATCH] Add support for shortdoc link type

2024-05-05 Thread Max Nikulin
On 05/05/2024 00:33, Bruno Cardoso wrote: I updated the patch to use the optional function argument instead. Thanks +++ b/doc/org-manual.org @@ -3372,6 +3372,10 @@ Here is the full set of built-in link types: Execute a shell command upon activation. +- =shortdoc= :: + +Link to short

Re: Importing "quoted" strings in `org-babel-import-elisp-from-file'

2024-05-04 Thread Max Nikulin
Max Nikulin writes: Quotes are not stripped: On 04/05/2024 18:51, Ihor Radchenko wrote: Fixed, on main. Another case: (org-babel-read "\"Newlines\"\n" t) "\"Newlines\" "

Re: Importing "quoted" strings in `org-babel-import-elisp-from-file'

2024-05-04 Thread Max Nikulin
On 04/05/2024 18:51, Ihor Radchenko wrote: Max Nikulin writes: Quotes are not stripped: (org-babel-read "\"abc\nsdf\"" t) "\"abc sdf\"" Fixed, on main. https://git.savannah.gnu.org/cgit/emacs/org-mode.git/commit/?id=edb5eaaac Thanks (org-babel-re

Re: Importing "quoted" strings in `org-babel-import-elisp-from-file'

2024-05-04 Thread Max Nikulin
On 04/05/2024 15:03, Ihor Radchenko wrote: Max Nikulin writes: I have no idea if "other\n\"string\"\nlines" may be passed `org-babel-read', but it is not discarded by the current regexp: "^[[:space:]]*\"\\(.*\\)\"[[:space:]]*$" I do not see why we sh

Re: Importing "quoted" strings in `org-babel-import-elisp-from-file'

2024-05-04 Thread Max Nikulin
On 03/05/2024 19:06, Ihor Radchenko wrote: Max Nikulin writes: What I do not like in `org-babel-read' is false positive for escaped quote when actually backslash is escaped: (org-babel-read "\"1\" 2 \"3\"" t) "1\\" Fixed, on main. https://git.

Re: [PATCH] Add support for shortdoc link type

2024-05-04 Thread Max Nikulin
On 04/05/2024 04:41, Bruno Cardoso wrote: On 2024-05-02, 17:41 +0700, Max Nikulin wrote: [...] I implemented the search option by borrowing the regexp from `ol-info': It is necessary to maintain backward compatibility for `ol-info'. I do not know what kind of separator was added earlier

Re: [BUG] HTML export does not preserve footnote label [9.6.15 (release_9.6.15 @ /usr/local/share/emacs/30.0.50/lisp/org/)]

2024-05-03 Thread Max Nikulin
On 03/05/2024 18:14, Ihor Radchenko wrote: Max Nikulin writes: Yes, sure. It is fine to reuse an existing user option. Though reading through its docstring and the code, I cannot tell what this is doing exactly. Is it applying to all HTML elements, or just headings? On my end, I have

Re: [BUG] HTML export does not preserve footnote label [9.6.15 (release_9.6.15 @ /usr/local/share/emacs/30.0.50/lisp/org/)]

2024-05-03 Thread Max Nikulin
On 03/05/2024 13:59, Protesilaos Stavrou wrote: From: Max Nikulin Fri, 26 Apr 2024 16:53:23 +0700 Another option may be to rely on the existing one: `org-html-prefer-user-labels' Yes, sure. It is fine to reuse an existing user option. Though reading through its docstring and the code, I

Re: link can not be created in a line with another link.

2024-05-03 Thread Max Nikulin
On 02/05/2024 18:15, Ihor Radchenko wrote: May you try the attached patch (on top of the latest main)? Does it feel better? I see regressions only (Emacs 28). [[#foo]] [[#bar]] SPC causes permanent appearance of square brackets for [[#foo]]. I do not like flashes especially since they do

Re: [BUG] #+CALL - permission error -bash

2024-05-03 Thread Max Nikulin
On 02/05/2024 21:55, Ihor Radchenko wrote: vitalij writes: in org-babel-sh-evaluate file:~/.emacs.d/elpa/org-9.6.28/ob-shell.el::300 this do apply: (process-file "/tmp/babel-NfRG9P/sh-script-jmKNA4" "/tmp/babel-NfRG9P/sh-stdin-o3CEm5" # nil nil) I don't allow executables in /tmp folder!

Re: Importing "quoted" strings in `org-babel-import-elisp-from-file'

2024-05-02 Thread Max Nikulin
On 01/05/2024 19:19, Ihor Radchenko wrote: And replacing the call to org-babel-string-read with org-babel-read does not break any tests... Since 005e68294 2009-06-11 17:04:42 -0700 Eric Schulte: making progress bringing org-babel-R.el into the new evaluation schema

Re: [PATCH] Add support for shortdoc link type

2024-05-02 Thread Max Nikulin
On 02/05/2024 02:24, Bruno Cardoso wrote: +++ b/lisp/ol.el @@ -1597,6 +1597,37 @@ PATH is a symbol name, as a string." (org-link-set-parameters "help" :follow #'org-link--open-help :store #'org-link--store-help) + "shortdoc" link type

[PATCH] test-ob-shell.el: Skip based on feature detection

2024-05-02 Thread Max Nikulin
On 28/04/2024 20:11, Ihor Radchenko wrote: https://git.savannah.gnu.org/cgit/emacs/org-mode.git/commit/?id=e4ab416fc Max Nikulin writes: Feature detection should be more reliable. [...] Feel free to submit a patch. See the attachment. From 46db53cebef5644be4abd8dcb18ca777ec0c3be3 Mon

Re: [BUG] ob-shell: internal representation of cmdline arguments the same

2024-05-01 Thread Max Nikulin
On 29/04/2024 19:22, Ihor Radchenko wrote: Matt writes: #+begin_src bash :cmdline "1 2 3" [...] To force quotes in the :cmdline one can do #+begin_src bash :cmdline "\"1 2 3\"" echo "$1" #+end_src I consider it as a kind of pitfall inconsistent with DWIM concept. An idea of a kludge is

Re: [PATCH] org.el: Call EXT-enable for `org-modules' (was Re: [PATCH] org-ctags.el: Do not activate on load)

2024-05-01 Thread Max Nikulin
On 01/05/2024 17:21, Ihor Radchenko wrote: Max Nikulin writes: However now I am in doubts why org-ctags is not a minor mode. What would be the benefit? Documented conventions how to enable or disable minor mode.

Re: [PATCH] ob-lua: Support all types and multiple values in results

2024-05-01 Thread Max Nikulin
On 30/04/2024 03:26, Rudolf Adamkovič wrote: +local value = result[1] +if string.find(value, '[%%(%%[{]') == 1 then + return quotes .. value .. quotes Am I wrong that quotes in value may cause issues? I expect some way of escaping " characters.

Re: link can not be created in a line with another link.

2024-05-01 Thread Max Nikulin
On 29/04/2024 19:45, Alexandros Prekates wrote: The same behavior happens with emacs -Q. bla [[foo]] If i add : [[boo]] bla [[foo]] org will create the two links. But if i add: [[boo][ bla [[foo]] with the intention to type: [[boo][description]] bla [[foo]] then what i will get is: a link

Re: [BUG] ob-shell: results missing leading quotes

2024-05-01 Thread Max Nikulin
On 29/04/2024 18:58, Ihor Radchenko wrote: The nested "..." are stripped on purpose via `org-babel-string-read'. Although this function dates back to R output processing and I do not fully understand why stripping nested quotes is useful for all possible babel backends. But that's a completely

Re: [PATCH] org-ctags.el: Do not activate on load

2024-04-30 Thread Max Nikulin
On 30/04/2024 19:59, Ihor Radchenko wrote: /bin/sh: 1: Syntax error: Bad function name (ert-test-failed ((should (test-org-ctags/list-elements-equal-p (list ...) (test-org-ctags/with-fake-ctags temp-dir "regexp" ...) '(2) "Regexp should be

[PATCH] org.el: Call EXT-enable for `org-modules' (was Re: [PATCH] org-ctags.el: Do not activate on load)

2024-04-30 Thread Max Nikulin
On 30/04/2024 17:02, Ihor Radchenko wrote: Max Nikulin writes: Isn't it better to modify buggy org-ctags than to add various kludges to tests? Applied, onto main. I was expecting that you would be against it since it is a breaking change. See a follow-up to Ihor Radchenko. Re: Autoloading

[PATCH] org-ctags.el: Do not activate on load

2024-04-29 Thread Max Nikulin
On 29/04/2024 20:12, Ihor Radchenko wrote: Max Nikulin writes: Notice that new tests for org-ctags do not require user interactions. [...] Of course, the cause is the known side effect of loading org-ctags. Maybe we can disable the tests until that bug is fixed. Or, ideally, load org-ctags

Re: [PATCH] org-ctags.el: Protect shell specials in directory name

2024-04-29 Thread Max Nikulin
On 29/04/2024 00:02, Ihor Radchenko wrote: Max Nikulin writes: Tests must be fully automated. We use make test in CI and things like project-compile are non-interactive. I do not mind. Requiring unexpected user interactions is a feature of org-ctags. So, may you update the patch to make

Re: [PATCH] org-ctags.el: Protect shell specials in directory name

2024-04-28 Thread Max Nikulin
On 28/04/2024 23:55, Ihor Radchenko wrote: Max Nikulin writes: I tried to run make test with your patch applied, but I am getting interactive prompt there: Visit tags table (default TAGS): You may press RET... It is due to invasive org-ctags behavior. Tests must be fully automated

Re: [PATCH] org-ctags.el: Protect shell specials in directory name

2024-04-28 Thread Max Nikulin
On 28/04/2024 19:53, Ihor Radchenko wrote: May you submit a patch? Max Nikulin writes: See the attachments. I tried to run make test with your patch applied, but I am getting interactive prompt there: Visit tags table (default TAGS): You may press RET... It is due to invasive org

[PATCH] org-ctags.el: Protect shell specials in directory name

2024-04-28 Thread Max Nikulin
On 20/03/2024 19:08, Ihor Radchenko wrote: Max Nikulin writes: The committed change is anyway incomplete. May you submit a patch? See the attachments.From 067dc590bb1c26c881f14d218da2cd502413ec5d Mon Sep 17 00:00:00 2001 From: Max Nikulin Date: Wed, 27 Mar 2024 23:04:07 +0700 Subject

Re: [PATCH] Re: [BUG] ob-shell: :shebang changes interpretation of :cmdline

2024-04-27 Thread Max Nikulin
On 27/04/2024 17:31, Max Nikulin wrote: On 26/04/2024 18:49, Ihor Radchenko wrote: +    shell-file-name ... +    (list shell-command-switch +  (concat (file-local-name script-file)  " " cmdline)))) Max Niku

Re: [PATCH] ob-lua: Support all types and multiple values in results

2024-04-27 Thread Max Nikulin
On 26/04/2024 20:40, Ihor Radchenko wrote: +++ b/lisp/ob-lua.el @@ -282,6 +282,8 @@ (defvar org-babel-lua-wrapper-method end end return result + elseif type(it) == 'string' then + return '\"' .. it .. '\"' If you are adding quotes around then quote characters

Re: [DISCUSSION] The meaning of :cmdline header argument across babel backends

2024-04-27 Thread Max Nikulin
On 26/04/2024 20:09, Ihor Radchenko wrote: Max Nikulin writes: However looking wider, I do not like that :cmdline for ob-shell has different meaning than for other languages, see e.g. ob-sql. Only for shell this parameter is treated as arguments of a *script*. In other cases :cmdline is used

Re: [PATCH] Re: [BUG] ob-shell: :shebang changes interpretation of :cmdline

2024-04-27 Thread Max Nikulin
On 26/04/2024 18:49, Ihor Radchenko wrote: +shell-file-name ... +(list shell-command-switch + (concat (file-local-name script-file) " " cmdline)))) Max Nikulin writes: Using `shell-comm

Re: FAILED test-ob-shell/bash-uses-assoc-arrays

2024-04-26 Thread Max Nikulin
On 26/04/2024 18:08, Ihor Radchenko wrote: Max Nikulin writes: My guess is that GPLv2 BASH on macOS does not support associative arrays. Perhaps these tests should be skipped if BASH_VERSION is not fresh enough (not supplied by Apple). https://git.savannah.gnu.org/cgit/emacs/org-mode.git

Re: [BUG] HTML export does not preserve footnote label [9.6.15 (release_9.6.15 @ /usr/local/share/emacs/30.0.50/lisp/org/)]

2024-04-26 Thread Max Nikulin
On 26/04/2024 15:17, Protesilaos Stavrou wrote: Since we are now using labels for the HTML export, I think it makes sense to optionally use those for the anchor tags as well. [...] +(defcustom org-html-footnote-use-label-for-anchor-text nil Another option may be to rely on the existing one:

Re: Trailing whitespace after export snippets without a transcoder

2024-04-25 Thread Max Nikulin
On 23/04/2024 02:01, Ihor Radchenko wrote: For example, consider an HTML exporter that aligns tags nicely and keeps blank lines between markup blocks for readability. If we remove such blank lines unconditionally, it will be problematic. I consider that just newlines are enough to make HTML

Re: [PATCH] Re: [BUG] ob-shell: :shebang changes interpretation of :cmdline

2024-04-25 Thread Max Nikulin
On 24/04/2024 19:52, Ihor Radchenko wrote: Max Nikulin writes: I believe, multiple arguments should be specified as '(1 a "b c"). Yes, but we do not, in general, know how to split them. Something should be changed anyway since current behavior is inconsistent and so is buggy.

Re: [BUG] No space after footnote with org-export-with-footnotes set to nil [9.6.1 ( @ /Users/test/.emacs.d/elpa/28.0/develop/org-9.6.1/)]

2024-04-24 Thread Max Nikulin
On 27/03/2023 21:16, Ihor Radchenko wrote: +++ b/etc/ORG-NEWS @@ -145,6 +145,27 @@ execution completes. The new ~:async~ header allows users to continue editing with Emacs while a ~:session~ block executes. ** Miscellaneous +*** Blank lines after removed objects are not retained during

Re: FAILED test-ob-shell/bash-uses-assoc-arrays

2024-04-24 Thread Max Nikulin
On 24/04/2024 19:54, Ihor Radchenko wrote: Max Nikulin writes: On 15/04/2024 23:46, Alexander Adolf wrote: FAILED test-ob-shell/bash-uses-assoc-arrays ((should (equal "two" (org-trim (org-babel-execute-src-block :form (equal "two" "three") :

FAILED test-ob-shell/bash-uses-assoc-arrays

2024-04-24 Thread Max Nikulin
On 15/04/2024 23:46, Alexander Adolf wrote: FAILED test-ob-shell/bash-uses-assoc-arrays ((should (equal "two" (org-trim (org-babel-execute-src-block :form (equal "two" "three") :value nil :explanation (arrays-of-different-length 3 5 "two" "three" first-mismatch-at 1)) FAILED

Re: [PATCH] Re: [BUG] ob-shell: :shebang changes interpretation of :cmdline

2024-04-24 Thread Max Nikulin
On 23/04/2024 17:28, Ihor Radchenko wrote: I propose the attached patch. +++ b/lisp/ob-shell.el @@ -322,14 +322,12 @@ (defun org-babel-sh-evaluate (session body params stdin cmdline) (with-temp-buffer (with-connection-local-variables (apply

Re: [PATCH] Re: [BUG] ob-shell: :shebang changes interpretation of :cmdline

2024-04-23 Thread Max Nikulin
On 23/04/2024 17:51, Max Nikulin wrote: I am in favor of dropping `shell-command-switch' in the latter case to pass arguments literally in both cases. Dropping "-c" may have side effects. Instead of :shebang, a source block may have shebang in the body #+begin_src bash #!/bin/ba

Re: [POLL] Should we enable or disable automatic tag alignment by default everywhere

2024-04-23 Thread Max Nikulin
On 23/04/2024 18:47, Ihor Radchenko wrote: Max Nikulin writes: There should be a way to align tags for a specific heading even when `org-auto-align-tags' is nil. I can convert org-align-tags into a command. I would prefer to have C-u C-c C-q (`org-set-tags-command') applied just

Re: [PATCH] Re: [BUG] ob-shell: :shebang changes interpretation of :cmdline

2024-04-23 Thread Max Nikulin
On 21/04/2024 22:09, Matt wrote: The proposed solution assumes we intend to parse the characters following :cmdline as space delimited and grouped by quotes. However, AFAICT, the parsing issue makes this solution ambiguous. Matt, I am sorry, but I do not agree with your proposal. I do not

Re: MathML and ODT export: inline possible?

2024-04-22 Thread Max Nikulin
On 22/04/2024 16:05, Fraga, Eric wrote: On Friday, 19 Apr 2024 at 23:19, Max Nikulin wrote: MathJax may be your friend. LaTeXML and katex do not add as well. MathJax works well but for websites; my context is that I need to prepare a Word document to share with others. I have not tried

Re: Trailing whitespace after export snippets without a transcoder

2024-04-22 Thread Max Nikulin
On 21/04/2024 20:00, Ihor Radchenko wrote: Max Nikulin writes: A space@@ascii:*@@ character. Hmm. We actually have a similar scenario in `org-export--prune-tree' with a slightly different logic - only keep spaces when previous object does not have any. I like this idea. At first I

Re: Trailing whitespace after export snippets without a transcoder

2024-04-20 Thread Max Nikulin
On 20/04/2024 18:14, Ihor Radchenko wrote: https://git.savannah.gnu.org/cgit/emacs/org-mode.git/commit/?id=a6d9fd82e Back-end developers should pay attention to the fact that white spaces before and after an ignored export snippet now are accumulated in the output. I have no

Re: [BUG] LaTeX preview should use a subdirectory in /tmp

2024-04-20 Thread Max Nikulin
On 19/04/2024 22:00, emm.charpent...@free.fr wrote: Embedding /home/charpent/WinFiles/Temporaire/Org/file:ltximg/EssaiSnippetsLaTeX_2923a1941c0e63eab69544afa1be2ec0d95a01c4.png as Images/0001.png... See Ihor Radchenko. Re: strange export problem with a file: link. Sat, 20 Apr 2024 12:25:56

Re: MathML and ODT export: inline possible?

2024-04-19 Thread Max Nikulin
On 18/04/2024 22:51, Fraga, Eric wrote: In LaTeX, something like $a_{b_c}$. Pandoc generates a b c (where the a, b, and c are ...) whereas, by hand, I use a b c MathJax may be your friend. LaTeXML and katex do not add as well. Actually I asked to confirm that your troubles are not

Re: [Pre-PATCH] Overhaul of the LaTeX preview system

2024-04-18 Thread Max Nikulin
On 19/04/2024 00:58, Корякин Артём wrote: shell-file-name ; "pwsh.exe" system-type ; windows-nt (w32-shell-dos-semantics) ; nil I have no idea what style of escaping is expected by pwsh.exe. If it is the same as for cmd.exe then I would try to add it to

Re: [Pre-PATCH] Overhaul of the LaTeX preview system

2024-04-18 Thread Max Nikulin
On 18/04/2024 03:11, Корякин Артём wrote: Maybe I could somehow rewrite shell-quote-argument using advices ? Maybe some more global changes is needed for emacs on windows ? Please, evaluate the following expressions e.g. in the scratch buffer: shell-file-name system-type

Re: MathML and ODT export: inline possible?

2024-04-18 Thread Max Nikulin
On 18/04/2024 22:03, Fraga, Eric wrote: PS - also thanks for details on latexmlmath. I am still curious if pandoc can handle your math expressions.

Re: strange export problem with a file: link

2024-04-18 Thread Max Nikulin
On 18/04/2024 21:37, Fraga, Eric wrote: On Thursday, 18 Apr 2024 at 19:08, Max Nikulin wrote: It is a side effect of 72b0e9ff0 2024-02-05 16:39:05 +0100 Ihor Radchenko: org-export: Do not strip link type by default during export I have now checked the ORG-NEWS file, which I probably should

Re: MathML and ODT export: inline possible?

2024-04-18 Thread Max Nikulin
around %i if you are on the main branch. On bugfix it is necessary to use an alternative command with %I. However LaTeXML has issues with latex3 Max Nikulin… Re: Warn about shell-expansion in the docstring of org-latex-to-html-convert-command. Tue, 5 Mar 2024 19:01:16 +0700. https

Re: strange export problem with a file: link

2024-04-18 Thread Max Nikulin
On 18/04/2024 17:50, Fraga, Eric wrote: #+begin_src latex :results file raw :exports results :file function.png \[ y = f(x) \] #+end_src org-odt-export-to-odt: OpenDocument export failed: Opening input file: No such file or directory, /tmp/file:function.png It is a side effect of 72b0e9ff0

org syntax: is \begin{equation} a latex-fragment?

2024-04-18 Thread Max Nikulin
Hi, Is it expected that 8< \begin{equation} >8 is parsed by (org-element-parse-buffer) as the following? (latex-fragment ... :value "\\begin{equation}") \begin{something} is quite special in LaTeX, so despite it is similar to `latex-fragment' definition, it almost

Re: [POLL] Should we enable or disable automatic tag alignment by default everywhere

2024-04-18 Thread Max Nikulin
get properly aligned tags after editing. In that thread tag alignment was just an example when minor user mistake can lead to global changes. Max Nikulin. Preventing inadvertent global modifications. Tue, 12 Sep 2023 23:47:30 +0700. https://list.orgmode.org/udq4n4$1me$1...@ciao.gmane.io

Re: [PATCH v2] org-faq.org: Inline comments

2024-04-17 Thread Max Nikulin
On 15/04/2024 19:17, Ihor Radchenko wrote: +#+begin_src org +The following line may become a patagraph separator. +@@comment: might give unexpected effect @@ +Put some text before @@comment: a better variant +@@ and after instread. +#+end_src I am no longer able to reproduce the problem with

Re: Verbatim content (inline-special-block)

2024-04-12 Thread Max Nikulin
On 11/04/2024 19:58, Ihor Radchenko wrote: Max Nikulin writes: @code{ unlike =code=, we can have leading and trailing spaces } @code{ @foo{is not interpreted inside}} I think, it should be controlled by some optional parameter like @kbd[:verbatim t]{ unlike =code

Re: [BUG] Customizing org-emphasis-alist does not work as expected [9.6.15 (release_9.6.15 @ d:/emacs_home/program/emacs/share/emacs/29.2/lisp/org/)]

2024-04-11 Thread Max Nikulin
On 10/04/2024 15:19, Ihor Radchenko wrote: Chang Xiaoduan writes: I am trying to customizing `org-emphasis-alist` so that ` can replace ~ as the code marker. I use the following code in my configuration file: Custom markup markers are not supported. This customization is only useful to

Re: [Q] startup hook: How do I detect if the current buffer has been opened programmatically?

2024-04-11 Thread Max Nikulin
On 09/04/2024 00:31, Rudi C wrote: I want to automatically preview all LaTeX fragments if the buffer was opened by the user Be careful and limit automatic preview to files you created yourself. Doing so for an arbitrary file downloaded from net might cause huge amount of garbage files in

Link to the repository (Re: inline-special-block: part2)

2024-04-11 Thread Max Nikulin
On 11/04/2024 17:49, Max Nikulin wrote: I think, it is better to create a new thread to discuss next bunch of questions related to the inline special block feature. I have realized that I forgot to add the link for those who wish to try the new feature in action: Juan Manuel Macías

Re: Experimental public branch for inline special blocks

2024-04-11 Thread Max Nikulin
On 02/03/2024 03:34, Juan Manuel Macías wrote: Finally, I have made public on GitLab my experimental branch for the new (possible) inline-special-block element: <https://gitlab.com/maciaschain/org-mode.git> Next part of this thread: Max Nikulin. inline-special-block: part2. Thu, 11 Ap

Verbatim content (inline-special-block)

2024-04-11 Thread Max Nikulin
Ihor Radchenko. Re: Experimental public branch for inline special blocks. Tue, 09 Apr 2024 08:52:38 +. https://list.orgmode.org/875xwqj4tl.fsf@localhost - We should be able to define special markup for code, where the contents is not parsed. Something like @code{ unlike

inline-special-block: part2

2024-04-11 Thread Max Nikulin
I think, it is better to create a new thread to discuss next bunch of questions related to the inline special block feature. Previous thread is rather huge: Juan Manuel Macías. Experimental public branch for inline special blocks. Fri, 01 Mar 2024 20:34:35 +.

Re: Experimental public branch for inline special blocks

2024-04-09 Thread Max Nikulin
On 09/04/2024 15:52, Ihor Radchenko wrote: Aside: It looks like your public branch is not up-to-date as the time of writing this email - I do not see commits changing the syntax to @foo{...} and @@{...} yet, Do you have in your local copy the following? latest commit ba2fc870c

Re: [BUG] Partially broken Org mode when remote setupfile is unavailable

2024-04-09 Thread Max Nikulin
On 09/04/2024 15:07, Ihor Radchenko wrote: Ihor Radchenko writes: #+setupfile: http://localhost:8000/setup-http-987.org * Heading [[https://orgmode.org/][Link]] [...] See https://debbugs.gnu.org/cgi/bugreport.cgi?bug=5 The Emacs bug has been fixed. There is no workaround we can

Re: [POLL] Dealing with +1m/y repeaters when jumping to impossible date (should 05-31 +1m be 07-01 or 06-30?)

2024-04-06 Thread Max Nikulin
On 06/04/2024 01:34, Ihor Radchenko wrote: Generally, I did see several requests to change the strategy when calculating next month/year. However, that would be a breaking change. I'd only go for it if people are strongly in favor of the change. So, changing this to a poll. I think the

Re: [PATCH] Unit tests for function calling MathML converters (Re: Warn about shell-expansion in the docstring of org-latex-to-html-convert-command)

2024-04-01 Thread Max Nikulin
On 31/03/2024 15:27, Ihor Radchenko wrote: Max Nikulin writes: +(ert-deftest test-org/format-latex-as-html () + "Test shell special characters escaping in `org-format-latex-as-html'." + (let ((org-latex-to-html-convert-command + "printf \"\" %i")) I

Re: Warn about shell-expansion in the docstring of org-latex-to-html-convert-command

2024-04-01 Thread Max Nikulin
On 31/03/2024 15:25, Ihor Radchenko wrote: Max Nikulin writes: I think it is in the right direction. - Manual needs update as well. - I would explicitly stress that quotes causes undefined or even dangerous behavior. See e.g. the last paragraph https://specifications.freedesktop.org/desktop

Re: [BUG] Partially broken Org mode when remote setupfile is unavailable

2024-04-01 Thread Max Nikulin
On 30/03/2024 17:02, Ihor Radchenko wrote: Ihor Radchenko writes: Unexpected errors: - When point is on any non-empty line, RET is broken save-excursion: Wrong type argument: stringp, nil - The link is not fontified (it is parsed properly however and even may be opened). This is because

Re: [BUG] ox-html output does not pass validation for html4-strict doctype [9.7-pre (release_9.6.23-1423-gcea6a1.dirty @ /home/yantar92/.emacs.d/straight/build/org/)]

2024-03-27 Thread Max Nikulin
On 25/03/2024 18:18, Ihor Radchenko wrote: I am wondering whether it is at all possible to use the same syntax and yet pass validation for all the allowed values of `org-html-doctype': "html4-strict", "html4-transitional", "html4-frameset", "xhtml-strict", "xhtml-transitional", "xhtml-frameset",

Re: [BUG] Partially broken Org mode when remote setupfile is unavailable

2024-03-27 Thread Max Nikulin
On 20/03/2024 19:15, Ihor Radchenko wrote: Max Nikulin writes: #+setupfile: /dav:localhost#8000:/setup-dav-123.org [...] Package ‘tramp-gvfs’ not supported File mode specification error: (user-error Package `tramp-gvfs' not supported) [...] - When point is on any non-empty line, RET

Re: inline-special-block: export rules

2024-03-26 Thread Max Nikulin
On 21/03/2024 17:26, Juan Manuel Macías wrote: Max Nikulin writes: I am afraid that :export will cause confusion with :exports for source code blocks. Its name differs by just "s" but possible values have nothing common. I agree. At the moment two alternative names come to mind:

Re: [PATCH] Run latex more than once for LaTeX src block evaluation

2024-03-26 Thread Max Nikulin
On 24/03/2024 15:58, Ihor Radchenko wrote: Max Nikulin writes: On 22/03/2024 05:55, Michael wrote: I have a small patch for `org-preview-latex-process-alist' making the default setting for LaTeX source block evaluation be running latex three times (instead of the current one). I suspect

Re: [BUG] org-clock-sum: Wrong type argument: fixnump, nil [9.7-pre (release_9.6.22-1309-g8507ef @ /home/grfz/src/org-mode/lisp/)]

2024-03-26 Thread Max Nikulin
On 25/03/2024 22:46, Gregor Zattler wrote: rm -rf *; git checkout -f git clean -dffx should be a bit safer. However there is still risk to lost files have not been committed to git.

Re: [BUG] LaTeX preview should use a subdirectory in /tmp

2024-03-26 Thread Max Nikulin
On 25/03/2024 19:49, Max Nikulin wrote: (defun org-ensure-tmp-dir (dir-symbol prefix)   (let ((dir (symbol-value dir-symbol)))     ;; Temporary directory has not been cleaned.     (or (and dir (file-directory-p dir) dir) `if' should be used instead of `or' here. (setf (symbol-value

Re: [BUG] LaTeX preview should use a subdirectory in /tmp

2024-03-25 Thread Max Nikulin
On 25/03/2024 18:40, Ihor Radchenko wrote: Max Nikulin writes: This feature should not write temporary files to /tmp directly See the attached tentative patch. Thanks for prompt reaction. +++ b/lisp/org.el @@ -16361,7 +16361,7 @@ (defun org-create-formula-image org-format

[BUG] LaTeX preview should use a subdirectory in /tmp

2024-03-25 Thread Max Nikulin
This is a follow-up to recent changes related to LaTeX preview. This feature should not write temporary files to /tmp directly, some subdirectory should be created for this purpose. The idea is to mitigate consequences if a user opens a file from a compromised or a malicious source and gets

Re: [PATCH] Run latex more than once for LaTeX src block evaluation

2024-03-24 Thread Max Nikulin
On 22/03/2024 05:55, Michael wrote: I have a small patch for `org-preview-latex-process-alist' making the default setting for LaTeX source block evaluation be running latex three times (instead of the current one). I suspect it may make the LaTeX preview feature unacceptably slow for

fixup! and git

2024-03-19 Thread Max Nikulin
On 19/03/2024 23:27, Ihor Radchenko wrote: Oops. https://git.savannah.gnu.org/cgit/emacs/org-mode.git/commit/?id=4f548f948 fixup! org-create-math-formula: Do not run file-related hooks when checking output "fixup!" is a keyword for "git rebase --interactive --autosquash" and usually added

Re: Warn about shell-expansion in the docstring of org-latex-to-html-convert-command

2024-03-19 Thread Max Nikulin
On 19/03/2024 21:49, Ihor Radchenko wrote: but I do not see downsides of using `insert-file-contents' instead of `find-file-noselect' and not running `find-file-hook' in this particular case (other cases in Org tree appears to be fine from a quick glance).

  1   2   3   4   5   6   7   8   9   10   >