-
Notifications
You must be signed in to change notification settings - Fork 158
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
Trouble building the fast pair library #3042
Comments
When attempting to build all the targets in fastpair package I get this output. I ran Please advise on how to proceed. Thanks |
I attempted to build it on WSL Fedora. I received a different error but a similar issue, with reference to non-existing source.
|
I am able to build the connections library for Linux however, by following the README.
|
I made more progress by removing some referenced but un-resolvable source in the BUILD files, the diff is below. Still isn't building, stuck on GenProto command failure.
|
Hi @Thomas-Campion-Bose, it looks like the imports in the fastpair proto files aren’t correct. Try removing “third_party/nearby” from the beginning of the imports. Unfortunately, we don’t have any GitHub actions for testing the fastpair build, so the GitHub code is prone to breaking. If you are able to get this building successfully, let me know and I can make the necessary changes and add a test to keep this package more stable in the future :) |
Hi @bourdakos1, I appreciate the prompt response. I think a few libraries/tests within fast pair are broken. I have documented a list of targets below that require non-trivial fixes (i.e. removal of auth manager, but continued usage). I have attached the output from the failing builds along with the current diff I have on my local branch.
diff_output.txt |
Link to my fork: https://github.com/Thomas-Campion-Bose/nearby.git |
Thanks! This is all very helpful :) I’ll work on getting these changes upstreamed |
Is there anyway you could point me to a commit that is working for the fastpair package? Trying to evaluate how useful it would be for automated fastpair validation. |
I’m not entirely sure if the fastpair code exported to GitHub has ever been buildable, so trying old commits might not provide much value. I’m working towards getting all of the open sourced packages in this repo to a buildable state, but we aren’t fully there yet. |
Okay good to know. So is this code used internally to any success? I am just trying to evaluate if contributing to this effort would bear any fruit any reasonable time frame. Also are there any other google supported frameworks for validating an OEM fast pair provider implementation other than the Validator app? |
It might not be worth your time. We are considering removing the package, since we don't have anyone committed to maintaining the external copy. |
I am attempting to build the fastpair library on Windows, but am having issues with the bazel build system. There doesn't seem to be much documentation on how to build each of the packages. Some clarification would be greatly appreciated!
The text was updated successfully, but these errors were encountered: