30154 ----- 10715 Spec update https://pharo.fogbugz.com/f/cases/10715 10701 Remove non-boolean setting https://pharo.fogbugz.com/f/cases/10701 10678 Failing Regressiontest: FLSortedCollectionSerializationTest https://pharo.fogbugz.com/f/cases/10678 10716 update shrinking methods https://pharo.fogbugz.com/f/cases/10716 Diff information: http://smalltalkhub.com/mc/Pharo/Pharo30/main/System-Support-MarcusDenker.84... http://smalltalkhub.com/mc/Pharo/Pharo30/main/Spec-Widgets-MarcusDenker.182.... http://smalltalkhub.com/mc/Pharo/Pharo30/main/Spec-Core-MarcusDenker.127.dif... http://smalltalkhub.com/mc/Pharo/Pharo30/main/Settings-Compiler-MarcusDenker... http://smalltalkhub.com/mc/Pharo/Pharo30/main/NautilusSpec-MarcusDenker.1.di... http://smalltalkhub.com/mc/Pharo/Pharo30/main/Morphic-MarcusDenker.1.diff http://smalltalkhub.com/mc/Pharo/Pharo30/main/Kernel-MarcusDenker.1460.diff http://smalltalkhub.com/mc/Pharo/Pharo30/main/FuelTests-MarcusDenker.316.dif...
Marcus, there are some problems with renamed Morphic package. In the Monticello packages list I see Morphic (MarcusDenker.1). In the class PluggableMenuSpec there is a category named *Morphic-Menus, that now should be *Morphic-Base-Menus. So I guess this mess has origin in the Spec update integration. I think we should revert whole this update. -- Pavel On Wed, May 22, 2013 at 8:30 AM, Marcus Denker <marcus.denker@inria.fr>wrote:
30154 -----
10715 Spec update https://pharo.fogbugz.com/f/cases/10715
10701 Remove non-boolean setting https://pharo.fogbugz.com/f/cases/10701
10678 Failing Regressiontest: FLSortedCollectionSerializationTest https://pharo.fogbugz.com/f/cases/10678
10716 update shrinking methods https://pharo.fogbugz.com/f/cases/10716
Diff information:
http://smalltalkhub.com/mc/Pharo/Pharo30/main/System-Support-MarcusDenker.84...
http://smalltalkhub.com/mc/Pharo/Pharo30/main/Spec-Widgets-MarcusDenker.182....
http://smalltalkhub.com/mc/Pharo/Pharo30/main/Spec-Core-MarcusDenker.127.dif...
http://smalltalkhub.com/mc/Pharo/Pharo30/main/Settings-Compiler-MarcusDenker...
http://smalltalkhub.com/mc/Pharo/Pharo30/main/NautilusSpec-MarcusDenker.1.di... http://smalltalkhub.com/mc/Pharo/Pharo30/main/Morphic-MarcusDenker.1.diff http://smalltalkhub.com/mc/Pharo/Pharo30/main/Kernel-MarcusDenker.1460.diff
http://smalltalkhub.com/mc/Pharo/Pharo30/main/FuelTests-MarcusDenker.316.dif...
On May 22, 2013, at 10:00 AM, Pavel Krivanek <pavel.krivanek@gmail.com> wrote:
Marcus, there are some problems with renamed Morphic package. In the Monticello packages list I see Morphic (MarcusDenker.1). In the class PluggableMenuSpec there is a category named *Morphic-Menus, that now should be *Morphic-Base-Menus. So I guess this mess has origin in the Spec update integration.
I think we should revert whole this update.
So does that mean as soon as we rename a package we just can never merge in anything anymore? Maybe then we should have spend some time to finish reviewing all the pending issues before doing the rename? This will now cost hours and hours of workâ¦.
-- Pavel
On Wed, May 22, 2013 at 8:30 AM, Marcus Denker <marcus.denker@inria.fr> wrote: 30154 -----
10715 Spec update https://pharo.fogbugz.com/f/cases/10715
10701 Remove non-boolean setting https://pharo.fogbugz.com/f/cases/10701
10678 Failing Regressiontest: FLSortedCollectionSerializationTest https://pharo.fogbugz.com/f/cases/10678
10716 update shrinking methods https://pharo.fogbugz.com/f/cases/10716
Diff information: http://smalltalkhub.com/mc/Pharo/Pharo30/main/System-Support-MarcusDenker.84... http://smalltalkhub.com/mc/Pharo/Pharo30/main/Spec-Widgets-MarcusDenker.182.... http://smalltalkhub.com/mc/Pharo/Pharo30/main/Spec-Core-MarcusDenker.127.dif... http://smalltalkhub.com/mc/Pharo/Pharo30/main/Settings-Compiler-MarcusDenker... http://smalltalkhub.com/mc/Pharo/Pharo30/main/NautilusSpec-MarcusDenker.1.di... http://smalltalkhub.com/mc/Pharo/Pharo30/main/Morphic-MarcusDenker.1.diff http://smalltalkhub.com/mc/Pharo/Pharo30/main/Kernel-MarcusDenker.1460.diff http://smalltalkhub.com/mc/Pharo/Pharo30/main/FuelTests-MarcusDenker.316.dif...
On May 22, 2013, at 10:00 AM, Pavel Krivanek <pavel.krivanek@gmail.com> wrote:
Marcus, there are some problems with renamed Morphic package. In the Monticello packages list I see Morphic (MarcusDenker.1). In the class PluggableMenuSpec there is a category named *Morphic-Menus, that now should be *Morphic-Base-Menus. So I guess this mess has origin in the Spec update integration.
I think we should revert whole this update.
So I retracted it. Now just *how* do I know that a SLICE is bad? It did not lead to a conflict, there is no way to realize that there is a problem. Marcus
On Wed, May 22, 2013 at 10:24 AM, Marcus Denker <marcus.denker@inria.fr>wrote:
On May 22, 2013, at 10:00 AM, Pavel Krivanek <pavel.krivanek@gmail.com> wrote:
Marcus, there are some problems with renamed Morphic package. In the Monticello packages list I see Morphic (MarcusDenker.1). In the class PluggableMenuSpec there is a category named *Morphic-Menus, that now should be *Morphic-Base-Menus. So I guess this mess has origin in the Spec update integration.
I think we should revert whole this update.
So I retracted it.
Now just *how* do I know that a SLICE is bad? It did not lead to a conflict, there is no way to realize that there is a problem.
It is very strange that we had such problems with this update because all of the slices do not modify Morphic package. I think that the problem is in the act of renaming where extension of the classes that are part of the renamed packages were not renamed. So maybe we will not have such problems with later merges (but I'm not sure) -- Pavel
Marcus
https://pharo.fogbugz.com/f/cases/10720 On Wed, May 22, 2013 at 12:06 PM, Pavel Krivanek <pavel.krivanek@gmail.com>wrote:
On Wed, May 22, 2013 at 10:24 AM, Marcus Denker <marcus.denker@inria.fr>wrote:
On May 22, 2013, at 10:00 AM, Pavel Krivanek <pavel.krivanek@gmail.com> wrote:
Marcus, there are some problems with renamed Morphic package. In the Monticello packages list I see Morphic (MarcusDenker.1). In the class PluggableMenuSpec there is a category named *Morphic-Menus, that now should be *Morphic-Base-Menus. So I guess this mess has origin in the Spec update integration.
I think we should revert whole this update.
So I retracted it.
Now just *how* do I know that a SLICE is bad? It did not lead to a conflict, there is no way to realize that there is a problem.
It is very strange that we had such problems with this update because all of the slices do not modify Morphic package. I think that the problem is in the act of renaming where extension of the classes that are part of the renamed packages were not renamed. So maybe we will not have such problems with later merges (but I'm not sure)
-- Pavel
Marcus
participants (2)
-
Marcus Denker -
Pavel Krivanek