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 of 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
Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.