SEO
vBulletin Search Engine Optimization
| |||||||
| Register | FAQ | Members List | Calendar | Search | Today's Posts | Mark Forums Read |
04-12-2008, 08:11 AM
| ||||
| ||||
 Buildfarm failures en masse Seems the change of the XML messages now causes more or less every buildfarm member to fail the XML test. Meaning all the ones that don't have libxml, but that seems to be the vast majority... Looks like a missing update to regression tests? //Magnus ---------------------------(end of broadcast)--------------------------- TIP 2: Don't 'kill -9' the postmaster      |
|
04-12-2008, 08:11 AM
| |||
| |||
| Re: Buildfarm failures en masse Magnus Hagander wrote: > Seems the change of the XML messages now causes more or less every > buildfarm member to fail the XML test. Meaning all the ones that don't have > libxml, but that seems to be the vast majority... > > Looks like a missing update to regression tests? I don't think the new wording complies with the message guidelines anyway, so my vote goes to revert the wording change and reach a consensus about it before committing anything else. I think this should be ERROR: unsupported XML feature DETAIL: The function foo requires libxml2 support. HINT: You need to re-compile PostgreSQL with --enable-whatever-xml. -- Alvaro Herrera http://www.CommandPrompt.com/ The PostgreSQL Company - Command Prompt, Inc. ---------------------------(end of broadcast)--------------------------- TIP 4: Have you searched our list archives? http://archives.postgresql.org |
|
04-12-2008, 08:11 AM
| |||
| |||
| Re: Buildfarm failures en masse Alvaro Herrera wrote: > Magnus Hagander wrote: > > Seems the change of the XML messages now causes more or less every > > buildfarm member to fail the XML test. Meaning all the ones that don't have > > libxml, but that seems to be the vast majority... > > > > Looks like a missing update to regression tests? > > I don't think the new wording complies with the message guidelines > anyway, so my vote goes to revert the wording change and reach a > consensus about it before committing anything else. > > I think this should be > > ERROR: unsupported XML feature > DETAIL: The function foo requires libxml2 support. > HINT: You need to re-compile PostgreSQL with --enable-whatever-xml. Yes, I like your wording better. I will make the updates to the code and regression tests. -- Bruce Momjian <bruce@momjian.us> http://momjian.us EnterpriseDB http://www.enterprisedb.com + If your life is a hard drive, Christ can be your backup. + ---------------------------(end of broadcast)--------------------------- TIP 1: if posting/reading through Usenet, please send an appropriate subscribe-nomail command to majordomo@postgresql.org so that your message can get through to the mailing list cleanly |
|
04-12-2008, 08:11 AM
| ||||
| ||||
| Re: Buildfarm failures en masse Bruce Momjian wrote: > Alvaro Herrera wrote: > > Magnus Hagander wrote: > > > Seems the change of the XML messages now causes more or less every > > > buildfarm member to fail the XML test. Meaning all the ones that don't have > > > libxml, but that seems to be the vast majority... > > > > > > Looks like a missing update to regression tests? > > > > I don't think the new wording complies with the message guidelines > > anyway, so my vote goes to revert the wording change and reach a > > consensus about it before committing anything else. > > > > I think this should be > > > > ERROR: unsupported XML feature > > DETAIL: The function foo requires libxml2 support. > > HINT: You need to re-compile PostgreSQL with --enable-whatever-xml. > > Yes, I like your wording better. I will make the updates to the code > and regression tests. OK, new wording: ERROR: unsupported XML feature DETAIL: This functionality requires libxml support. HINT: You need to re-compile PostgreSQL using --with-libxml. The error is all in one macro, so it has to be generic and not mention the specific function, or xml data type. Regression tests updated. -- Bruce Momjian <bruce@momjian.us> http://momjian.us EnterpriseDB http://www.enterprisedb.com + If your life is a hard drive, Christ can be your backup. + ---------------------------(end of broadcast)--------------------------- TIP 4: Have you searched our list archives? http://archives.postgresql.org |
 |
« Last minute mini-proposal (I know, I know) for PQexecf()
|
Re: pg_index updates and SI invalidation »
| Thread Tools | |
| Display Modes | |
Linear Mode
|
|
All times are GMT. The time now is 11:43 PM.
