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

 
 

bug #39896: building development snapshot fails without flex

Submitted by:  Mike Miller <mtmiller>
Submitted on:  Fri 30 Aug 2013 04:27:24 AM UTC  
 
Category: Configuration and Build SystemSeverity: 3 - Normal
Priority: 5 - NormalItem Group: Build 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.

 

(Jump to the original submission 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>
Project Administrator
Fri 30 Aug 2013 02:53:44 PM UTC, comment #8:

Please try this changeset:

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

Michael Goffioul <goffioul>
Project Member
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>
Project Administrator
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>
Project Member
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>
Project Member
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>
Project 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>
Project 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>
Project Member
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:

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>
Project 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>
Project Administrator

 

(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 goffioul (Posted a comment)
  • -unavailable- added by jwe (Posted a comment)
  • -unavailable- added by mtmiller (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
    Fri 30 Aug 2013 02:53:44 PM UTCgoffioulStatusNone=>Fixed
      Open/ClosedOpen=>Closed

    Back to the top


    Powered by Savane 3.1-cleanup1