Issue 3912 in pharo: Circular dependency with GLMOrangeUITheme

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

Issue 3912 in pharo: Circular dependency with GLMOrangeUITheme

pharo
Status: Accepted
Owner: ----
Labels: Type-ReportDefect Milestone-1.2

New issue 3912 by renggli: Circular dependency with GLMOrangeUITheme
http://code.google.com/p/pharo/issues/detail?id=3912

The issue is the class GLMOrangeUITheme being in the 'Morphic' package and  
an indirect subclass of UITheme that is in the 'Polymorph' package which  
itself depends on the 'Morphic' package ==> circular dependency ==>  
loading/unloading impossible.

Igor mentions that a fix was proposed and integrated for Pharo 1.3.


Reply | Threaded
Open this post in threaded view
|

Re: Issue 3912 in pharo: Circular dependency with GLMOrangeUITheme

pharo
Updates:
        Labels: -Milestone-1.2 Milestone-1.2.2 Milestone-1.3

Comment #1 on issue 3912 by [hidden email]: Circular dependency with  
GLMOrangeUITheme
http://code.google.com/p/pharo/issues/detail?id=3912

(No comment was entered for this change.)


Reply | Threaded
Open this post in threaded view
|

Re: Issue 3912 in pharo: Circular dependency with GLMOrangeUITheme

pharo
Updates:
        Labels: -Milestone-1.2.2

Comment #2 on issue 3912 by [hidden email]: Circular dependency with  
GLMOrangeUITheme
http://code.google.com/p/pharo/issues/detail?id=3912

This is the same in 1.3

I propose, as this is a purely cosmetic problem, to postpone this to 1.3  
and keep 1.2 as it is.


Reply | Threaded
Open this post in threaded view
|

Re: Issue 3912 in pharo: Circular dependency with GLMOrangeUITheme

pharo
Updates:
        Status: Closed

Comment #3 on issue 3912 by [hidden email]: Circular dependency with  
GLMOrangeUITheme
http://code.google.com/p/pharo/issues/detail?id=3912

So far I move all the themes into Polymorph. Removed SoftSqueak, started to  
rename Glamour (should rename the classes). So I will close this issue  
since the circular dependency should be over now.
In 13131