bugGNU Octave - Bugs: bug #43603, ltfat 2.0.1: fails to build and...

 
 

bug #43603: ltfat 2.0.1: fails to build and install on mxe-octave installation

Submitter:  John Donoghue <lostbard>
Submitted:  Thu 13 Nov 2014 08:49:42 PM UTC
   
 
Category:  Octave Package Severity:  3 - Normal
Priority:  5 - Normal Item Group:  Build Failure
Status:  Fixed Assigned to:  lostbard
Originator Name:  JohnD Open/Closed:  * Closed
Release:  * 3.8.2 Operating System:  * Microsoft Windows
Fixed Release:  None Planned Release:  None
* Mandatory Fields

Add a New Comment Rich Markup
   

Thu 22 Jan 2015 02:07:01 PM UTC, comment #5: 

Patch added to repo http://sourceforge.net/p/ltfat/bugs/112/#6948 so will be fixed in next release.

mxe-sources patched.

John Donoghue <lostbard>
Group Member
Fri 21 Nov 2014 06:43:15 PM UTC, comment #4: 

Also posted to ltfat bugs page: http://sourceforge.net/p/ltfat/bugs/112/

John Donoghue <lostbard>
Group Member
Fri 21 Nov 2014 01:51:04 PM UTC, comment #3: 

Patch for current git-repo code

(file #32515)

John Donoghue <lostbard>
Group Member
Fri 21 Nov 2014 01:49:22 PM UTC, comment #2: 

Updated patch to stop warning on -PIC flag

(file #32514)

John Donoghue <lostbard>
Group Member
Thu 13 Nov 2014 08:54:37 PM UTC, comment #1: 

Adding maintainer of package to CC

John Donoghue <lostbard>
Group Member
Thu 13 Nov 2014 08:49:42 PM UTC, original submission:  

Attempting to install ltfat package from octave-forge fails:

>> pkg -forge -verbose install ltfat
mkdir (C:\Users\JDONOG~1.ECE\AppData\Local\Temp\oct-5)
untar (C:\Users\jdonoghue.ECE\ltfat-2.0.1.tar.gz, C:\Users\JDONOG~1.ECE\AppData\Local\Temp\oct-5)
checking for gcc... gcc
checking whether the C compiler works... yes
checking for C compiler default output file name... a.exe
checking for suffix of executables... .exe
checking whether we are cross compiling... no
checking for suffix of object files... o
checking whether we are using the GNU C compiler... yes
checking whether gcc accepts -g... yes
checking for gcc option to accept ISO C89... none needed
checking for gcc option to accept ISO C99... -std=gnu99
checking for g++... g++
checking whether we are using the GNU C++ compiler... yes
checking whether g++ accepts -g... yes
checking for mkoctfile... yes
checking for Pa_GetHostApiCount in -lportaudio... no
configure: WARNING: Portaudio lib not found. Disabling support of the block processing framework.
configure: creating ./config.status
config.status: creating Makefile
make: Entering directory `/tmp/oct-5/ltfat/src'
make -f Makefile_unix
make[1]: Entering directory `/tmp/oct-5/ltfat/src'
cc -O3 -fPIC -std=c99 -Ithirdparty  -DLTFAT_DOUBLE  -c dgt.c -o ddgt.o
make[1]: cc: Command not found
make[1]: *** [ddgt.o] Error 127
make[1]: Leaving directory `/tmp/oct-5/ltfat/src'
make: *** [oct] Error 2
make: Leaving directory `/tmp/oct-5/ltfat/src'


The issue appears to be that it attempts to use the unix makefile instead of the mingw one, and doesn't know uses 'cc' instead of gcc, even though configure found gcc.

Making it run the Makefile_mingwoct also appears to have issues that appear to be related with setting the SHELL variable to COMSPEC.

Attached is a patch to the package that ensures it uses CC from configure, only sets SHELL to COMSPEC if SHELL is  not already set.

With this patch, ltfat installs for me using mxe-octave 3.8.2 and hopefully doesn't break the linux build.


John Donoghue <lostbard>
Group Member

 

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

Attach Files:
   
   
Comment:
   

Attached Files
file #32515:  0001-Update-to-compile-with-mingw-mxe-octave.patch added by lostbard (2KiB - application/octet-stream - current gitrepo patch)
file #32514:  ltfat_mingw.diff added by lostbard (1KiB - application/octet-stream - update for -PIC)
file #32428:  ltfat.diff added by lostbard (690B - 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 lostbard
  • -email is unavailable- added by lostbard (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 8 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2015-01-22 lostbard StatusPatch Submitted Fixed
        Assigned toNone lostbard
        Open/ClosedOpen Closed
    2014-11-21 lostbard Attached File- Added 0001-Update-to-compile-with-mingw-mxe-octave.patch, #32515
        StatusNone Patch Submitted
    2014-11-21 lostbard Attached File- Added ltfat_mingw.diff, #32514
    2014-11-13 lostbard Carbon-Copy- Added -email is unavailable-
    2014-11-13 lostbard Attached File- Added ltfat.diff, #32428

    Back to the top

    Powered by Savane 3.13-f8d8.
    Corresponding source code