* [Babel] Padlines
@ 2013-09-17 22:36 aditya siram
2013-09-18 7:32 ` Sebastien Vauban
2013-09-18 21:23 ` Eric Schulte
0 siblings, 2 replies; 14+ messages in thread
From: aditya siram @ 2013-09-17 22:36 UTC (permalink / raw)
To: emacs-orgmode
[-- Attachment #1: Type: text/plain, Size: 272 bytes --]
Hi all,
What's the rationale for having padlines by default in tangled source? It
generates wrong programs for languages where whitespace is significant
(Haskell) and, for me, doesn't noticeably improve the look of the tangled
file in cases where it isn't.
Thanks!
-deech
[-- Attachment #2: Type: text/html, Size: 334 bytes --]
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [Babel] Padlines
2013-09-17 22:36 [Babel] Padlines aditya siram
@ 2013-09-18 7:32 ` Sebastien Vauban
2013-09-18 21:23 ` Eric Schulte
1 sibling, 0 replies; 14+ messages in thread
From: Sebastien Vauban @ 2013-09-18 7:32 UTC (permalink / raw)
To: emacs-orgmode-mXXj517/zsQ
aditya siram wrote:
> What's the rationale for having padlines by default in tangled source?
It generates more readable source-code, as blocks are not "glued" next to each
other.
> It generates wrong programs for languages where whitespace is significant
> (Haskell) and, for me, doesn't noticeably improve the look of the tangled
> file in cases where it isn't.
Though, I admit that "padlines yes" should be smarter and add only one line
*between* blocks, but NOT before the first one!
Best regards,
Seb
--
Sebastien Vauban
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [Babel] Padlines
2013-09-17 22:36 [Babel] Padlines aditya siram
2013-09-18 7:32 ` Sebastien Vauban
@ 2013-09-18 21:23 ` Eric Schulte
2013-09-18 22:02 ` Sebastien Vauban
1 sibling, 1 reply; 14+ messages in thread
From: Eric Schulte @ 2013-09-18 21:23 UTC (permalink / raw)
To: aditya siram; +Cc: emacs-orgmode
aditya siram <aditya.siram@gmail.com> writes:
> Hi all,
> What's the rationale for having padlines by default in tangled source? It
> generates wrong programs for languages where whitespace is significant
> (Haskell) and, for me, doesn't noticeably improve the look of the tangled
> file in cases where it isn't.
> Thanks!
> -deech
It is possible to change the value of default header arguments on a
per-language basis because e.g., while (:padlines "yes") may make sense
for sh, it probably doesn't for Haskell.
I just changed the value of org-babel-default-header-args:haskell, so
that :padlines will be turned off by default for that language.
Cheers,
--
Eric Schulte
https://cs.unm.edu/~eschulte
PGP: 0x614CA05D
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [Babel] Padlines
2013-09-18 21:23 ` Eric Schulte
@ 2013-09-18 22:02 ` Sebastien Vauban
2013-09-20 21:26 ` Eric Schulte
0 siblings, 1 reply; 14+ messages in thread
From: Sebastien Vauban @ 2013-09-18 22:02 UTC (permalink / raw)
To: emacs-orgmode-mXXj517/zsQ
Hi Eric,
Eric Schulte wrote:
> aditya siram <aditya.siram-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> writes:
>> What's the rationale for having padlines by default in tangled source? It
>> generates wrong programs for languages where whitespace is significant
>> (Haskell) and, for me, doesn't noticeably improve the look of the tangled
>> file in cases where it isn't.
>
> It is possible to change the value of default header arguments on a
> per-language basis because e.g., while (:padlines "yes") may make sense
> for sh, it probably doesn't for Haskell.
Could it be possible that ":padline yes" does not insert a blank line in front
of the very first block, only *between* all blocks?
Best regards,
Seb
--
Sebastien Vauban
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [Babel] Padlines
2013-09-18 22:02 ` Sebastien Vauban
@ 2013-09-20 21:26 ` Eric Schulte
2013-09-23 7:31 ` Sebastien Vauban
[not found] ` <87siwz5gml.fsf-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
0 siblings, 2 replies; 14+ messages in thread
From: Eric Schulte @ 2013-09-20 21:26 UTC (permalink / raw)
To: Sebastien Vauban; +Cc: emacs-orgmode
[-- Attachment #1: Type: text/plain, Size: 858 bytes --]
"Sebastien Vauban" <sva-news@mygooglest.com> writes:
> Hi Eric,
>
> Eric Schulte wrote:
>> aditya siram <aditya.siram@gmail.com> writes:
>>> What's the rationale for having padlines by default in tangled source? It
>>> generates wrong programs for languages where whitespace is significant
>>> (Haskell) and, for me, doesn't noticeably improve the look of the tangled
>>> file in cases where it isn't.
>>
>> It is possible to change the value of default header arguments on a
>> per-language basis because e.g., while (:padlines "yes") may make sense
>> for sh, it probably doesn't for Haskell.
>
> Could it be possible that ":padline yes" does not insert a blank line in front
> of the very first block, only *between* all blocks?
>
> Best regards,
> Seb
I just pushed up a commit which implements this behavior. See the
attached file for an example.
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: padline-example.org --]
[-- Type: text/x-org, Size: 1103 bytes --]
#+Title: Examples with the new padline behavior
#+headers: :tangle pad-yes-with-shebang.sed
#+headers: :shebang "#!/bin/sed -f"
#+headers: :padline "yes"
#+begin_src sed
1 {N;s/\n//1}
#+end_src
#+headers: :tangle pad-no-with-shebang.sed
#+headers: :shebang "#!/bin/sed -f"
#+headers: :padline "no"
#+begin_src sed
1 {N;s/\n//1}
#+end_src
#+headers: :tangle pad-yes-without-shebang.sed
#+headers: :padline "yes"
#+begin_src sed
1 {N;s/\n//1}
#+end_src
#+headers: :tangle pad-no-without-shebang.sed
#+headers: :padline "no"
#+begin_src sed
1 {N;s/\n//1}
#+end_src
1. Tangle the above four blocks with =C-c C-v t=.
2. Execute the following code block to view the contents of the
resulting sed files.
#+begin_src sh :results scalar
head pad*sed
#+end_src
#+RESULTS:
: ==> padline-example.sed <==
: #!/bin/sed -f
:
: 1 {N;s/\n//1}
:
: ==> pad-no-without-shebang.sed <==
: 1 {N;s/\n//1}
:
: ==> pad-no-with-shebang.sed <==
: #!/bin/sed -f
: 1 {N;s/\n//1}
:
: ==> pad-yes-without-shebang.sed <==
: 1 {N;s/\n//1}
:
: ==> pad-yes-with-shebang.sed <==
: #!/bin/sed -f
:
: 1 {N;s/\n//1}
[-- Attachment #3: Type: text/plain, Size: 63 bytes --]
--
Eric Schulte
https://cs.unm.edu/~eschulte
PGP: 0x614CA05D
^ permalink raw reply [flat|nested] 14+ messages in thread* Re: [Babel] Padlines
2013-09-20 21:26 ` Eric Schulte
@ 2013-09-23 7:31 ` Sebastien Vauban
[not found] ` <87siwz5gml.fsf-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
1 sibling, 0 replies; 14+ messages in thread
From: Sebastien Vauban @ 2013-09-23 7:31 UTC (permalink / raw)
To: emacs-orgmode-mXXj517/zsQ
Hi Eric,
Eric Schulte wrote:
> "Sebastien Vauban" <sva-news-D0wtAvR13HarG/iDocfnWg@public.gmane.org> writes:
>> Eric Schulte wrote:
>>> aditya siram <aditya.siram-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> writes:
>>>> What's the rationale for having padlines by default in tangled source? It
>>>> generates wrong programs for languages where whitespace is significant
>>>> (Haskell) and, for me, doesn't noticeably improve the look of the tangled
>>>> file in cases where it isn't.
>>>
>>> It is possible to change the value of default header arguments on a
>>> per-language basis because e.g., while (:padlines "yes") may make sense
>>> for sh, it probably doesn't for Haskell.
>>
>> Could it be possible that ":padline yes" does not insert a blank line in front
>> of the very first block, only *between* all blocks?
>
> I just pushed up a commit which implements this behavior. See the
> attached file for an example.
>
> #+Title: Examples with the new padline behavior
>
> #+headers: :tangle pad-yes-with-shebang.sed
> #+headers: :shebang "#!/bin/sed -f"
> #+headers: :padline "yes"
> #+begin_src sed
> 1 {N;s/\n//1}
> #+end_src
>
> #+headers: :tangle pad-no-with-shebang.sed
> #+headers: :shebang "#!/bin/sed -f"
> #+headers: :padline "no"
> #+begin_src sed
> 1 {N;s/\n//1}
> #+end_src
>
> #+headers: :tangle pad-yes-without-shebang.sed
> #+headers: :padline "yes"
> #+begin_src sed
> 1 {N;s/\n//1}
> #+end_src
> #+headers: :tangle pad-no-without-shebang.sed
> #+headers: :padline "no"
> #+begin_src sed
> 1 {N;s/\n//1}
> #+end_src
>
> 1. Tangle the above four blocks with =C-c C-v t=.
> 2. Execute the following code block to view the contents of the
> resulting sed files.
>
> #+begin_src sh :results scalar
> head pad*sed
> #+end_src
>
> #+RESULTS:
> : ==> padline-example.sed <==
> : #!/bin/sed -f
> :
> : 1 {N;s/\n//1}
> :
> : ==> pad-no-without-shebang.sed <==
> : 1 {N;s/\n//1}
> :
> : ==> pad-no-with-shebang.sed <==
> : #!/bin/sed -f
> : 1 {N;s/\n//1}
> :
> : ==> pad-yes-without-shebang.sed <==
> : 1 {N;s/\n//1}
> :
> : ==> pad-yes-with-shebang.sed <==
> : #!/bin/sed -f
> :
> : 1 {N;s/\n//1}
That seems perfect, and solves corner-cases I wouldn't have thought of (with
shebang). Thanks a lot!
Best regards,
Seb
--
Sebastien Vauban
^ permalink raw reply [flat|nested] 14+ messages in thread[parent not found: <87siwz5gml.fsf-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>]
* Re: [Babel] Padlines
[not found] ` <87siwz5gml.fsf-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
@ 2013-09-27 14:53 ` Sebastien Vauban
2013-09-28 12:17 ` Eric Schulte
0 siblings, 1 reply; 14+ messages in thread
From: Sebastien Vauban @ 2013-09-27 14:53 UTC (permalink / raw)
To: Eric Schulte; +Cc: emacs-orgmode-mXXj517/zsQ
Hi Eric,
Eric Schulte wrote:
> "Sebastien Vauban" <sva-news-D0wtAvR13HarG/iDocfnWg@public.gmane.org> writes:
>> Eric Schulte wrote:
>>> aditya siram <aditya.siram-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> writes:
>>>> What's the rationale for having padlines by default in tangled source? It
>>>> generates wrong programs for languages where whitespace is significant
>>>> (Haskell) and, for me, doesn't noticeably improve the look of the tangled
>>>> file in cases where it isn't.
>>>
>>> It is possible to change the value of default header arguments on a
>>> per-language basis because e.g., while (:padlines "yes") may make sense for
>>> sh, it probably doesn't for Haskell.
>>
>> Could it be possible that ":padline yes" does not insert a blank line in
>> front of the very first block, only *between* all blocks?
>
> I just pushed up a commit which implements this behavior. See the
> attached file for an example.
>
> #+Title: Examples with the new padline behavior
The blank line which was inserted between blocks isn't anymore for me.
ECM:
--8<---------------cut here---------------start------------->8---
* Tangle these blocks
:PROPERTIES:
:tangle: yes
:padline: yes
:END:
#+begin_src emacs-lisp :file test.csv
"data"
#+end_src
#+begin_src emacs-lisp :file test.csv
"datb"
#+end_src
--8<---------------cut here---------------end--------------->8---
results in:
--8<---------------cut here---------------start------------->8---
"data"
"datb"
--8<---------------cut here---------------end--------------->8---
Note that I tried adding ":padline" to yes, but I normally should not, as it is
the default.
Best regards,
Seb
--
Sebastien Vauban
^ permalink raw reply [flat|nested] 14+ messages in thread* Re: [Babel] Padlines
2013-09-27 14:53 ` Sebastien Vauban
@ 2013-09-28 12:17 ` Eric Schulte
2013-09-30 8:54 ` Sebastien Vauban
2013-10-01 5:57 ` Vladimir Lomov
0 siblings, 2 replies; 14+ messages in thread
From: Eric Schulte @ 2013-09-28 12:17 UTC (permalink / raw)
To: Sebastien Vauban; +Cc: emacs-orgmode
>
> The blank line which was inserted between blocks isn't anymore for me.
>
> ECM:
>
> --8<---------------cut here---------------start------------->8---
> * Tangle these blocks
> :PROPERTIES:
> :tangle: yes
> :padline: yes
> :END:
>
> #+begin_src emacs-lisp :file test.csv
> "data"
> #+end_src
>
> #+begin_src emacs-lisp :file test.csv
> "datb"
> #+end_src
> --8<---------------cut here---------------end--------------->8---
>
> results in:
>
> --8<---------------cut here---------------start------------->8---
> "data"
> "datb"
> --8<---------------cut here---------------end--------------->8---
>
> Note that I tried adding ":padline" to yes, but I normally should not, as it is
> the default.
>
> Best regards,
> Seb
Hi Seb,
Thanks for this bug report, there was a problem in my previous patch in
this thread. I've just pushed up a fix which should solve this problem.
Best,
--
Eric Schulte
https://cs.unm.edu/~eschulte
PGP: 0x614CA05D
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [Babel] Padlines
2013-09-28 12:17 ` Eric Schulte
@ 2013-09-30 8:54 ` Sebastien Vauban
2013-10-01 10:28 ` Rainer M Krug
2013-10-01 5:57 ` Vladimir Lomov
1 sibling, 1 reply; 14+ messages in thread
From: Sebastien Vauban @ 2013-09-30 8:54 UTC (permalink / raw)
To: emacs-orgmode-mXXj517/zsQ
Hi Eric,
Eric Schulte wrote:
>> The blank line which was inserted between blocks isn't anymore for me.
>>
>> [...] results in:
>>
>> --8<---------------cut here---------------start------------->8---
>> "data"
>> "datb"
>> --8<---------------cut here---------------end--------------->8---
>>
>> Note that I tried adding ":padline" to yes, but I normally should not, as it
>> is the default.
>
> Thanks for this bug report, there was a problem in my previous patch in
> this thread. I've just pushed up a fix which should solve this problem.
Confirmed!
Thanks a lot.
Best regards,
Seb
--
Sebastien Vauban
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [Babel] Padlines
2013-09-30 8:54 ` Sebastien Vauban
@ 2013-10-01 10:28 ` Rainer M Krug
2013-10-01 11:59 ` Eric Schulte
0 siblings, 1 reply; 14+ messages in thread
From: Rainer M Krug @ 2013-10-01 10:28 UTC (permalink / raw)
To: emacs-orgmode
Hi
I assume this is related to this discussion and the patches: after
updating this morning, get an error message after tangling:
variable padline void. This goes away after reverting to 8.2 2 release.
Default settings for padline used.
Cheers,
Rainer
"Sebastien Vauban" <sva-news@mygooglest.com>
writes:
> Hi Eric,
>
> Eric Schulte wrote:
>>> The blank line which was inserted between blocks isn't anymore for me.
>>>
>>> [...] results in:
>>>
>>> --8<---------------cut here---------------start------------->8---
>>> "data"
>>> "datb"
>>> --8<---------------cut here---------------end--------------->8---
>>>
>>> Note that I tried adding ":padline" to yes, but I normally should not, as it
>>> is the default.
>>
>> Thanks for this bug report, there was a problem in my previous patch in
>> this thread. I've just pushed up a fix which should solve this problem.
>
> Confirmed!
>
> Thanks a lot.
>
> Best regards,
> Seb
<#secure method=pgpmime mode=sign>
--
Rainer M. Krug
email: RMKrug<at>gmail<dot>com
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [Babel] Padlines
2013-10-01 10:28 ` Rainer M Krug
@ 2013-10-01 11:59 ` Eric Schulte
2013-10-01 12:53 ` Rainer M Krug
0 siblings, 1 reply; 14+ messages in thread
From: Eric Schulte @ 2013-10-01 11:59 UTC (permalink / raw)
To: Rainer M Krug; +Cc: emacs-orgmode
Rainer M Krug <Rainer@krugs.de> writes:
> Hi
>
> I assume this is related to this discussion and the patches: after
> updating this morning, get an error message after tangling:
> variable padline void. This goes away after reverting to 8.2 2 release.
>
> Default settings for padline used.
>
> Cheers,
>
> Rainer
>
Vladimir's patch in this thread fixes this issue and has been applied.
>
>
> "Sebastien Vauban" <sva-news@mygooglest.com>
> writes:
>
>> Hi Eric,
>>
>> Eric Schulte wrote:
>>>> The blank line which was inserted between blocks isn't anymore for me.
>>>>
>>>> [...] results in:
>>>>
>>>> --8<---------------cut here---------------start------------->8---
>>>> "data"
>>>> "datb"
>>>> --8<---------------cut here---------------end--------------->8---
>>>>
>>>> Note that I tried adding ":padline" to yes, but I normally should not, as it
>>>> is the default.
>>>
>>> Thanks for this bug report, there was a problem in my previous patch in
>>> this thread. I've just pushed up a fix which should solve this problem.
>>
>> Confirmed!
>>
>> Thanks a lot.
>>
>> Best regards,
>> Seb
> <#secure method=pgpmime mode=sign>
>
--
Eric Schulte
https://cs.unm.edu/~eschulte
PGP: 0x614CA05D
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [Babel] Padlines
2013-10-01 11:59 ` Eric Schulte
@ 2013-10-01 12:53 ` Rainer M Krug
0 siblings, 0 replies; 14+ messages in thread
From: Rainer M Krug @ 2013-10-01 12:53 UTC (permalink / raw)
To: Eric Schulte; +Cc: emacs-orgmode
Eric Schulte <schulte.eric@gmail.com> writes:
> Rainer M Krug <Rainer@krugs.de> writes:
>
>> Hi
>>
>> I assume this is related to this discussion and the patches: after
>> updating this morning, get an error message after tangling:
>> variable padline void. This goes away after reverting to 8.2 2 release.
>>
>> Default settings for padline used.
>>
>> Cheers,
>>
>> Rainer
>>
>
> Vladimir's patch in this thread fixes this issue and has been applied.
Thanks - will try it out then
Rainer
>
>>
>>
>> "Sebastien Vauban" <sva-news@mygooglest.com>
>> writes:
>>
>>> Hi Eric,
>>>
>>> Eric Schulte wrote:
>>>>> The blank line which was inserted between blocks isn't anymore for me.
>>>>>
>>>>> [...] results in:
>>>>>
>>>>> --8<---------------cut here---------------start------------->8---
>>>>> "data"
>>>>> "datb"
>>>>> --8<---------------cut here---------------end--------------->8---
>>>>>
>>>>> Note that I tried adding ":padline" to yes, but I normally should not, as it
>>>>> is the default.
>>>>
>>>> Thanks for this bug report, there was a problem in my previous patch in
>>>> this thread. I've just pushed up a fix which should solve this problem.
>>>
>>> Confirmed!
>>>
>>> Thanks a lot.
>>>
>>> Best regards,
>>> Seb
>> <#secure method=pgpmime mode=sign>
>>
>
<#secure method=pgpmime mode=sign>
--
Rainer M. Krug, PhD (Conservation Ecology, SUN), MSc (Conservation Biology, UCT), Dipl. Phys. (Germany)
Centre of Excellence for Invasion Biology
Stellenbosch University
South Africa
Tel : +33 - (0)9 53 10 27 44
Cell: +33 - (0)6 85 62 59 98
Fax : +33 - (0)9 58 10 27 44
Fax (D): +49 - (0)3 21 21 25 22 44
email: Rainer@krugs.de
Skype: RMkrug
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [Babel] Padlines
2013-09-28 12:17 ` Eric Schulte
2013-09-30 8:54 ` Sebastien Vauban
@ 2013-10-01 5:57 ` Vladimir Lomov
2013-10-01 11:59 ` Eric Schulte
1 sibling, 1 reply; 14+ messages in thread
From: Vladimir Lomov @ 2013-10-01 5:57 UTC (permalink / raw)
To: Eric Schulte; +Cc: emacs-orgmode
[-- Attachment #1: Type: text/plain, Size: 1408 bytes --]
Hello,
** Eric Schulte [2013-09-28 06:17:51 -0600]:
>> The blank line which was inserted between blocks isn't anymore for me.
>> ECM:
>> --8<---------------cut here---------------start------------->8---
>> * Tangle these blocks
>> :PROPERTIES:
>> :tangle: yes
>> :padline: yes
>> :END:
>> #+begin_src emacs-lisp :file test.csv
>> "data"
>> #+end_src
>> #+begin_src emacs-lisp :file test.csv
>> "datb"
>> #+end_src
>> --8<---------------cut here---------------end--------------->8---
>> results in:
>> --8<---------------cut here---------------start------------->8---
>> "data"
>> "datb"
>> --8<---------------cut here---------------end--------------->8---
>> Note that I tried adding ":padline" to yes, but I normally should not, as it is
>> the default.
>> Best regards,
>> Seb
> Hi Seb,
> Thanks for this bug report, there was a problem in my previous patch in
> this thread. I've just pushed up a fix which should solve this problem.
And seems that you miss another 'padline' in function
'org-babel-spec-to-string', line 323. I attached small patch which
removes that line so I can tangle Org document with shell scripts.
---
WBR, Vladimir Lomov
--
Coming to Stores Near You:
101 Grammatically Correct Popular Tunes Featuring:
(You Aren't Anything but a) Hound Dog
It Doesn't Mean a Thing If It Hasn't Got That Swing
I'm Not Misbehaving
And A Whole Lot More...
[-- Attachment #2: ob-tangle.patch --]
[-- Type: text/x-diff, Size: 532 bytes --]
diff --git a/lisp/ob-tangle.el b/lisp/ob-tangle.el
index 63d0a47..f655711 100644
--- a/lisp/ob-tangle.el
+++ b/lisp/ob-tangle.el
@@ -322,7 +322,6 @@ that the appropriate major-mode is set. SPEC has the form:
(insert-comment (lambda (text)
(when (and comments (not (string= comments "no"))
(> (length text) 0))
- (when padline (insert "\n"))
(comment-region (point) (progn (insert text) (point)))
(end-of-line nil) (insert "\n")))))
(when comment (funcall insert-comment comment))
^ permalink raw reply related [flat|nested] 14+ messages in thread* Re: [Babel] Padlines
2013-10-01 5:57 ` Vladimir Lomov
@ 2013-10-01 11:59 ` Eric Schulte
0 siblings, 0 replies; 14+ messages in thread
From: Eric Schulte @ 2013-10-01 11:59 UTC (permalink / raw)
To: Vladimir Lomov; +Cc: emacs-orgmode
Vladimir Lomov <lomov.vl@gmail.com> writes:
> Hello,
> ** Eric Schulte [2013-09-28 06:17:51 -0600]:
>
>
>>> The blank line which was inserted between blocks isn't anymore for me.
>
>>> ECM:
>
>>> --8<---------------cut here---------------start------------->8---
>>> * Tangle these blocks
>>> :PROPERTIES:
>>> :tangle: yes
>>> :padline: yes
>>> :END:
>
>>> #+begin_src emacs-lisp :file test.csv
>>> "data"
>>> #+end_src
>
>>> #+begin_src emacs-lisp :file test.csv
>>> "datb"
>>> #+end_src
>>> --8<---------------cut here---------------end--------------->8---
>
>>> results in:
>
>>> --8<---------------cut here---------------start------------->8---
>>> "data"
>>> "datb"
>>> --8<---------------cut here---------------end--------------->8---
>
>>> Note that I tried adding ":padline" to yes, but I normally should not, as it is
>>> the default.
>
>>> Best regards,
>>> Seb
>
>> Hi Seb,
>
>> Thanks for this bug report, there was a problem in my previous patch in
>> this thread. I've just pushed up a fix which should solve this problem.
>
> And seems that you miss another 'padline' in function
> 'org-babel-spec-to-string', line 323. I attached small patch which
> removes that line so I can tangle Org document with shell scripts.
>
Applied, Thanks for catching this and for posting the patch.
p.s. Please format future patches with "git format-patch".
>
> ---
> WBR, Vladimir Lomov
--
Eric Schulte
https://cs.unm.edu/~eschulte
PGP: 0x614CA05D
^ permalink raw reply [flat|nested] 14+ messages in thread
end of thread, other threads:[~2013-10-01 12:53 UTC | newest]
Thread overview: 14+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-09-17 22:36 [Babel] Padlines aditya siram
2013-09-18 7:32 ` Sebastien Vauban
2013-09-18 21:23 ` Eric Schulte
2013-09-18 22:02 ` Sebastien Vauban
2013-09-20 21:26 ` Eric Schulte
2013-09-23 7:31 ` Sebastien Vauban
[not found] ` <87siwz5gml.fsf-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2013-09-27 14:53 ` Sebastien Vauban
2013-09-28 12:17 ` Eric Schulte
2013-09-30 8:54 ` Sebastien Vauban
2013-10-01 10:28 ` Rainer M Krug
2013-10-01 11:59 ` Eric Schulte
2013-10-01 12:53 ` Rainer M Krug
2013-10-01 5:57 ` Vladimir Lomov
2013-10-01 11:59 ` Eric Schulte
Code repositories for project(s) associated with this public 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).