On Tue, Apr 24, 2012 at 03:44:48PM -0700, Danek Duvall wrote:
> Edward Pilatowicz wrote:
>
> > src/zoneproxy/zoneproxy-adm/zoneproxy-adm.c
> > src/zoneproxy/zoneproxyd/zoneproxy_impl.h
> >
> > - i don't see any changes in here other than copyright?
>
> You're right for zoneproxy-adm.c -- while checking the copyrights of the
> files I was changing, I noticed that this one had the form wrong, so I
> corrected it.  But zoneproxy_impl.h has more changes.
>

not that i can see in the "sdiffs" portion of the webrev.  although it's
weird because the index.html says 4 lines change for this file, and
looking at the wdiffs it shows some differences.  (looks like whitespace
changes.)  strange.

> > wrt other modules, the last time i looked at them they generated a ton
> > of list errors because of 7018856, which was closed as a dup of 7025627.
> > but roger fixed 7025627 in snv_166 and i assumed they'd like clean after
> > that, but i guess not.
>
> Interestingly, the modules still have a bunch of errors about fstat64(),
> pread64(), and so on, being declared inconsistently, which sounds precisely
> like 7025627.  I haven't investigated those at all, but clearly something
> is going on somewhere.
>
> Thanks,
> Danek
_______________________________________________
pkg-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/pkg-discuss

Reply via email to