[Pkg-ace-devel] Re: [ace-users] Bug #1854 RESOLVED/FIXED: Reactor separation

[Marek Brudka]

U=BFytkownik Raphael Bossek napisa=B3:

>Hi,
>
>I've finished the reactor separation effort reported some month ago.
>All related informations are saved in DOC Bugzilla:
>
>http://deuce.doc.wustl.edu/bugzilla/show_bug.cgi?id=3D1854
>
>You can also find the corresponding patch against CVS HEAD here
>(applied to bug #1854):
>
>http://deuce.doc.wustl.edu/bugzilla/showattachment.cgi?attach_id=3D290
>
>Any comments are welcome.
>
>For all Debian GNU/Linux users: This patch will find its way to
>5.4.2.1-2 of the ACE+TAO+CIAO Debian GNU/Linux packages.
>
>--
>Raphael Bossek
> =20
>
Hi Raphael,
    I consider the patches you sent as very reasonable, not only from=20
Debian point of view but
also other operating systems. However I cannot find any changes you made=20
in cvs.doc.wustl.edu
repository. Why?
    We have already several cases in our company of dependencies on qt=20
in application using ace only,
just because we had a ACE distribution prepared for qt. This was only=20
not only ugly,
but also could lead to some additional costs ($3000 per programmer)=20
related with qt licence.
We solved the problem, but it would be better to have ACE prepared for=20
such situations. That's
why I think you made a great work.
    In your patch I see you made changes on MPC level. That's good. I=20
can made the next step in
resolving  1854 bug, namely I can remove ACE_HAS_[QT,XT,GTK,TK]  as they=20
not only pollute
a namespace now, but also lead to some configuration management problems=20
. ACE_HAS_*
macros force developer to have either config.h with appropriate=20
definitions, but this makes hard to
have system wide ace/config.h header, or to define these macros on=20
project level, but this complicates
projects build system. Certainly, these considerations cannot be applied=20
to MPC based projects,
but there applications using ACE, but not MPC :-)

BTW. Is FL available for Debian? If so I can also try to remove ACE_HAS_F=
L.

Thanks
Marek Brudka