bugGNU Octave - Bugs: bug #42900, build failure on invalid symbol...

 
 

bug #42900: build failure on invalid symbol "OCTAVE-DEBUG"

Submitter:  Mike Miller <mtmiller>
Submitted:  Fri 01 Aug 2014 08:30:18 PM UTC
   
 
Category:  Configuration and Build System Severity:  3 - Normal
Priority:  5 - Normal Item Group:  Build Failure
Status:  Fixed Assigned to:  None
Originator Name:  Open/Closed:  * Closed
Release:  * dev Operating System:  * GNU/Linux
Fixed Release:  None Planned Release:  None
* Mandatory Fields

Add a New Comment Rich Markup
   

Mon 04 Aug 2014 02:58:35 AM UTC, comment #2: 

And http://hg.savannah.gnu.org/hgweb/octave/rev/8de35ac99e80 on stable. I have not verified myself but seems to be fixed on the hydra builder, closing.

Mike Miller <mtmiller>
Group Member
Sun 03 Aug 2014 06:56:06 AM UTC, comment #1: 

This should be fixed by jwe's cset here http://hg.savannah.gnu.org/hgweb/octave/rev/7761c281aac0.  The report can be closed after this is verified.

Rik <rik5>
Group administrator
Fri 01 Aug 2014 08:30:18 PM UTC, original submission:  

The latest hydra build of the default branch failed to compile due to the following error:


In file included from parse-tree/lex.ll:98:0:
parse-tree/oct-parse.h:36:15: warning: extra tokens at end of #ifndef directive [enabled by default]
 #ifndef OCTAVE-DEBUG
               ^
parse-tree/oct-parse.h:44:17: warning: missing whitespace after the macro name [enabled by default]
 #  define OCTAVE-DEBUG 0
                 ^
parse-tree/oct-parse.h:44:24: error: missing binary operator before token "0"
 #  define OCTAVE-DEBUG 0
                        ^
parse-tree/oct-parse.h:47:5: note: in expansion of macro 'OCTAVE'
 #if OCTAVE-DEBUG
     ^
parse-tree/oct-parse.h:52:15: warning: extra tokens at end of #ifndef directive [enabled by default]
 #ifndef OCTAVE-TOKENTYPE


See http://hydra.nixos.org/build/13025761 for details.

I think this is due to a typo in this change made on the stable branch today:

http://hg.savannah.gnu.org/hgweb/octave/rev/d8abf813c69f

If I'm reading the change right, the %define api.prefix should be "octave_" instead of "octave-".

Mike Miller <mtmiller>
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 rik5 (Posted a comment)
  • -email is unavailable- added by mtmiller (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 3 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2014-08-04 mtmiller StatusReady For Test Fixed
        Open/ClosedOpen Closed
    2014-08-03 rik5 StatusNone Ready For Test

    Back to the top

    Powered by Savane 3.13-caa5.
    Corresponding source code