bugGNU Octave - Bugs: bug #57735, pkg: package names should be...

 
 

bug #57735: pkg: package names should be handled case insensitively

Submitted by:  Mike Miller <mtmiller>
Submitted on:  Mon 03 Feb 2020 11:21:21 PM UTC  
 
Category:  Octave Function Severity:  1 - Wish
Priority:  1 - Later Item Group:  Feature Request
Status:  None Assigned to:  None
Originator Name:  Open/Closed:  Open
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.

 

Mon 03 Feb 2020 11:22:37 PM UTC, comment #1: 

This does not need to be addressed for Octave 6, and we can discuss and collect comments either here or on the maintainers mailing list.

Mike Miller <mtmiller>
Project Administrator
Mon 03 Feb 2020 11:21:21 PM UTC, original submission:  

The package name tends to be lowercase, but the code in pkg.m and its subfunctions handles the package name inconsistently. Sometimes it needs to match exactly, including case, and sometimes the case does not matter.

In my opinion, case should be ignored in all cases, all comparisons with package names should be case insensitive. This would be consistent with the 'pip' package manager for Python.

Example:

>> pkg install -forge Signal
For information about changes from previous versions of the signal package, run 'news signal'.
>> pkg install -forge SIGNAL
For information about changes from previous versions of the signal package, run 'news signal'.
>> pkg install -forge sIgNaL
For information about changes from previous versions of the signal package, run 'news signal'.
>> pkg load sIgNaL
error: package sIgNaL is not installed

>> pkg uninstall sIgNaL
warning: some of the packages you want to uninstall are not installed

>> pkg load SIGNAL
error: package SIGNAL is not installed

>> pkg unload SIGNAL
error: pkg: package(s): SIGNAL not installed

>> pkg uninstall SIGNAL
warning: some of the packages you want to uninstall are not installed

>> pkg describe Signal
error: some packages are not installed:  Signal


I think all of these should work equivalently regardless of the case the user types or the case that the package maintainer chose to put in the DESCRIPTION file.

Mike Miller <mtmiller>
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

 

CC list is empty

 

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 2 latest changes.

Date Changed by Updated Field Previous Value => Replaced by
2020-02-03 mtmiller Priority5 - Normal => 1 - Later
2020-02-03 mtmiller Carbon-CopyRemoved 80942 => -

Back to the top


Powered by Savane 3.5