April 2013 - Pharo-dev - lists.pharo.org

Re: [Pharo-project] RE : cannot modify class comments on Linux

by Eliot Miranda

Hi Sven, On Fri, Apr 26, 2013 at 4:15 AM, Sven Van Caekenberghe <sven(a)stfx.eu> wrote: > > On 26 Apr 2013, at 12:44, Igor Stasenko <siguctua(a)gmail.com> wrote: > > > On 26 April 2013 10:27, Pavel Krivanek <pavel.krivanek(a)gmail.com> wrote: > >> The reason is in the optimalizations in the latest glibc. It really > does not > >> write the new content to the changes file immediately. We need to call > flush > >> at the end of nextChunkPut: > >> > > This is nasty one. > > What i worry that it affects much more than code working with .changes > file. > > Why ? Isn't that standard behavior/semantics: unless you flush you cannot > be sure anything is actually out of the OS' buffers. And even then, disks > and RAID cards are known to lie and sometimes don't write data out even > when flush is called. > But the issue is whether the read is through the same file pointer as the write or not. If the read is made through the same file pointer (or a dup'ed file pointer) then the data should be in the buffer and the read should see it, and if so, then this is a bug in either glibc (if the FILE * info is not properly synchronised) or (unlikely) the kernel. If it is through a separate file pointer then yes, the problem is a missing flush. Which is it? > In networking (TCP streams) this is similar. > > >> Cheers, > >> -- Pavel > >> > >> > >> On Fri, Apr 26, 2013 at 9:57 AM, GOUBIER Thierry < > thierry.goubier(a)cea.fr> > >> wrote: > >>> > >>> Reproduced on Ubuntu 13.04. > >>> > >>> Class organization remoteString filePositionHi is past the end of the > >>> changes file (i.e. exactly equal to the file size if the comment is > the last > >>> operation). > >>> > >>> Thierry > >>> ________________________________ > >>> De : pharo-project-bounces(a)lists.gforge.inria.fr > >>> [pharo-project-bounces(a)lists.gforge.inria.fr] de la part de Pavel > Krivanek > >>> [pavel.krivanek(a)gmail.com] > >>> Date d'envoi : jeudi 25 avril 2013 21:32 > >>> Ã : Pharo-project(a)lists.gforge.inria.fr > >>> Objet : Re: [Pharo-project] cannot modify class comments on Linux > >>> > >>> Jaroslav wrote this: > >>> This does work on F17, but it doesn't work on F18, F19. Maybe glibc > issue? > >>> I guess the reproducer needs glibc-2.16 and up, IMHO 12.04 has 2.15 > same > >>> as F17, so I guess this would be also reproducible on Ubuntu 13.04. > >>> > >>> -- Pavel > >>> > >>> On Thu, Apr 25, 2013 at 11:16 AM, Gisela Decuzzi < > giseladecuzzi(a)gmail.com> > >>> wrote: > >>>> > >>>> I'm on 64-bit Ubuntu 12.04 > >>>> > >>>> > >>>> 2013/4/25 Pavel Krivanek <pavel.krivanek(a)gmail.com> > >>>>> > >>>>> Hi, > >>>>> > >>>>> what architecture and distro do you use because I tried it on a > 32-bit > >>>>> Debian and it worked. I'm on 64-bit Fedora 18. That may be the > reason why > >>>>> the tests are not failing on CI. This error appears on all old system > >>>>> classes and on newely created classes too. > >>>>> Some next info: https://bugzilla.redhat.com/show_bug.cgi?id=956376 > >>>>> > >>>>> -- Pavel > >>>>> > >>>>> On Thu, Apr 25, 2013 at 10:58 AM, Gisela Decuzzi > >>>>> <giseladecuzzi(a)gmail.com> wrote: > >>>>>> > >>>>>> Hi Pavel, I tried in Pharo 2.0 and 3.0 and it work well for me. > (just > >>>>>> try in a random class comment) > >>>>>> Are you trying to modify a particular class comment? > >>>>>> > >>>>>> > >>>>>> 2013/4/25 Pavel Krivanek <pavel.krivanek(a)gmail.com> > >>>>>>> > >>>>>>> Hi, > >>>>>>> > >>>>>>> if you will try to modify a class comment on Linux, the error > >>>>>>> "RemoteString past end of file" will appear. On windows VM it > seems to work > >>>>>>> well. Can someone confirm this behavior on Linux. Squeak has the > same > >>>>>>> problem. > >>>>>>> > >>>>>>> Cheers, > >>>>>>> -- Pavel > >>>>>> > >>>>>> > >>>>> > >>>> > >>> > >> > > > > > > > > -- > > Best regards, > > Igor Stasenko. > > > > > -- best, Eliot

April 26, 2013

Re: [Pharo-project] RE : cannot modify class comments on Linux

by Nicolas Cellier

Camillo, this is not the best reason, is it? Quand on veut tuer son chien, on l'accuse de la rage... With this kind of argumentation, I hope you won't find a bug in Integer ;) 2013/4/26 Camillo Bruni <camillobruni(a)gmail.com> > On 2013-04-26, at 12:44, Igor Stasenko <siguctua(a)gmail.com> wrote: > > On 26 April 2013 10:27, Pavel Krivanek <pavel.krivanek(a)gmail.com> wrote: > >> The reason is in the optimalizations in the latest glibc. It really > does not > >> write the new content to the changes file immediately. We need to call > flush > >> at the end of nextChunkPut: > > > yet another reason to get rid of .changes and .sources! :) >

April 26, 2013

[Pharo-project] [update 3.0] #30066

by Marcus Denker

30066 ----- 10401 Add Amber BlockClosure asynchronous methods into Pharo https://pharo.fogbugz.com/f/cases/10401 10418 cleanUp #methodHeaderFor: https://pharo.fogbugz.com/f/cases/10418 10419 Remove CompiledMethodWithNode (part 2) https://pharo.fogbugz.com/f/cases/10419 Diff information: http://smalltalkhub.com/#!/~Pharo/Pharo30/diff/Traits-MarcusDenker.475 http://smalltalkhub.com/#!/~Pharo/Pharo30/diff/Tools-MarcusDenker.1083 http://smalltalkhub.com/#!/~Pharo/Pharo30/diff/Kernel-MarcusDenker.1382 http://smalltalkhub.com/#!/~Pharo/Pharo30/diff/Compiler-MarcusDenker.400

April 26, 2013

[Pharo-project] [regression reporter]regression occurred

by no-reply＠ci.inria.fr

https://ci.inria.fr/pharo/job/Pharo-3.0-Update-Step-2-Validation/./label=li… 1 regressions found. Tests.Release.ReleaseTest.testObsoleteBehaviors

April 26, 2013

Re: [Pharo-project] [regression reporter]regression occurred

by Marcus Denker

Yes, staged build killed this updateâ¦ (would have been #30066â¦) On Apr 26, 2013, at 5:24 PM, no-reply(a)ci.inria.fr wrote: > https://ci.inria.fr/pharo/job/Pharo-3.0-Update-Step-2-Validation/./label=li… > > 15 regressions found. > FuelTests.FLGlobalTraitSerializationTest.testClassSideMethodNotFoundAfterRemoval > FuelTests.FLGlobalTraitSerializationTest.testClassSideMethodPreservesIdentity > FuelTests.FLGlobalTraitSerializationTest.testMethodNotFoundAfterRemoval > FuelTests.FLGlobalTraitSerializationTest.testMethodPreservesIdentity > RPackage.SystemIntegration.RPackageMethodsSynchronisationTest.testAddMethodInClassicCategoryAddMethodToTheParentPackageOfItsTrait > RPackage.SystemIntegration.RPackageTraitSynchronisationTest.testTraitCompositionMethodsArePackagedWithTheirTraits > RPackage.SystemIntegration.RPackageTraitSynchronisationTest.testRemoveTraitMethodOverridenByClassDoesRemoveMethodFromClassPackage > RPackage.SystemIntegration.RPackageTraitSynchronisationTest.testRemoveTraitMethod > RPackage.SystemIntegration.RPackageTraitSynchronisationTest.testRemoveMethodComingFromTraitDoesNotRemoveMethodFromTraitPackage > RPackage.SystemIntegration.RPackageTraitSynchronisationTest.testAddMethodByUsingATraitDoesNotAddTheMethodToThePackage > RPackage.Tests.RPackageTraitTest.testStartingSituation > RPackage.Tests.RPackageTraitTest.testPackageOfTraitMethodIsTraitPackage > RPackage.Tests.RPackageTraitTest.testPackageOfClassMethodFromTraitIsTraitPackage > RPackage.Tests.RPackageTraitTest.testPackageOfClassMethodIsClassPackage > RPackage.Tests.RPackageTraitTest.testPackageOfClassMethodFromTraitExtensionIsExtendingPackage >

April 26, 2013

[Pharo-project] [regression reporter]regression occurred

by no-reply＠ci.inria.fr

https://ci.inria.fr/pharo/job/Pharo-3.0-Update-Step-2-Validation/./label=wi… 15 regressions found. FuelTests.FLGlobalTraitSerializationTest.testClassSideMethodNotFoundAfterRemoval FuelTests.FLGlobalTraitSerializationTest.testClassSideMethodPreservesIdentity FuelTests.FLGlobalTraitSerializationTest.testMethodNotFoundAfterRemoval FuelTests.FLGlobalTraitSerializationTest.testMethodPreservesIdentity RPackage.SystemIntegration.RPackageMethodsSynchronisationTest.testAddMethodInClassicCategoryAddMethodToTheParentPackageOfItsTrait RPackage.SystemIntegration.RPackageTraitSynchronisationTest.testTraitCompositionMethodsArePackagedWithTheirTraits RPackage.SystemIntegration.RPackageTraitSynchronisationTest.testRemoveTraitMethodOverridenByClassDoesRemoveMethodFromClassPackage RPackage.SystemIntegration.RPackageTraitSynchronisationTest.testRemoveTraitMethod RPackage.SystemIntegration.RPackageTraitSynchronisationTest.testRemoveMethodComingFromTraitDoesNotRemoveMethodFromTraitPackage RPackage.SystemIntegration.RPackageTraitSynchronisationTest.testAddMethodByUsingATraitDoesNotAddTheMethodToThePackage RPackage.Tests.RPackageTraitTest.testStartingSituation RPackage.Tests.RPackageTraitTest.testPackageOfTraitMethodIsTraitPackage RPackage.Tests.RPackageTraitTest.testPackageOfClassMethodFromTraitIsTraitPackage RPackage.Tests.RPackageTraitTest.testPackageOfClassMethodIsClassPackage RPackage.Tests.RPackageTraitTest.testPackageOfClassMethodFromTraitExtensionIsExtendingPackage

April 26, 2013

[Pharo-project] [regression reporter]regression occurred

by no-reply＠ci.inria.fr

https://ci.inria.fr/pharo/job/Pharo-3.0-Update-Step-2-Validation/./label=ma… 15 regressions found. FuelTests.FLGlobalTraitSerializationTest.testClassSideMethodNotFoundAfterRemoval FuelTests.FLGlobalTraitSerializationTest.testClassSideMethodPreservesIdentity FuelTests.FLGlobalTraitSerializationTest.testMethodNotFoundAfterRemoval FuelTests.FLGlobalTraitSerializationTest.testMethodPreservesIdentity RPackage.SystemIntegration.RPackageMethodsSynchronisationTest.testAddMethodInClassicCategoryAddMethodToTheParentPackageOfItsTrait RPackage.SystemIntegration.RPackageTraitSynchronisationTest.testTraitCompositionMethodsArePackagedWithTheirTraits RPackage.SystemIntegration.RPackageTraitSynchronisationTest.testRemoveTraitMethodOverridenByClassDoesRemoveMethodFromClassPackage RPackage.SystemIntegration.RPackageTraitSynchronisationTest.testRemoveTraitMethod RPackage.SystemIntegration.RPackageTraitSynchronisationTest.testRemoveMethodComingFromTraitDoesNotRemoveMethodFromTraitPackage RPackage.SystemIntegration.RPackageTraitSynchronisationTest.testAddMethodByUsingATraitDoesNotAddTheMethodToThePackage RPackage.Tests.RPackageTraitTest.testStartingSituation RPackage.Tests.RPackageTraitTest.testPackageOfTraitMethodIsTraitPackage RPackage.Tests.RPackageTraitTest.testPackageOfClassMethodFromTraitIsTraitPackage RPackage.Tests.RPackageTraitTest.testPackageOfClassMethodIsClassPackage RPackage.Tests.RPackageTraitTest.testPackageOfClassMethodFromTraitExtensionIsExtendingPackage

April 26, 2013

[Pharo-project] [regression reporter]regression occurred

by no-reply＠ci.inria.fr

https://ci.inria.fr/pharo/job/Pharo-3.0-Update-Step-2-Validation/./label=li… 15 regressions found. FuelTests.FLGlobalTraitSerializationTest.testClassSideMethodNotFoundAfterRemoval FuelTests.FLGlobalTraitSerializationTest.testClassSideMethodPreservesIdentity FuelTests.FLGlobalTraitSerializationTest.testMethodNotFoundAfterRemoval FuelTests.FLGlobalTraitSerializationTest.testMethodPreservesIdentity RPackage.SystemIntegration.RPackageMethodsSynchronisationTest.testAddMethodInClassicCategoryAddMethodToTheParentPackageOfItsTrait RPackage.SystemIntegration.RPackageTraitSynchronisationTest.testTraitCompositionMethodsArePackagedWithTheirTraits RPackage.SystemIntegration.RPackageTraitSynchronisationTest.testRemoveTraitMethodOverridenByClassDoesRemoveMethodFromClassPackage RPackage.SystemIntegration.RPackageTraitSynchronisationTest.testRemoveTraitMethod RPackage.SystemIntegration.RPackageTraitSynchronisationTest.testRemoveMethodComingFromTraitDoesNotRemoveMethodFromTraitPackage RPackage.SystemIntegration.RPackageTraitSynchronisationTest.testAddMethodByUsingATraitDoesNotAddTheMethodToThePackage RPackage.Tests.RPackageTraitTest.testStartingSituation RPackage.Tests.RPackageTraitTest.testPackageOfTraitMethodIsTraitPackage RPackage.Tests.RPackageTraitTest.testPackageOfClassMethodFromTraitIsTraitPackage RPackage.Tests.RPackageTraitTest.testPackageOfClassMethodIsClassPackage RPackage.Tests.RPackageTraitTest.testPackageOfClassMethodFromTraitExtensionIsExtendingPackage

April 26, 2013

[Pharo-project] [regression reporter]regression occurred

by no-reply＠ci.inria.fr

https://ci.inria.fr/pharo/job/Pharo-3.0-Update-Step-2-Validation/./label=wi… 1 regressions found. Tests.Release.ReleaseTest.testObsoleteBehaviors

April 26, 2013

Re: [Pharo-project] Smart Suggestion Version 1

by Gisela Decuzzi

2013/4/24 stephane ducasse <stephane.ducasse(a)free.fr> > What would be cool is to have a way to select the enclosing expression but > you know it:) Now I'm selecting the node from the expression where you asked for suggestions... I'm looking to for underline the scope but it's an advance. Also if you want to try it's working in Rubric text editor example (but only throw the menu).

April 26, 2013