Date: Fri, 20 Jan 2012 19:04:55 -0500 From: Eitan Adler <eadler@freebsd.org> To: Hiroki Sato <hrs@freebsd.org> Cc: svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org Subject: Re: svn commit: r230354 - head/usr.sbin/makefs Message-ID: <CAF6rxg=VWzUcDuk_oN_wepJPejg8JnGBKm5oLXB7Y8JCW52t3w@mail.gmail.com> In-Reply-To: <20120120.123256.1432718473132856309.hrs@allbsd.org> References: <201201200138.q0K1cSou016739@svn.freebsd.org> <20120120.123256.1432718473132856309.hrs@allbsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
On Thu, Jan 19, 2012 at 10:32 PM, Hiroki Sato <hrs@freebsd.org> wrote: > Eitan Adler <eadler@FreeBSD.org> wrote > =C2=A0in <201201200138.q0K1cSou016739@svn.freebsd.org>: > > ea> Author: eadler > ea> Date: Fri Jan 20 01:38:28 2012 > ea> New Revision: 230354 > ea> URL: http://svn.freebsd.org/changeset/base/230354 > ea> > ea> Log: > ea> =C2=A0 Fix a variety of warnings when compiling with gcc46 > ea> > ea> =C2=A0 Approved by: =C2=A0 =C2=A0 =C2=A0dim, cperciva (mentor, blanke= t for pre-mentorship already-approved commits) > ea> =C2=A0 MFC after: =C2=A0 =C2=A0 =C2=A0 =C2=A03 days > ea> > ea> Modified: > ea> =C2=A0 head/usr.sbin/makefs/cd9660.c > > =C2=A0Removing the dot handling part and leaving a comment in lines > =C2=A01106-1117 make people confused. I missed that - sorry. > =C2=A0In addition to that, I personally don't think this should be remove= d > =C2=A0because our cd9660.c is still based on NetBSD's one in any > =C2=A0way---bugfixes on our side have been reported to the upstream and w= e > =C2=A0will import useful changes from there if any. =C2=A0Although the cu= rrent > =C2=A0dot handling is useless, keeping the difference between the two sma= ll > =C2=A0still has a meaning. I was was unaware this code was contributed. I just looked at the NetBSD version and I don't think it suffers from the same problem - the loop appears to be used later. If that is because of some other bug fix which could be upstreamed that would be great. On the other hand I would like to continue with my goal of making the non-contrib world compilable with CC=3Dgcc46. Should I revert this commit? --=20 Eitan Adler Ports committer X11, Bugbusting teams
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CAF6rxg=VWzUcDuk_oN_wepJPejg8JnGBKm5oLXB7Y8JCW52t3w>