Hi Matt, I'm unfamilliar with patch by mail but i try to apply your patch to my melpa local org 9.4 version used by doom emacs. Patch hang on Hunk #3, i attach the .rej file.
patch -p1 < 0001-ob-core.el-Add-ability-to-use-closures-as-default-he.patch File to patch: ob-core.el patching file ob-core.el Hunk #1 succeeded at 467 (offset -6 lines). Hunk #2 succeeded at 596 (offset -16 lines). Hunk #3 FAILED at 2745. Hunk #4 succeeded at 2714 (offset -51 lines). 1 out of 4 hunks FAILED -- saving rejects to file ob-core.el.rej patch unexpectedly ends in middle of line Best Le mercredi 14 octobre 2020 à 07:16 -0700, Matt Huszagh a écrit : > rey-coyrehourcq <[email protected]> writes: > > > I'm interested by this functionality, do you know if it was merged or i > > need to apply patch locally ? > > Hi SR, > > This hasn't been merged yet. I believe it's ready, but we're just > waiting on a maintainer to apply it upstream. If you're able to apply > the patch locally and provide feedback that always helps. > > Matt -- Sébastien Rey-Coyrehourcq Research Engineer UMR IDEES 02.35.14.69.30 {Stronger security for your email, follow EFF tutorial : https://ssd.eff.org/}
--- lisp/ob-core.el
+++ lisp/ob-core.el
@@ -2745,12 +2786,21 @@ parameters when merging lists."
results-exclusive-groups
results
(split-string
- (if (stringp value) value (eval value t))))))
+ (if (stringp value)
+ value
+ (if (functionp value)
+ (funcall value)
+ (eval value t)))))))
(`(:exports . ,value)
(setq exports (funcall merge
exports-exclusive-groups
exports
- (split-string (or value "")))))
+ (split-string (or
+ (if value
+ (if (functionp value)
+ (funcall value)
+ value)
+ ""))))))
;; Regular keywords: any value overwrites the previous one.
(_ (setq params (cons pair (assq-delete-all (car pair) params)))))))
;; Handle `:var' and clear out colnames and rownames for replaced
signature.asc
Description: This is a digitally signed message part
