Opened 13 years ago
Closed 13 years ago
#1156 closed enhancement (fixed)
Tiger geocoder README (and unix loader suggestions)
Reported by: | forkandwait | Owned by: | robe |
---|---|---|---|
Priority: | medium | Milestone: | PostGIS 2.0.0 |
Component: | tiger geocoder | Version: | master |
Keywords: | Cc: |
Description
Attached is a slightly edited version of the README, with any major new text in CAPS.
Besides stylistic things, I broke paragraphs by 2 newlines and re-wrapped the text to be 80 character or less lines (except for some code). Now it "looks right" when you open in it emacs, and it didn't before.
I also added instructions on outputting the script to a file from psql (basically "\a \t \o foo.sh" — but look in the text for details)
Also, I added text that recommended that the generated script be edited with "set -e -u" at the top to make it debug easier (this is basically like "use strict" in Perl — otherwise the script keeps running even if wget can't find a server, for example). This change should be folded into the SQL code, by someone who knows how that works.
Attachments (1)
Change History (3)
by , 13 years ago
comment:1 by , 13 years ago
In my README I suggested that "set -e -u" be added to the generated script because it will make the script stop if there is an error.
While this is great advice, I get errors when constraints are created that already exist if I run with "set -e", and the script just blows past them otherwise. There *may* be a problem with the generated code that "set -e" is uncovering (or it might be running the loader on a totally fresh DB would work, while I am trying to restart it).
comment:2 by , 13 years ago
Resolution: | → fixed |
---|---|
Status: | new → closed |
Fixed at r7742. I also added a couple extra notes and changed the file to LF instead of carriage return so that should make it easier to read under Unix.
Fork, What is your real name?
Thanks, Regina
Updated readme