Xft has a lot of private symbols left exported to the world - marking these to be
hidden at link time can reduce linker overhead. Patch will follow momentarily to
mark functions defined in xftint.h with _X_HIDDEN and Xft.h as _X_EXPORT.
Created attachment 3512 [details] [review]
Patch against Xorg CVS head
Should probably be reviewed by KeithP before committing to CVS to verify none
the symbols I've marked _X_HIDDEN need to be visible outside libXft.so.
keith, comments? i guess it would need to be updated to the new world order, though.
Looks like a reasonable change to me. One question is whether xlibs CVS for Xft
has been moved to xorg/lib yet or not; right now the canonical version of at
least some of the 'xlibs' libraries remains over there. I'd prefer to avoid the
same disaster as the x server migration had.
which disaster? i packaged for quite a while under the assumption that your
statement that xft/xcursor/xrender mainline would move to xorg/lib when r7 was
released, which it is ... some clarity would probably be welcome.
the X.org CVS was built by importing from XFree86, and then the kdrive bits were
grafted in place. The result is a CVS history which is muddied beyond the
ability of cvsps to comprehend, which means migrating from CVS to any other SCM
is somewhat challenging.
What I'd like to see for these libraries is that the existing X.org history be
nuked and a single patch applied to the xlibs bits to bring them in line with
X.org release processes. I've done that with compositeproto, fixesproto and
damageproto and have migrated those to git. Would it be reasonable to publish
these .git repositories?
Added patch keyword
committed, 2.1.9 released