disguise-one / hap-encoder-adobe-cc

Community-supported exporter of HAP codec family for Adobe CC applications
Other
169 stars 15 forks source link

Snappy unit tests using wrong header #43

Open GregBakker opened 4 years ago

GregBakker commented 4 years ago

Platform: macos Prerequisites: brew installed Snappy [which should be ignored], re-enable snappy tests in top level CMakeLists.txt Symptom: When snappy unit tests are built, they find the system installed snappy header instead of the locally generated snappy header and fail to build Expected: The tests would find the locally generated header

This is likely a problem with our local Snappy's CMakeLists.txt. Also it has a weird 'typedef std::string string' in its own namespace, whose absence causes the compile fail.

We should update our local Snappy; also check that the new version doesn't have the issue.