Ticket #819 (closed defect: fixed)

Opened 2 years ago

Last modified 15 months ago

topology regression testsuite: localized messages raise false positives

Reported by: aperi2007 Owned by: strk
Priority: medium Milestone: PostGIS 2.0.0
Component: topology Version: trunk
Keywords: Cc:

Description

Hi, compiling the trunk r I see an error in make check of createtopogeom. Attach the logs.

Attachments

pgis_reg_13065.zip Download (1.5 KB) - added by aperi2007 2 years ago.
log errors
pgis_reg_24275.zip Download (1.5 KB) - added by aperi2007 2 years ago.
log erorr after make clean all check
pgis_reg_19659.zip Download (1.5 KB) - added by aperi2007 2 years ago.
log error with lang=C
error_log.zip Download (4.7 KB) - added by aperi2007 2 years ago.
two error log files

Change History

Changed 2 years ago by aperi2007

log errors

follow-up: ↓ 2   Changed 2 years ago by strk

Please try 'make clean all check' from top-level dir.

Changed 2 years ago by aperi2007

log erorr after make clean all check

in reply to: ↑ 1   Changed 2 years ago by aperi2007

Replying to strk:

Please try 'make clean all check' from top-level dir.

I try, but still error

log attached

follow-up: ↓ 5   Changed 2 years ago by strk

Ah! localization problem:

-PL/pgSQL function "createtopogeom" line 91 at EXECUTE statement
+funzione PL/pgSQL "createtopogeom" linea 91 a istruzione EXECUTE

Weird, as the run_test script sets LC_ALL=C Do you have LANG defined ? Or some other localization variable ?

  Changed 2 years ago by strk

It's weird because the db creation log does speak english instead:

psql:/home/postgres/software/postgis/postgis-2.0.0SVN/regress/../topology/topology.sql:3019: NOTICE:  aggregate topology.topoelementarray_agg(topology.topoelement) does not exist, skipping

in reply to: ↑ 3   Changed 2 years ago by aperi2007

Replying to strk:

Ah! localization problem: {{{ -PL/pgSQL function "createtopogeom" line 91 at EXECUTE statement +funzione PL/pgSQL "createtopogeom" linea 91 a istruzione EXECUTE }}} Weird, as the run_test script sets LC_ALL=C Do you have LANG defined ? Or some other localization variable ?

yes, in the shell there is LANG=it_IT.UTF-8

  Changed 2 years ago by strk

can you please try setting LANG=C ?

  Changed 2 years ago by aperi2007

I try setting export LANG=C in the shell. After start postgres db and run make clean all check, always in the same shell. But still the error is here. log attach.

Changed 2 years ago by aperi2007

log error with lang=C

follow-up: ↓ 9   Changed 2 years ago by strk

Try running: $ env | grep it_IT and paste the result

in reply to: ↑ 8   Changed 2 years ago by aperi2007

Replying to strk:

Try running: $ env | grep it_IT and paste the result

Here the result.

postgres@deb:~$ env | grep it_IT LANG=it_IT.UTF-8 postgres@deb:~$

  Changed 2 years ago by strk

There's nothing else then. Please try with r6786

  Changed 2 years ago by strk

  • status changed from new to closed
  • resolution set to fixed

Assuming fixed ...

  Changed 2 years ago by aperi2007

  • status changed from closed to reopened
  • resolution fixed deleted

Hi, I'm having still this error I don't know if it is a false positive. I guess it was never disappear.

I add my log error.

Changed 2 years ago by aperi2007

two error log files

  Changed 2 years ago by aperi2007

I don't sure to understand what error was issuing. I see this error message.

 regress/createtopogeom. failed (diff expected obtained: /tmp/pgis_reg_9628/test_16_diff)
Run tests: 21
Failed: 1
make[2]: *** [check] Error 1
make[2]: Leaving directory `/home/postgres/software/postgis/postgis-2.0.0SVN/topology/test'
make[1]: *** [check] Error 2
make[1]: Leaving directory `/home/postgres/software/postgis/postgis-2.0.0SVN/topology'
make: *** [check] Error 1

But the logs generated are two "pgis_reg_5400 " and "pgis_reg_9628" I put both in the zip added.

  Changed 2 years ago by strk

checked, but it's still a localization issue. Does: env | grep it return anything else ?

  Changed 23 months ago by strk

  • summary changed from topology: error on regress of createtopogeom to topology regression testsuite: localized messages raise false positives

Please consider appending RUNTESTFLAGS=-v to your 'make check' call next time:

make check RUNESTFLAGS=-v

That way you would get errors inline and avoid attachements.

  Changed 17 months ago by strk

Andrea: could you please try with r8555 ?

  Changed 15 months ago by strk

r9088 may have made this even stronger. Andrea: chances you can test again?

  Changed 15 months ago by aperi2007

  • status changed from reopened to closed
  • resolution set to fixed

I try it.

It work for me.

close the ticket.

Note: See TracTickets for help on using tickets.