bugGNU Octave - Bugs: bug #61668, [octave-forge] (gsl) configure...

 
 

bug #61668: [octave-forge] (gsl) configure tests fail because main is defined with no return type

Submitter:  Carlo de Falco <cdf>
Submitted:  Mon 13 Dec 2021 03:25:59 PM UTC
   
 
Category:  Octave Forge Package Severity:  3 - Normal
Priority:  5 - Normal Item Group:  Build Failure
Status:  Patch Submitted Assigned to:  None
Originator Name:  Open/Closed:  * Open
Release:  * 7.0.90 Operating System:  * Any
Fixed Release:  None Planned Release:  None
* Mandatory Fields

Add a New Comment Rich Markup
   

Wed 11 Jan 2023 09:49:48 AM UTC, comment #2: 

Has the patch been applied?
Who is maintaining the GSL package?

Carlo de Falco <cdf>
Group Member
Mon 13 Dec 2021 03:28:43 PM UTC, comment #1: 

original submission:

> the following change fixes the problem for me


the patch didn't show in the original submission, trying again ...



$ diff -u configure.ac.in configure.ac.in.new
--- configure.ac.in        2021-12-13 16:16:41.000000000 +0100
+++ configure.ac.in.new        2021-12-13 16:15:25.000000000 +0100
@@ -82,7 +82,7 @@
   [AC_LANG_SOURCE(
      [[
 #include "ov-scalar.h"
-main()
+int main()
 {
   octave_scalar x (1.234);
   bool y = x.isreal ();
@@ -102,7 +102,7 @@
 #include "ov-scalar.h"
 #include "ovl.h"
 #include "parse.h"
-main()
+int main()
 {
   octave_value_list ovl = octave::feval ("sin", octave_value (1.234));
   return 0;


Carlo de Falco <cdf>
Group Member
Mon 13 Dec 2021 03:25:59 PM UTC, original submission:  

Installing the gsl on Octave 6.3 fails because
the configure script fails to detect 6.3 features.

the reason for the failure is that the tests defined
in configure define main with no return type, which causes
the tests to fail with the error


 error: C++ requires a type specifier for all declarations


regardless of the Octave version being used

I tested this with clang++ on macos but suspect the same
would happen on other systems

the following change fixes the problem for me


$ diff -u configure.ac.in configure.ac.in.new
--- configure.ac.in        2021-12-13 16:16:41.000000000 +0100
+++ configure.ac.in.new        2021-12-13 16:15:25.000000000 +0100
@@ -82,7 +82,7 @@
   [AC_LANG_SOURCE(
      [[
 #include "ov-scalar.h"
-main()
+int main()
 {
   octave_scalar x (1.234);
   bool y = x.isreal ();
@@ -102,7 +102,7 @@
 #include "ov-scalar.h"
 #include "ovl.h"
 #include "parse.h"
-main()
+int main()
 {
   octave_value_list ovl = octave::feval ("sin", octave_value (1.234));
   return 0;
-varbatim-

Carlo de Falco <cdf>
Group Member

 

(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 cdf (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 project members can vote.

     

    Follows 1 latest change.

    Date Changed by Updated Field Previous Value => Replaced by
    2023-01-11 mmuetzel Dependencies- bugs #59159 is dependent

    Back to the top

    Powered by Savane 3.12