Re: 7.3b5 contrib compile problem

I saw this when compiling 7.3b4 as well and also with 7.3b5

cd contrib
make
...
make[1]: Leaving directory
`/nfs/visor/u/software/postgres/postgresql-7.3b5/contrib/rtree_gist'
make[1]: Entering directory
`/nfs/visor/u/software/postgres/postgresql-7.3b5/contrib/seg'
sed 's,MODULE_PATHNAME,$libdir/seg,g' seg.sql.in >seg.sql
gcc -O2 -g -Wall -Wmissing-prototypes -Wmissing-declarations -fpic -I.
-I../../src/include -I/usr/local/include -c -o seg.o seg.c
bison -y -d -p seg_yy segparse.y
mv -f y.tab.c segparse.c
mv -f y.tab.h segparse.h
/usr/bin/flex -Pseg_yy -o'segscan.c' segscan.l
gcc -O2 -g -Wall -Wmissing-prototypes -Wmissing-declarations -fpic -I.
-I../../src/include -I/usr/local/include -c -o segparse.o segparse.c
In file included from segscan.l:8,
from segparse.y:182:
segparse.h:2: redefinition of `struct BND'
segparse.h:8: conflicting types for `YYSTYPE'
segparse.y:48: previous declaration of `YYSTYPE'
segparse.h:15: conflicting types for `seg_yylval'
/usr/lib/bison.simple:138: previous declaration of `seg_yylval'
make[1]: *** [segparse.o] Error 1
make[1]: Leaving directory `/nfs/visor/u/software/postgres/postgresql-7.3b5/contrib/seg'
make: *** [all] Error 2

Any ideas?

Thanks,

--
Laurette Cisneros
The Database Group
(510) 420-3137
NextBus Information Systems, Inc.
www.nextbus.com
----------------------------------
My other vehicle is my imagination.
- bumper sticker

Wow, that is strange. I have bison 1.75 here and it compiles fine.
What version of bison do you have?

bison --version

---------------------------------------------------------------------------

Laurette Cisneros wrote:
>
> I saw this when compiling 7.3b4 as well and also with 7.3b5
>
> cd contrib
> make
> ...
> make[1]: Leaving directory
> `/nfs/visor/u/software/postgres/postgresql-7.3b5/contrib/rtree_gist'
> make[1]: Entering directory
> `/nfs/visor/u/software/postgres/postgresql-7.3b5/contrib/seg'
> sed 's,MODULE_PATHNAME,$libdir/seg,g' seg.sql.in >seg.sql
> gcc -O2 -g -Wall -Wmissing-prototypes -Wmissing-declarations -fpic -I.
> -I../../src/include -I/usr/local/include -c -o seg.o seg.c
> bison -y -d -p seg_yy segparse.y
> mv -f y.tab.c segparse.c
> mv -f y.tab.h segparse.h
> /usr/bin/flex -Pseg_yy -o'segscan.c' segscan.l
> gcc -O2 -g -Wall -Wmissing-prototypes -Wmissing-declarations -fpic -I.
> -I../../src/include -I/usr/local/include -c -o segparse.o segparse.c
> In file included from segscan.l:8,
> from segparse.y:182:
> segparse.h:2: redefinition of `struct BND'
> segparse.h:8: conflicting types for `YYSTYPE'
> segparse.y:48: previous declaration of `YYSTYPE'
> segparse.h:15: conflicting types for `seg_yylval'
> /usr/lib/bison.simple:138: previous declaration of `seg_yylval'
> make[1]: *** [segparse.o] Error 1
> make[1]: Leaving directory `/nfs/visor/u/software/postgres/postgresql-7.3b5/contrib/seg'
> make: *** [all] Error 2
>
> Any ideas?
>
> Thanks,
>
> --
> Laurette Cisneros
> The Database Group
> (510) 420-3137
> NextBus Information Systems, Inc.
> www.nextbus.com
> ----------------------------------
> My other vehicle is my imagination.
> - bumper sticker
>
>
> ---------------------------(end of broadcast)---------------------------
> TIP 1: subscribe and unsubscribe commands go to majordomo(at)postgresql(dot)org
>

--
Bruce Momjian | http://candle.pha.pa.us
pgman(at)candle(dot)pha(dot)pa(dot)us | (610) 359-1001
+ If your life is a hard drive, | 13 Roberts Road
+ Christ can be your backup. | Newtown Square, Pennsylvania 19073

bison --version
GNU Bison version 1.28

Should I update it?

This just started with 7.3b4.

Thanks,

L.
On Wed, 6 Nov 2002, Bruce Momjian wrote:

>
> Wow, that is strange. I have bison 1.75 here and it compiles fine.
> What version of bison do you have?
>
> bison --version
>
> ---------------------------------------------------------------------------
>
> Laurette Cisneros wrote:
> >
> > I saw this when compiling 7.3b4 as well and also with 7.3b5
> >
> > cd contrib
> > make
> > ...
> > make[1]: Leaving directory
> > `/nfs/visor/u/software/postgres/postgresql-7.3b5/contrib/rtree_gist'
> > make[1]: Entering directory
> > `/nfs/visor/u/software/postgres/postgresql-7.3b5/contrib/seg'
> > sed 's,MODULE_PATHNAME,$libdir/seg,g' seg.sql.in >seg.sql
> > gcc -O2 -g -Wall -Wmissing-prototypes -Wmissing-declarations -fpic -I.
> > -I../../src/include -I/usr/local/include -c -o seg.o seg.c
> > bison -y -d -p seg_yy segparse.y
> > mv -f y.tab.c segparse.c
> > mv -f y.tab.h segparse.h
> > /usr/bin/flex -Pseg_yy -o'segscan.c' segscan.l
> > gcc -O2 -g -Wall -Wmissing-prototypes -Wmissing-declarations -fpic -I.
> > -I../../src/include -I/usr/local/include -c -o segparse.o segparse.c
> > In file included from segscan.l:8,
> > from segparse.y:182:
> > segparse.h:2: redefinition of `struct BND'
> > segparse.h:8: conflicting types for `YYSTYPE'
> > segparse.y:48: previous declaration of `YYSTYPE'
> > segparse.h:15: conflicting types for `seg_yylval'
> > /usr/lib/bison.simple:138: previous declaration of `seg_yylval'
> > make[1]: *** [segparse.o] Error 1
> > make[1]: Leaving directory `/nfs/visor/u/software/postgres/postgresql-7.3b5/contrib/seg'
> > make: *** [all] Error 2
> >
> > Any ideas?
> >
> > Thanks,
> >
> > --
> > Laurette Cisneros
> > The Database Group
> > (510) 420-3137
> > NextBus Information Systems, Inc.
> > www.nextbus.com
> > ----------------------------------
> > My other vehicle is my imagination.
> > - bumper sticker
> >
> >
> > ---------------------------(end of broadcast)---------------------------
> > TIP 1: subscribe and unsubscribe commands go to majordomo(at)postgresql(dot)org
> >
>
>

--
Laurette Cisneros
The Database Group
(510) 420-3137
NextBus Information Systems, Inc.
www.nextbus.com
----------------------------------
My other vehicle is my imagination.
- bumper sticker

I think bison 1.75 will fix it. I am just not sure why earlier releases
fail that way.

---------------------------------------------------------------------------

Laurette Cisneros wrote:
> bison --version
> GNU Bison version 1.28
>
> Should I update it?
>
> This just started with 7.3b4.
>
> Thanks,
>
> L.
> On Wed, 6 Nov 2002, Bruce Momjian wrote:
>
> >
> > Wow, that is strange. I have bison 1.75 here and it compiles fine.
> > What version of bison do you have?
> >
> > bison --version
> >
> > ---------------------------------------------------------------------------
> >
> > Laurette Cisneros wrote:
> > >
> > > I saw this when compiling 7.3b4 as well and also with 7.3b5
> > >
> > > cd contrib
> > > make
> > > ...
> > > make[1]: Leaving directory
> > > `/nfs/visor/u/software/postgres/postgresql-7.3b5/contrib/rtree_gist'
> > > make[1]: Entering directory
> > > `/nfs/visor/u/software/postgres/postgresql-7.3b5/contrib/seg'
> > > sed 's,MODULE_PATHNAME,$libdir/seg,g' seg.sql.in >seg.sql
> > > gcc -O2 -g -Wall -Wmissing-prototypes -Wmissing-declarations -fpic -I.
> > > -I../../src/include -I/usr/local/include -c -o seg.o seg.c
> > > bison -y -d -p seg_yy segparse.y
> > > mv -f y.tab.c segparse.c
> > > mv -f y.tab.h segparse.h
> > > /usr/bin/flex -Pseg_yy -o'segscan.c' segscan.l
> > > gcc -O2 -g -Wall -Wmissing-prototypes -Wmissing-declarations -fpic -I.
> > > -I../../src/include -I/usr/local/include -c -o segparse.o segparse.c
> > > In file included from segscan.l:8,
> > > from segparse.y:182:
> > > segparse.h:2: redefinition of `struct BND'
> > > segparse.h:8: conflicting types for `YYSTYPE'
> > > segparse.y:48: previous declaration of `YYSTYPE'
> > > segparse.h:15: conflicting types for `seg_yylval'
> > > /usr/lib/bison.simple:138: previous declaration of `seg_yylval'
> > > make[1]: *** [segparse.o] Error 1
> > > make[1]: Leaving directory `/nfs/visor/u/software/postgres/postgresql-7.3b5/contrib/seg'
> > > make: *** [all] Error 2
> > >
> > > Any ideas?
> > >
> > > Thanks,
> > >
> > > --
> > > Laurette Cisneros
> > > The Database Group
> > > (510) 420-3137
> > > NextBus Information Systems, Inc.
> > > www.nextbus.com
> > > ----------------------------------
> > > My other vehicle is my imagination.
> > > - bumper sticker
> > >
> > >
> > > ---------------------------(end of broadcast)---------------------------
> > > TIP 1: subscribe and unsubscribe commands go to majordomo(at)postgresql(dot)org
> > >
> >
> >
>
> --
> Laurette Cisneros
> The Database Group
> (510) 420-3137
> NextBus Information Systems, Inc.
> www.nextbus.com
> ----------------------------------
> My other vehicle is my imagination.
> - bumper sticker
>
>

--
Bruce Momjian | http://candle.pha.pa.us
pgman(at)candle(dot)pha(dot)pa(dot)us | (610) 359-1001
+ If your life is a hard drive, | 13 Roberts Road
+ Christ can be your backup. | Newtown Square, Pennsylvania 19073

Oh, I see it now, I think there was a change on November 1st to rule
generation, but I can't see how that would cause your problem.

If you 'patch -R' the attached patch, does it fix the problem?

---------------------------------------------------------------------------

Bruce Momjian wrote:
>
> I think bison 1.75 will fix it. I am just not sure why earlier releases
> fail that way.
>
> ---------------------------------------------------------------------------
>
> Laurette Cisneros wrote:
> > bison --version
> > GNU Bison version 1.28
> >
> > Should I update it?
> >
> > This just started with 7.3b4.
> >
> > Thanks,
> >
> > L.
> > On Wed, 6 Nov 2002, Bruce Momjian wrote:
> >
> > >
> > > Wow, that is strange. I have bison 1.75 here and it compiles fine.
> > > What version of bison do you have?
> > >
> > > bison --version
> > >
> > > ---------------------------------------------------------------------------
> > >
> > > Laurette Cisneros wrote:
> > > >
> > > > I saw this when compiling 7.3b4 as well and also with 7.3b5
> > > >
> > > > cd contrib
> > > > make
> > > > ...
> > > > make[1]: Leaving directory
> > > > `/nfs/visor/u/software/postgres/postgresql-7.3b5/contrib/rtree_gist'
> > > > make[1]: Entering directory
> > > > `/nfs/visor/u/software/postgres/postgresql-7.3b5/contrib/seg'
> > > > sed 's,MODULE_PATHNAME,$libdir/seg,g' seg.sql.in >seg.sql
> > > > gcc -O2 -g -Wall -Wmissing-prototypes -Wmissing-declarations -fpic -I.
> > > > -I../../src/include -I/usr/local/include -c -o seg.o seg.c
> > > > bison -y -d -p seg_yy segparse.y
> > > > mv -f y.tab.c segparse.c
> > > > mv -f y.tab.h segparse.h
> > > > /usr/bin/flex -Pseg_yy -o'segscan.c' segscan.l
> > > > gcc -O2 -g -Wall -Wmissing-prototypes -Wmissing-declarations -fpic -I.
> > > > -I../../src/include -I/usr/local/include -c -o segparse.o segparse.c
> > > > In file included from segscan.l:8,
> > > > from segparse.y:182:
> > > > segparse.h:2: redefinition of `struct BND'
> > > > segparse.h:8: conflicting types for `YYSTYPE'
> > > > segparse.y:48: previous declaration of `YYSTYPE'
> > > > segparse.h:15: conflicting types for `seg_yylval'
> > > > /usr/lib/bison.simple:138: previous declaration of `seg_yylval'
> > > > make[1]: *** [segparse.o] Error 1
> > > > make[1]: Leaving directory `/nfs/visor/u/software/postgres/postgresql-7.3b5/contrib/seg'
> > > > make: *** [all] Error 2
> > > >
> > > > Any ideas?
> > > >
> > > > Thanks,
> > > >
> > > > --
> > > > Laurette Cisneros
> > > > The Database Group
> > > > (510) 420-3137
> > > > NextBus Information Systems, Inc.
> > > > www.nextbus.com
> > > > ----------------------------------
> > > > My other vehicle is my imagination.
> > > > - bumper sticker
> > > >
> > > >
> > > > ---------------------------(end of broadcast)---------------------------
> > > > TIP 1: subscribe and unsubscribe commands go to majordomo(at)postgresql(dot)org
> > > >
> > >
> > >
> >
> > --
> > Laurette Cisneros
> > The Database Group
> > (510) 420-3137
> > NextBus Information Systems, Inc.
> > www.nextbus.com
> > ----------------------------------
> > My other vehicle is my imagination.
> > - bumper sticker
> >
> >
>
> --
> Bruce Momjian | http://candle.pha.pa.us
> pgman(at)candle(dot)pha(dot)pa(dot)us | (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
>

--
Bruce Momjian | http://candle.pha.pa.us
pgman(at)candle(dot)pha(dot)pa(dot)us | (610) 359-1001
+ If your life is a hard drive, | 13 Roberts Road
+ Christ can be your backup. | Newtown Square, Pennsylvania 19073

Laurette Cisneros <laurette(at)nextbus(dot)com> writes:
> bison --version
> GNU Bison version 1.28

> Should I update it?

Yes.

It's interesting that bison 1.28's output is sufficiently different to
cause a problem, but we are not going to worry about supporting use of
old bisons.

regards, tom lane

> Laurette Cisneros <laurette(at)nextbus(dot)com> writes:
> > bison --version
> > GNU Bison version 1.28
>
> > Should I update it?
>
> Yes.
>
> It's interesting that bison 1.28's output is sufficiently different to
> cause a problem, but we are not going to worry about supporting use of
> old bisons.

Do I have to install the CVS version of Bison to get the new compile to
work?

Chris

"Christopher Kings-Lynne" <chriskl(at)familyhealth(dot)com(dot)au> writes:
> Do I have to install the CVS version of Bison to get the new compile to
> work?

No, you can use their current release, 1.75. (Reportedly 1.50 works
too, but I never tried it.)

regards, tom lane

Tom Lane wrote:
> Laurette Cisneros <laurette(at)nextbus(dot)com> writes:
> > bison --version
> > GNU Bison version 1.28
>
> > Should I update it?
>
> Yes.
>
> It's interesting that bison 1.28's output is sufficiently different to
> cause a problem, but we are not going to worry about supporting use of
> old bisons.

Just to clarify, do the tarballs for /contrib/seg have the pre-processed
bison output, or are people required to have more current bisons to
compile the code?

--
Bruce Momjian | http://candle.pha.pa.us
pgman(at)candle(dot)pha(dot)pa(dot)us | (610) 359-1001
+ If your life is a hard drive, | 13 Roberts Road
+ Christ can be your backup. | Newtown Square, Pennsylvania 19073

Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us> writes:
> Just to clarify, do the tarballs for /contrib/seg have the pre-processed
> bison output, or are people required to have more current bisons to
> compile the code?

AFAIK we do not provide prebuilt bison or flex output for any of the
contrib modules.

regards, tom lane

I said:
>> It's interesting that bison 1.28's output is sufficiently different to
>> cause a problem, but we are not going to worry about supporting use of
>> old bisons.

Well, it turned out to be reasonably easy to fix this, so I did. It
seems that bison 1.28 generates a .h file that cannot be included into
the .c file it generates :-(. Experimentation shows this is fixed in
bison 1.35, possibly earlier; but it's easy enough to just not include
the .h file.

regards, tom lane

FYI: I easily updated to 1.75 and all is well.

Thanks for all the help,

L.
On Thu, 7 Nov 2002, Tom Lane wrote:

> I said:
> >> It's interesting that bison 1.28's output is sufficiently different to
> >> cause a problem, but we are not going to worry about supporting use of
> >> old bisons.
>
> Well, it turned out to be reasonably easy to fix this, so I did. It
> seems that bison 1.28 generates a .h file that cannot be included into
> the .c file it generates :-(. Experimentation shows this is fixed in
> bison 1.35, possibly earlier; but it's easy enough to just not include
> the .h file.
>
> regards, tom lane
>

--
Laurette Cisneros
The Database Group
(510) 420-3137
NextBus Information Systems, Inc.
www.nextbus.com
----------------------------------
My other vehicle is my imagination.
- bumper sticker