On 10/15/2013 06:19 PM, Jérôme Fenal wrote:
kk2013/10/15 Martin Kosek <[email protected]>:
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.
--
Petr³
_______________________________________________
Freeipa-devel mailing list
[email protected]
https://www.redhat.com/mailman/listinfo/freeipa-devel
