bugGNU Octave - Bugs: bug #53849, [octave forge] (signal) error...

 
 

bug #53849: [octave forge] (signal) error building medfilt1.cc on Windows

Submitter:  Charles Praplan <charprap>
Submitted:  Sun 06 May 2018 10:53:46 AM UTC
   
 
Category:  Octave Package Severity:  3 - Normal
Priority:  5 - Normal Item Group:  Build Failure
Status:  Fixed Assigned to:  None
Originator Name:  Charles Praplan 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

Wed 26 Sep 2018 06:01:29 AM UTC, comment #9: 

Hello, I have the same issue and the link seems not available now. How can I install the package now? thanks

Jay Huang <hj411335>
Wed 09 May 2018 05:43:40 PM UTC, comment #8: 

Thanks for the testing, this fix will be part of the 1.4.1 bug fix release.

Mike Miller <mtmiller>
Group Member
Wed 09 May 2018 04:21:16 PM UTC, comment #7: 
Doug Stewart <dastew>
Wed 09 May 2018 04:09:12 PM UTC, comment #6: 

OK in Win10 and Octave 4.4.0 with the command

>> pkg install octave-signal-3cbff3de1ebc.tar.gz


BUT

installing directly from octave-forge using the command you mentioned fails:

>> pkg install -forge https://hg.mtmxr.com/octave-signal/archive/3cbff3de1ebc.tar.gz

error: get_forge_pkg: invalid package NAME: https://hg.mtmxr.com/octave-signal/archive/3cbff3de1ebc.ta
r.gz
error: called from
    get_forge_pkg at line 34 column 5
    get_forge_download at line 26 column 12
    pkg at line 403 column 29

Charles Praplan <charprap>
Wed 09 May 2018 06:28:44 AM UTC, comment #5: 

OK in Win7 and Octave 4.4.0

>> pkg install octave-signal-3cbff3de1ebc.tar.gz

For information about changes from previous versions of the signal package, run 'news signal'.

>> pkg list signal

Package Name  | Version | Installation directory
--------------+---------+-----------------------
      signal  |   1.4.0 | E:\octave\OCTAVE~1.0\...

Renato <yamane>
Tue 08 May 2018 09:46:18 PM UTC, comment #4: 

I have written a fix for this build error, please test with revision 3cbff3de1ebc, available at

https://hg.mtmxr.com/octave-signal/rev/3cbff3de1ebc

or by installing a test package with

pkg install -forge https://hg.mtmxr.com/octave-signal/archive/3cbff3de1ebc.tar.gz

Mike Miller <mtmiller>
Group Member
Mon 07 May 2018 10:22:34 AM UTC, comment #3: 

Also confirmed in Windows 7 Professional.

Renato <yamane>
Mon 07 May 2018 05:32:42 AM UTC, comment #2: 

Yes I can test it! Just tell me when!

Charles Praplan <charprap>
Sun 06 May 2018 03:50:47 PM UTC, comment #1: 

Thanks. If I provide a patch or update the mercurial repo can you test whether the fix works for you?

The medfilt1 function compiles fine on GNU/Linux, seems to only be a problem on Windows.

It would be very helpful to have regular Windows testers for these kinds of issues in the future.

Mike Miller <mtmiller>
Group Member
Sun 06 May 2018 10:53:46 AM UTC, original submission:  

When trying to update the packages, errors occurred with the new signal 1.4.0 package. In the error report below, one can see that the io package was updated but that the signal package not.

I also tried to download the package (signal-1.4.0.tar.gz) and to install it locally, still without success.

This is on Windows 10.



>> pkg update

For information about changes from previous versions of the io package, run 'news io'.
warning: pkg: package odepkg not found on forge - skipping update
medfilt1.cc: In member function 'void sorted_window::init(const double*, octave_idx_type, octave_idx_type, octave_idx_type)':
medfilt1.cc:91:7: error: 'bzero' was not declared in this scope
       bzero (buf, zeros * sizeof (double));
       ^~~~~
medfilt1.cc:91:7: note: suggested alternative: 'zeros'
       bzero (buf, zeros * sizeof (double));
       ^~~~~
       zeros
make: * [medfilt1.oct] Error 1
make: Entering directory `/tmp/oct-Ec9drO/signal-1.4.0/src'
C:/Programs/Octave/OCTAVE~1.0/bin/mkoctfile-4.4.0.exe -Wall -Wno-deprecated-declarations  _fwht_.cc -o _fwht_.oct
C:/Programs/Octave/OCTAVE~1.0/bin/mkoctfile-4.4.0.exe -Wall -Wno-deprecated-declarations  _ultrwin_.cc -o _ultrwin_.oct
C:/Programs/Octave/OCTAVE~1.0/bin/mkoctfile-4.4.0.exe -Wall -Wno-deprecated-declarations  -c cl2bp.cc -o cl2bp.o
C:/Programs/Octave/OCTAVE~1.0/bin/mkoctfile-4.4.0.exe -Wall -Wno-deprecated-declarations  -c cl2bp_lib.cc -o cl2bp_lib.o
C:/Programs/Octave/OCTAVE~1.0/bin/mkoctfile-4.4.0.exe -Wall -Wno-deprecated-declarations  cl2bp.o cl2bp_lib.o -o cl2bp.oct
C:/Programs/Octave/OCTAVE~1.0/bin/mkoctfile-4.4.0.exe -Wall -Wno-deprecated-declarations  medfilt1.cc -o medfilt1.oct
make: Leaving directory `/tmp/oct-Ec9drO/signal-1.4.0/src'

pkg: error running `make' for the signal package.
error: called from
    configure_make at line 95 column 9
    install at line 184 column 7
    pkg at line 437 column 9
    pkg at line 598 column 11

Charles Praplan <charprap>

 

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

Attach Files:
   
   
Comment:
   

No files currently attached

 

Depends on the following items: None found

Digest:
   bug dependencies.

 

Carbon-Copy List
  • -email is unavailable- added by hj411335 (Posted a comment)
  • -email is unavailable- added by dastew (Posted a comment)
  • -email is unavailable- added by yamane (Posted a comment)
  • -email is unavailable- added by charprap (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 9 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2018-09-26 mtmiller Carbon-CopyRemoved 80942 -
    2018-06-20 mtmiller Dependencies- bugs #54155 is dependent
    2018-06-05 mtmiller Dependencies- bugs #54066 is dependent
    2018-05-09 mtmiller StatusReady For Test Fixed
        Open/ClosedOpen Closed
    2018-05-08 mtmiller StatusNone Ready For Test
    2018-05-06 mtmiller Item GroupInstallation Failure Build Failure
        Release4.4.0 dev
        SummaryErrors when updating signal package to 1.4.0 [octave forge] (signal) error building medfilt1.cc on Windows

    Back to the top

    Powered by Savane 3.13-f8d8.
    Corresponding source code