Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

do "resolve_overlap_and_gap-pre.pl" failed, '.' is no longer in @INC #6

Closed
strk opened this issue Apr 20, 2020 · 6 comments
Closed
Assignees

Comments

@strk
Copy link
Collaborator

strk commented Apr 20, 2020

Output of make check on my system:

PostgreSQL 9.6.9 on x86_64-pc-linux-gnu, compiled by gcc (Ubuntu 7.2.0-8ubuntu3.2) 7.2.0, 64-bit
  Postgis 2.5.4 - r17978 - 2020-04-20 15:04:07
  scripts 2.5.4 r17978
  GEOS: 3.9.0dev-CAPI-1.14.0
  PROJ: Rel. 7.0.0, March 1st, 2020

Running tests

do "resolve_overlap_and_gap-pre.pl" failed, '.' is no longer in @INC; did you mean do "./resolve_overlap_and_gap-pre.pl"? at ./run_test.pl line 617.
 resolve_overlap_and_gap ... ok 
 uninstall .. ok (4581)

Run tests: 2
Failed: 0

Two concerns:

  1. The "failed" message (why it fails ? should we fix?)
  2. The final report of 0 failures (did we not have a failure?)

It looks to me that the run_test.pl script was copied from PostGIS so possibly the second concern should be addressed upstream (handle that failure from within run_test.pl). The first concern most probably belongs here.

@strk strk self-assigned this Apr 20, 2020
@strk
Copy link
Collaborator Author

strk commented Apr 20, 2020

I've updated original submission of this ticket, for those receiving emails upon ticket creation (I dubt github sends another mail for the correction).

@strk
Copy link
Collaborator Author

strk commented Apr 20, 2020

I've pushed a fix for the failure (first issue) as part of PR #5

@strk
Copy link
Collaborator Author

strk commented Apr 20, 2020

As per second issue, it looks like a newer version of {{{ run_test.pl }}} from upstream PostGIS had that one fixed already, but upgrading would make adaptation slightly harder so not sure it's worth the trouble at the moment.

@larsop
Copy link
Owner

larsop commented Apr 20, 2020

Since I use commit of subtrans and Postgres procedures it will not run on Postgres 9.6. I have mostly tested on Postgres 11 and 12.

@larsop larsop closed this as completed in bd8c86d Apr 21, 2020
@strk
Copy link
Collaborator Author

strk commented Apr 24, 2020

Since I use commit of subtrans and Postgres procedures it will not run on Postgres 9.6. I have mostly tested on Postgres 11 and 12.

You mean tests are silently skip if I'm running 9.6 ? Becuase you can see something did run.
Shall I file another ticket to tackle that "all-success" response ?

@strk
Copy link
Collaborator Author

strk commented Apr 24, 2020

Ok, with this fix I actually see the test fails, correctly:

+ERROR: syntax error at or near "CALL" at character 1

So no need to another issue for that. Will just install and switch to PostgreSQL 12

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants