Hi Guille, On 15 November 2017 at 10:39, Guillermo Polito <guillermopolito@gmail.com> wrote:
Made a small review ;)
Good idea. I've updated the PR. Thanks, Alistair
On Wed, Nov 15, 2017 at 10:34 AM, Alistair Grant <akgrant0710@gmail.com> wrote:
On 14 November 2017 at 22:36, Alistair Grant <akgrant0710@gmail.com> wrote:
On 10 November 2017 at 22:30, Alistair Grant <akgrant0710@gmail.com> wrote:
On 9 November 2017 at 22:12, Norbert Hartl <norbert@hartl.name> 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...
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).
PR #481 submitted (the Iceberg problems were pre-keyboard :-))
Thanks, Alistair
--
Guille Polito
Research Engineer
Centre de Recherche en Informatique, Signal et Automatique de Lille
CRIStAL - UMR 9189
French National Center for Scientific Research - http://www.cnrs.fr
Phone: +33 06 52 70 66 13