On Wed, Dec 27, 2006 at 09:45:35AM -0800, Martin Knoblauch wrote: > > I used NetBSD as a base from my port (as it is the closest), sadly > > they are not that similar as to just work with the other source > > as you can see by the diff. > > Understand. Btw. you should check the use of the strings NetBSD / > FreeBSD in you patch :-)
not sure what you mean, but since NetBSD was based on FreeBSD and my port was based in NetBSD I am not surprised at all if there are extra references in between them from the old code, and based on the license (BSD) removing some of them might not be possible. if I recall correctly, the same happens between Cygwin and Linux, and most other platforms that were based on the Linux metrics code (probably including FreeBSD). If you meant to say that there is a lot of duplicated code in between all of them I agree, but having an independent (per platform) metrics.c file where there is no need to use portable code has also its own advantages as it simplifies the maintenance of that platform without concerns of interdependency with other platforms even if it requires that the same bug be fixed several times in some cases (which is why as part of porting OpenBSD I ended up fixing and testing NetBSD and FreeBSD). > > DragonflyBSD will be most likely closer to FreeBSD and the same for > > MacOS X (AKA Darwin), but I have no interest on adding those yet > > (DragonFlyBSD could be an interesting option for clusters, but > > I'd heard of no one using it in a cluster yet). > > You realize that we already have a Darwin port, although I do not know > the quality/completeness of the metrics code. I do, and I really meant to say (except that my idea got somehow compressed): "I have no interest on adding DragonflyBSD or testing Darwin as I have no use for those yet" BTW on my private development tree I made a port to Debian/kFreeBSD (uses Linux) and NexentaOS (uses Solaris) just for fun, and too do some torture testing to the autoconf rules used in libmetrics and explore the impact that different kernel/userspace do to it. Carlo
