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

Allow output-relevant switches to be provided in a config file #107

Closed
gravitystorm opened this issue Dec 10, 2013 · 2 comments
Closed

Allow output-relevant switches to be provided in a config file #107

gravitystorm opened this issue Dec 10, 2013 · 2 comments

Comments

@gravitystorm
Copy link
Contributor

I can split the command-line parameters into two groups - those that a given style needs to have set, and those not relevant to the style itself.

For example,

  • --extra-attributes, --multi-geometry, --keep-coastlines, --hstore might be required by a style
  • --num-processes, --unlogged, --input-reader, --cache have no influence on the output.

Styles currently need to have a readme entry stating which particular flags need to be set, otherwise the user will find a non-working system after hours of importing data. Users also need to know enough about how osm2pgsql works in order to successfully merge those options with other ones that they've read elsewhere, usually regarding optimisations.

So a better solution would be to allow the style to control the switches, ideally by extending the standard foo.style osm2pgsql config file.

@lonvia
Copy link
Collaborator

lonvia commented Feb 7, 2020

The new 'flex' output has started to move the style-relevant into the lua style file. So if you want that 'flex' is the way to go.

@gravitystorm
Copy link
Contributor Author

OK, that seems reasonable to me.

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

No branches or pull requests

2 participants