Thanks for the overview. Can you go through the list and see if the defines
autoconf doesn't set are really needed in general (they could be left over
after cleanup). If you think autoconf check for a certain feature, can you
report each macro seperately in bugzilla? Then we get a clean overview in
bugzilla what has to be done.
> I've written the attached script to compare autogenerated
> config.h with
> regular one.
> $ export ACE_ROOT=/path/to/ACE_wrappers
> $ cd $ACE_ROOT
> $ ./ace-cmp.sh ./objdir/ace/config.h ./ace/config-linux.h
> will compare autogenerated config from $ACE_ROOT/objdir/ace/config.h
> with the regular $ACE_ROOT/ace/config-linux.h
> This script is buggy and probably not portable - for instance it uses
> `cpp -dM'. I still have to find out why it does not reveal the
> ACE_HAS_POSIX_SEM_TIMEOUT difference pointed out last month.
> Anyway, on my box it shows 115 differences, and these will probably
> require some work.
> I'll be away for the week-end, hence the early post.
Right, I will do this.
I have just found out why I was not getting the expected difference on
ACE_HAS_POSIX_SEM_TIMEOUT: ACE_HAS_LINUX_NPTL was *not* defined in the
regular $ACE_ROOT/ace/config-linux.h, hence all posix stuff was not
checked. It this intended?
Adding -DACE_HAS_LINUX_NPTL to the already fragile script gives the
attached diff I'll use to report bugs.
> Right, I will do this.
> I have just found out why I was not getting the expected difference on
> ACE_HAS_POSIX_SEM_TIMEOUT: ACE_HAS_LINUX_NPTL was *not* defined in the
> regular $ACE_ROOT/ace/config-linux.h, hence all posix stuff was not
> checked. It this intended?
No, I don't think so. ACE_HAS_LINUX_NPTL seems something that could be
defined, but the include of config-posix.h shouldn't depend on that
> Adding -DACE_HAS_LINUX_NPTL to the already fragile script gives the
> attached diff I'll use to report bugs.
If you have things that need to be changed, let us know.
The script was not including ace/config-g++-common.h because it was
using cpp instead of g++. A modified script, fixing this issue and
defining ACE_HAS_LINUX_NPTL, is attached.
I'll try to improve this script when I have time so that:
* it does not need -DACE_HAS_LINUX_NPTL (not needed for ACE > 5.5.10)
* it works with other compilers
A new diff is attached as well. Lines starting with `-' are defined by
the autotools method while line starting with `+' are by the regular
method. There are a few false positive (e.g. the difference on gperf,
#define ACE_GPERF "gperf-ace", which is manually set for Debian
packages), but there are also interesting lines, like the difference on
ACE_DEFAULT_BASE_ADDR, yet to be explained.