Re: Please upload: postgresql-8.0.0cvs-1 [test] (gold star alert)

2004-10-06 Thread Jason Tishler
On Tue, Oct 05, 2004 at 12:15:47PM -0400, Igor Pechtchanski wrote:
 On Mon, 4 Oct 2004, Christopher Faylor wrote:
  Igor, can we get two gold stars for Jason?  One for his years of
  Postgres support, and another one for just being a long time
  contributor to Cygwin.
 
  [snip]
 
 Done.

Thanks!

Jason

-- 
PGP/GPG Key: http://www.tishler.net/jason/pubkey.asc or key servers
Fingerprint: 7A73 1405 7F2B E669 C19D  8784 1AFD E4CC ECF4 8EF6


Re: Please upload: postgresql-8.0.0cvs-1 [test]

2004-10-05 Thread Gerrit P. Haase
Hello Reini,

Am Montag, 4. Oktober 2004 um 23:39 schriebst du:

 Gerrit P. Haase schrieb:
The tablespace feature is known to fail.
 
 What does this mean exactly?  (Link to docu should do it).

 http://developer.postgresql.org/docs/postgres/catalog-pg-tablespace.html
 just specify the physical layout for a table. taken from oracle.
 implemented as simple symlink on directories.
 to share a db across discs probably.
 added just lately.

Great feature.  Lets hope it will work;)


Thanks,
Gerrit
-- 
=^..^=




Re: Please upload: postgresql-8.0.0cvs-1 [test]

2004-10-05 Thread Gerrit P. Haase
Hallo Gerrit,

Am Dienstag, 5. Oktober 2004 um 04:25 schriebst du:

 Hello Reini,

 getting this when I try to build:

 make[1]: Leaving directory
 `/ftproot/pub/PostgreSQL/postgresql/postgresql-8.0.0cvs/.build/contrib/dbsize'
 make: *** earthdistance: No such file or directory.  Stop.
 make: Entering an unknown directorymake: Leaving an unknown directorymake: *** [all] 
 Error 2
 make: Leaving directory
 `/ftproot/pub/PostgreSQL/postgresql/postgresql-8.0.0cvs/.build/contrib'

Getting another one (missing -lpgport here?):
dlltool --export-all  --output-def ltree.def ltree_io.o ltree_op.o lquery_op.o 
_ltree_op.o crc32.o ltxtquery_io.o ltxtquery_op.o ltree_gist.o _ltree_gist.o
dllwrap -o ltree.dll --dllname ltree.dll  --def ltree.def ltree_io.o ltree_op.o 
lquery_op.o _ltree_op.o crc32.o ltxtquery_io.o ltxtquery_op.o ltree_gist.o 
_ltree_gist.o ../../src/utils/dllinit.o -L../../src/port -L/usr/local/lib 
-L../../src/backend -lpostgres 
lquery_op.o(.text+0x485):lquery_op.c: undefined reference to `_pg_strncasecmp'
ltxtquery_op.o(.text+0x18c):ltxtquery_op.c: undefined reference to `_pg_strncasecmp'
collect2: ld returned 1 exit status
dllwrap: gcc exited with status 1
make[1]: *** [libltree.a] Error 1


Gerrit
-- 
=^..^=




Re: Please upload: postgresql-8.0.0cvs-1 [test]

2004-10-05 Thread Reini Urban
Gerrit P. Haase schrieb:
getting this when I try to build:
make[1]: Leaving directory 
`/ftproot/pub/PostgreSQL/postgresql/postgresql-8.0.0cvs/.build/contrib/dbsize'
make: *** earthdistance: No such file or directory.  Stop.
make: Entering an unknown directorymake: Leaving an unknown directorymake: *** [all] 
Error 2
make: Leaving directory 
`/ftproot/pub/PostgreSQL/postgresql/postgresql-8.0.0cvs/.build/contrib'
Any idea?
Yes. They had troubles yesterday with this contrib (hidden in CVS) when 
I packaged it. So I disabled it in the contrib/Makefile, but obviously 
forgot to add this Makefile patch to the patch. Sorry.

Just remove the earthdistance \ line from contrib/Makefile. Will be in 
the next update again.

--
Reini Urban
http://xarch.tu-graz.ac.at/home/rurban/


Re: Please upload: postgresql-8.0.0cvs-1 [test]

2004-10-05 Thread Reini Urban
Gerrit P. Haase schrieb:
Am Dienstag, 5. Oktober 2004 um 04:25 schriebst du:
getting this when I try to build:
Getting another one (missing -lpgport here?):
Yes.
I'm sure I had this in one version of my patches, but where is it gone ...
DLL's missing -lpgport on cygwin is my most concern to get fixed 
upstream, BTW. Besides tablespace and IPC.

dlltool --export-all  --output-def ltree.def ltree_io.o ltree_op.o lquery_op.o _ltree_op.o crc32.o ltxtquery_io.o ltxtquery_op.o ltree_gist.o _ltree_gist.o
dllwrap -o ltree.dll --dllname ltree.dll  --def ltree.def ltree_io.o ltree_op.o lquery_op.o _ltree_op.o crc32.o ltxtquery_io.o ltxtquery_op.o ltree_gist.o _ltree_gist.o ../../src/utils/dllinit.o -L../../src/port -L/usr/local/lib -L../../src/backend -lpostgres 
lquery_op.o(.text+0x485):lquery_op.c: undefined reference to `_pg_strncasecmp'
ltxtquery_op.o(.text+0x18c):ltxtquery_op.c: undefined reference to `_pg_strncasecmp'
collect2: ld returned 1 exit status
dllwrap: gcc exited with status 1
make[1]: *** [libltree.a] Error 1
Add
  SHLIB_LINK += -lpgport
to the Makefile as last line.
For me it built without, strangely.
--
Reini Urban
http://xarch.tu-graz.ac.at/home/rurban/


Re: Please upload: postgresql-8.0.0cvs-1 [test]

2004-10-05 Thread Reini Urban
Christopher Faylor schrieb:
On Mon, Oct 04, 2004 at 10:51:07PM +0200, Reini Urban wrote:
Reini Urban schrieb:
ok, setup.hint updated as attached. package otherwise unchanged.
oops, that attachment was the wrong version. sorry.

wget -O - http://xarch.tu-graz.ac.at/publ/cygwin/postgresql/get.sh | sh
This has been installed in the cygwin release area.
Thanks.  I appreciate being provided with the exact command needed to
download this.  
Ok, I'll do next time.
It's odd, though.
It's from Jaari. The rule is:
URL=http://xarch.tu-graz.ac.at/publ/cygwin
wget $URL/$PKG/$FULLPKG.tar.bz2 $URL/$PKG/$FULLPKG-src.tar.bz2 
$URL/$PKG/setup.hint

I wouldn't have expected that you'd need me to fill in the root password
and my credit card number.  :-)
You do have one? :)
--
Reini Urban
http://xarch.tu-graz.ac.at/home/rurban/


Re: Please upload: postgresql-8.0.0cvs-1 [test]

2004-10-05 Thread Reini Urban
Gerrit P. Haase schrieb:
Hello Reini,
Am Dienstag, 5. Oktober 2004 um 11:42 schriebst du:

Gerrit P. Haase schrieb:
Am Dienstag, 5. Oktober 2004 um 04:25 schriebst du:
getting this when I try to build:
Getting another one (missing -lpgport here?):

Yes.
I'm sure I had this in one version of my patches, but where is it gone ...
DLL's missing -lpgport on cygwin is my most concern to get fixed 
upstream, BTW. Besides tablespace and IPC.

dlltool --export-all  --output-def ltree.def ltree_io.o
ltree_op.o lquery_op.o _ltree_op.o crc32.o ltxtquery_io.o
ltxtquery_op.o ltree_gist.o _ltree_gist.o
dllwrap -o ltree.dll --dllname ltree.dll  --def ltree.def
ltree_io.o ltree_op.o lquery_op.o _ltree_op.o crc32.o
ltxtquery_io.o ltxtquery_op.o ltree_gist.o _ltree_gist.o
../../src/utils/dllinit.o -L../../src/port -L/usr/local/lib
-L../../src/backend -lpostgres 
lquery_op.o(.text+0x485):lquery_op.c: undefined reference to `_pg_strncasecmp'
ltxtquery_op.o(.text+0x18c):ltxtquery_op.c: undefined reference to `_pg_strncasecmp'
collect2: ld returned 1 exit status
dllwrap: gcc exited with status 1
make[1]: *** [libltree.a] Error 1

Add
  SHLIB_LINK += -lpgport
to the Makefile as last line.
For me it built without, strangely.
Actually I added it to BE_DLLLIBS in Makefile.cygwin:
BE_DLLLIBS= -L$(top_builddir)/src/backend -lpostgres -L$(top_builddir)/src/port 
-lpg_port
They will not accept this, though it makes sense to add -lpgport to the 
backend libs. In my first set of patches I did this also.

They decided recently to split modules into normal MODULES and 
MODULE_big, where big will get all the libs, and some contrib modules 
have wrong Zuordnungen.
  src/makefiles/pgxs.mk
but then they turned off USE_PGXS and switched to
$(top_srcdir)/contrib/contrib-global.mk
I'll need some time to understand this.

And another one, there is a missing CPPFLAG for the src/bin/psql
directory 

(at least for me, I'm building outside the source directory), 
Ah, good to to know, that I can try this also now. Didn't work last 
month. I don't believe that they know it also.

I added -I$(top_srcdir)/src/bin/psql here in
src/bin/psql/Makefile: 

override CPPFLAGS := -I$(libpq_srcdir) $(CPPFLAGS) -I$(top_srcdir)/src/bin/psql -DFRONTEND
Thanks. I'll test and fix upstream.
--
Reini Urban
http://xarch.tu-graz.ac.at/home/rurban/


Re: Please upload: postgresql-8.0.0cvs-1 [test] (gold star alert)

2004-10-05 Thread Igor Pechtchanski
On Mon, 4 Oct 2004, Christopher Faylor wrote:

 On Mon, Oct 04, 2004 at 04:42:25PM -0400, Jason Tishler wrote:
 Reini,
 
 On Mon, Oct 04, 2004 at 10:03:54PM +0200, Reini Urban wrote:
  I've taken over from Jason Tishler with the 8.x series with his
  permission. Thanks Jason!
 
 You are very welcome.  Actually, I should be thanking *you*! :,)

 Igor, can we get two gold stars for Jason?  One for his years of
 Postgres support, and another one for just being a long time contributor
 to Cygwin.

 And one for Reini for taking up the Postgres yoke, too, please.

 cgf

Done.
Igor
-- 
http://cs.nyu.edu/~pechtcha/
  |\  _,,,---,,_[EMAIL PROTECTED]
ZZZzz /,`.-'`'-.  ;-;;,_[EMAIL PROTECTED]
 |,4-  ) )-,_. ,\ (  `'-'   Igor Pechtchanski, Ph.D.
'---''(_/--'  `-'\_) fL a.k.a JaguaR-R-R-r-r-r-.-.-.  Meow!

Happiness lies in being privileged to work hard for long hours in doing
whatever you think is worth doing.  -- Dr. Jubal Harshaw


Re: Please upload: postgresql-8.0.0cvs-1 [test]

2004-10-04 Thread Christopher Faylor
On Mon, Oct 04, 2004 at 10:03:54PM +0200, Reini Urban wrote:
Note: maintainer change.

wget -q -O - http://xarch.tu-graz.ac.at/publ/cygwin/postgresql/get.sh | sh

Please mark this as experimental (test).
Or am I supposed to add the setup.hint lines for test?

Yes.  Just update the setup.hint file as you want.  Remember that if you
add any curr, prev, or, test tag you have to add all of them.

cgf


Re: Please upload: postgresql-8.0.0cvs-1 [test]

2004-10-04 Thread Reini Urban
Christopher Faylor schrieb:
On Mon, Oct 04, 2004 at 10:03:54PM +0200, Reini Urban wrote:
Note: maintainer change.
 wget -q -O - http://xarch.tu-graz.ac.at/publ/cygwin/postgresql/get.sh | sh
Please mark this as experimental (test).
Or am I supposed to add the setup.hint lines for test?

Yes.  Just update the setup.hint file as you want.  Remember that if you
add any curr, prev, or, test tag you have to add all of them.
ok, setup.hint updated as attached. package otherwise unchanged.
wget -O - http://xarch.tu-graz.ac.at/publ/cygwin/postgresql/get.sh | sh
--
Reini Urban
http://xarch.tu-graz.ac.at/home/rurban/
sdesc: PostgreSQL Data Base Management System
ldesc: PostgreSQL is an advanced Object-Relational database management system
(DBMS) that supports almost all SQL constructs (including transactions,
subselects and user-defined types and functions). 

With contrib and full docs.
This is based on a cvs snapshot post-beta3, which fixes most 
of the cygwin issues. Only tablespace doesn't work.
category: Database
requires: ash crypt cygrunsrv cygwin libncurses7 libreadline5 libintl3 openssl zlib
version: 7.4.5-1
[prev]
version: 7.4.3-1
[test]
version: 8.0.0cvs-1

Re: Please upload: postgresql-8.0.0cvs-1 [test]

2004-10-04 Thread Jason Tishler
Reini,

On Mon, Oct 04, 2004 at 10:03:54PM +0200, Reini Urban wrote:
 I've taken over from Jason Tishler with the 8.x series with his
 permission. Thanks Jason!

You are very welcome.  Actually, I should be thanking *you*! :,)

Thanks,
Jason

-- 
PGP/GPG Key: http://www.tishler.net/jason/pubkey.asc or key servers
Fingerprint: 7A73 1405 7F2B E669 C19D  8784 1AFD E4CC ECF4 8EF6


Re: Please upload: postgresql-8.0.0cvs-1 [test]

2004-10-04 Thread Christopher Faylor
On Mon, Oct 04, 2004 at 10:34:11PM +0200, Reini Urban wrote:
Christopher Faylor schrieb:
On Mon, Oct 04, 2004 at 10:03:54PM +0200, Reini Urban wrote:
Note: maintainer change.

 wget -q -O - http://xarch.tu-graz.ac.at/publ/cygwin/postgresql/get.sh | 
 sh

Please mark this as experimental (test).
Or am I supposed to add the setup.hint lines for test?


Yes.  Just update the setup.hint file as you want.  Remember that if you
add any curr, prev, or, test tag you have to add all of them.

ok, setup.hint updated as attached. package otherwise unchanged.

wget -O - http://xarch.tu-graz.ac.at/publ/cygwin/postgresql/get.sh | sh

The setup.hint syntax is:

curr: 7.4.5-1

You don't use [prev], [curr], or [test].  That's setup.ini syntax (don't
ask me why, it's just a hysterical artifact).

In your setup.hint below, you have no curr so setup.exe would only
display a previous and a test version.  That's not what you want, I'm
sure.

cgf


Re: Please upload: postgresql-8.0.0cvs-1 [test] (gold star alert)

2004-10-04 Thread Christopher Faylor
On Mon, Oct 04, 2004 at 04:42:25PM -0400, Jason Tishler wrote:
Reini,

On Mon, Oct 04, 2004 at 10:03:54PM +0200, Reini Urban wrote:
 I've taken over from Jason Tishler with the 8.x series with his
 permission. Thanks Jason!

You are very welcome.  Actually, I should be thanking *you*! :,)

Igor, can we get two gold stars for Jason?  One for his years of
Postgres support, and another one for just being a long time contributor
to Cygwin.

And one for Reini for taking up the Postgres yoke, too, please.

cgf


Re: Please upload: postgresql-8.0.0cvs-1 [test]

2004-10-04 Thread Reini Urban
Reini Urban schrieb:
ok, setup.hint updated as attached. package otherwise unchanged.
oops, that attachment was the wrong version. sorry.
wget -O - http://xarch.tu-graz.ac.at/publ/cygwin/postgresql/get.sh | sh
sdesc: PostgreSQL Data Base Management System
ldesc: PostgreSQL is an advanced Object-Relational database management system
(DBMS) that supports almost all SQL constructs, including transactions,
subselects and user-defined types and functions.
category: Database
requires: ash crypt cygrunsrv cygwin libncurses7 libreadline5 libintl3 openssl zlib
curr: 7.4.5-1
prev: 7.4.3-1
test: 8.0.0cvs-1


Re: Please upload: postgresql-8.0.0cvs-1 [test]

2004-10-04 Thread Gerrit P. Haase
Hello Reini,

 The tablespace feature is known to fail.

What does this mean exactly?  (Link to docu should do it).


Gerrit
-- 
=^..^=




Re: Please upload: postgresql-8.0.0cvs-1 [test]

2004-10-04 Thread Reini Urban
Gerrit P. Haase schrieb:
The tablespace feature is known to fail.
What does this mean exactly?  (Link to docu should do it).
http://developer.postgresql.org/docs/postgres/catalog-pg-tablespace.html
just specify the physical layout for a table. taken from oracle.
implemented as simple symlink on directories.
to share a db across discs probably.
added just lately.
This is the expected result:
CREATE TABLESPACE badspace LOCATION '/no/such/location';
ERROR:  could not set permissions on directory /no/such/location: No 
such file or directory
-- No such tablespace
CREATE TABLE bar (i int) TABLESPACE nosuchspace;
ERROR:  tablespace nosuchspace does not exist
-- Fail, not empty
DROP TABLESPACE testspace;
ERROR:  tablespace testspace is not empty
DROP SCHEMA testschema CASCADE;
NOTICE:  drop cascades to table testschema.foo
-- Should succeed
DROP TABLESPACE testspace;

our culprit:
CREATE TABLESPACE testspace LOCATION 
'/usr/src/postgresql/postgresql-8.0.0cvs/src/test/regress/testtablespace';
+ ERROR:  could not create symbolic link 
/usr/src/postgresql/postgresql-8.0.0cvs/src/test/regress/./tmp_check/data/pg_tblspc/155183: 
Permission denied
  -- create a schema in the tablespace

It failed to hardlink a directory (probably bogus error message).
Some win32 magician changed the existing symlink code to use junctions 
lately. But I have to check the actual codepath.
Attached are the failing tests.
--
Reini Urban
http://xarch.tu-graz.ac.at/home/rurban/
*** ./expected/tablespace.out   Mon Oct  4 21:15:55 2004
--- ./results/tablespace.outMon Oct  4 21:19:59 2004
***
*** 1,34 
  -- create a tablespace we can use
  CREATE TABLESPACE testspace LOCATION 
'/usr/src/postgresql/postgresql-8.0.0cvs/src/test/regress/testtablespace';
  -- create a schema in the tablespace
  CREATE SCHEMA testschema TABLESPACE testspace;
  -- sanity check
  SELECT nspname, spcname FROM pg_catalog.pg_tablespace t, pg_catalog.pg_namespace n
where n.nsptablespace = t.oid and n.nspname = 'testschema';
nspname   |  spcname  
! +---
!  testschema | testspace
! (1 row)
  
  -- try a table
  CREATE TABLE testschema.foo (i int);
  SELECT relname, spcname FROM pg_catalog.pg_tablespace t, pg_catalog.pg_class c
  where c.reltablespace = t.oid AND c.relname = 'foo';
   relname |  spcname  
! -+---
!  foo | testspace
! (1 row)
  
  INSERT INTO testschema.foo VALUES(1);
  INSERT INTO testschema.foo VALUES(2);
  -- index
  CREATE INDEX foo_idx on testschema.foo(i);
  SELECT relname, spcname FROM pg_catalog.pg_tablespace t, pg_catalog.pg_class c
  where c.reltablespace = t.oid AND c.relname = 'foo_idx';
   relname |  spcname  
! -+---
!  foo_idx | testspace
! (1 row)
  
  -- Will fail with bad path
  CREATE TABLESPACE badspace LOCATION '/no/such/location';
--- 1,37 
  -- create a tablespace we can use
  CREATE TABLESPACE testspace LOCATION 
'/usr/src/postgresql/postgresql-8.0.0cvs/src/test/regress/testtablespace';
+ ERROR:  could not create symbolic link 
/usr/src/postgresql/postgresql-8.0.0cvs/src/test/regress/./tmp_check/data/pg_tblspc/155183:
 Permission denied
  -- create a schema in the tablespace
  CREATE SCHEMA testschema TABLESPACE testspace;
+ ERROR:  tablespace testspace does not exist
  -- sanity check
  SELECT nspname, spcname FROM pg_catalog.pg_tablespace t, pg_catalog.pg_namespace n
where n.nsptablespace = t.oid and n.nspname = 'testschema';
   nspname | spcname 
! -+-
! (0 rows)
  
  -- try a table
  CREATE TABLE testschema.foo (i int);
+ ERROR:  schema testschema does not exist
  SELECT relname, spcname FROM pg_catalog.pg_tablespace t, pg_catalog.pg_class c
  where c.reltablespace = t.oid AND c.relname = 'foo';
   relname | spcname 
! -+-
! (0 rows)
  
  INSERT INTO testschema.foo VALUES(1);
+ ERROR:  schema testschema does not exist
  INSERT INTO testschema.foo VALUES(2);
+ ERROR:  schema testschema does not exist
  -- index
  CREATE INDEX foo_idx on testschema.foo(i);
+ ERROR:  schema testschema does not exist
  SELECT relname, spcname FROM pg_catalog.pg_tablespace t, pg_catalog.pg_class c
  where c.reltablespace = t.oid AND c.relname = 'foo_idx';
   relname | spcname 
! -+-
! (0 rows)
  
  -- Will fail with bad path
  CREATE TABLESPACE badspace LOCATION '/no/such/location';
***
*** 38,45 
  ERROR:  tablespace nosuchspace does not exist
  -- Fail, not empty
  DROP TABLESPACE testspace;
! ERROR:  tablespace testspace is not empty
  DROP SCHEMA testschema CASCADE;
! NOTICE:  drop cascades to table testschema.foo
  -- Should succeed
  DROP TABLESPACE testspace;
--- 41,49 
  ERROR:  tablespace nosuchspace does not exist
  -- Fail, not empty
  DROP TABLESPACE testspace;
! ERROR:  tablespace testspace does not exist
  DROP SCHEMA testschema CASCADE;
! ERROR:  schema testschema does not exist
  -- Should succeed
  DROP TABLESPACE testspace;
+ ERROR:  tablespace 

Re: Please upload: postgresql-8.0.0cvs-1 [test] (gold star alert)

2004-10-04 Thread Reini Urban
Christopher Faylor schrieb:
And one for Reini for taking up the Postgres yoke, too, please.
Thanks, but that's kind of unfair, since building libming is MUCH harder 
than building and testing postgresql on a regular basis. (for about one 
month now). The postgresql folks are very knowledgable.

libming needs a complete autotool'izing overhaul, DLL building is 
fragile, and has much more supported language extensions than postgres. 
Even harder than php and apache.
Sigh. All this for such an unneccessary stupid little package.

A small note to future wannabe package maintainers:
  Never pick a small innocently looking package  1.0 you might need,
  when libtoolized DLL's are wanted and multiple languages
  (or just tcl) are involved.
--
Reini


Re: Please upload: postgresql-8.0.0cvs-1 [test]

2004-10-04 Thread Gerrit P. Haase
Hello Reini,

getting this when I try to build:

make[1]: Leaving directory 
`/ftproot/pub/PostgreSQL/postgresql/postgresql-8.0.0cvs/.build/contrib/dbsize'
make: *** earthdistance: No such file or directory.  Stop.
make: Entering an unknown directorymake: Leaving an unknown directorymake: *** [all] 
Error 2
make: Leaving directory 
`/ftproot/pub/PostgreSQL/postgresql/postgresql-8.0.0cvs/.build/contrib'

Any idea?

Gerrit
-- 
=^..^=




Re: Please upload: postgresql-8.0.0cvs-1 [test]

2004-10-04 Thread Christopher Faylor
On Mon, Oct 04, 2004 at 10:51:07PM +0200, Reini Urban wrote:
Reini Urban schrieb:
ok, setup.hint updated as attached. package otherwise unchanged.

oops, that attachment was the wrong version. sorry.

wget -O - http://xarch.tu-graz.ac.at/publ/cygwin/postgresql/get.sh | sh

This has been installed in the cygwin release area.

Thanks.  I appreciate being provided with the exact command needed to
download this.  It's odd, though.

I wouldn't have expected that you'd need me to fill in the root password
and my credit card number.  :-)

cgf