bugGNU Octave - Bugs: bug #41218, socket package: functions do not...

 
 

bug #41218: socket package: functions do not removed from path after unload

Submitted by:  Carnë Draug <carandraug>
Submitted on:  Mon 13 Jan 2014 03:50:37 PM UTC  
 
Category:  Octave Forge Package Severity:  3 - Normal
Priority:  5 - Normal Item Group:  None
Status:  Fixed Assigned to:  None
Originator Name:  Open/Closed:  Closed
Release:  dev Operating System:  Any

Add a New Comment (Rich Markup)
   

You are not logged in

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

 

( Jump to the original submission)

Tue 21 Jan 2014 08:19:35 PM UTC, comment #6: 

this should be remedied in commit c4c977

Paul Dreik <pauldreik>
Tue 21 Jan 2014 02:22:46 PM UTC, comment #5: 

There was no method to "undo" autoload before. Versions before the inclusion of "remove" will keep the function until the end of the Octave session.

Carnë Draug <carandraug>
Project Member
Mon 20 Jan 2014 09:14:53 PM UTC, comment #4: 

ok the system ate my code. new try, all is on one line.

 PKG_DEL: try ; autoload ("disconnect", which ("socket"),
"remove") ; catch ; end;

Paul Dreik <pauldreik>
Mon 20 Jan 2014 09:13:06 PM UTC, comment #3: 

ok, I made a test with

// PKG_DEL: try ; autoload ("disconnect", which ("socket"), "remove") ; catch ; end;

which works in 3.8 and 3.6, have not tried in 3.4 yet.
It is a bit ugly, but hey it can be removed in a distant future:-)

What was the method to remove functions in 3.6?

Is there anything bad with doing it like this?

Paul Dreik <pauldreik>
Tue 14 Jan 2014 11:14:53 AM UTC, comment #2: 

Yes. pkg() will create PKG_ADD and PKG_DEL scripts from such comments in the source.

The problem is that such change would bump the next release dependencies by a lot.

Carnë Draug <carandraug>
Project Member
Mon 13 Jan 2014 06:52:20 PM UTC, comment #1: 

The .cc file is full of // PKG_ADD directives, do you mean
there should be complemented with // PKG_DEL as well?

Paul Dreik <pauldreik>
Mon 13 Jan 2014 03:50:37 PM UTC, original submission:  

The sockets package defines a bunch of functions from its single oct file. The problem is that once the package is unloaded, they stay around.

This can be fixed by adding PKG_DEL instructions to unload them with

autoload ("foos", "socket.oct", "remove")

The problem is that this feature was only added to a recent version (not sure if 3.8 or 3.6).

Carnë Draug <carandraug>
Project 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

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by pauldreik (Posted a comment)
  • -email is unavailable- added by carandraug (Submitted the item)
  • -email is unavailable- added by carandraug (Paul Dreik (maintainer of the sockets package))
  •  

    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
    2014-01-21 carandraug StatusNone => Fixed
        Open/ClosedOpen => Closed
    2014-01-13 carandraug Carbon-Copy- => Added slask@pauldreik.se

    Back to the top


    Powered by Savane 3.5