[Pharo-dev] FileSystem fix integration

Alistair Grant akgrant0710 at gmail.com
Tue Aug 15 05:07:54 EDT 2017


Hi Stef,

On Tue, Aug 15, 2017 at 10:04:31AM +0200, Stephane Ducasse wrote:
> tx a lot for the summary, this is super useful :)
> I will over them later (because I must work on my new lectures :).
> 
> Stef

I've just realised that:

- https://pharo.fogbugz.com/f/cases/20165/Support-segment-path-printing

has merge conflicts, and that #20307 adds more merge conflicts.

The easiest thing to do is to defer this issue until after #20307 has
been merged.  I'll then update the patch to remove the merge conflicts
and resubmit.

Cheers,
Alistair




> On Tue, Aug 15, 2017 at 7:58 AM, Alistair Grant <akgrant0710 at gmail.com> wrote:
> > Hi Stef,
> >
> > I've added the PR to fix automatic canonicalisation of path references
> > to the parent directory (".."):
> >
> > - https://pharo.fogbugz.com/f/cases/20307/File-names-should-not-be-canonicalised-by-default
> >   - https://github.com/pharo-project/pharo/pull/206
> >
> > Other open issues (to help keep track):
> >
> > - https://pharo.fogbugz.com/f/cases/20165/Support-segment-path-printing
> > - https://pharo.fogbugz.com/f/cases/18084/FileReference-EnsureCreateFile
> >     https://github.com/pharo-project/pharo/pull/133
> > - https://pharo.fogbugz.com/f/cases/19609/FileReference-base-should-be-before-last-separator
> >     https://github.com/pharo-project/pharo/pull/137
> > - https://pharo.fogbugz.com/f/cases/20294/Add-FileAttributesPlugin-to-the-linux-32-bit-VM
> >   This is the first step in fixing FileReference>>isSymlink and
> >   extending support for all file attributes returned by stat() and
> >   lstat().
> >   The parent issue is:
> >   https://pharo.fogbugz.com/f/cases/18279/isSymlink-seems-to-be-broken-on-Linux
> >
> >
> > Since #13217 has been accepted, #18042 and its associated PR should be
> > closed (won't fix):
> >
> > - https://pharo.fogbugz.com/f/cases/18042
> >   - https://github.com/pharo-project/pharo/pull/75
> >
> >
> > Thanks!
> > Alistair
> >
> >
> >
> > On Thu, Aug 10, 2017 at 08:23:49PM +0000, Alistair Grant wrote:
> >> Hi Stef,
> >>
> >> On Wed, Aug 09, 2017 at 10:12:17PM +0200, Stephane Ducasse wrote:
> >> > Hi alistair
> >> >
> >> > I went on fogbugz to find the fix to be included and I'm not sure not
> >> > that find them.
> >> > I looked for FileSystem.
> >> >
> >> > I found
> >> >      - https://pharo.fogbugz.com/f/cases/20165/Support-segment-path-printing
> >> >
> >> >      - https://pharo.fogbugz.com/f/cases/18084/FileReference-EnsureCreateFile
> >> >            https://github.com/pharo-project/pharo/pull/133
> >> >
> >> >      - https://pharo.fogbugz.com/f/cases/19609/FileReference-base-should-be-before-last-separator
> >> >            https://github.com/pharo-project/pharo/pull/137
> >> >
> >> > Did I miss some others?
> >> >
> >> > Stef
> >>
> >> - https://pharo.fogbugz.com/f/cases/20294/Add-FileAttributesPlugin-to-the-linux-32-bit-VM
> >>   This is the first step in fixing FileReference>>isSymlink and
> >>   extending support for all file attributes returned by stat() and
> >>   lstat().
> >>   The parent issue is:
> >>   https://pharo.fogbugz.com/f/cases/18279/isSymlink-seems-to-be-broken-on-Linux
> >>
> >> Just for completeness, the two issues relating to FileReference>>/:
> >>
> >> - https://pharo.fogbugz.com/f/cases/13217/FS-basename-with-compound-path-string
> >>   - https://github.com/pharo-project/pharo/pull/192
> >> - https://pharo.fogbugz.com/f/cases/18042
> >>   - https://github.com/pharo-project/pharo/pull/75
> >>
> >>
> >> Cheers,
> >> Alistair
> >
> 



More information about the Pharo-dev mailing list