bugGNU Octave - Bugs: bug #35572, ./configure doesn't test for GNU...

 
 

bug #35572: ./configure doesn't test for GNU sh, but build files depend on it.

Submitter:  None
Submitted:  Mon 20 Feb 2012 07:55:49 PM UTC
   
 
Category:  Configuration and Build System Severity:  3 - Normal
Priority:  5 - Normal Item Group:  None
Status:  Fixed Assigned to:  None
Originator Name:  Rick Originator Email:  -email is unavailable-
Open/Closed:  * Closed Release:  * 3.4.3
Operating System:  * Solaris/SunOS Fixed Release:  None
Planned Release:  None
* Mandatory Fields

Add a New Comment Rich Markup
   

Thu 23 Feb 2012 08:52:08 PM UTC, comment #4: 

I checked in the following changeset:

http://hg.savannah.gnu.org/hgweb/octave/rev/08e48e7a4c8a

John W. Eaton <jwe>
Group administrator
Thu 23 Feb 2012 06:24:02 AM UTC, comment #3: 

The issue is that the output stored in $MX_INC is a newline delimited list.  Using 'echo' turns the list into a space-separated list.  Any way that accomplishes that action will be good enough.  This might be possible back in the original awk script, or you could use sed, or you could use the following kludge:


VX_INC=$($AWK -f $mk_ops prefix=vx list_h_files=1 $liboctave_dir/vx-ops)
VX_INC=$(echo $VX_INC)
echo "VX_OP_INC = $VX_INC" > $liboctave_dir/vx-op-inc.mk-t


This gets rids of 'echo -n' which you said was problematic and it converts the list in a separate command rather than embedding it within a double-quoted string.

Rik <rik5>
Group administrator
Thu 23 Feb 2012 12:55:53 AM UTC, comment #2: 

I'm wondering why I wrote


echo "MX_OP_INC = $(echo $MX_INC)" ...


instead of just using


echo "MX_OP_INC = $MX_INC" ...


Doesn't the latter work correctly?

I don't think echo -n is portable.  There is some discussion of that in the autoconf manual.

John W. Eaton <jwe>
Group administrator
Wed 22 Feb 2012 09:34:55 PM UTC, comment #1: 

I recoded the file in question to avoid the GNU sh construct that I think is causing the problem.  Could you test it as I don't have access to a Solaris system?

Steps to take:
1) Replace liboctave/config-ops.sh with the one attached to this bug report
2) In liboctave directory, 'rm op.mk'
3) In top-level directory, './configure --ANY_OPTIONS_YOU_HAVE--'
4) 'make'

(file #25121)

Rik <rik5>
Group administrator
Mon 20 Feb 2012 07:55:49 PM UTC, original submission:  

Build failed at liboctave/config-ops.sh with message:

Unexpected $ in VX_OP_INC = $, line 36

I changed the first line of that file  from
#! /bin/sh to

to

#! /path/to/gnu/sh

and the build continued successfully.

Thanks,
Rick

Anonymous

 

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

Attach Files:
   
   
Comment:
   

Attached Files
file #25121:  config-ops.sh added by rik5 (3KiB - application/x-sh)

 

Depends on the following items: None found

Items that depend on this one: None found

 

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

    Date Changed by Updated Field Previous Value => Replaced by
    2012-02-23 jwe StatusNone Fixed
        Open/ClosedOpen Closed
    2012-02-22 rik5 Attached File- Added config-ops.sh, #25121

    Back to the top

    Powered by Savane 3.13-02a9.
    Corresponding source code