[Buildroot] [PATCH 04/11 v5] package/opengl-registry: new package

[Yann E. MORIN <yann.morin.1998@free.fr>]

J?rome, All,

On 2015-01-21 18:42 +0100, J?r?me Pouiller spake thusly:
> On Wednesday 21 January 2015 18:34:54 Yann E. MORIN wrote:
> > On 2015-01-20 14:51 +0100, J?r?me Pouiller spake thusly:
> > > On Monday 19 January 2015 21:40:34 Yann E. MORIN wrote:
> > > > The OpenGL Registry provides all GL headers, save for gl.h itself. We
> > > > vampirise the one from mesa3d (at the version we currently package),
> > > > which is as good as any other.
> > > > 
> > > > Those headers are needed when an OpenGL/EGL/GLES implementation does
> > > > not provide its own headers, like the NVidia binary blob (to come in
> > > > follow-up patches.)
> > > > 
> > > > Note that, even though this is a generic package, and we could
> > > > everything in the BUILD_CMDS, we decided to split the tweaking of glx.h
> > > > in a post-build script, to really make it clear we are tweaking a
> > > > generated file.
> > > > 
> > > > Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
> > > [...]
> > > 
> > > > +# Always install gl.h since the others may depend on it, and
> > > > +# we are sure at least one OpenGL variants will be installed.
> > > > +define OPENGL_REGISTRY_INSTALL_STAGING_CMDS
> > > > +	$(INSTALL) -D -m 0644 package/opengl/khronos-registry/opengl-registry/gl.h \
> > > > +		   $(STAGING_DIR)/usr/include/GL/gl.h
> > > > +	for d in $(OPENGL_REGISTRY_DIRS); do \
> > > > +		for h in $(@D)/$${d}/*.h; do \
> > > > +			$(INSTALL) -D -m 0644 $${h} $(STAGING_DIR)/usr/include/$${d}/$${h##*/} \
> > > > +			|| exit 1; \
> > > > +		done \
> > > > +	done
> > > > +endef
> > > I think we should also provide gl.pc, egl.pc, glesv1_cm.pc, etc... no?
> > 
> > I think the providers of the GL libraries are to provide the .pc files,
> > because only them know what they really depend on.
> > 
> > For example, the Rpi stuff wants to link with -lvcos for example, while
> > the iMX6 may want to link with -limx6-hal  (for example), so we can not
> > provide the .pc files from the registry, which only installs headers.
> I agree, but you don't either provide with nvidia-driver, no?

No, because I did not need them for my use-case! ;-)

Regards,
Yann E. MORIN.

-- 
.-----------------.--------------------.------------------.--------------------.
|  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
| +33 223 225 172 `------------.-------:  X  AGAINST      |  \e/  There is no  |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
'------------------------------^-------^------------------^--------------------'