Re: [libvirt] [PATCH] docs: document that C & Python are the preferred languages
On Thu, Sep 05, 2019 at 04:22:17PM +0200, Andrea Bolognani wrote:
On Thu, 2019-09-05 at 12:42 +0100, Daniel P. Berrangé wrote:
> + <p>
> + The libvirt repository makes use of a large number of programming
> + languages. There is a general desire to phase out some of the
> + existing languages used to reduce the knowledge burden on
> + developers, and facilitate introduction of new languages in
> + the future.
Reducing the number of languages used by the project and facilitating
the introduction of more languages seem to be contrasting goals.
Accordingly, I would leave out the last part of the sentence.
That are actually directly related. The aim is to eliminate some
existing languages, so that when we add more languages, we've not
increased the overall burden.
> + <li>C - for the main libvirt codebase. Dialect supported by
> + GCC/CLang only.</li>
> + <li>Python - for supporting build scripts / tools. Code must
> + run with both version 2.7 and 3.x at this time.</li>
Instead of esplicitly singling out 2.7 and 3.x, I would just say that
for both C and Python there it is required to support all platforms
listed in "platforms.html".
I think its important to call our both python versions explicitly
as most distros support both versions in one way or another, so
its not clear from platforms.html that we want to support both.
Adopting Meson will eventually force the matter as that requires
python 3, so we'll have to drop py2 at that point.
> + <p>
> + Languages that should not be used for any new contributions.
s/contributions./contributions:/
Regards,
Daniel
--
|: https://berrange.com -o- https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org -o- https://fstop138.berrange.com :|
|: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|