emacs-orgmode
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH] Add light argument to org-babel-lob-get-info


From: Ihor Radchenko
Subject: Re: [PATCH] Add light argument to org-babel-lob-get-info
Date: Wed, 19 Oct 2022 07:01:35 +0000

Ferdinand Pieper <fer@pie.tf> writes:

> Ihor Radchenko <yantar92@posteo.net> writes:
>
>> I'd prefer to change LIGHT to NO-EVAL, including in
>> org-babel-get-src-block-info. Changing argument name in function does
>> not affect its caller in any way. Just need to update the function body
>> and docstring carefully.
>>
>> NO-EVAL is already used by org-babel-parse-header-arguments and
>> org-babel-params-from-properties.
>
> I also prefer the NO-EVAL naming. I updated the patch to use no-eval and also 
> added a second patch to rename the light argument of 
> org-babel-get-src-block-info and all its occurrences (I changed the 'light to 
> 'no-eval in all its calls only for consistency and future proofing reasons, 
> altough it is unnecessary right now). Feel free to squash them into one 
> commit. I was not sure wether one or two commits would be preferred.

Thanks!
Applied onto main.
https://git.savannah.gnu.org/cgit/emacs/org-mode.git/commit/?id=bbec9aafee8f0cd022a8e2b782ac1f3f920fdb8f
https://git.savannah.gnu.org/cgit/emacs/org-mode.git/commit/?id=e58bd039e3c885cc3a3e6fd422b5a06a9ad1eea4

>> May I know if you are proposing this for a specific purpose?
>
> Thanks for asking. I use it in a function that creates quasi unique names 
> similar to org-babel-temp-file. But instead of random names it uses the 
> source block content and header info to create a unique hash. This makes it 
> very easy to create unique but from call to call consistent filenames in 
> header arguments, when you don't want to worry about the filename, but also 
> don't want to clutter your result directory (especially if its not /tmp) or 
> care about constant filenames, e.g. for linking. Without the no-eval/light 
> argument the hash generation gets stuck in a recursive loop. I attached the 
> functions for reference.
> If there is wider interest in this, these could be added to org either as a 
> variant of org-babel-temp-file or as an extension.

Have you seen org-babel-temp-stable-file?

-- 
Ihor Radchenko // yantar92,
Org mode contributor,
Learn more about Org mode at <https://orgmode.org/>.
Support Org development at <https://liberapay.com/org-mode>,
or support my work at <https://liberapay.com/yantar92>



reply via email to

[Prev in Thread] Current Thread [Next in Thread]