bugGNUstep - Bugs: bug #53223, [xlib] font_cacher should not be...

Group
 
 

bug #53223: [xlib] font_cacher should not be built if WITH_XFT=yes

Submitter:  Yavor Doganov <yavor>
Submitted:  Sun 25 Feb 2018 11:09:53 AM UTC
   
 
Category:  Backend Severity:  3 - Normal
Item Group:  Change Request Status:  Fixed
Privacy:  Public Assigned to:  FredKiefer
Open/Closed:  Closed
* Mandatory Fields

Add a New Comment Rich Markup
   

Sun 25 Feb 2018 05:54:22 PM UTC, comment #1: 

Thank you for this patch. I added xpds to the list of the deprecated backends and committed it.

Fred Kiefer <FredKiefer>
Group Member
Sun 25 Feb 2018 11:09:53 AM UTC, original submission:  

I am sorry to report another xlib-related bug.  What "inspired" my interest in xlib is this bug report:

https://bugs.debian.org/885786

When I first looked at the xlib directory I didn't realize there were two different variants and that font_cacher is completely unused by the "modern" flavor.  The attached patch fixes this and also adds a manpage which I had to write for Debian (feel free to discard it).

Ideally, we would like to follow strictly upstream's preference and not package art and xlib.  But this would require packaging opal which is currently not possible.

Ever since cairo was made the default backend we made special effort to enforce it for new installations.  That was a difficult thing to do because "art" sorts before "cairo" so we had to find a different approach.  It seems to be working; according to popcon there are 1109 machines with the cairo package installed and only 5 with art:

https://qa.debian.org/popcon.php?package=gnustep-back

Of these 5 machines that have art installed, 3 are mine (permanently connected and with the popularity-contest package installed).

I couldn't find any information in the README/NEWS files or the official GNUstep website that art and xlib are deprecated.  It seems to be common knowledge among regular GNUstep users but I think it's reasonable to expect that new users are unaware of this.
The second part of the patch adds a deprecation warning which is printed at the end of the configure run.  Hopefully this will decrease the usage of the deprecated backends even further.

Yavor Doganov <yavor>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach Files:
   
   
Comment:
   

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by FredKiefer (Posted a comment)
  • -email is unavailable- added by yavor (Submitted the item)
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

    Only logged-in users can vote.

     

    Follow 4 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2018-02-25 FredKiefer StatusNone Fixed
        Assigned toNone FredKiefer
        Open/ClosedOpen Closed
    2018-02-25 yavor Attached File- Added 0001-Add-font_cacher-manpage-install-font_cacher-conditio.patch, #43398

    Back to the top

    Powered by Savane 3.13-02a9.
    Corresponding source code