Opened 12 years ago
Closed 11 years ago
#20798 closed Cleanup/optimization (wontfix)
create_template_postgis should use -e shell parameter
| Reported by: | Mitar | Owned by: | nobody |
|---|---|---|---|
| Component: | GIS | Version: | 1.5 |
| Severity: | Normal | Keywords: | |
| Cc: | Triage Stage: | Accepted | |
| Has patch: | no | Needs documentation: | no |
| Needs tests: | no | Patch needs improvement: | no |
| Easy pickings: | no | UI/UX: | no |
Description
create_template_postgis scripts should use #!/bin/bash -e as hashbang so that the script fails if any of the commands fails. This is useful for automation (like Travis CI installation of PostGIS).
Change History (3)
comment:1 by , 12 years ago
| Triage Stage: | Unreviewed → Accepted |
|---|---|
| Type: | Uncategorized → Cleanup/optimization |
comment:2 by , 11 years ago
comment:3 by , 11 years ago
| Resolution: | → wontfix |
|---|---|
| Status: | new → closed |
Thanks for looking. Considering that this script is not used any more with Postgis 2, and the minor effect it has, I suggest to "won't fix" the issue.
Note:
See TracTickets
for help on using tickets.
After looking into it I'm not sure if it's such a good idea, because the script don't clean after itself in case of failure.
For example if a user doesn't have postgis installed, the first time the script run it will create the database template_postgis then fail to install the postgis extension in it. Then the user install postgis but the script will directly fail in subsequents runs since template_postgis will already exists. So this isn't really user friendly.