Re: [PATCH] ob-tangle: Add flag to optionally remove files before writing

2024-02-29 Thread Ihor Radchenko
Olivier Lischer writes: >> Please, let me know if you want to change anything. >> Also, if you can, please test my patch with your use-case. > > I tried it and it works for me. > Thank you for your patience. Thanks for testing! Applied, onto main. https://git.savannah.gnu.org/cgit/emacs/org-mode

Re: [PATCH] ob-tangle: Add flag to optionally remove files before writing

2024-02-28 Thread Olivier Lischer
Ihor Radchenko writes: > > Thanks for the update! > Unfortunately, your patch has unbalanced parenthesis and will not work. > I have modified your patch to make things work and to make it conform to > Elisp code conventions. I also removed spurious change in the existing > variable. > > See the a

Re: [PATCH] ob-tangle: Add flag to optionally remove files before writing

2024-02-28 Thread Ihor Radchenko
e-case. >From b8d2a8506c9b63ad9fefbf8caaefb5de94001eb7 Mon Sep 17 00:00:00 2001 Message-ID: From: Olivier Lischer Date: Tue, 23 Jan 2024 21:02:20 +0100 Subject: [PATCH] ob-tangle: Add flag to optionally remove files before writing * lisp/ob-tangle.el: Add new custom option `org-babel-tangle-re

Re: [PATCH] ob-tangle: Add flag to optionally remove files before writing

2024-01-30 Thread Ihor Radchenko
Olivier Lischer writes: >> I suggest changing your patch, setting the default value of >> `org-babel-tangle-remove-file-before-write' to 'auto. >> This will keep the current behaviour but fall back to delete + write new >> file when the tangle target is read-only. >> That will avoid feature regre

Re: [PATCH] ob-tangle: Add flag to optionally remove files before writing

2024-01-29 Thread Olivier Lischer
'auto. > This will keep the current behaviour but fall back to delete + write new > file when the tangle target is read-only. > That will avoid feature regression. I changed the path as you proposed. >From a464592f622ba42732a5449d2c4db51f1975444a Mon Sep 17 00:00:00 2001 From: Oli

Re: [PATCH] ob-tangle: Add flag to optionally remove files before writing

2024-01-28 Thread Ihor Radchenko
Olivier Lischer writes: >> Thanks for the patch, but may you please explain why introducing such >> variable is useful? > > Sure. > > I configure all my .dotfiles in an Org mode file and tangle the > configuration in the right places. The tangled files are all > read-only to prevent accidentally

Re: [PATCH] ob-tangle: Add flag to optionally remove files before writing

2024-01-28 Thread Olivier Lischer
Ihor Radchenko writes: > Olivier Lischer writes: > >> In December last year was a change introduced, that a file should not be >> removed before >> tangling (https://lists.gnu.org/r/emacs-orgmode/2021-05/msg00337.html). >> In an older bug report >> (https://lists.gnu.org/r/emacs-orgmode/2021-0

Re: [PATCH] ob-tangle: Add flag to optionally remove files before writing

2024-01-24 Thread Ihor Radchenko
Olivier Lischer writes: > In December last year was a change introduced, that a file should not be > removed before > tangling (https://lists.gnu.org/r/emacs-orgmode/2021-05/msg00337.html). > In an older bug report > (https://lists.gnu.org/r/emacs-orgmode/2021-05/msg00337.html) > the decision was

[PATCH] ob-tangle: Add flag to optionally remove files before writing

2024-01-23 Thread Olivier Lischer
added a variable to switch between both behaviors. Any feedback or advice is welcome. >From 4f2f2a0dcca82a934144636da478a9243e514f66 Mon Sep 17 00:00:00 2001 From: Olivier Lischer Date: Tue, 23 Jan 2024 21:02:20 +0100 Subject: [PATCH] ob-tangle: Add flag to optionally remove files before writ