[Buildroot] [PATCH v2 1/2] qt5multimedia: fix compile without opengl
Peter Seiderer
ps.report at gmx.net
Tue Feb 3 18:52:25 UTC 2015
Hello Arnout,
> Gesendet: Dienstag, 03. Februar 2015 um 17:21 Uhr
> Von: "Arnout Vandecappelle" <arnout at mind.be>
> An: "Peter Seiderer" <ps.report at gmx.net>, "Thomas Petazzoni" <thomas.petazzoni at free-electrons.com>
> Cc: buildroot at busybox.net
> Betreff: Re: [Buildroot] [PATCH v2 1/2] qt5multimedia: fix compile without opengl
>
> On 28/01/15 22:48, Peter Seiderer wrote:
> > But I searched upstream git, there is already a fix in 5.4.1/dev branch
> > fixing this problem ([1]), but as I think doing it the wrong way,
> > disabling Format_RGB24 for the non-OpenGL case ([2]), but a follow up
> > patch fixing the issue is on its way upstream ([3])...
>
> Could you then submit those two upstream patches? Either separately or
> squashed, but with a reference to the upstream commit and your SoB.
>
Mhh, I understand the reasoning for grabbing upstream patches, but in this case
I am not sure its worth the work to possibly rebase the upstream version (in case
they do not apply cleanly, or to squash them), to get nearly the same result as
with the suggested patch, the fix for [2] with [3] is derived work of my suggested
buildroot patch ;-)
Less work would be to just add the upstream references to the commit message
and/or the patch...
Any preferred solution?
Regards,
Peter
> Regards,
> Arnout
>
> >
> > Regards,
> > Peter
> >
> > [1] https://qt.gitorious.org/qt/qtmultimedia/commit/2b181d546970d18a48a0f36f5d1a22418b61cd4d
> > [2] https://codereview.qt-project.org/101817
> > [3] https://codereview.qt-project.org/104811
> >
> >>
> >> Thanks,
> >>
> >> Thomas
> >> --
> >> Thomas Petazzoni, CTO, Free Electrons
> >> Embedded Linux, Kernel and Android engineering
> >> http://free-electrons.com
> > _______________________________________________
> > buildroot mailing list
> > buildroot at busybox.net
> > http://lists.busybox.net/mailman/listinfo/buildroot
> >
> >
>
>
> --
> Arnout Vandecappelle arnout at mind be
> Senior Embedded Software Architect +32-16-286500
> Essensium/Mind http://www.mind.be
> G.Geenslaan 9, 3001 Leuven, Belgium BE 872 984 063 RPR Leuven
> LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle
> GPG fingerprint: 7CB5 E4CC 6C2E EFD4 6E3D A754 F963 ECAB 2450 2F1F
>
More information about the buildroot
mailing list