bugGNU Octave - Bugs: bug #44524, pkg uninstall invalidname removes...

 
 

bug #44524: pkg uninstall invalidname removes global package entries

Submitter:  John Donoghue <lostbard>
Submitted:  Thu 12 Mar 2015 06:08:53 PM UTC
   
 
Category:  Octave Function Severity:  3 - Normal
Priority:  5 - Normal Item Group:  Incorrect Result
Status:  Fixed Assigned to:  None
Originator Name:  JohnD Open/Closed:  * Closed
Release:  * dev Operating System:  * Any
Fixed Release:  None Planned Release:  None
* Mandatory Fields

Add a New Comment Rich Markup
   

Fri 13 Mar 2015 02:41:17 AM UTC, comment #4: 

FYI this was also previously reported as bug #36453. Thanks for fixing!

Mike Miller <mtmiller>
Group Member
Thu 12 Mar 2015 09:19:53 PM UTC, comment #3: 

I modified the cset to just use isempty rather than length (...) == 0 and pushed it (http://hg.savannah.gnu.org/hgweb/octave/rev/6670df97456c).

Rik <rik5>
Group administrator
Thu 12 Mar 2015 06:39:04 PM UTC, comment #2: 

Fixed typo

(file #33321)

John Donoghue <lostbard>
Group Member
Thu 12 Mar 2015 06:34:25 PM UTC, comment #1: 

This patch checks delete_idx before continuing with the rest of the pkg delete.


(file #33320)

John Donoghue <lostbard>
Group Member
Thu 12 Mar 2015 06:08:53 PM UTC, original submission:  

When an unknown package name is used in uninstall, all names in the pkg list disappear:


>> pkg list
Package Name  | Version | Installation directory
--------------+---------+-----------------------
      divand  |   1.1.2 | C:\Octave\Octave-4.0.0-rc1\share\octave\packages\divand-1.1.2
          ga *|  0.10.0 | C:\Octave\Octave-4.0.0-rc1\share\octave\packages\ga-0.10.0
          io  |   2.2.7 | C:\Octave\Octave-4.0.0-rc1\share\octave\packages\io-2.2.7

>> pkg uninstall nopackagenamedthis
warning: some of the packages you want to uninstall are not installed
>> pkg list
no packages installed.
>>



The installed files all appear to still be there.
It happens in Windows and in Linux - although appears to be just global installed packages that disappears, which is what the windows packages default to.

John Donoghue <lostbard>
Group Member

 

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

Attach Files:
   
   
Comment:
   

Attached Files
file #33321:  dontdelpkg2.patch added by lostbard (6KiB - application/octet-stream)
file #33320:  dontdelpkg.patch added by lostbard (6KiB - application/octet-stream)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by mtmiller (Posted a comment)
  • -email is unavailable- added by rik5 (Posted a comment)
  • -email is unavailable- added by lostbard (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 5 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2015-03-12 rik5 StatusNone Fixed
        Open/ClosedOpen Closed
    2015-03-12 lostbard Attached File- Added dontdelpkg2.patch, #33321
    2015-03-12 lostbard Attached File- Added dontdelpkg.patch, #33320
    2015-03-12 lostbard Summarypkg uninstall invalidname pkg uninstall invalidname removes global package entries

    Back to the top

    Powered by Savane 3.13-02a9.
    Corresponding source code