[Pkg-corba-devel] filename of omniORB configuration file

[Thomas Girard]

Hello Floris,

On Fri, Mar 28, 2008 at 09:51:27PM +0000, Floris Bruynooghe wrote:
> > It seems there's something wrong in file:
> >   doc/tex/omniNames.tex
> > 
> > It looks like the file in SVN is the one from version 4.1.1.
> 
> Oops, hope that is the only file that I overlooked.  I blame the odd
> mergeWithUpstream behaviour of just being overlay to tarball and
> svn-upgrade's inablility to handle it as a "merge"...

I found this having a look at the generated .diff.gz while preparing the
source package.

If you feel it's simpler to keep the whole omniORB source tree in our
SVN repo then feel free to do it.

> > Another question is: do we want to handle this omniorb4 -> omniorb
> > transition before lenny release?
> 
> According to the last release update[1] we have until mid before we go
> into freeze.  Provided we don't run into any issues like with 4.1.0/1
> that is well doable, so I would vote for before lenny I think.
> 
> However it's probably best to make sure the current version works on
> all build hosts, so we might want to leave that for -2?  As for -1 I
> think that is ready for an upload right now if we don't want to make
> any more changes (and you change the distribution in the changelog
> file).

Agreed. Let's keep this transition for 4.1.2-2. I'm currently reviewing
changes between 4.1.1 and 4.1.2 to make sure the ABI was not changed.

I have doubts on some changes:
 * Fix idl/COS/CosTime.idl => depending library should be bumped
 * include/omniORB4/IOP.h: in class IOP:
   + static _core_attr const ComponentId TAG_CSI_SEC_MECH_LIST;
   + static _core_attr const ComponentId TAG_NULL_TAG;
   + static _core_attr const ComponentId TAG_TLS_SEC_TRANS;
 * include/omniORB4/internal/orbParameters.h:
   + _CORBA_MODULE_VAR _core_attr CORBA::Boolean immediateRopeSwitch
 * include/omniORB4/omniIOR.h:
   + static void unmarshal_TAG_CSI_SEC_MECH_LIST
   + static char* dump_TAG_CSI_SEC_MECH_LIST
 * include/omniORB4/sslContext.h: in class sslContext
   + CORBA::Boolean    pd_ssl_owner; => ABI break; check use.

I need to dig further with the help of [1].

Also, the license was changed from LGPL 2 to LGPL 2.1.

Regards,

Thomas

[1] http://techbase.kde.org/index.php?title=Policies/Binary_Compatibility_Issues_With_C%2B%2B