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

 
 

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

Submitter:  Avinoam Kalma <avinoam>
Submitted:  Mon 20 Jul 2015 06:43:49 PM UTC
   
 
Category:  Configuration and Build System Severity:  3 - Normal
Priority:  5 - Normal Item Group:  Build Failure
Status:  Fixed Assigned to:  lostbard
Originator Name:  Avinoam Open/Closed:  * Closed
Release:  * dev Operating System:  * GNU/Linux
Fixed Release:  None Planned Release:  None
* Mandatory Fields

Add a New Comment Rich Markup
   

Jump to the original submission

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

marking as fixed

John Donoghue <lostbard>
Group Member
Tue 28 Jul 2015 08:11:31 PM UTC, comment #12: 

It works for me. Thank you!

Avinoam Kalma <avinoam>
Group 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>
Group Member
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>
Group Member
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>
Group Member
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>
Group Member
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>
Group Member
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>
Group Member
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>
Group Member
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>
Group Member
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>
Group Member
Sat 25 Jul 2015 02:03:36 AM UTC, comment #2: 
John Donoghue <lostbard>
Group Member
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.
 


ibtool: link: Could not determine the host path corresponding to
libtool: link:   `/home/jade/mxe-octave/usr/x86_64-w64-mingw32/lib:/home/jade/mxe-octave/usr/x86_64-w64-mingw32/bin:/home/jade/mxe-octave/tmp-default-octave/octave-4.1.0+/.build/libgui/.libs:/home/jade/mxe-octave/tmp-default-octave/octave-4.1.0+/.build/libinterp/.libs:/home/jade/mxe-octave/tmp-default-octave/octave-4.1.0+/.build/liboctave/.libs:/home/jade/mxe-octave/usr/lib/gcc/x86_64-w64-mingw32/4.9.3:/home/jade/mxe-octave/usr/x86_64-w64-mingw32/lib/octave/4.1.0+'
libtool: link: Continuing, but uninstalled executables may not work.
rm -f src/octave-gui-4.1.0+.exe && \
        cd src && ln -s octave-gui.exe octave-gui-4.1.0+.exe
make[4]: *** No rule to make target `src/octave-gui', needed by `doc/interpreter/voronoi.txt'.  Stop.
make[4]: Leaving directory `/home/jade/mxe-octave/tmp-default-octave/octave-4.1.0+/.build'


John Donoghue <lostbard>
Group Member
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:


make[1]: Entering directory `/home/avinoam/mxe-octave'
rm -rf   '/home/avinoam/mxe-octave/tmp-default-octave' '/home/avinoam/mxe-octave
/tmp-default-octave-install'
mkdir -p '/home/avinoam/mxe-octave/tmp-default-octave'
( cd '/home/avinoam/mxe-octave/tmp-default-octave' &&  tar xzf '/home/avinoam/mx
e-octave/pkg/octave-4.1.0+.tar.gz' ) ||  false
test ! -d '/home/avinoam/mxe-octave/src/default-octave' || cp -a '/home/avinoam/
mxe-octave/src/default-octave' '/home/avinoam/mxe-octave/tmp-default-octave'
cd '/home/avinoam/mxe-octave/tmp-default-octave/octave-4.1.0+'
(cd '/home/avinoam/mxe-octave/tmp-default-octave/octave-4.1.0+' && patch -p1 -u)
 < /home/avinoam/mxe-octave/src/default-octave-1-docinstall.patch
can't find file to patch at input line 4
Perhaps you used the wrong -p or --strip option?
The text leading up to this was:
--------------------------
|diff -ur a/doc/interpreter/Makefile.in b/doc/interpreter/Makefile.in
|--- a/doc/interpreter/Makefile.in        2013-12-27 17:01:52.000000000 -0500
|+++ b/doc/interpreter/Makefile.in        2014-01-21 12:47:59.618865568 -0500
--------------------------
File to patch:
Skip this patch? [y]
Skipping patch.
1 out of 1 hunk ignored
make[1]: *** [build-only-default-octave] Error 1
make[1]: Leaving directory `/home/avinoam/mxe-octave'


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>
Group Member

 

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

Attach Files:
   
   
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
  • -email is unavailable- added by mtmiller (Posted a comment)
  • -email is unavailable- added by lostbard (Posted a comment)
  • -email is unavailable- added by avinoam (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 6 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2015-07-28 lostbard StatusReady For Test Fixed
        Open/ClosedOpen Closed
    2015-07-28 lostbard StatusNone Ready For Test
        Assigned toNone lostbard
    2015-07-28 mtmiller Attached File- Added oct-exe-ext.diff, #34543
    2015-07-28 lostbard Attached File- Added xross-fix.patch, #34542

    Back to the top

    Powered by Savane 3.13-02a9.
    Corresponding source code