emacs-orgmode@gnu.org archives
 help / color / mirror / code / Atom feed
* Fwd: minor ob-sed.el patch for org-mode
       [not found] <67576C6A-2FA3-4416-A8F1-08BFD4A395BC@contoso.com>
@ 2018-04-04 11:28 ` Bjarte Johansen
  2018-04-04 12:15   ` Marco Wahl
  2018-04-05 14:49   ` Nicolas Goaziou
  0 siblings, 2 replies; 3+ messages in thread
From: Bjarte Johansen @ 2018-04-04 11:28 UTC (permalink / raw)
  To: emacs-orgmode; +Cc: Brad Knotwell

[-- Attachment #1: Type: text/plain, Size: 139 bytes --]

Hello,

This is a minor patch suggested by Brad Knotwell so that ob-sed can support BSD sed and not only GNU sed.

Regards,
Bjarte


[-- Attachment #2: 0002-ob-sed-Use-f-instead-of-file-to-support-BSD-sed.patch --]
[-- Type: application/octet-stream, Size: 1139 bytes --]

From 7609ee600dff8ed7b2f528f7ec15e167ee2b8d3b Mon Sep 17 00:00:00 2001
From: Brad Knotwell <b.knotwell@f5.com>
Date: Wed, 4 Apr 2018 13:19:32 +0200
Subject: [PATCH] ob-sed: Use -f instead of --file to support BSD sed

lisp/ob-sed.el (org-babel-execute:sed): Change file flag to "-f"
instead of "--file" to support BSD sed as well.

Signed-off-by: Bjarte Johansen <bjarte.johansen@gmail.com>
---
 lisp/ob-sed.el | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/lisp/ob-sed.el b/lisp/ob-sed.el
index b2c80f88c..ca8b7545f 100644
--- a/lisp/ob-sed.el
+++ b/lisp/ob-sed.el
@@ -4,7 +4,7 @@
 
 ;; Author: Bjarte Johansen
 ;; Keywords: literate programming, reproducible research
-;; Version: 0.1.0
+;; Version: 0.1.1
 
 ;; This file is part of GNU Emacs.
 
@@ -79,7 +79,7 @@ function is called by `org-babel-execute-src-block'."
          (cmd (mapconcat #'identity
 			 (remq nil
 			       (list org-babel-sed-command
-				     (format "--file=\"%s\"" code-file)
+				     (format "-f \"%s\"" code-file)
 				     cmd-line
 				     in-file))
 			 " ")))
-- 
2.15.1 (Apple Git-101)


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: Fwd: minor ob-sed.el patch for org-mode
  2018-04-04 11:28 ` Fwd: minor ob-sed.el patch for org-mode Bjarte Johansen
@ 2018-04-04 12:15   ` Marco Wahl
  2018-04-05 14:49   ` Nicolas Goaziou
  1 sibling, 0 replies; 3+ messages in thread
From: Marco Wahl @ 2018-04-04 12:15 UTC (permalink / raw)
  To: emacs-orgmode

Bjarte Johansen <bjarte.johansen@gmail.com> writes:

> This is a minor patch suggested by Brad Knotwell so that ob-sed can support BSD sed and not only GNU sed.

+1

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: Fwd: minor ob-sed.el patch for org-mode
  2018-04-04 11:28 ` Fwd: minor ob-sed.el patch for org-mode Bjarte Johansen
  2018-04-04 12:15   ` Marco Wahl
@ 2018-04-05 14:49   ` Nicolas Goaziou
  1 sibling, 0 replies; 3+ messages in thread
From: Nicolas Goaziou @ 2018-04-05 14:49 UTC (permalink / raw)
  To: Bjarte Johansen; +Cc: Brad Knotwell, emacs-orgmode

Hello,

Bjarte Johansen <bjarte.johansen@gmail.com> writes:

> This is a minor patch suggested by Brad Knotwell so that ob-sed can
> support BSD sed and not only GNU sed.

Applied. Thank you.

BTW, what's your status wrt FSF papers? I could not find you in list of
contributors.

Regards,

-- 
Nicolas Goaziou

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2018-04-05 21:48 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [not found] <67576C6A-2FA3-4416-A8F1-08BFD4A395BC@contoso.com>
2018-04-04 11:28 ` Fwd: minor ob-sed.el patch for org-mode Bjarte Johansen
2018-04-04 12:15   ` Marco Wahl
2018-04-05 14:49   ` Nicolas Goaziou

Code repositories for project(s) associated with this inbox:

	https://git.savannah.gnu.org/cgit/emacs/org-mode.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).