Thread
-
Re: [HACKERS] Re: [SQL] plpgsql error
Tom Lane <tgl@sss.pgh.pa.us> — 1999-05-11T23:26:15Z
Brook Milligan <brook@trillium.NMSU.Edu> writes: > Isn't the correct solution to have the Makefile contain a rule that > creates the file from a template (e.g., with sed -e > 's/@xxx@/${xxx}/g')? That way make resolves the variable references > and you needn't worry about it. (after further thought...) Oh, right, I see what you're saying: don't generate mklang.sql in configure at all, but let pl/plpgsql/src/Makefile be responsible for it. Yeah, that'd be a cleaner solution. However, what I just committed works ;-). If you feel like improving it, be my guest; I have other items on the to-do list... regards, tom lane -
Re: [HACKERS] Re: [SQL] plpgsql error
Jan Wieck <jwieck@debis.com> — 1999-05-12T10:37:24Z
> > Brook Milligan <brook@trillium.NMSU.Edu> writes: > > Isn't the correct solution to have the Makefile contain a rule that > > creates the file from a template (e.g., with sed -e > > 's/@xxx@/${xxx}/g')? That way make resolves the variable references > > and you needn't worry about it. > > (after further thought...) Oh, right, I see what you're saying: don't > generate mklang.sql in configure at all, but let pl/plpgsql/src/Makefile > be responsible for it. Yeah, that'd be a cleaner solution. However, > what I just committed works ;-). If you feel like improving it, be > my guest; I have other items on the to-do list... I've just committed a little change to initdb and it's Makefile. The initdb Makefile now expands __DLSUFFIX__ into it and initdb uses $PGLIB/plpgsql__DLSUFFIX__ to test if it is there and then runs the appropriate queries against template1. Same for PL/Tcl. If anyone agrees we can get rid of these mklang.sql scripts totally. Jan -- #======================================================================# # It's easier to get forgiveness for being wrong than for being right. # # Let's break this rule - forgive me. # #======================================== jwieck@debis.com (Jan Wieck) # -
Re: [HACKERS] Re: [SQL] plpgsql error
Bruce Momjian <maillist@candle.pha.pa.us> — 1999-05-12T12:50:17Z
> > > > Brook Milligan <brook@trillium.NMSU.Edu> writes: > > > Isn't the correct solution to have the Makefile contain a rule that > > > creates the file from a template (e.g., with sed -e > > > 's/@xxx@/${xxx}/g')? That way make resolves the variable references > > > and you needn't worry about it. > > > > (after further thought...) Oh, right, I see what you're saying: don't > > generate mklang.sql in configure at all, but let pl/plpgsql/src/Makefile > > be responsible for it. Yeah, that'd be a cleaner solution. However, > > what I just committed works ;-). If you feel like improving it, be > > my guest; I have other items on the to-do list... > > I've just committed a little change to initdb and it's > Makefile. The initdb Makefile now expands __DLSUFFIX__ into > it and initdb uses $PGLIB/plpgsql__DLSUFFIX__ to test if it > is there and then runs the appropriate queries against > template1. Same for PL/Tcl. > > If anyone agrees we can get rid of these mklang.sql scripts > totally. Sure. -- Bruce Momjian | http://www.op.net/~candle maillist@candle.pha.pa.us | (610) 853-3000 + If your life is a hard drive, | 830 Blythe Avenue + Christ can be your backup. | Drexel Hill, Pennsylvania 19026