[Freeipa-devel] [PATCH] [DOC] 0003 Split text commands & descriptions into XML tables.

[Jérôme Fenal]

kk2013/10/15 Martin Kosek <mkosek at redhat.com>:
> Thanks. It would be ideal, if this table is (in future) generated somehow
> semi-automatically as practically all this info can be gathered from FreeIPA
> code. But for now, this is great.
>
> I see some issues with the patch though:
>
> 1) Whitespaces before tabs

OK, I fixed my attached script my removing leading spaces in the
second part of s///.

> I fixed some of them with
> sed -i "s/\+\t$/+/g" /tmp/freeipa-jfenal-0003-Split-commands-in-proper-tables.patch
>
> Bug there is the second issue:
>
> 2) Test&build fails:
>
> $ git am /tmp/freeipa-jfenal-0003-Split-commands-in-proper-tables-1.patch
> Applying: Split commands in proper tables
> /home/mkosek/freeipa-docs/.git/rebase-apply/patch:211: space before tab in indent.
>                                                 <row>
> /home/mkosek/freeipa-docs/.git/rebase-apply/patch:212: space before tab in indent.
>                                                         <entry>
> /home/mkosek/freeipa-docs/.git/rebase-apply/patch:213: space before tab in indent.
>                                                                  automountkey-add
> /home/mkosek/freeipa-docs/.git/rebase-apply/patch:214: space before tab in indent.
>                                                         </entry>
> /home/mkosek/freeipa-docs/.git/rebase-apply/patch:215: space before tab in indent.
>                                                         <entry>
> warning: squelched 1849 whitespace errors
> warning: 1854 lines add whitespace errors.

Will look into that.

Side question, how did you get the initial text command list?

Regards,

J.
-- 
Jérôme Fenal
-------------- next part --------------
A non-text attachment was scrubbed...
Name: t.pl
Type: application/octet-stream
Size: 323 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20131015/d427bf73/attachment.obj>