Martin Burnicki <martin.burni...@meinberg.de> wrote: > Without having looked at the code base, I'm sure there are already > predefined macros available for the current build target/architecture. > > So it should not be a problem to include something like > > #if defined( SOLARIS ) > #include <timepps_solaris.h> > #elif defined ( SCO ) > #include <timepps_sco.h> > #elif defined ( LINUX ) || defined( FREEBSD ) > #include <timepps.h> > #endif > > If this piece of code is required more than once it should be put into a > common header file which is included in all places where this is > required. No duplicate code. > > Martin
The above is already there in the code, Martin. The only point is that these timepps-SunOS.h, timepps-SCO.h and timepps-Solaris.h files are included but the Linux version is not. But we better stop discussing with Harlan. He is a manager type that knows zilch about the code yet always is commenting on the proposed changes. He wants money to hire people, of course because he cannot get volunteers to work under his management. Pity that the ntp project has a character like that involved. We better lose him. _______________________________________________ questions mailing list questions@lists.ntp.org http://lists.ntp.org/listinfo/questions