kicad-developers team mailing list archive
-
kicad-developers team
-
Mailing list archive
-
Message #12542
Re: PATCH: Tidy up documentation of CMake options
-
From:
Wayne Stambaugh <stambaughw@xxxxxxxxxxx>
-
Date:
Sun, 23 Feb 2014 20:03:18 -0500
-
Cc:
KiCad Developers <kicad-developers@xxxxxxxxxxxxxxxxxxx>
-
In-reply-to:
<CAOuK9Liv7MqniWR8An6jH=GjV-YCJhx1TOQQEeMTZ5VgsUxHFA@mail.gmail.com>
-
User-agent:
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:24.0) Gecko/20100101 Thunderbird/24.3.0
On 2/22/2014 9:05 PM, Nick Østergaard wrote:
> When looking at this, I noticed some duplicate text (comments) in
> CMakeLists.txt, attached is a patch.
Fix committed in r4717. Thanks for the patch.
Wayne
>
> 2014-02-23 0:19 GMT+01:00 Wayne Stambaugh <stambaughw@xxxxxxxxxxx>:
>> On 2/22/2014 3:35 PM, Blair Bonnett wrote:
>>> Hi all,
>>>
>>> In a recent thread, it was pointed out that the documentation of the
>>> CMake configuration options had not been updated to reflect the change
>>> from OFF to ON for the default of KICAD_KEEPCASE. There were also an
>>> obsolete option (USE_FP_LIB_TABLE) and some undocumented options
>>> (BUILD_GITHUB_PLUGIN, KICAD_REPO_NAME, DOWNLOAD_DIR).
>>>
>>> The attached patch (generated against revision 4713) tidies up
>>> Documentation/compiling/build-config.txt. As well as fixing the above
>>> oversights, I have add extra whitespace to make it more readable, and
>>> made the default settings more prominent.
>>>
>>> It also fixes a minor spelling mistake in CMakeLists.txt (specifig =>
>>> specific) and points the user to the build option documentation from
>>> other documentation files.
>>>
>>> I *think* I got all the default settings correct, but a second pair of
>>> eyes would be appreciated.
>>>
>>> Thanks
>>> Blair
>>
>> Patch committed in r4714. Thank you for the help.
>>
>> Regards,
>>
>> Wayne
>>
>>
>> _______________________________________________
>> Mailing list: https://launchpad.net/~kicad-developers
>> Post to : kicad-developers@xxxxxxxxxxxxxxxxxxx
>> Unsubscribe : https://launchpad.net/~kicad-developers
>> More help : https://help.launchpad.net/ListHelp
References