#18246 closed Cleanup/optimization (fixed)
Geodjango documentation improvement
| Reported by: | José Luis Patiño Andrés | Owned by: | nobody |
|---|---|---|---|
| Component: | Documentation | Version: | dev |
| Severity: | Normal | Keywords: | gis |
| Cc: | Triage Stage: | Accepted | |
| Has patch: | no | Needs documentation: | no |
| Needs tests: | no | Patch needs improvement: | no |
| Easy pickings: | no | UI/UX: | no |
Description
Hello!
I am just starting with Geodjango, so reading the documentation and learning how to set up a GIS environment in PostgreSQL.
In the 'Geodjango installation' page, in the 'Debian' section, you kindly add a shell script to set up everything in the Post-installation notes. Script is 'create_template_postgis-debian.sh'
In line 41 of the script, the if ((GEOGRAPHY)) is failing and returns the next error:
create_template_postgis-debian.sh: 41: create_template_postgis-debian.sh: GEOGRAPHY: not found
I think it will be a good idea to change this line for a more explicit comparison, in order to prevent this error:
if [ $GEOGRAPHY -eq 1 ]
That works better, and is more exact, I guess.
Hope this little tip to be useful. Thanks for your time!
Change History (3)
comment:1 by , 13 years ago
| Triage Stage: | Unreviewed → Accepted |
|---|
follow-up: 3 comment:2 by , 13 years ago
| Keywords: | gis added |
|---|---|
| Resolution: | → fixed |
| Status: | new → closed |
Thanks for the report. Fixed in aaa3382fdc35bfa5e26c029225a2eb190da66b79
comment:3 by , 13 years ago
Replying to claudep:
Thanks for the report. Fixed in aaa3382fdc35bfa5e26c029225a2eb190da66b79
Thanks to you for taking into account.
Cheers.
OK, it seems that the
((syntax is not supported by dash, for example.