Re: [HACKERS] contrib/earthdistance missing regression test files
On Tue, Sep 24, 2002 at 23:57:29 -0400, Tom Lane [EMAIL PROTECTED] wrote: Bruce Momjian [EMAIL PROTECTED] writes: How do I run the regression tests for /contrib stuff? make make install make installcheck AFAICT, earthdistance is nowhere near passing yet :-(. It looks to me like the regression test is depending on the cube-based features that we decided to hold off for 7.4. Bruno, is that right? It shouldn't be. When I resubmitted the patch I intended to take out all of the cube related tests. If there is a reference to cube in there it is by mistake. ---(end of broadcast)--- TIP 4: Don't 'kill -9' the postmaster
Re: [HACKERS] contrib/earthdistance missing regression test files
AFAICT, earthdistance is nowhere near passing yet :-(. It looks to me like the regression test is depending on the cube-based features that we decided to hold off for 7.4. Bruno, is that right? It shouldn't be. When I resubmitted the patch I intended to take out all of the cube related tests. If there is a reference to cube in there it is by mistake. I took a look at the diff file I submitted and the only reference to cube in the regression test was in a comment I didn't change after removing the tests for the cube based distance stuff. ---(end of broadcast)--- TIP 6: Have you searched our list archives? http://archives.postgresql.org
Re: [HACKERS] contrib/earthdistance missing regression test files
OK, I reinstalled the proper earthdistance.out/sql files and it passes regession now. Sorry for the mistake. --- Bruno Wolff III wrote: AFAICT, earthdistance is nowhere near passing yet :-(. It looks to me like the regression test is depending on the cube-based features that we decided to hold off for 7.4. Bruno, is that right? It shouldn't be. When I resubmitted the patch I intended to take out all of the cube related tests. If there is a reference to cube in there it is by mistake. I took a look at the diff file I submitted and the only reference to cube in the regression test was in a comment I didn't change after removing the tests for the cube based distance stuff. -- Bruce Momjian| http://candle.pha.pa.us [EMAIL PROTECTED] | (610) 359-1001 + If your life is a hard drive, | 13 Roberts Road + Christ can be your backup.| Newtown Square, Pennsylvania 19073 ---(end of broadcast)--- TIP 3: if posting/reading through Usenet, please send an appropriate subscribe-nomail command to [EMAIL PROTECTED] so that your message can get through to the mailing list cleanly
Re: [HACKERS] contrib/earthdistance missing regression test files
Bruce Momjian [EMAIL PROTECTED] writes: OK, I reinstalled the proper earthdistance.out/sql files and it passes regession now. Sorry for the mistake. Looks good here too. Thanks. regards, tom lane ---(end of broadcast)--- TIP 6: Have you searched our list archives? http://archives.postgresql.org
[HACKERS] contrib/earthdistance missing regression test files
The Makefile for contrib/earthdistance indicates that there should be a regression test, but the files seem to be missing from CVS. The change to the Makefile was made here: http://developer.postgresql.org/cvsweb.cgi/contrib/earthdistance/Makefile.diff?r1=1.11r2=1.12 Was the Makefile change a mistake, or are there files missing? Joe ---(end of broadcast)--- TIP 3: if posting/reading through Usenet, please send an appropriate subscribe-nomail command to [EMAIL PROTECTED] so that your message can get through to the mailing list cleanly
Re: [HACKERS] contrib/earthdistance missing regression test files
On Tue, Sep 24, 2002 at 10:43:51 -0700, Joe Conway [EMAIL PROTECTED] wrote: The Makefile for contrib/earthdistance indicates that there should be a regression test, but the files seem to be missing from CVS. The change to the Makefile was made here: http://developer.postgresql.org/cvsweb.cgi/contrib/earthdistance/Makefile.diff?r1=1.11r2=1.12 Was the Makefile change a mistake, or are there files missing? There is supposed to be a regression test. I may have forgotten to use -N or -r on the diff. If it is confirmed that the files needed for the regression test didn't make it into the submitted diff file, I can send in a diff versus current cvs. ---(end of broadcast)--- TIP 2: you can get off all lists at once with the unregister command (send unregister YourEmailAddressHere to [EMAIL PROTECTED])
Re: [HACKERS] contrib/earthdistance missing regression test files
On Tue, Sep 24, 2002 at 15:02:20 -0500, Bruno Wolff III [EMAIL PROTECTED] wrote: There is supposed to be a regression test. I may have forgotten to use -N or -r on the diff. If it is confirmed that the files needed for the regression test didn't make it into the submitted diff file, I can send in a diff versus current cvs. I still have a copy of the diff file (at least I think it is the one I sent in) and it has the regression sql and output files defined. ---(end of broadcast)--- TIP 2: you can get off all lists at once with the unregister command (send unregister YourEmailAddressHere to [EMAIL PROTECTED])
Re: [HACKERS] contrib/earthdistance missing regression test files
Bruno Wolff III wrote: On Tue, Sep 24, 2002 at 15:02:20 -0500, Bruno Wolff III [EMAIL PROTECTED] wrote: There is supposed to be a regression test. I may have forgotten to use -N or -r on the diff. If it is confirmed that the files needed for the regression test didn't make it into the submitted diff file, I can send in a diff versus current cvs. I still have a copy of the diff file (at least I think it is the one I sent in) and it has the regression sql and output files defined. Yep, I missed adding earthdistance.out. Is that the only file. I usually do a 'gmake distclean' and 'cvs update' after a batch of patches to see that there aren't any new files in my CVS tree. I missed it this time. Does that fix the problem? -- Bruce Momjian| http://candle.pha.pa.us [EMAIL PROTECTED] | (610) 359-1001 + If your life is a hard drive, | 13 Roberts Road + Christ can be your backup.| Newtown Square, Pennsylvania 19073 ---(end of broadcast)--- TIP 6: Have you searched our list archives? http://archives.postgresql.org
Re: [HACKERS] contrib/earthdistance missing regression test files
On Tue, Sep 24, 2002 at 16:10:19 -0400, Bruce Momjian [EMAIL PROTECTED] wrote: Bruno Wolff III wrote: On Tue, Sep 24, 2002 at 15:02:20 -0500, Bruno Wolff III [EMAIL PROTECTED] wrote: There is supposed to be a regression test. I may have forgotten to use -N or -r on the diff. If it is confirmed that the files needed for the regression test didn't make it into the submitted diff file, I can send in a diff versus current cvs. I still have a copy of the diff file (at least I think it is the one I sent in) and it has the regression sql and output files defined. Yep, I missed adding earthdistance.out. Is that the only file. No. The new files are: expected/earthdistance.out (which you metion above) sql/earthdistance.sql ---(end of broadcast)--- TIP 5: Have you checked our extensive FAQ? http://www.postgresql.org/users-lounge/docs/faq.html
Re: [HACKERS] contrib/earthdistance missing regression test files
Bruno Wolff III wrote: On Tue, Sep 24, 2002 at 16:10:19 -0400, Bruce Momjian [EMAIL PROTECTED] wrote: Bruno Wolff III wrote: On Tue, Sep 24, 2002 at 15:02:20 -0500, Bruno Wolff III [EMAIL PROTECTED] wrote: There is supposed to be a regression test. I may have forgotten to use -N or -r on the diff. If it is confirmed that the files needed for the regression test didn't make it into the submitted diff file, I can send in a diff versus current cvs. I still have a copy of the diff file (at least I think it is the one I sent in) and it has the regression sql and output files defined. Yep, I missed adding earthdistance.out. Is that the only file. No. The new files are: expected/earthdistance.out (which you metion above) sql/earthdistance.sql OK, here's what I see now in CVS: #$ pwd /pgtop/contrib/earthdistance #$ lf CVS/README.earthdistanceearthdistance.out Makefileearthdistance.c earthdistance.sql.in What should be changed? -- Bruce Momjian| http://candle.pha.pa.us [EMAIL PROTECTED] | (610) 359-1001 + If your life is a hard drive, | 13 Roberts Road + Christ can be your backup.| Newtown Square, Pennsylvania 19073 ---(end of broadcast)--- TIP 4: Don't 'kill -9' the postmaster
Re: [HACKERS] contrib/earthdistance missing regression test files
Bruce Momjian [EMAIL PROTECTED] writes: No. The new files are: expected/earthdistance.out (which you metion above) sql/earthdistance.sql OK, here's what I see now in CVS: #$ pwd /pgtop/contrib/earthdistance #$ lf CVS/README.earthdistanceearthdistance.out Makefileearthdistance.c earthdistance.sql.in What should be changed? The earthdistance.out file should be in an expected/ subdirectory, not directly in the contrib/earthdistance directory. Also, there is a missing regression input script file earthdistance.sql (this is not related to earthdistance.sql.in), which should be in a sql/ subdirectory. regards, tom lane ---(end of broadcast)--- TIP 3: if posting/reading through Usenet, please send an appropriate subscribe-nomail command to [EMAIL PROTECTED] so that your message can get through to the mailing list cleanly
Re: [HACKERS] contrib/earthdistance missing regression test files
Tom Lane wrote: Bruce Momjian [EMAIL PROTECTED] writes: No. The new files are: expected/earthdistance.out (which you metion above) sql/earthdistance.sql OK, here's what I see now in CVS: #$ pwd /pgtop/contrib/earthdistance #$ lf CVS/README.earthdistanceearthdistance.out Makefileearthdistance.c earthdistance.sql.in What should be changed? The earthdistance.out file should be in an expected/ subdirectory, not directly in the contrib/earthdistance directory. Also, there is a missing regression input script file earthdistance.sql (this is not related to earthdistance.sql.in), which should be in a sql/ subdirectory. OK, done. I thought the earthdistance.sql file was derived from earthdistance.sql.in, and I thought the out was just a test file. I got them fixed now, in their proper directory. How do I run the regression tests for /contrib stuff? -- Bruce Momjian| http://candle.pha.pa.us [EMAIL PROTECTED] | (610) 359-1001 + If your life is a hard drive, | 13 Roberts Road + Christ can be your backup.| Newtown Square, Pennsylvania 19073 ---(end of broadcast)--- TIP 3: if posting/reading through Usenet, please send an appropriate subscribe-nomail command to [EMAIL PROTECTED] so that your message can get through to the mailing list cleanly
Re: [HACKERS] contrib/earthdistance missing regression test files
Bruce Momjian [EMAIL PROTECTED] writes: How do I run the regression tests for /contrib stuff? make make install make installcheck AFAICT, earthdistance is nowhere near passing yet :-(. It looks to me like the regression test is depending on the cube-based features that we decided to hold off for 7.4. Bruno, is that right? regards, tom lane ---(end of broadcast)--- TIP 1: subscribe and unsubscribe commands go to [EMAIL PROTECTED]