[Buildroot] [PATCH 3 of 4 RFC] manual: add section about depending on toolchain options

Thomas Petazzoni thomas.petazzoni at free-electrons.com
Wed Sep 18 16:34:14 UTC 2013


Dear Thomas De Schampheleire,

On Wed, 18 Sep 2013 13:15:41 +0200, Thomas De Schampheleire wrote:

> In the original thread, there were a few outstanding questions:
> - capitalization
>   In this series I have opted for the mixed capitalization (one of the
> options mentioned by ThomasP): mostly lowercase except for acronyms
> like RPC, IPv6, C++.
> 
> - which features to mention in comments
>   After thinking about it, it makes sense to me to mention C++,
> thread, wchar, RPC, dynamic library, and IPv6 in comments, and not to
> mention MMU and target architecture. My reasoning is that the last two
> are not freely choosable for a given project, while the other options
> can pretty easily be turned on when desired.
> 
> I'm open to discussing this, if you do not agree.

I agree with both. I believe the documentation should explicitly state
that a BR2_USE_MMU dependency or an architecture dependency should not
be mentioned in a comment (with the reason that you explain).

However, for the reason that you explain, showing a comment when the
problem is a lack of dynamic library is not completely good. On some
noMMU architectures, there no support for dynamic libraries at all, so
the user cannot "enable" that at all. But well, despite this, I believe
it's better to show a comment when a package is not available due to
the lack of shared libraries.

Thomas
-- 
Thomas Petazzoni, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com



More information about the buildroot mailing list