On 10 November 2017 at 22:30, Alistair Grant <[email protected]> wrote:
> On 9 November 2017 at 22:12, Norbert Hartl <[email protected]> wrote:
>> is there a good explanation why
>>
>> ‚foo‘ asFileReference parent basename
>>
>> gives ‚/‘ ?
>
> I'd like to think about this a bit more, but at first glance the
> problem seems to be that Path>>basename was written only taking
> Absolute paths in to consideration.
>
> So RelativePath>>basename needs to be added:
>
> basename
>     "Returns the base of the basename,
>     i.e.
>     foo/gloops.taz basename is 'gloops.taz'
>     . basename is '.'"
>     self size == 0
>         "the current directory"
>         ifTrue: [ ^ '.'].
>     ^ self at: self size

Fogbugz 20693: 
https://pharo.fogbugz.com/f/cases/20693/Incorrect-basename-of-empty-relative-path

I've got the code changes done, but am having troubles with Iceberg
(which I'll post about if I can't get it working).

Thanks,
Alistair

Reply via email to