taskGNU Octave - Tasks: task #12561, forge: conflicts due to strsplit

 
 

You are not allowed to post comments on this tracker with your current authentification level.

task #12561: forge: conflicts due to strsplit

Submitted by:  Carnë Draug <carandraug>
Submitted on:  Tue Apr 2 22:33:06 2013  
 
Should Start On: Tue Apr 2 00:00:00 2013Should be Finished on: Wed Apr 2 00:00:00 2014
Category: NonePriority: 5 - Normal
Status: NonePrivacy: Public
Assigned to: NoneOriginator Name: 
Open/Closed: OpenFixed Release: None
Planned Release: None

Thu Apr 14 11:19:11 2016, comment #5:

You can scrap ncarray and optim from the previous list. ncarray wants to be compatible with octave 3.4 and older versions Matlab so has its own strsplit that works in both.

Carnë Draug <carandraug>
Project Member
Thu Apr 14 10:41:24 2016, comment #4:

This task tracker forces a deadline so just ignore that field. In hindsight, maybe I should have reported this as a bug instead.

The following packages still have a copy of the old strsplit:

  • ncarray (nccoord)
  • miscellaneous (read_options)
  • dataframe (dataframe.m ,private/df_name2idx, and private/df_matassign)
  • fixed (fixedpoint)
  • system-identification (mutualinf, pspec_me)
  • plot (dxfwrite)
  • gsl (test_ellint and test_hyperg)

I wouldn't consider copying the old file a fix, it was a workaround for unmaintained packages. It also means that someone working on that package not knowing about that private file won't be able to use the core strsplit.

In some of the cases above, they might not even make use of the old strsplit features so maybe it's not needed at all. Still requires someone going through the functions.

Carnë Draug <carandraug>
Project Member
Thu Apr 14 03:16:05 2016, comment #3:

The deadline for this was two years ago.

Can it be closed, or does something still need to be done?

Lachlan Andrew <lachlan>
Project Member
Fri Apr 17 15:36:29 2015, comment #2:

I was just about the write up a bug reports on the misc package when I saw this.
I think will cover the errors on misc, dataframe shown on [1]

[1] http://wiki.octave.org/Octave-Forge#GNU_Octave_4.0_compatibility_assessment

John Donoghue <lostbard>
Project Member
Sat Apr 27 21:21:46 2013, comment #1:

Carnë,

Knowing the code, and after having run some tests, I think there are no changes needed in the io package (at least not in any of the spreadsheet functions).
Note: this is with changeset 16572:17941fedd4ce (tip of Apr. 27, 2013)

Philip Nienhuis <philipnienhuis>
Project Member
Tue Apr 2 22:33:06 2013, original submission:

Because of the problem with strsplit, I have added a copy of the old function as a private function to the packages that made use of it. Once 3.8 is released, they should be fixed to use the new version. Here's the list of packages

  • cgi
  • dataframe
  • ncArray
  • comm
  • fixed
  • fuzzy-logic-toolkit
  • gsl
  • io
  • miscellaneous
  • plot
  • system-identification

I don't think they will actually be released before Octave but the fix is there. Also, the mechanics and geometry make use of it on PKG_ADD and PKG_DEL scripts so those need to to be fixed in some other way.

Carnë Draug <carandraug>
Project Member

 

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by lachlan (Posted a comment)
  • -unavailable- added by lostbard (Posted a comment)
  • -unavailable- added by carandraug (Submitted the item)
  •  

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

    Only logged-in users can vote.

     

    Please enter the title of George Orwell's famous dystopian book (it's a date):

     

     

    No Changes Have Been Made to This Item

    Back to the top


    Powered by Savane 3.1-cleanup