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

[Alexander Barth]

On Wed, Apr 16, 2014 at 5:40 AM, Orion Poplawski <address@hidden> wrote:

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?

It would be good to know if this restriction (only lower case letters in package names) is intentional or a bug. But as we do not get an answer, I will change the package name to lower case in future releases to avoid (or work around) this issue.

Thanks for coming back on this,

Alex