On Wed, 7 Sep 2011, Norbert Thiebaud wrote: > On Wed, Sep 7, 2011 at 6:47 PM, Peter Foley <pefol...@verizon.net> wrote: > > On Wed, 7 Sep 2011, Bjoern Michaelsen wrote: > > > >> On Wed, 7 Sep 2011 15:32:21 -0500 > >> Norbert Thiebaud <nthieb...@gmail.com> > >> wrote: > >> > >> > I wonder would that work dep wise to just have a packaging step ? > >> > >> $(call gb_Library_get_headers_target,libthatneedsmesa) : \ > >> $(call gb_Package_get_target,Mesa_inc) > >> > >> Of course, we could create a new name for that, but IMHO that is simple > >> enough as is. > > > > I tried adding that but still got a build error. I've attached the build > > log and my current patch. > try (unteseted) > > $(eval $(call gb_LinkTarget_get_headers_target,OGLTrans) : $(call > gb_Package_get_target,Mesa_inc)) > > note LinkTarget not Library and the $eval > > or actually, try Michael's version > > define gb_LinkTarget_add_external_headers > $(call gb_LinkTarget_get_external_headers_target,$(1)) :| \ > $(call gb_Package_get_target,$(2)) > endef > > (to be added somewhere in LinkTarget.mk > > and used as > > $(eval $(call gb_LinkTarget_add_external_headers(OGLTrans,Mesa_inc)) > > I suppose that to be perfect that should bu tuck under > gb_LinkTarget_use_Mesa in RepositoryExternal.mk to properly deal with > the case when using system-mesa vs internal-mesa... > > and then just add Mesa to the list of 'use_externals' of OGLTrans ...
I tried both suggestions but still got the same build error. I've attached a updated diff for the add_external_headers one. Peter _______________________________________________ LibreOffice mailing list LibreOffice@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/libreoffice
