rweather / noise-c

Noise-C, a plain C implementation of the Noise protocol
MIT License
306 stars 86 forks source link

Includes in src/backend/ref #18

Closed nazar-pc closed 7 years ago

nazar-pc commented 7 years ago

I'm not a C/C++ developer, but still I'm trying to port this library to JavaScript using Emscripten (compile it into WebAssembly).

I'm having some issues with files in src/backend/ref. For instance, #include "internal.h" doesn't work here. Next few lines also seem to require changes.

From my limited understanding following:

#include "internal.h"
#include "crypto/aes/rijndael-alg-fst.h"
#include "crypto/ghash/ghash.h"

Should in fact be like this:

#include <protocol/internal.h>
#include <crypto/aes/rijndael-alg-fst.h>
#include <crypto/ghash/ghash.h>

Emscripten uses clang if it makes any difference.

Am I missing something or this is indeed a bug?

nazar-pc commented 7 years ago

OK, so it basically work, but requires includes directories to be configured like in makefile

rweather commented 7 years ago

No worries - thanks for the feedback anyway. :-)