bugGNU Octave - Bugs: bug #54165, (octave forge) [control]...

 
 

bug #54165: (octave forge) [control] Cross-build errors for several of package with octave-5.0.0

Submitter:  Philip Nienhuis <philipnienhuis>
Submitted:  Thu 21 Jun 2018 08:16:10 PM UTC
   
 
Category:  Octave Package Severity:  3 - Normal
Priority:  5 - Normal Item Group:  Build Failure
Status:  Fixed Assigned to:  lostbard
Originator Name:  Philip Nienhuis Open/Closed:  * Closed
Release:  * dev Operating System:  * Microsoft Windows
Fixed Release:  None Planned Release:  None
* Mandatory Fields

Add a New Comment Rich Markup
   

Jump to the original submission

Fri 29 Jun 2018 12:03:12 PM UTC, comment #14: 

All of-packages now build without error for me. Closing bug report as fixed.

Markus M├╝tzel <mmuetzel>
Group administrator
Wed 27 Jun 2018 06:03:55 PM UTC, comment #13: 

I pushed the following changeset:

http://hg.octave.org/mxe-octave/rev/f54322738d5f

John W. Eaton <jwe>
Group administrator
Wed 27 Jun 2018 05:26:24 PM UTC, comment #12: 

Looks good to me

John Donoghue <lostbard>
Group Member
Wed 27 Jun 2018 01:47:33 PM UTC, comment #11: 

John:

In that case, then maybe default-octave.mk should just use


--with-blas="-lblas -lxerbla"


when running configure for Octave, similar to what you suggested in comment #7.  How about the attached change?  I'm testing now but it will probably take a few more hours.

(file #44444)

John W. Eaton <jwe>
Group administrator
Tue 26 Jun 2018 10:54:05 PM UTC, comment #10: 

control uses the output of mkoctfile -p BLAS_LIBS for the libs to link with.

It might be one of the few that also include fortran sources ?

CUrrently only provides a makefile for building so if it went with a configure option it could detect a little better.

John Donoghue <lostbard>
Group Member
Tue 26 Jun 2018 08:20:07 PM UTC, comment #9: 

Just for completeness:
control is the only OF package that showed the -lxerbla error. All other OF packages get cross-built fine. So maybe control is the only package directly referencing *BLAS/Lapack libs?

Philip Nienhuis <philipnienhuis>
Group Member
Tue 26 Jun 2018 08:07:56 PM UTC, comment #8: 

Should the configure check for BLAS determine whether XERBLA needs to be provided separately?


John W. Eaton <jwe>
Group administrator
Tue 26 Jun 2018 07:47:03 PM UTC, comment #7: 

We could provide the blas libs explicitly to octave in configure as --with-blas=yes BLAS_LIBS='-lblas -lxerbla'


John Donoghue <lostbard>
Group Member
Tue 26 Jun 2018 07:44:43 PM UTC, comment #6: 

For completeness, control also fails doing a native compile in windows using pkg install -forge control, with same error

John Donoghue <lostbard>
Group Member
Sun 24 Jun 2018 09:56:43 PM UTC, comment #5: 

I would think perhaps that it should be pulled in with the blas libs, but currently in windows:
mkoctfile -p BLAS_LIBS is only -lblas



John Donoghue <lostbard>
Group Member
Fri 22 Jun 2018 08:14:17 PM UTC, comment #4: 

thank JohnD, that fix in comment #2 works.

However, cross-building of-control still doesn't work due to;

:
slicotlibrary.a(AB01OD.o): In function `ab01od_':
/home/philip/devel/octdev/mxe/mxe_64b20180514/tmp-of-control/tmpbYzVSW-pkg/control-3.1.0/src/sltmp
/AB01OD.f:352: undefined reference to `xerbla_'
slicotlibrary.a(AB04MD.o): In function `ab04md_':
/home/philip/devel/octdev/mxe/mxe_64b20180514/tmp-of-control/tmpbYzVSW-pkg/control-3.1.0/src/sltmp
/AB04MD.f:247: undefined reference to `xerbla_'
slicotlibrary.a(AB08ND.o): In function `ab08nd_':
/home/philip/devel/octdev/mxe/mxe_64b20180514/tmp-of-control/tmpbYzVSW-pkg/control-3.1.0/src/sltmp/AB08ND.f:334: undefined reference to `xerbla_'
slicotlibrary.a(AB08NX.o): In function `ab08nx_':
/home/philip/devel/octdev/mxe/mxe_64b20180514/tmp-of-control/tmpbYzVSW-pkg/control-3.1.0/src/sltmp/AB08NX.f:257: undefined reference to `xerbla_'
slicotlibrary.a(AB09HD.o): In function `ab09hd_':
/home/philip/devel/octdev/mxe/mxe_64b20180514/tmp-of-control/tmpbYzVSW-pkg/control-3.1.0/src/sltmp/AB09HD.f:492: undefined reference to `xerbla_'
slicotlibrary.a(AB09HY.o):/home/philip/devel/octdev/mxe/mxe_64b20180514/tmp-of-control/tmpbYzVSW-pkg/control-3.1.0/src/sltmp/AB09HY.f:227: more undefined references to `xerbla_' follow
:


Must be related to the recent fix for bug #39000 I suppose.
I fixed it by adding "-lxerbla" to the various mkoctfile references in the Makefile but I'm unsure if that's the right fix.

Philip Nienhuis <philipnienhuis>
Group Member
Fri 22 Jun 2018 12:35:11 PM UTC, comment #3: 

And pushed http://hg.octave.org/mxe-octave/rev/11a3d14a1562 for struct and database

John Donoghue <lostbard>
Group Member
Fri 22 Jun 2018 01:56:09 AM UTC, comment #2: 

I fixed that earlier today in the octave repo.

http://hg.octave.org/octave/rev/b3c68525ed82


Different issues currently make struct and database also fail, but beuase they are missing a include <iostream> from other changes to octave.



John Donoghue <lostbard>
Group Member
Thu 21 Jun 2018 08:36:10 PM UTC, comment #1: 

Several other packages with binary module fail to build as well. I just did 'touch installed-packages/of-<package name>'  to proceed to the next.

Logs for:
general
io
optim
specfun
are attached, after specfun I figured it must be one and the same issue and then I quit the crossbuild attempt


(file #44407, file #44408, file #44409, file #44410)

Philip Nienhuis <philipnienhuis>
Group Member
Thu 21 Jun 2018 08:16:10 PM UTC, original submission:  

The control package cannot be built in mxe-octave with octave-5.0.0 configured with
--enable-octave=default \
--enable-binary-packages \
--enable-devel-tools \
--enable-64 \
--with-ccache

Until one or two weeks ago it just got cross-built fine.

Octave tip at 9feda91c8592 "merge stable to default"
mxe-octave at 80461a9e516c "override default xerbla with version from Octave"

Relevant part of log attached, including the last two of a long list of deprecation warnings.
It looks a bit to me as if some include is lacking, but that's just a hunch.



Philip Nienhuis <philipnienhuis>
Group Member

 

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

Attach Files:
   
   
Comment:
   

Attached Files
file #44444:  diffs.txt added by jwe (750B - text/plain)
file #44407:  of-general added by philipnienhuis (13KiB - application/octet-stream)
file #44408:  of-io added by philipnienhuis (22KiB - application/octet-stream)
file #44409:  of-optim added by philipnienhuis (57KiB - application/octet-stream)
file #44410:  of-specfun added by philipnienhuis (33KiB - application/octet-stream)
file #44406:  of-control.log.part added by philipnienhuis (219KiB - application/octet-stream)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by mmuetzel (Posted a comment)
  • -email is unavailable- added by jwe (Updated the item)
  • -email is unavailable- added by lostbard (Posted a comment)
  • -email is unavailable- added by philipnienhuis (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 12 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2018-06-29 mmuetzel StatusReady For Test Fixed
        Open/ClosedOpen Closed
    2018-06-27 jwe Attached File- Added diffs.txt, #44444
    2018-06-22 lostbard StatusNone Ready For Test
        Assigned toNone lostbard
    2018-06-22 jwe Summary(octave-forge)[control] Cross-build errors for several of package with octave-5.0.0 (octave forge) [control] Cross-build errors for several of package with octave-5.0.0
    2018-06-21 philipnienhuis Attached File- Added of-general, #44407
        Attached File- Added of-io, #44408
        Attached File- Added of-optim, #44409
        Attached File- Added of-specfun, #44410
        Summary(octave-forge)[control] Cross-build errors for control package with octave-5.0.0 (octave-forge)[control] Cross-build errors for several of package with octave-5.0.0
    2018-06-21 philipnienhuis Attached File- Added of-control.log.part, #44406

    Back to the top

    Powered by Savane 3.13-12ab.
    Corresponding source code