Christopher Mi wrote: > Lizhong Li wrote: >> Hi All, >> >> I am porting the package 'mrtg' to opensolaris, which can monitor >> SNMP network >> devices and draw pretty pictures showing how much traffic has passed >> through >> each interface. >> >> Could you please review it ? >> >> http://cr.opensolaris.org/~uniopen/mrtg >> >> _______________________________________________ >> sfwnv-discuss mailing list >> sfwnv-discuss at opensolaris.org >> http://mail.opensolaris.org/mailman/listinfo/sfwnv-discuss >> > Hi Lizhong, > > In my perspective, you did very well :-) . Just two questions. > > 1. You don't have a depend file. It seems that you need one. Since > when configure, > it ask for the GL library. > Yes, I just checked it and added the file depend. > 2. Maybe you take the SUNWtop as the template. Since I found > 47 # SUNWtop > in file usr/src/pkgdefs/SUNWmrtg/prototype_sparc > Please change it and others if exists. > Exactly, I've fixed it. And the new webrev is available, please kindly help check, thanks. http://cr.opensolaris.org/~uniopen/mrtg
> Thanks, > Chris -- Thanks, Lizhong
