bugGNU Octave - Bugs: bug #39896, building development snapshot...

 
 

bug #39896: building development snapshot fails without flex

Submitter:  Mike Miller <mtmiller>
Submitted:  Fri 30 Aug 2013 04:27:24 AM 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:  * Any
Fixed Release:  None Planned Release:  None
* Mandatory Fields

Add a New Comment Rich Markup
   

Jump to the original submission

Sat 31 Aug 2013 03:22:48 AM UTC, comment #9: 

Looks to be working now, thanks for the quick fix.

http://hydra.nixos.org/build/5905318

Mike Miller <mtmiller>
Group Member
Fri 30 Aug 2013 02:53:44 PM UTC, comment #8: 
Michael Goffioul <goffioul>
Fri 30 Aug 2013 02:43:40 PM UTC, comment #7: 

Yes, that's exactly the problem, changing the dependency to Makefile.am should work. I agree that it's probably not worth it to tease out all those tiny awk scripts into separate files.

Mike Miller <mtmiller>
Group Member
Fri 30 Aug 2013 02:20:37 PM UTC, comment #6: 

I embedded the script into Makefile.am, because they're so simple that I didn't think a separate script was worth it. And even if they're generated due to other changes in Makefile.am, the impact on the rebuild is quite low (it's not like a change in config.h...).

Michael Goffioul <goffioul>
Fri 30 Aug 2013 02:18:11 PM UTC, comment #5: 

Because they're auto-generated, and the generation rule (simple awk script) is in Makefile.am.

Michael Goffioul <goffioul>
Fri 30 Aug 2013 02:17:07 PM UTC, comment #4: 

Oh, because of the embedded awk script?

Hmm, we have other things like this.

Maybe the embedded scripts should go in separate files so we can depend on those instead and not have to depend on Makefile or Makefile.am, which could change for many reasons not related to the files that are generated using the embedded scripts.

I can take care of that.

John W. Eaton <jwe>
Group administrator
Fri 30 Aug 2013 02:14:16 PM UTC, comment #3: 

Why do they need to depend on Makefile.am?

John W. Eaton <jwe>
Group administrator
Fri 30 Aug 2013 02:06:01 PM UTC, comment #2: 

Mike's right, I souldn't have made oct-tex-symbols.cc and oct-tex-lexer.ll dependent on Makefile. There's no way you can avoid reconstructing oct-tex-lexer.ll as Makefile will always be recreated by configure. The dependency should be on Makefile.am instead.

Michael Goffioul <goffioul>
Fri 30 Aug 2013 01:50:36 PM UTC, comment #1: 

In the 3.7.6 tar file that I created, I see the following files in the corefcn directory:


-rw-r--r-- 1 jwe jwe  5270 Aug 19 17:29 oct-tex-parser.yy
-rw-r--r-- 1 jwe jwe  2875 Aug 19 17:29 oct-tex-lexer.ll
-rw-r--r-- 1 jwe jwe  3173 Aug 20 16:21 oct-tex-parser.h
-rw-r--r-- 1 jwe jwe 59119 Aug 20 16:21 oct-tex-parser.cc
-rw-r--r-- 1 jwe jwe 62081 Aug 20 16:21 oct-tex-lexer.cc


and my build does not run flex or bison to regenerate these files.

How did you create the dist file (in or out of the source tree, in a fresh clone, etc.)?

What are the timestamps for these files in the tar file you created?

John W. Eaton <jwe>
Group administrator
Fri 30 Aug 2013 04:27:24 AM UTC, original submission:  

As of fe6518a1d87c, building Octave from a tar snapshot without flex availble fails. The source distribution has been able to be built without flex and bison. This change introduced a dependency on the Makefile that causes the oct-tex-lexer.ll to be regenerated, which in turn requires flex to be run at build time.

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

http://hydra.nixos.org/build/5784406/log/raw

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 jwe (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 2 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2013-08-30 goffioul StatusNone Fixed
        Open/ClosedOpen Closed

    Back to the top

    Powered by Savane 3.13-02a9.
    Corresponding source code