Re: upper case latter in package name [was Re: ncarray 1.0.2 released]

[Orion Poplawski]

On 02/25/2014 11:07 AM, Orion Poplawski wrote:
> On 02/20/2014 06:35 AM, Alexander Barth wrote:
>>
>> Am 19.02.2014 17:31 schrieb "Orion Poplawski" <address@hidden
>> <mailto:address@hidden>>:
>>  >
>>  > On 02/19/2014 09:23 AM, Alexander Barth wrote:
>>  > > Hi all,
>>  > >
>>  > > It seems that pkg cannot install a package with an upper case
>> letter in the
>>  > > filename. Is this a bug in "pkg" or intended?
>>  > >
>>  > >>> pkg install dummyPackage-1.0.0.tar.gz
>>  > > package name 'dummypackage' doesn't correspond to its filename
>>  > > 'dummyPackage-1.0.0.tar'
>>  > > error: called from 'install' in file
>>  > >
>> /home/abarth/opt/octave-3.8.0/share/octave/3.8.0/m/pkg/private/install.m
>> near
>>  > > line 119, column 11
>>  > >
>>  > > Thanks and regards
>>  > > Alexander
>>  >
>>  > Well, in pkg.m parsing the DESCRIPTION file:
>>  >
>>  > desc.name <http://desc.name> = tolower (desc.name <http://desc.name>);
>>  >
>>  > And in all dep handling:
>>  >
>>  >   deps = split_by (tolower (depends), ",");
>>  >
>>  > So as written it seems like all octave packages really should be
>> all lower
>>  > case.  Or we need another tolower() in the install check.
>>
>> Does somebody knows the reason for not supporting uppercase letters in
>> the
>> package filename? pkg could just take the name as it is without
>> convertion to
>> lowercase.
>>
>> Thanks and regards,
>> Alexander
> 
> Filed https://savannah.gnu.org/bugs/index.php?41691 to see if we can get
> some more traction on this issue.

Hmm, not much more traction it seems :(.  I'm happy to create a patch,
but I have no idea what the desired behavior around mixed case packages
is.  But this is currently blocking me from packaging ncarray in Fedora.

However:

octave:4> pkg install -forge ncArray
package name 'ncarray' doesn't correspond to its filename
'ncArray-1.0.2.tar'
error: called from 'install' in file
/usr/share/octave/3.8.1/m/pkg/private/install.m near line 121, column 11
error: called from:
error:   /usr/share/octave/3.8.1/m/pkg/private/install.m at line 148,
column 5
error:   /usr/share/octave/3.8.1/m/pkg/pkg.m at line 394, column 9
octave:4> pkg install -forge ncarray
For information about changes from previous versions of the ncarray
package, run 'news ncarray'.
octave:5> pkg list
Package Name  | Version | Installation directory
--------------+---------+-----------------------
          io  |   2.0.2 | /home/orion/octave/io-2.0.2
     ncarray  |   1.0.2 | /home/orion/octave/ncarray-1.0.2
      netcdf  |   1.0.2 | /usr/share/octave/packages/netcdf-1.0.2
  statistics  |   1.2.3 | /home/orion/octave/statistics-1.2.3


So maybe we should just declare that all packages need to be all lower
case names?

- Orion

-- 
Orion Poplawski
Technical Manager                     303-415-9701 x222
NWRA/CoRA Division                    FAX: 303-415-9702
3380 Mitchell Lane                  address@hidden
Boulder, CO 80301              http://www.cora.nwra.com