Skip to content

Commit

Permalink
Document root file workaround for docker. #82
Browse files Browse the repository at this point in the history
  • Loading branch information
dabreegster committed Apr 13, 2021
1 parent 096acd1 commit dd6bdfc
Showing 1 changed file with 4 additions and 0 deletions.
4 changes: 4 additions & 0 deletions convert_osm/src/elevation.rs
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,10 @@ pub fn add_data(map: &mut RawMap) -> Result<()> {
std::fs::create_dir_all(abstio::path_shared_input("elevation"))?;
let pwd = std::env::current_dir()?.display().to_string();
// Because elevation_lookups has so many dependencies, just depend on Docker.
// TODO This is only going to run on Linux, unless we can also build images for other OSes.
// TODO On Linux, data/input/shared/elevation files wind up being owned by root, due to how
// docker runs. For the moment, one workaround is to manually fix the owner afterwards:
// find data/ -user root -exec sudo chown $USER:$USER '{}' \;
let status = Command::new("docker")
.arg("run")
// Bind the input directory to the temporary place we just created
Expand Down

0 comments on commit dd6bdfc

Please sign in to comment.