[ksh93-integration-discuss] Re: Updating the ksh93-integrationprototype002 codebase from B37 to B48 ?

[Roland Mainz]

Mike Kupfer wrote:
> 
> >>>>> "Roland" == Roland Mainz <roland.mainz at nrubsig.org> writes:
> 
> Roland> can we still get a webrev from the current version of the
> Roland> ksh93-integration prototype002 (the current revision is "431"
> Roland> ...) ?
> 
> Is 430 okay?

Yes... (the latest is now "432" but I guess people can view
http://polaris.blastwave.org/changeset/431 and
http://polaris.blastwave.org/changeset/432 "manually" ... :-) ) ...

> Up until now I've been running into bugs in webrev.  Steve Lau has fixed
> them, and I'm currently reviewing the webrev to make sure it's sane
> (i.e., no other webrev bugs).

Thanks! :-)
 
> Roland> Sightly offtopic: What do you think about the idea to move the
> Roland> global CERRWARN settings to per-file CERRWARN settings (see
> Roland> http://polaris.blastwave.org/changeset/431 for an example) if
> Roland> possible ?
> 
> It's certainly an improvement over global settings, though I'm still
> nervous about disabling warnings like this.

I know... and the lack of a gcc option to turn warnings for duplicate
CPP symbols off means we have to kill them all (the situation has become
much better since March thanks to Glenn&&David) ...
... however for libcmd it's only one duplicte symbol (and already fixed
by upstream) called "SS" which comes from a system header (welcome to
http://bugs.opensolaris.org/bugdatabase/view_bug.do?bug_id=6400595) ...
somehow I think that xx@@!!!-system-headers should never define CPP
symbols with less then five letters ... ;-(

The good news is that all of the important warnings are gone (64bit
problems etc.) ... :-)

----

Bye,
Roland

-- 
  __ .  . __
 (o.\ \/ /.o) roland.mainz at nrubsig.org
  \__\/\/__/  MPEG specialist, C&&JAVA&&Sun&&Unix programmer
  /O /==\ O\  TEL +49 641 7950090
 (;O/ \/ \O;)