backslashchar in texinfo

backslashchar in texinfo

[Takaaki ISHIKAWA]

Dear all,

I found two build errors in the latest org manual(HEAD of git).

1. Using ``@backslashcar{}'' command in texinfo(L5114).
2. Missing ``@''(L3091).

The texinfo command was introduced at the version 4.14,
see http://permalink.gmane.org/gmane.comp.gnu.lilypond.devel/51827.

I think the command should not be used because many users
currently use the previous version 4.13.

Any comments?

Best,
Takaaki


---- patch ----

diff --git a/doc/org.texi b/doc/org.texi
index 83f5939..5464830 100644
--- a/doc/org.texi
+++ b/doc/org.texi
@@ -3088,7 +3088,7 @@ lines will be left alone by this command.
 Selects this line for global recalculation with @kbd{C-u C-c *}, but
 not for automatic recalculation.  Use this when automatic
 recalculation slows down editing too much.
-@item
+@item @@
 Unmarked lines are exempt from recalculation with @kbd{C-u C-c *}.
 All lines that should be recalculated should be marked with @samp{#}
 or @samp{*}.
@@ -5111,7 +5111,7 @@ FILE         @r{The filename the entry is located in.}
 To create sparse trees and special lists with selection based on properties,
 the same commands are used as for tag searches (@pxref{Tag searches}).
 @table @kbd
-@orgcmdkkc{C-c / m,C-c \,org-match-sparse-tree}
+@orgcmdkkc{C-c / m,C-c \\,org-match-sparse-tree}
 Create a sparse tree with all matching entries.  With a
 @kbd{C-u} prefix argument, ignore headlines that are not a TODO line.
 @orgcmd{C-c a m,org-tags-view}

---- patch ----

--
Takaaki ISHIKAWA <takaxp@ieee.org>

Re: backslashchar in texinfo

[Bastien]

Hi Takaaki,

Takaaki ISHIKAWA <takaxp@ieee.org> writes:

> I found two build errors in the latest org manual(HEAD of git).
>
> 1. Using ``@backslashcar{}'' command in texinfo(L5114).
> 2. Missing ``@''(L3091).

Thanks for reporting this!

> The texinfo command was introduced at the version 4.14,
> see http://permalink.gmane.org/gmane.comp.gnu.lilypond.devel/51827.
>
> I think the command should not be used because many users
> currently use the previous version 4.13.
>
> Any comments?

I backported a diff from the Emacs trunk erroneously.
It is fixed now.

Thanks!

-- 
 Bastien

Re: backslashchar in texinfo

[Takaaki ISHIKAWA]

Hi Bastien,

Could you change the line 3091 from ``@item'' to ``@item @@'' by hand?
I don't know why my patch was not generated correctly.

Best,
Takaaki

On 2013/01/06, at 14:19, Bastien <bzg@altern.org> wrote:

> Hi Takaaki,
> 
> Takaaki ISHIKAWA <takaxp@ieee.org> writes:
> 
>> I found two build errors in the latest org manual(HEAD of git).
>> 
>> 1. Using ``@backslashcar{}'' command in texinfo(L5114).
>> 2. Missing ``@''(L3091).
> 
> Thanks for reporting this!
> 
>> The texinfo command was introduced at the version 4.14,
>> see http://permalink.gmane.org/gmane.comp.gnu.lilypond.devel/51827.
>> 
>> I think the command should not be used because many users
>> currently use the previous version 4.13.
>> 
>> Any comments?
> 
> I backported a diff from the Emacs trunk erroneously.
> It is fixed now.
> 
> Thanks!
> 
> -- 
> Bastien



--
Takaaki ISHIKAWA <takaxp@ieee.org>

Re: backslashchar in texinfo

[Bastien]

Hi Takaaki,

sorry, I didn't see the patch in your previous email.

Takaaki ISHIKAWA <takaxp@ieee.org> writes:

> Could you change the line 3091 from ``@item'' to ``@item @@'' by hand?

I think it should be @item, to mean "no special char in this table
line."  Or am I missing something?

> I don't know why my patch was not generated correctly.

The patch was okay, I didn't apply it -- but it was not attached in
a way that lets the patchwork catch it.

Thanks!

-- 
 Bastien

Re: backslashchar in texinfo

[Takaaki ISHIKAWA]

Hi Bastien,

>> Could you change the line 3091 from ``@item'' to ``@item @@'' by hand?
> 
> I think it should be @item, to mean "no special char in this table
> line."  Or am I missing something?

Oh, I was so wrong. Please keep it as ``@item''.
Thanks.

>> I don't know why my patch was not generated correctly.
> 
> The patch was okay, I didn't apply it -- but it was not attached in
> a way that lets the patchwork catch it.

That's Ok! I understood.

Best,
Takaaki

--
Takaaki ISHIKAWA <takaxp@ieee.org>

Re: backslashchar in texinfo

[Andy Moreton]

On Sun 06 Jan 2013, Bastien wrote:

> Hi Takaaki,
>
> Takaaki ISHIKAWA <takaxp@ieee.org> writes:
>
>> I found two build errors in the latest org manual(HEAD of git).
>>
>> 1. Using ``@backslashcar{}'' command in texinfo(L5114).
>> 2. Missing ``@''(L3091).
>
> Thanks for reporting this!
>
>> The texinfo command was introduced at the version 4.14,
>> see http://permalink.gmane.org/gmane.comp.gnu.lilypond.devel/51827.
>>
>> I think the command should not be used because many users
>> currently use the previous version 4.13.
>>
>> Any comments?
>
> I backported a diff from the Emacs trunk erroneously.
> It is fixed now.

This still appears to be broken in emacs trunk - could you please update
the org manual there to fix this too ?

Thanks,

    AndyM

Re: backslashchar in texinfo

[Bastien]

Hi Andy,

Andy Moreton <andrewjmoreton@gmail.com> writes:

> This still appears to be broken in emacs trunk - could you please update
> the org manual there to fix this too ?

This has been fixed in the emacs-24 branch and will be merged into
trunk soon.

http://git.savannah.gnu.org/cgit/emacs.git/commit/?h=emacs-24&id=7c68ba

Best,

-- 
 Bastien

Re: backslashchar in texinfo

[Andy Moreton]

On 06/01/2013 20:20, Bastien wrote:
> Hi Andy,
>
> Andy Moreton <andrewjmoreton@gmail.com> writes:
>
>> This still appears to be broken in emacs trunk - could you please update
>> the org manual there to fix this too ?
>
> This has been fixed in the emacs-24 branch and will be merged into
> trunk soon.

Thanks for sorting this out,

    AndyM