[Pharo-project] issue 5048 broke Pharo Kernel
Hi, the issue http://code.google.com/p/pharo/issues/detail?id=5048 broke Pharo-Kernel build on CI server. -- Pavel
how can we fix it? Because we believe that Transcript should not be packaged in Stream. Now may be in its own package. Stef On Dec 6, 2011, at 3:40 PM, Pavel Krivanek wrote:
Hi,
the issue http://code.google.com/p/pharo/issues/detail?id=5048 broke Pharo-Kernel build on CI server.
-- Pavel
We need it in the Kernel because some classes log system messages there (like Compiler during recompilation of the image). I think that own separate package would be fine. -- Pavel On Tue, Dec 6, 2011 at 5:36 PM, Stéphane Ducasse <stephane.ducasse@inria.fr> wrote:
how can we fix it? Because we believe that Transcript should not be packaged in Stream. Now may be in its own package.
Stef
On Dec 6, 2011, at 3:40 PM, Pavel Krivanek wrote:
Hi,
the issue http://code.google.com/p/pharo/issues/detail?id=5048 broke Pharo-Kernel build on CI server.
-- Pavel
Ok I will do that. Note that we started to remove systematically hard coded references to transcript so that we can even remove it. We have std output now. Stef On Dec 6, 2011, at 6:50 PM, Pavel Krivanek wrote:
We need it in the Kernel because some classes log system messages there (like Compiler during recompilation of the image). I think that own separate package would be fine.
-- Pavel
On Tue, Dec 6, 2011 at 5:36 PM, Stéphane Ducasse <stephane.ducasse@inria.fr> wrote:
how can we fix it? Because we believe that Transcript should not be packaged in Stream. Now may be in its own package.
Stef
On Dec 6, 2011, at 3:40 PM, Pavel Krivanek wrote:
Hi,
the issue http://code.google.com/p/pharo/issues/detail?id=5048 broke Pharo-Kernel build on CI server.
-- Pavel
Great. Please add the new package name to kernelPackageNames too. -- Pavel On Tue, Dec 6, 2011 at 10:40 PM, Stéphane Ducasse <stephane.ducasse@inria.fr> wrote:
Ok I will do that. Note that we started to remove systematically hard coded references to transcript so that we can even remove it. We have std output now.
Stef
On Dec 6, 2011, at 6:50 PM, Pavel Krivanek wrote:
We need it in the Kernel because some classes log system messages there (like Compiler during recompilation of the image). I think that own separate package would be fine.
-- Pavel
On Tue, Dec 6, 2011 at 5:36 PM, Stéphane Ducasse <stephane.ducasse@inria.fr> wrote:
how can we fix it? Because we believe that Transcript should not be packaged in Stream. Now may be in its own package.
Stef
On Dec 6, 2011, at 3:40 PM, Pavel Krivanek wrote:
Hi,
the issue http://code.google.com/p/pharo/issues/detail?id=5048 broke Pharo-Kernel build on CI server.
-- Pavel
participants (2)
-
Pavel Krivanek -
Stéphane Ducasse