Opened 13 years ago
Closed 13 years ago
#1716 closed defect (fixed)
SVN snapshot tarball lacks postgis_svn_revision.h
| Reported by: | strk | Owned by: | chodgson |
|---|---|---|---|
| Priority: | critical | Milestone: | Management 1.0 |
| Component: | management | Version: | master |
| Keywords: | Cc: |
Description
postgis_svn_revision.h should be in the tarball, it is not in the SVN snapshot (it is in the beta4 download).
Change History (5)
comment:1 by , 13 years ago
| Milestone: | PostGIS 2.0.0 → PostGIS 2.0.1 |
|---|
comment:2 by , 13 years ago
| Milestone: | PostGIS 2.0.1 → PostGIS 2.0.0 |
|---|---|
| Priority: | medium → critical |
comment:3 by , 13 years ago
| Component: | postgis → management |
|---|---|
| Owner: | changed from to |
This is also about management as it's the packaging procedure used. make_dist should do fine.
comment:4 by , 13 years ago
| Milestone: | PostGIS 2.0.0 → Management 1.0 |
|---|
comment:5 by , 13 years ago
| Resolution: | → fixed |
|---|---|
| Status: | new → closed |
Hudson can't use make_dist.sh because (1) make_dist.sh doesn't really support SVN revisions (it still has the word "cvs" in several places) and (2) Hudson needs a special ./configure line that points to all the right versions of the pre-reqs. So I just copied the perl command that makes the postgis_svn_revision.h into the build script used by Hudson (it's otherwise quite similar to make_dist.sh, perhaps one rainy day I would consider consolidating them). The new snapshot has the svn revision header included.
Note that this bug breaks postgis_full_version() reporting of what needs upgrade, so it's pretty critical