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

[Jérôme Fenal]

2013/10/18 Petr Viktorin <pviktori at redhat.com>:
> On 10/15/2013 06:19 PM, Jérôme Fenal wrote:
>>
>> 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?
>
>
> It looks like `ipa help commands` output.

OK, I'll try give it a try to automate the creation of included files
here this week-end.

Regards,

J.
-- 
Jérôme Fenal