FileSystem fix integration

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
24 messages Options
12
Reply | Threaded
Open this post in threaded view
|

Re: FileSystem fix integration

alistairgrant
Hi Stef,

On Wed, Aug 16, 2017 at 12:16:52PM +0200, Stephane Ducasse wrote:
> I'm integrating 20307 and I would like to really thank you for
> - the bug entry description
> - all the comments
> - and tests in the PR.
>
> Tx!!!

Thanks for your kind words.

And for integrating the PR :-)


I've re-submitted 20165 Support segment path printing.

So the open list is:

- https://pharo.fogbugz.com/f/cases/20165/Support-segment-path-printing
  - https://github.com/pharo-project/pharo/pull/210
    Note that this is a different PR from before, see the comments in
    the issue.
- 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


Still to be closed (won't fix):

- https://pharo.fogbugz.com/f/cases/18042
  - https://github.com/pharo-project/pharo/pull/75


And waiting on FFI / kernel resolution:

- https://pharo.fogbugz.com/f/cases/5723/
  - https://github.com/pharo-project/pharo/pull/92


Cheers,
Alistair


Reply | Threaded
Open this post in threaded view
|

Re: FileSystem fix integration

Stephane Ducasse-3
May be you already see it but
- https://pharo.fogbugz.com/f/cases/19609/FileReference-base-should-be-before-last-separator
    https://github.com/pharo-project/pharo/pull/137
has a merge conflict

I will integrate now

- https://pharo.fogbugz.com/f/cases/18084/FileReference-EnsureCreateFile
    https://github.com/pharo-project/pharo/pull/133

and after when esteban is back we will integrate

https://pharo.fogbugz.com/f/cases/20294/Add-FileAttributesPlugin-to-the-linux-32-bit-VM

On Wed, Aug 16, 2017 at 3:42 PM, Alistair Grant <[hidden email]> wrote:

> Hi Stef,
>
> On Wed, Aug 16, 2017 at 12:16:52PM +0200, Stephane Ducasse wrote:
>> I'm integrating 20307 and I would like to really thank you for
>> - the bug entry description
>> - all the comments
>> - and tests in the PR.
>>
>> Tx!!!
>
> Thanks for your kind words.
>
> And for integrating the PR :-)
>
>
> I've re-submitted 20165 Support segment path printing.
>
> So the open list is:
>
> - https://pharo.fogbugz.com/f/cases/20165/Support-segment-path-printing
>   - https://github.com/pharo-project/pharo/pull/210
>     Note that this is a different PR from before, see the comments in
>     the issue.
> - 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
>
>
> Still to be closed (won't fix):
>
> - https://pharo.fogbugz.com/f/cases/18042
>   - https://github.com/pharo-project/pharo/pull/75
>
>
> And waiting on FFI / kernel resolution:
>
> - https://pharo.fogbugz.com/f/cases/5723/
>   - https://github.com/pharo-project/pharo/pull/92
>
>
> Cheers,
> Alistair
>
>

Reply | Threaded
Open this post in threaded view
|

Re: FileSystem fix integration

alistairgrant
Hi Stef,

On Wed, Aug 16, 2017 at 08:43:24PM +0200, Stephane Ducasse wrote:
> May be you already see it but
> - https://pharo.fogbugz.com/f/cases/19609/FileReference-base-should-be-before-last-separator
>     https://github.com/pharo-project/pharo/pull/137
> has a merge conflict

I haven't looked at this at all.  I'll try and take a look it at soon
(my daughter's birthday party this weekend, so time is limited :-)).


> I will integrate now
>
> - https://pharo.fogbugz.com/f/cases/18084/FileReference-EnsureCreateFile
>     https://github.com/pharo-project/pharo/pull/133
>
> and after when esteban is back we will integrate
>
> https://pharo.fogbugz.com/f/cases/20294/Add-FileAttributesPlugin-to-the-linux-32-bit-VM

I'm looking forward to this one :-)

Cheers,
Alistair


> On Wed, Aug 16, 2017 at 3:42 PM, Alistair Grant <[hidden email]> wrote:
> > Hi Stef,
> >
> > On Wed, Aug 16, 2017 at 12:16:52PM +0200, Stephane Ducasse wrote:
> >> I'm integrating 20307 and I would like to really thank you for
> >> - the bug entry description
> >> - all the comments
> >> - and tests in the PR.
> >>
> >> Tx!!!
> >
> > Thanks for your kind words.
> >
> > And for integrating the PR :-)
> >
> >
> > I've re-submitted 20165 Support segment path printing.
> >
> > So the open list is:
> >
> > - https://pharo.fogbugz.com/f/cases/20165/Support-segment-path-printing
> >   - https://github.com/pharo-project/pharo/pull/210
> >     Note that this is a different PR from before, see the comments in
> >     the issue.
> > - 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
> >
> >
> > Still to be closed (won't fix):
> >
> > - https://pharo.fogbugz.com/f/cases/18042
> >   - https://github.com/pharo-project/pharo/pull/75
> >
> >
> > And waiting on FFI / kernel resolution:
> >
> > - https://pharo.fogbugz.com/f/cases/5723/
> >   - https://github.com/pharo-project/pharo/pull/92
> >
> >
> > Cheers,
> > Alistair
> >
> >
>

Reply | Threaded
Open this post in threaded view
|

Re: FileSystem fix integration

Guillermo Polito
FileSystem needs love, Thanks!

On Wed, Aug 16, 2017 at 9:22 PM, Alistair Grant <[hidden email]> wrote:
Hi Stef,

On Wed, Aug 16, 2017 at 08:43:24PM +0200, Stephane Ducasse wrote:
> May be you already see it but
> - https://pharo.fogbugz.com/f/cases/19609/FileReference-base-should-be-before-last-separator
>     https://github.com/pharo-project/pharo/pull/137
> has a merge conflict

I haven't looked at this at all.  I'll try and take a look it at soon
(my daughter's birthday party this weekend, so time is limited :-)).


> I will integrate now
>
> - https://pharo.fogbugz.com/f/cases/18084/FileReference-EnsureCreateFile
>     https://github.com/pharo-project/pharo/pull/133
>
> and after when esteban is back we will integrate
>
> https://pharo.fogbugz.com/f/cases/20294/Add-FileAttributesPlugin-to-the-linux-32-bit-VM

I'm looking forward to this one :-)

Cheers,
Alistair


> On Wed, Aug 16, 2017 at 3:42 PM, Alistair Grant <[hidden email]> wrote:
> > Hi Stef,
> >
> > On Wed, Aug 16, 2017 at 12:16:52PM +0200, Stephane Ducasse wrote:
> >> I'm integrating 20307 and I would like to really thank you for
> >> - the bug entry description
> >> - all the comments
> >> - and tests in the PR.
> >>
> >> Tx!!!
> >
> > Thanks for your kind words.
> >
> > And for integrating the PR :-)
> >
> >
> > I've re-submitted 20165 Support segment path printing.
> >
> > So the open list is:
> >
> > - https://pharo.fogbugz.com/f/cases/20165/Support-segment-path-printing
> >   - https://github.com/pharo-project/pharo/pull/210
> >     Note that this is a different PR from before, see the comments in
> >     the issue.
> > - 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
> >
> >
> > Still to be closed (won't fix):
> >
> > - https://pharo.fogbugz.com/f/cases/18042
> >   - https://github.com/pharo-project/pharo/pull/75
> >
> >
> > And waiting on FFI / kernel resolution:
> >
> > - https://pharo.fogbugz.com/f/cases/5723/
> >   - https://github.com/pharo-project/pharo/pull/92
> >
> >
> > Cheers,
> > Alistair
> >
> >
>




--

   

Guille Polito


Research Engineer

French National Center for Scientific Research - http://www.cnrs.fr



Web: http://guillep.github.io

Phone: +33 06 52 70 66 13

12