Skip to content
This repository has been archived by the owner on Sep 30, 2024. It is now read-only.

The ParsePatients path needs to be updated #8

Open
ymnliu opened this issue Sep 25, 2020 · 2 comments
Open

The ParsePatients path needs to be updated #8

ymnliu opened this issue Sep 25, 2020 · 2 comments

Comments

@ymnliu
Copy link

ymnliu commented Sep 25, 2020 •
edited
Loading

The build path of bazel build cc:ParsePatients should be updated as bazel build //cc/google/fhir_examples:ParsePatients in the README.md.

The thing is similar for bazel-bin/ParsePatients $WORKSPACE which I found eventually working as:

bazel-bin/cc/google/fhir_examples/ParsePatients.runfiles/com_google_fhir_examples/cc/google/fhir_examples/ParsePatients $WORKSPACE
nickgeorge added a commit that referenced this issue Sep 25, 2020
@nickgeorge
Fixes cc build path in fhir-examples//README.md per #8
742efb9 
PiperOrigin-RevId: 333668157
@nickgeorge
Copy link
Contributor

Great catch! I really appreciate it. Fixed in 742efb9

@ymnliu
Copy link
Author

ymnliu commented Sep 25, 2020 •
edited
Loading

Thank you for the prompt update, @nickgeorge. I found the second command working as bazel-bin/cc/google/fhir_examples/ParasePatients $WORKSPACE on my MacOS Majave (10.14.6). It did not work as ::ParsePatient to me but could be just a mac issue.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@nickgeorge @ymnliu