bugGNU Octave - Bugs: bug #36950, configure warning summary contains...

 
 

bug #36950: configure warning summary contains extra "yes"

Submitted by:  Carnë Draug <carandraug>
Submitted on:  Wed 25 Jul 2012 03:57:11 PM UTC  
 
Category: Configuration and Build SystemSeverity: 3 - Normal
Priority: 5 - NormalItem Group: Installation Failure
Status: FixedAssigned to: None
Originator Name: Open/Closed: Closed
Release: devOperating System: Any

Add a New Comment(Rich Markup)
   

You are not logged in

Please log in, so followups can be emailed to you.

 

Wed 25 Jul 2012 04:07:27 PM UTC, comment #1:

Thanks, I've pulled in both changes:

http://hg.savannah.gnu.org/hgweb/octave/graph/c753ee228708

Jordi Gutiérrez Hermoso <jordigh>
Project Administrator
Wed 25 Jul 2012 03:57:11 PM UTC, original submission:

Hi

right at the end of running configure, on the warnings section (about not having gnuplot installed etc), if someone does not have gnuplot installed, there's an extra line that simply says "yes".

This does not occur on the stable branch. I have tracked this down to the revision 8a84849ad986 and prepared a changeset. Please pull the revision c753ee228708 from https://bitbucket.org/carandraug/octave

Also, not a bug but on the list of libraries (at the end of the configure output), while some say "libraries" others simply say "LIBS". I have made this consistent. Please pull the revision a25937adede9 from the same place.

I have also found that the multiline warning messages on m4/acinclude.m4 only have "configure: WARNING" on the first line (the others have one per line (the ones at the end of configure.ac). These warns are first stored in warn_thing variable and then passed to OCTAVE_CONFIGURE_WARNING(). Should I replace it by something like

OCTAVE_CONFIGURE_WARNING([gnuplot not found. It isn't necessary to have gnuplot installed, but])
OCTAVE_CONFIGURE_WARNING([without native graphics or gnuplot you won't be able to use any of])
OCTAVE_CONFIGURE_WARNING([Octave's plotting commands.])

? I don't see the warn variables being used anymore but if they exist to know that we have already warned about it, They could maybe be replaced by warn_thing=yes ? I can prepare a changeset for this.

Carnë

Carnë Draug <carandraug>
Project Member

 

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

Attach File(s):
   
   
Comment:
   

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by jordigh (Posted a comment)
  • -unavailable- added by carandraug (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 project members can vote.

     

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

     

     

    Follow 2 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Wed 25 Jul 2012 04:07:27 PM UTCjordighStatusNone=>Fixed
      Open/ClosedOpen=>Closed

    Back to the top


    Powered by Savane 3.1-cleanup1