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.

Sign up for GitHub

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

Jump to bottom

WIP - Test nested callbacks with vcx_set_logger #87

Open
wants to merge 78 commits into
base: master
Choose a base branch
from
Open

WIP - Test nested callbacks with vcx_set_logger #87

wants to merge 78 commits into from

Conversation

nsivraj
Copy link

@nsivraj nsivraj commented Apr 22, 2019

No description provided.

Fixed the Android test to have the logging work and added testing log…
182d478 
…ic to add the same key to the wallet twice to verify that it does not crash.
@nsivraj nsivraj changed the title [WIP] - Test for adding duplicate key to wallet WIP - Test for adding duplicate key to wallet Apr 22, 2019
@nsivraj nsivraj changed the title WIP - Test for adding duplicate key to wallet WIP - Test nested callbacks with vcx_set_logger Apr 23, 2019
Norman Jarvis added 21 commits April 23, 2019 14:40
Just made change to see if getting the logger locally and writing to …
0c0a887 
…that logger locally would make a difference.
Changes to LibvcxLogger to see if it will pass the test.
7fe21c2
Fixed the code to be correct.
c35ca07
Added back in the map_err call but do nothing with err.
35514d0
Put back the map_err functions.
cdb3016
Fix the compile errors.
49c6e0c
Changed from u8 to i8.
14094d9
Still trying to fix the string constant build issue.
0013763
Still fixing the build issues.
a8db198
Commented out the calling of the log callback.
0a0aeea
Put back in the invocation of the logging callback.
25d9854
Wrapped the logger callback with spawn move.
7fd727c
Added the use statement for spawn.
d04925d
Put back in the correct logging code as the other changes still cause…
923543f 
… a crash.
Changed the building of the android libvcx.so artifact to NOT merge i…
7872aea 
…n existing .so files.
Removed an accidental extra fi.
cf7bcc5
Switching to the default android api level to be 24.
b771b0e
Changed the parameters to be of type Pointer and commented out the lo…
5b5e0f4 
…gging code to see if it passes.
Merge branch 'master' into norms/fix-kipb-12-and-13-p7
128dfe2
Merge branch 'master' into norms/fix-kipb-12-and-13-p7
9e1f96b
Trying the std::mem::forget call
a915e6b
Norman Jarvis and others added 30 commits April 29, 2019 10:36
Revert "Fixing next compile error."
15a9926 
This reverts commit 932b481.
Revert "Fixed the compile error."
54adcb4 
This reverts commit 4b63ded.
Revert "Since default logger works then passing in the callback to th…
588dd4b 
…e default logger to see if it can work."

This reverts commit 7231ebf.
Reverting last 4 commits and then put back in spawn move in rust code.
bfdc12b
Commented out the rust code that we do not need.
bc7b66e
Changes to manage docker images and a change to allow the build.nondo…
0b41a0d 
…cker.sh script to be invoked multiple time.
Don't remove the built libraries and added code to see if the calls s…
00760b6 
…till fail.
The move command does not do anything.
236c661
Added an if statement to avoid needing to rebuild the toolchain tools…
555eb1a 
… each time.
Adding script that gives steps for building libvcx.so on an android d…
279842c 
…ocker image.
Added the executable bit on the shell script.
47822a7
Added back in the rm and mv commands for the libvcx_arch folder.
98e152c
Fixed the script to use the rm and mv commands in the androidBuild.sh…
f5f1187 
… script.
Script changes to optimize the building of rust code and running andr…
d718c5e 
…oid tests.
Adjusting for the vagrant environment.
f6eae33
@nsivraj
A few scripting changes
f6dad1a
Changed from /home/android to /home/vagrant
3aa5315
Switching back to /home/android from /home/vagrant
e75aa79
Testing the script to build and test android to verify that the defau…
0cedc85 
…lt logger does not cause the crash.
Got rid of the space character after the slash character right next t…
f93dcf5 
…o the ruby-dev package name.
Added three more ignores.
99edf7c
Adding in code to pass in a logging callback as part of the default l…
ae49c86 
…ogger.
Trying out changing the libindy logger.
d1661dd
Forgot to commit this file.
2150ad3
changed the path back to /tmp
ddd1241
Instructions for how to start the emulator image.
63bade2
Added command to see if emulator is running.
0975288
Adding android logger to custom logger.
4ba6e93
Added a prefix to recognize the log statement.
0bfe9c5
Putting back the logger to cause the crash.
2cab979
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@nsivraj