bugGNU Octave - Bugs: bug #45588, mxe-octave default-octave build...

 
 

bug #45588: mxe-octave default-octave build failure

Submitted by:  Avinoam Kalma <avinoam>
Submitted on:  Mon 20 Jul 2015 06:43:49 PM UTC  
 
Category: Configuration and Build SystemSeverity: 3 - Normal
Priority: 5 - NormalItem Group: Build Failure
Status: FixedAssigned to: John Donoghue <lostbard>
Originator Name: AvinoamOpen/Closed: Closed
Release: devOperating System: GNU/Linux

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)

Tue 28 Jul 2015 08:13:04 PM UTC, comment #13:

marking as fixed

John Donoghue <lostbard>
Project MemberIn charge of this item.
Tue 28 Jul 2015 08:11:31 PM UTC, comment #12:

It works for me. Thank you!

Avinoam Kalma <avinoam>
Project Member
Tue 28 Jul 2015 10:19:38 AM UTC, comment #11:

Pushed http://hg.savannah.gnu.org/hgweb/octave/rev/be6720faab96

The combination of this patch with the change pushed in comment 7 gets default octave working in mxe for me.

John Donoghue <lostbard>
Project MemberIn charge of this item.
Tue 28 Jul 2015 02:28:39 AM UTC, comment #10:

A windows cross build was completed without errors. I had only done a partial linux native build and it worked, so am doing a full new linux build just to make sure it didnt break anything else before pushing it up.

John Donoghue <lostbard>
Project MemberIn charge of this item.
Tue 28 Jul 2015 02:23:20 AM UTC, comment #9:

Ok your patch looks more complete and sounds like you've already tested the build with it.

Mike Miller <mtmiller>
Project Administrator
Tue 28 Jul 2015 02:19:43 AM UTC, comment #8:

I think comment #1 is a different problem.

Can you try the attached patch and see if that resolves that error? This needs to be applied before bootstrap is run.

(file #34543)

Mike Miller <mtmiller>
Project Administrator
Tue 28 Jul 2015 02:16:28 AM UTC, comment #7:

So it does not need patch any more for windows docs.
I have pushed to mxe to allow it to build.: http://hg.octave.org/mxe-octave/rev/3a38544f2a8f

octave does need a change to the makefile for general cross building - patch attached for this.

(file #34542)

John Donoghue <lostbard>
Project MemberIn charge of this item.
Mon 27 Jul 2015 01:23:56 PM UTC, comment #6:

I don't know if the patch is needed, with it removed, see comment 1 for what happens.

John Donoghue <lostbard>
Project MemberIn charge of this item.
Mon 27 Jul 2015 12:42:56 PM UTC, comment #5:

If this patch was really needed, then it should be reapplied to the top-level Makefile.in (where the Automake install rule is still the same).

If someone can explain why this install rule fails and needs to be patched for Windows, maybe it can be submitted as a bug against Automake.

Mike Miller <mtmiller>
Project Administrator
Mon 27 Jul 2015 12:28:21 PM UTC, comment #4:

Yes - all the changes recently for non recursive builds has removed all the Makefile.in files, so the current patch is no longer relavenent.

John Donoghue <lostbard>
Project MemberIn charge of this item.
Mon 27 Jul 2015 03:51:22 AM UTC, comment #3:

It seems that the problem is that there is no Makefile.in
in the library doc/interpreter, so the patch
mxe-octave/src/default-octave-1-docinstall.patch cannot work.

Avinoam Kalma <avinoam>
Project Member
Sat 25 Jul 2015 02:03:36 AM UTC, comment #2:

Probably related to https://savannah.gnu.org/bugs/index.php?45567

John Donoghue <lostbard>
Project MemberIn charge of this item.
Sat 25 Jul 2015 02:02:28 AM UTC, comment #1:

With old patch removed, make attempts to use the octave binary to generate the documentation.

John Donoghue <lostbard>
Project MemberIn charge of this item.
Mon 20 Jul 2015 06:43:49 PM UTC, original submission:

Another problem, different from bug #45578:

Attempting to build a current default branch octave using mxe-octave, I get:

It seems that there is a problem with the patch in
mxe-octave/src/default-octave-1-docinstall.patch, which is
probably not compatible to Makefile.in

Avinoam Kalma <avinoam>
Project Member

 

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

Attach File(s):
   
   
Comment:
   

Attached Files
file #34543:  oct-exe-ext.diff added by mtmiller (566B - text/x-diff)
file #34542:  xross-fix.patch added by lostbard (2KiB - text/x-patch)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by mtmiller (Posted a comment)
  • -unavailable- added by lostbard (Posted a comment)
  • -unavailable- added by avinoam (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 6 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Tue 28 Jul 2015 08:13:04 PM UTClostbardStatusReady For Test=>Fixed
      Open/ClosedOpen=>Closed
    Tue 28 Jul 2015 10:19:38 AM UTClostbardStatusNone=>Ready For Test
      Assigned toNone=>lostbard
    Tue 28 Jul 2015 02:19:43 AM UTCmtmillerAttached File-=>Added oct-exe-ext.diff, #34543
    Tue 28 Jul 2015 02:16:28 AM UTClostbardAttached File-=>Added xross-fix.patch, #34542

    Back to the top


    Powered by Savane 3.1-cleanup1