bugGNU TeXmacs - Bugs: bug #33302, Incorrect use of gs to condition...

 
 

You are not allowed to post comments on this tracker with your current authentication level.

bug #33302: Incorrect use of gs to condition output of qt_image_to_eps causes problems with gs 9

Submitted by:  Sam Liddicott <samjam>
Submitted on:  Fri 13 May 2011 09:19:14 AM UTC  
 
Category: ConversionPriority: 9 - Immediate
Item Group: ErrorStatus: Fixed
Privacy: PublicAssigned to: Francois Poulain <fpoulain>
Originator Name: Open/Closed: Closed
Release: Release: None
Fixed Release: NoneFixed Release: 
Keywords: 

Sat 10 Nov 2012 12:00:23 PM UTC, comment #5:

Fixed in r5800.

Francois Poulain <fpoulain>
Project MemberIn charge of this item.
Mon 01 Oct 2012 07:29:52 PM UTC, comment #4:

It might be better to leave it as a runtime option, I don't know. In any case, if you feel like implementing it, I'll be happy to review the code if you want/need me to.

Because I really hate matters of dependencies and the build system, I'm assigning this item to Joris to (eventually) get his attention.

Miguel de Benito <mdbenito>
Project Member
Thu 23 Aug 2012 08:41:53 AM UTC, comment #3:

I propose to add a patch to the build system that detects gs9, and, if imagemagick is not used will fail to configure unless --enable-force-all-pdf-to-be-very-large=yes is also passed to configure

This will at least inform the package builders effectively

Sam Liddicott <samjam>
Project Member
Thu 14 Jul 2011 09:33:33 PM UTC, comment #2:

There is no bug in gs, this is incorrect use of gs that just happened to work satisfactorily before version 9. This is not my view, but the view of the gs developers.

gs after version 9 will work if and when the eps2write target is implemented (and used).

Until then, this buggy use of gs is causing PDF files to be 20 times larger than normal and to render slowly.

Let us have the dependency on imagemagick rather than this or we drive people to lyx.

Sam Liddicott <samjam>
Project Member
Thu 14 Jul 2011 08:14:56 PM UTC, comment #1:

The patch cannot be committed to the main repository if it forces a dependency on imagemagick. As it is a workaround some GS' bug, it can be used if some checks (GS version and presence of bug, imagemagick present, maybe config option) are added.

Miguel de Benito <mdbenito>
Project Member
Fri 13 May 2011 09:19:14 AM UTC, original submission:

Since using gs 9 my exported post-script files have been 2-3 times larger than normal and my pdf files have been 20 times larger than normal.

If I disable the use of gs to "condition" the eps files written by qt_image_to_eps then the resulting .ps file seems to not be conformant:
1. gv can only display page 1
2. evince can only display some page in the middle.
3. basic old gs can display all pages (althought of course it requires me to press enter after each page).

I guess this is why gs was being used to condition these .eps images.

I note that when ps2ps invoked on the non-conformant output ps file, it reduces it from 158MB to 9MB and causes it to work just fine in evince and so on.

Also, ps2pdf when invoked on the non-conformant output ps file produces a 2.5MB pdf file that also seems to work just fine.

Primary conclusions
1. The .eps written by qt_image_to_eps is not satisfactory.
2. The .eps written by gs is not satisfactory.
3. The .eps written by the imagemagick "convert" utility is satisfactory.

Secondary conclusions
1. ps2ps might be invoked to some advantage on the output .ps in all cases.
2. qt_image_to_eps might do well to copy the format produced by imagemagick or that produced by gs version 8 when processing the imagemagick format.
3. gs 9 tools should not be used to output intermediate post-script files

Actions - patch provided, tested on Ubuntu 11.04 - requires imagemagick to be present
1. In image_to_eps
a. Disable qt_image_to_eps
b. Disable gs_to_eps

Future actions:
1. Fix qt_image_to_eps to produce conformant eps files directly instead of using gs to sanitize
2. Maybe the gs eps2write target instead of the epswrite target when it exists (the ps2write target doesn't work).
3. Consider invoking ps2ps on the generated ps to condition and trim the resultant .ps file. This can reduce a .ps file from 167MB to 10MB

Sam Liddicott <samjam>
Project Member

 

Attached Files
file #23396:  0001-Disable-qt_image_to_eps-and-gs_to_eps-in-image_to_ep.patch added by samjam (2KiB - text/x-patch - Patch to disable use of qt_image_to_eps and gs_to_eps in image_to_eps)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by mdbenito (Posted a comment)
  • -unavailable- added by samjam (Submitted the item)
  •  

    Do you think this task is very important?
    If so, you can click here to add your encouragement to it.
    This task has 0 encouragements so far.

    Only logged-in users can vote.

     

    Please enter the title of George Orwell's famous dystopian book (it's a date):

     

     

    Follow 11 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Sat 10 Nov 2012 12:00:23 PM UTCfpoulainStatusConfirmed=>Fixed
      Assigned tovdhoeven=>fpoulain
      Open/ClosedOpen=>Closed
      Release1.0.7.10=>
    Mon 01 Oct 2012 07:29:52 PM UTCmdbenitoPriority5 - Normal=>9 - Immediate
      StatusPostponed=>Confirmed
      Assigned tomdbenito=>vdhoeven
    Thu 14 Jul 2011 08:14:56 PM UTCmdbenitoStatusNone=>Postponed
      Assigned toNone=>mdbenito
      Release=>1.0.7.10
    Fri 13 May 2011 09:20:38 AM UTCsamjamAttached File-=>Added 0001-Disable-qt_image_to_eps-and-gs_to_eps-in-image_to_ep.patch, #23396

    Back to the top


    Powered by Savane 3.1-cleanup1