On Thu, 2005-12-29 at 18:46 +0100, Carlos Garnacho wrote:
> On Mon, 2005-12-19 at 16:28 -0500, Juan Luis Baptiste wrote:
> > Hi,
> >
> > Here I send a patch that adds support to the network backend for the
> > following
> > platforms:
> >
> > - Yoper Linux 2.2
> > - Ark Linux
> > - Mandriva 2006.0 and 2006.1 (Cooker)
> > - Fedora 4
> > - Ubuntu/Kubuntu 5.10 and 6.04.
> >
> > Also the Yoper Linux logo is attached. PLEASE, PLEASE, PLEASE review the
> > patch soon, more than six months ago I sent the yoper linux patch and I
> > never
> > got any feedback about it, and now when I applied it again against the
>
> My fault, sorry...
>
> > current CVS tree it conflicted and I had to redo it all over again, I don't
> > want to have to do the same with this patch that is a lot bigger than that
> > old one.
> >
> > The patch is against 12-18-2005.
>
> The patch seems Ok, a couple of comments:
>
> - I agree with you that the indenting in the backends is far from what
> it ought to be, but please, don't mix reindenting of large pieces of
> code with changes in it, it took me >1h to review it, and I'm still not
> sure that i've spotted all the typos.
> - In platform.pl.in, given that the base distro for ubuntu/kubuntu is
> Ubuntu itself, I'd label the description "Ubuntu Linux foo", not
> "Kubuntu ..." (or, at least, label it "(K)Ubuntu")
> - In platform.pl.in, the description for ark should be "Ark Linux", not
> "Arkark Linux"
>
> I'll review it again later today and commit it, thanks for the patch :)
I've reviewed it again, found more typos (related to ubuntu-5.10 not
being included in all places it should) and committed the patch, thanks
again, and please, don't do that again to patches :P
Carlos
>
> Regards
>
> >
> > Cheers
> > _______________________________________________
> > system-tools-list mailing list
> > [email protected]
> > http://mail.gnome.org/mailman/listinfo/system-tools-list
> _______________________________________________
> system-tools-list mailing list
> [email protected]
> http://mail.gnome.org/mailman/listinfo/system-tools-list
_______________________________________________
system-tools-list mailing list
[email protected]
http://mail.gnome.org/mailman/listinfo/system-tools-list
