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

Fix hash/encrypt library searching #5804

Merged
merged 2 commits into from
Nov 10, 2016
Merged

Fix hash/encrypt library searching #5804

merged 2 commits into from
Nov 10, 2016

Conversation

Gobberwart
Copy link
Contributor

Short Description:

  • Fixes file name issue for macos users (libniantichash-osx-64.so not found)
  • Searches multiple locations for required files... encrypt_location as specified in config, bot root dir, and ./src/pgoapi/pgoapi/lib.
  • Update pgoapi src in requirements.txt to known working copy.

Filename is libniantichash-macos-64.dylib not libniantichash-osx-64.so
Instead of just looking in root, or encrypt_location, or whatever the
heck the last person did... will first search local or encrypt_location.
If not found, will search pgoapi/lib. If still not found, abort.
@mention-bot
Copy link

@Gobberwart, thanks for your PR! By analyzing the history of the files in this pull request, we identified @pablossp, @seehuily and @Heihachi to be potential reviewers.

@solderzzc
Copy link
Contributor

solderzzc commented Nov 10, 2016

👍

Approved with PullApprove

@solderzzc solderzzc merged commit f6fec58 into PokemonGoF:dev Nov 10, 2016
solderzzc pushed a commit that referenced this pull request Nov 11, 2016
* Corrected maxos niantic lib filename

Filename is libniantichash-macos-64.dylib not libniantichash-osx-64.so

* Better search for encrypt/hash files

Instead of just looking in root, or encrypt_location, or whatever the
heck the last person did... will first search local or encrypt_location.
If not found, will search pgoapi/lib. If still not found, abort.
@krummrey
Copy link
Contributor

libniantichash-osx-64.so can't be found.
The merge is broken on OS X!!! Even after a reset and/or clean install.
#5798

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.

4 participants