bugGNU Octave - Bugs: bug #44152, print("plot_file.pdf")...

 
 

bug #44152: print("plot_file.pdf") generates warning message

Submitter:  Michael Godfrey <godfrey>
Submitted:  Sun 01 Feb 2015 05:35:00 PM UTC
   
 
Category:  Plotting with OpenGL Severity:  4 - Important
Priority:  3 - Low Item Group:  Regression
Status:  Fixed Assigned to:  None
Originator Name:  Godfrey Open/Closed:  * Closed
Release:  * dev Operating System:  * GNU/Linux
Fixed Release:  None Planned Release:  None
* Mandatory Fields

Add a New Comment Rich Markup
   

Fri 01 Sep 2017 06:42:39 PM UTC, comment #3: 

This turned out to be a long patch.  There was a lot of cruft surrounding the warning messages.  I checked something in here (http://hg.savannah.gnu.org/hgweb/octave/rev/fcf7543e0d49).

Rik <rik5>
Group administrator
Wed 29 Jun 2016 10:20:24 AM UTC, comment #2: 

Omitting all of the checks seems a good option.  I assume that the error from the missing binary states which binary is missing.

Lachlan Andrew <lachlan>
Fri 24 Apr 2015 05:16:42 AM UTC, comment #1: 

The relevant lines from _print_parse_opts_.m


  if (warn_on_missing_binary)
    if (isempty (arg_st.ghostscript.binary))
      warning ("print:missing_gs", "print.m: Ghostscript binary is not available.\nOnly eps output is available.");
    else
      if (isempty (arg_st.epstool_binary))
        warning ("print:missing_epstool", "print.m: epstool binary is not available.\nSome output formats are not available.");
      endif
      if (isempty (arg_st.fig2dev_binary))
        warning ("print:missing_fig2dev", "print.m: fig2dev binary is not available.\nSome output formats are not available.");
      endif
      if (isempty (arg_st.pstoedit_binary))
        warning ("print:missing_pstoedit", "print.m: pstoedit binary is not available.\nSome output formats are not available.");
      endif
    endif
    warn_on_missing_binary = false;
  endif


Should we get rid of all of these messages entirely and just use the error reporting that will happen if you try to print something that requires a missing binary?

Or maybe we should keep the first one because Ghostscript is pretty crucial to most of the printing.

Rik <rik5>
Group administrator
Sun 01 Feb 2015 05:35:00 PM UTC, original submission:  

The warning is:
warning: print.m: pstoedit binary is not available.
Some output formats are not available.
warning: called from
    _print_parse_opts_ at line 383 column 9
    print at line 292 column 8
====================================
This warning is not appropriate since pstoedit is not used
to generate PDF,

Of course, installing pstoedit causes the warning to not appear,
but the plot is identical to the one without pstoedit.

pstoedit is not tested for in ./configure. If it is needed
for some plot formats, it should be tested for, or the
warning should only be issued if the plot requires it.

This is a minor item.

Michael Godfrey <godfrey>
Group Member

 

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

Attach Files:
   
   
Comment:
   

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by lachlan (Posted a comment)
  • -email is unavailable- added by rik5 (Updated the item)
  • -email is unavailable- added by godfrey (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 group members can vote.

     

    Follow 5 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2017-09-01 rik5 StatusNone Fixed
        Open/ClosedOpen Closed
    2016-07-01 rik5 Severity2 - Minor 4 - Important
    2015-02-01 rik5 Severity3 - Normal 2 - Minor
        Priority5 - Normal 3 - Low

    Back to the top

    Powered by Savane 3.13-d3ae.
    Corresponding source code