Re: [HACKERS] [pgsql-www] Contrib/earthdistance missing from cvsweb.

2004-10-04 Thread Marc G. Fournier
On Mon, 4 Oct 2004, Dave Page wrote:

-Original Message-
From: Bruce Momjian [mailto:[EMAIL PROTECTED]
Sent: 04 October 2004 17:17
To: Marc G. Fournier
Cc: Dave Page; PostgreSQL WWW Mailing List
Subject: Re: [pgsql-www] Contrib/earthdistance missing from cvsweb.
Marc G. Fournier wrote:
On Mon, 4 Oct 2004, Dave Page wrote:
Hi Marc,
I always seem to get problems checking out the
earthdistance contrib
module from CVS. Tom mentioned in the past that this was
because you
had experimented with it when trying to split the repository.
I can get at it if I checkout pgsql vs. pgsql-server (or
vice-versa,
I forget which), but as this module is also missing from
cvsweb, it
would be good if you could fix it. Please :-)
Unfortunately, to 'fix it', I'm not sure of the risks,
since it causes
some major headaches when i pulled it out in the first place :(  If
nothing else, should probably wait until *after* the release, not
middle of beta ...
This just a CVS checkout issue.  It shouldn't affect the
actual CVS files.  I think it has been pushed off too long
that we should just do it now.  How many years must it be broken?
Which reminds me of why I reported it in the first place - I already get
errors about it being missing. I'm not sure things could get much worse!
If I correctly remember the issue, if I merge earthdistance back into the 
main source tree (which is a simple mv operation and cvs update on the 
modules file), it will potentially break everyone's currently checked out 
CVS source, since the 'paths' will change in the CVS/Root file ...

I'm willing to do it, since all the work I generally do, I do with a fresh 
check out ...

Am CC'ng in -hackers, as they will potentially be *the most* affected by 
doing this ...


Marc G. Fournier   Hub.Org Networking Services (http://www.hub.org)
Email: [EMAIL PROTECTED]   Yahoo!: yscrappy  ICQ: 7615664
---(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] [pgsql-www] Contrib/earthdistance missing from cvsweb.

2004-10-04 Thread Peter Eisentraut
Marc G. Fournier wrote:
 If I correctly remember the issue, if I merge earthdistance back into
 the main source tree (which is a simple mv operation and cvs update
 on the modules file), it will potentially break everyone's currently
 checked out CVS source, since the 'paths' will change in the CVS/Root
 file ...

This whole thing has been a mess that has been dragging on for too long.  
I suggest that after the release we make a clean break and put things 
back the way they were before (that is, one single pgsql module).

-- 
Peter Eisentraut
http://developer.postgresql.org/~petere/


---(end of broadcast)---
TIP 4: Don't 'kill -9' the postmaster


Re: [HACKERS] [pgsql-www] Contrib/earthdistance missing from cvsweb.

2004-10-04 Thread Marc G. Fournier
On Mon, 4 Oct 2004, Peter Eisentraut wrote:
Marc G. Fournier wrote:
If I correctly remember the issue, if I merge earthdistance back into
the main source tree (which is a simple mv operation and cvs update
on the modules file), it will potentially break everyone's currently
checked out CVS source, since the 'paths' will change in the CVS/Root
file ...
This whole thing has been a mess that has been dragging on for too long.
I suggest that after the release we make a clean break and put things
back the way they were before (that is, one single pgsql module).
Agreed, but Bruce is calling for this *during* beta, which I'm not 
comfortable with ... if anyone else wants to agree with Bruce, I'll do it, 
as I know it won't affect me ...


Marc G. Fournier   Hub.Org Networking Services (http://www.hub.org)
Email: [EMAIL PROTECTED]   Yahoo!: yscrappy  ICQ: 7615664
---(end of broadcast)---
TIP 9: the planner will ignore your desire to choose an index scan if your
 joining column's datatypes do not match


Re: [HACKERS] [pgsql-www] Contrib/earthdistance missing from cvsweb.

2004-10-04 Thread Bruce Momjian
Marc G. Fournier wrote:
 On Mon, 4 Oct 2004, Peter Eisentraut wrote:
 
  Marc G. Fournier wrote:
  If I correctly remember the issue, if I merge earthdistance back into
  the main source tree (which is a simple mv operation and cvs update
  on the modules file), it will potentially break everyone's currently
  checked out CVS source, since the 'paths' will change in the CVS/Root
  file ...
 
  This whole thing has been a mess that has been dragging on for too long.
  I suggest that after the release we make a clean break and put things
  back the way they were before (that is, one single pgsql module).
 
 Agreed, but Bruce is calling for this *during* beta, which I'm not 
 comfortable with ... if anyone else wants to agree with Bruce, I'll do it, 
 as I know it won't affect me ...
 

Seems we have 100% agreement so far.  Beta is a better time to require a
cvs fresh checkout than during development.

-- 
  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 7: don't forget to increase your free space map settings


Re: [HACKERS] [pgsql-www] Contrib/earthdistance missing from cvsweb.

2004-10-04 Thread Marc G. Fournier
On Mon, 4 Oct 2004, Tom Lane wrote:
Marc G. Fournier [EMAIL PROTECTED] writes:
as far as I know, the only thing that is 'broken' is cvsweb ... and since
it looks directly *at* teh cvs repository, not sure how it breaks that ...
The problem is that
http://developer.postgresql.org/cvsweb.cgi/pgsql-server/
works, but
http://developer.postgresql.org/cvsweb.cgi/pgsql/
pgsql is an 'alias' that merges pgsql-server and contrib modules ... 
cvsweb doesn't see it, as it only looks as the disk layout :(

I'm all for merging earthdistance back in and renaming the module back 
to just pgsql.  If it forces a fresh checkout, that's no big deal from 
my end, and I think it would get rid of a lot of confusion in the long 
run.

You had better give -hackers some notice, of course.
Baring anyone being against this, I'll make the changes tomorrow and post 
to the list once finished ... shouldn't take more then 5 minutes, but this 
gives ~24hrs notice for anyone working on something that they'd like to 
save :)


Marc G. Fournier   Hub.Org Networking Services (http://www.hub.org)
Email: [EMAIL PROTECTED]   Yahoo!: yscrappy  ICQ: 7615664
---(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] [pgsql-www] Contrib/earthdistance missing from cvsweb.

2004-10-04 Thread Andrew Dunstan
I trust those of us using CVSup will see the right thing, if we pick up 
the new CVSROOT/modules file?

cheers
andrew
Marc G. Fournier wrote:
On Mon, 4 Oct 2004, Tom Lane wrote:
Marc G. Fournier [EMAIL PROTECTED] writes:
as far as I know, the only thing that is 'broken' is cvsweb ... and 
since
it looks directly *at* teh cvs repository, not sure how it breaks 
that ...

The problem is that
http://developer.postgresql.org/cvsweb.cgi/pgsql-server/
works, but
http://developer.postgresql.org/cvsweb.cgi/pgsql/

pgsql is an 'alias' that merges pgsql-server and contrib modules ... 
cvsweb doesn't see it, as it only looks as the disk layout :(

I'm all for merging earthdistance back in and renaming the module 
back to just pgsql.  If it forces a fresh checkout, that's no big 
deal from my end, and I think it would get rid of a lot of confusion 
in the long run.

You had better give -hackers some notice, of course.

Baring anyone being against this, I'll make the changes tomorrow and 
post to the list once finished ... shouldn't take more then 5 minutes, 
but this gives ~24hrs notice for anyone working on something that 
they'd like to save :)


Marc G. Fournier   Hub.Org Networking Services 
(http://www.hub.org)
Email: [EMAIL PROTECTED]   Yahoo!: yscrappy  ICQ: 
7615664

---(end of broadcast)---
TIP 2: you can get off all lists at once with the unregister command
   (send unregister YourEmailAddressHere to [EMAIL PROTECTED])
---(end of broadcast)---
TIP 2: you can get off all lists at once with the unregister command
   (send unregister YourEmailAddressHere to [EMAIL PROTECTED])