On 10/15/2013 06:19 PM, Jérôme Fenal wrote:
> kk2013/10/15 Martin Kosek <mko...@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"
>> 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
>> /home/mkosek/freeipa-docs/.git/rebase-apply/patch:212: space before tab in
>> /home/mkosek/freeipa-docs/.git/rebase-apply/patch:213: space before tab in
>> /home/mkosek/freeipa-docs/.git/rebase-apply/patch:214: space before tab in
>> /home/mkosek/freeipa-docs/.git/rebase-apply/patch:215: space before tab in
>> 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?
This is a work that Deon did, not sure if she used some automated way though...
Freeipa-devel mailing list