From: | "Rocco Altier" <RoccoA(at)Routescape(dot)com> |
---|---|
To: | "Andrew Dunstan" <andrew(at)dunslane(dot)net>, "Tom Lane" <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | "Chris Browne" <cbbrowne(at)acm(dot)org>, <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Buildfarm & cvsignore files |
Date: | 2006-09-27 16:53:33 |
Message-ID: | 6E0907A94904D94B99D7F387E08C4F5701832FF3@FALCON.INSIGHT |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
I noticed that the build farm is only looking for the cvs-ignore'd files
for a vpath build.
Attached is a patch that will stop at the CVS stage if there are any
cvs-ignore'd files in the clean repository.
Its not triggered by a from-source build, only what should have been a
clean check out.
Thanks,
-rocco
> -----Original Message-----
> From: pgsql-hackers-owner(at)postgresql(dot)org
> [mailto:pgsql-hackers-owner(at)postgresql(dot)org] On Behalf Of
> Andrew Dunstan
> Sent: Sunday, September 03, 2006 11:45 AM
> To: Tom Lane
> Cc: Chris Browne; pgsql-hackers(at)postgresql(dot)org
> Subject: Re: [HACKERS] [COMMITTERS] pgsql: Second try
> committing the path
>
>
>
>
> Andrew Dunstan wrote:
> > Tom Lane wrote:
> >>
> >> The buildfarm script is supposed to complain about
> unexpected files in
> >> the repository --- I wonder if it is fooled by the
> .cvsignore entries
> >> for these files?
> >>
> >
> > Yes, we do. A patch made in July 2005 has this comment:
> >
> > "ignore files listed in cvsignore files - this will stop
> inappropriate
> > triggering of vpath builds."
> >
> >
> > Perhaps I should only do that for vpath builds. Or perhaps I should
> > even remove them at the end of a build, since we don't
> expect any of
> > those files in a clean repo, do we?
> >
> > Also, in case anyone has not got the message yet: Don't
> ever build by
> > hand in the buildfarm repo. Ever. I mean it. Use a copy.
> >
> >
>
> I have just committed a patch that removes the cvsignore trap. This
> should be safe as we now remove them at the end of a
> buildfarm vpath run.
>
> cheers
>
> andrew
>
> ---------------------------(end of
> broadcast)---------------------------
> TIP 5: don't forget to increase your free space map settings
>
Attachment | Content-Type | Size |
---|---|---|
pgbf-cvsignore.patch | application/octet-stream | 1.9 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | David Fetter | 2006-09-27 16:53:41 | Re: psql service parameter |
Previous Message | Jim C. Nasby | 2006-09-27 16:35:48 | Re: Cross-table statistics idea |