[Buildroot] [PATCH 1/2] qt5 add qmake.conf.in for eglfs

Thomas Petazzoni thomas.petazzoni at free-electrons.com
Wed Nov 8 12:22:19 UTC 2017


Hello,

Thanks for this new iteration. Sorry, but I have a lot of
pedantic/presentation comments, see below :)

On Wed,  8 Nov 2017 12:30:34 +0100, Giulio Benetti wrote:
> EGLFS_DEVICE_INTEGRATION = eglfs_* doesn't work correctly with previous patch
> EGLFS_DEVICE_INTEGRATION = eglfs_viv/eglfs_* must be written before include(../common/linux_device_post.conf)
> 
> On first patch, I've moved qmake.conf to qmake.conf.in suggested by Arnout,
> tagging EGLFS_DEVICE_INTEGRATION = @EGLFS_DEVICE at .
> Then with sed I substitute @EGLFS_DEVICE@ if any eglfs_* are possible,
> otherwise I delete that line and the subsequent too.
> 
> On second patch, I add support for sunxi-mali and upcoming sunxi-mali-mainline.
> In this way they behave the same way of vivante.
> 
> Hope I've done everything correctly.
> I've had to used sed without $(SED) because of its argument,
> to redirect to qmake.conf instead of changing qmake.conf.in.
> 
> After copied to qt5base build directory,
> I remove qmake.conf to avoid problem with git.
> 
> Thanks in advance for any response.
> 
> Kind regard to everyone

A commit log is not an e-mail, it is a text that describes the commit.
So:

 - It should not be written at the first person

 - It should not contain things like "Thanks in advance", "Kind
   regards", etc.

 - Also, things like "After copied to qt5base build directory, I remove
   qmake.conf to avoid problem with git" sound weird.

 - Don't talk about the second patch in the commit log of the first
   patch!

Also, since this is a new version of patches you have already sent, it
should be sent as PATCHv2, and have a changelog that details the
changes since the first iteration.

Best regards,

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com



More information about the buildroot mailing list