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

update from upstream #4

Open
wants to merge 154 commits into
base: master
Choose a base branch
from
Open

update from upstream #4

wants to merge 154 commits into from

Conversation

Andykmcc
Copy link

@Andykmcc Andykmcc commented Mar 6, 2023

No description provided.

iAlex97 and others added 30 commits February 24, 2022 23:07
* use gzipped csv instead of sql dumps for uk and us postcodes

* add test case for GB postcode import
Fix/gb us postcode import (#1)
Not just "nominatim import"
This enables passing NOMINATIM_DATABASE_DSN
#307 Add datrie to enable new ICU tokenizer
#245 Pass environment variables to all sudo commands
add table of contents to README.md [skip ci]
Hi,

I've found out that config.sh contains a bug while retrieving $REPLICATION_URL in order to update the database. Turns out, shell language was not respected on line 26 (spaces between [, ] and expressions). Starting the container with the pre-requisite parameters give you this warning / error message : 

`./config.sh: line 26: [https://download.geofabrik.de/europe/monaco-updates/: No such file or directory`

Simple fix here.
Second line that's erronous in bash. See #313
./config.sh: line 26: [{FILE}: No such file or directory
Point documentation to issues, closes #245
Add planet instance documentation
remove duplicated git package from install
* remove duplicated package

* support nominatim 4.1

* add NOMINATIM_TOKENIZER=icu to default env

* updated missed references to old postgresql

* test setting postgres_version in action workflows

* increase sleep time

* cleanup & references in /README.md

* remove —rm parameter from README

* README formatting [skip ci]

* Update 4.1/contrib/docker-compose.yml

Co-authored-by: Leonard Ehrenfried <[email protected]>

Co-authored-by: Leonard Ehrenfried <[email protected]>
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

Successfully merging this pull request may close these issues.