Date: Sun, 30 Apr 2023 07:21:59 -0600 From: Warner Losh <imp@bsdimp.com> To: Jessica Clarke <jrtc27@freebsd.org> Cc: Kirk McKusick <mckusick@freebsd.org>, src-committers <src-committers@freebsd.org>, "<dev-commits-src-all@freebsd.org>" <dev-commits-src-all@freebsd.org>, "<dev-commits-src-main@freebsd.org>" <dev-commits-src-main@freebsd.org> Subject: Re: git: 2e7797cd9bba - main - Yet another try to fix printf format conflict. Message-ID: <CANCZdfrvA_RKX2KPkH=suHdfKT4W6r9YbRbv%2BWoBk2sMcL=U_g@mail.gmail.com> In-Reply-To: <3ECD7073-5F45-4C62-BE47-D1C8CEC61883@freebsd.org> References: <202304300700.33U70cF9082164@gitrepo.freebsd.org> <3ECD7073-5F45-4C62-BE47-D1C8CEC61883@freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
[-- Attachment #1 --] On Sun, Apr 30, 2023, 1:08 AM Jessica Clarke <jrtc27@freebsd.org> wrote: > On 30 Apr 2023, at 08:00, Kirk McKusick <mckusick@FreeBSD.org> wrote: > > > > The branch main has been updated by mckusick: > > > > URL: > https://cgit.FreeBSD.org/src/commit/?id=2e7797cd9bba4fa82391b53a7dd7aac074baf716 > > > > commit 2e7797cd9bba4fa82391b53a7dd7aac074baf716 > > Author: Kirk McKusick <mckusick@FreeBSD.org> > > AuthorDate: 2023-04-30 06:59:45 +0000 > > Commit: Kirk McKusick <mckusick@FreeBSD.org> > > CommitDate: 2023-04-30 07:00:23 +0000 > > > > Yet another try to fix printf format conflict. > > > > I find it very annoying that there is no FreeBSD infrastructure to > > determine failures across architectures other than to check in > > changes and then have Jenkins find them. > > That’s what make universe/tinderbox/worlds/kernels are for? (See build(7)) > There's other ways too. The long way is to 'make buildworld TARGET=i386' which will catch things like this. However, that can take a while... If you've done one of those in the past, there will be enough of a tree for the next trick to work: cd sbin/dumpfs make buildenv TARGET=i386 make # in a subshell exit This will also build things and see if you got the printf formatting right and is pretty quick. So of course I always do this before I commit.... the fix that the tinderbox tells me I broke (since I don't always remember that I've changed a printf in larger commits). it would be nice if clang had a magic flag to catch this.... Warner Jess > > > Suggested by: Jessica Clarke > > MFC after: 1 week > > --- > > sbin/dumpfs/dumpfs.c | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/sbin/dumpfs/dumpfs.c b/sbin/dumpfs/dumpfs.c > > index 743734660e57..a0767c39e6bf 100644 > > --- a/sbin/dumpfs/dumpfs.c > > +++ b/sbin/dumpfs/dumpfs.c > > @@ -245,7 +245,7 @@ dumpfs(const char *name, int dosb) > > default: > > goto err; > > } > > - printf("old_cpg\t%d\tsize_cg\t%d\tCGSIZE\t%d\n", > > + printf("old_cpg\t%d\tsize_cg\t%zu\tCGSIZE\t%zu\n", > > afs.fs_old_cpg, sizeof(struct cg), CGSIZE(&afs)); > > printf("sblkno\t%d\tcblkno\t%d\tiblkno\t%d\tdblkno\t%d\n", > > afs.fs_sblkno, afs.fs_cblkno, afs.fs_iblkno, afs.fs_dblkno); > > [-- Attachment #2 --] <div dir="auto"><div><br><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sun, Apr 30, 2023, 1:08 AM Jessica Clarke <<a href="mailto:jrtc27@freebsd.org">jrtc27@freebsd.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 30 Apr 2023, at 08:00, Kirk McKusick <mckusick@FreeBSD.org> wrote:<br> > <br> > The branch main has been updated by mckusick:<br> > <br> > URL: <a href="https://cgit.FreeBSD.org/src/commit/?id=2e7797cd9bba4fa82391b53a7dd7aac074baf716" rel="noreferrer noreferrer" target="_blank">https://cgit.FreeBSD.org/src/commit/?id=2e7797cd9bba4fa82391b53a7dd7aac074baf716</a><br>; > <br> > commit 2e7797cd9bba4fa82391b53a7dd7aac074baf716<br> > Author: Kirk McKusick <mckusick@FreeBSD.org><br> > AuthorDate: 2023-04-30 06:59:45 +0000<br> > Commit: Kirk McKusick <mckusick@FreeBSD.org><br> > CommitDate: 2023-04-30 07:00:23 +0000<br> > <br> > Yet another try to fix printf format conflict.<br> > <br> > I find it very annoying that there is no FreeBSD infrastructure to<br> > determine failures across architectures other than to check in<br> > changes and then have Jenkins find them.<br> <br> That’s what make universe/tinderbox/worlds/kernels are for? (See build(7))<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto">There's other ways too. The long way is to 'make buildworld TARGET=i386' which will catch things like this. However, that can take a while... If you've done one of those in the past, there will be enough of a tree for the next trick to work:</div><div dir="auto"><br></div><div dir="auto">cd sbin/dumpfs</div><div dir="auto">make buildenv TARGET=i386</div><div dir="auto">make # in a subshell</div><div dir="auto">exit</div><div dir="auto"><br></div><div dir="auto">This will also build things and see if you got the printf formatting right and is pretty quick.</div><div dir="auto"><br></div><div dir="auto">So of course I always do this before I commit.... the fix that the tinderbox tells me I broke (since I don't always remember that I've changed a printf in larger commits). it would be nice if clang had a magic flag to catch this....</div><div dir="auto"><br></div><div dir="auto">Warner</div><div dir="auto"><br></div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"> Jess<br> <br> > Suggested by: Jessica Clarke<br> > MFC after: 1 week<br> > ---<br> > sbin/dumpfs/dumpfs.c | 2 +-<br> > 1 file changed, 1 insertion(+), 1 deletion(-)<br> > <br> > diff --git a/sbin/dumpfs/dumpfs.c b/sbin/dumpfs/dumpfs.c<br> > index 743734660e57..a0767c39e6bf 100644<br> > --- a/sbin/dumpfs/dumpfs.c<br> > +++ b/sbin/dumpfs/dumpfs.c<br> > @@ -245,7 +245,7 @@ dumpfs(const char *name, int dosb)<br> > default:<br> > goto err;<br> > }<br> > - printf("old_cpg\t%d\tsize_cg\t%d\tCGSIZE\t%d\n",<br> > + printf("old_cpg\t%d\tsize_cg\t%zu\tCGSIZE\t%zu\n",<br> > afs.fs_old_cpg, sizeof(struct cg), CGSIZE(&afs));<br> > printf("sblkno\t%d\tcblkno\t%d\tiblkno\t%d\tdblkno\t%d\n",<br> > afs.fs_sblkno, afs.fs_cblkno, afs.fs_iblkno, afs.fs_dblkno);<br> <br> </blockquote></div></div></div>
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CANCZdfrvA_RKX2KPkH=suHdfKT4W6r9YbRbv%2BWoBk2sMcL=U_g>