bugGNU Octave - Bugs: bug #54384, unicode conversion fails with...

 
 

bug #54384: unicode conversion fails with error on Solaris system

Submitted by:  John W. Eaton <jwe>
Submitted on:  Thu 26 Jul 2018 01:03:18 PM UTC  
 
Category:  Libraries Severity:  3 - Normal
Priority:  5 - Normal Item Group:  Unexpected Error
Status:  Fixed Assigned to:  None
Originator Name:  Open/Closed:  Closed
Release:  dev Operating System:  Solaris/SunOS

Add a New Comment(Rich Markup)
   

You are not logged in

Please log in, so followups can be emailed to you.

 

Fri 27 Jul 2018 12:53:18 PM UTC, comment #5:

I grafted the change to stable here:
http://hg.savannah.gnu.org/hgweb/octave/rev/e866bb226d38

For a test I manually commented out "#define HAVE_ICONV 1" from config.h after configure and re-compiled. Calling "native2unicode (234, 'foo')" showed the error message and the BISTs were skipped on "test native2unicode" and "test unicode2native". So I guess it works.

Closing as fixed.

Markus Mützel <mmuetzel>
Project Member
Thu 26 Jul 2018 09:24:55 PM UTC, comment #4:

If this works, the patch should be backported to stable.

Rik <rik5>
Project Administrator
Thu 26 Jul 2018 05:56:14 PM UTC, comment #3:

I pushed a changeset here that hopefully somewhat improves the error message and makes the failing BISTs conditional on HAVE_ICONV:
http://hg.savannah.gnu.org/hgweb/octave/rev/5401dfadd2f6

Marking as Ready For Test.

Markus Mützel <mmuetzel>
Project Member
Thu 26 Jul 2018 02:54:34 PM UTC, comment #2:

I see this in config.h on the Solaris system:

John W. Eaton <jwe>
Project Administrator
Thu 26 Jul 2018 02:32:30 PM UTC, comment #1:

This snippet from the gnulib source code might be related:

This is if HAVE_ICONV is not defined.
Is this macro also available in Octave? Maybe we could show a more meaningful error message.

Markus Mützel <mmuetzel>
Project Member
Thu 26 Jul 2018 01:03:18 PM UTC, original submission:

The test suite shows the following errors on Solaris:

John W. Eaton <jwe>
Project Administrator

 

(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

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by rik5 (Posted a comment)
  • -email is unavailable- added by mmuetzel (Posted a comment)
  • -email is unavailable- added by jwe (Submitted the item)
  •  

    Do you think this task is very important?
    If so, you can add your encouragement to it.
    This task has 0 encouragements so far.

    Only project members can vote.

     

     

     

    Follow 3 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2018-07-27 mmuetzel StatusReady For Test => Fixed
        Open/ClosedOpen => Closed
    2018-07-26 mmuetzel StatusNone => Ready For Test

    Back to the top


    Powered by Savane 3.3