nitrogen / simple_bridge

A simple, standardized interface library to Erlang HTTP Servers.
MIT License
112 stars 76 forks source link

Don't bundle own version of yaws_api.hrl. #9

Closed jadahl closed 12 years ago

jadahl commented 12 years ago

By bundling yaws_api.hrl it's likely that it will be incompatible with the one installed by yaws. Instead use the one installed.

Added /usr/lib include path so that it will find yaws include files by default on Debian based machines.

choptastic commented 12 years ago

Thanks Jonas,

I've actually been in the process of removing the yaws header altogether and using the yaws api calls instead, as on the commit from Torbjorn Tornkvist here:

https://github.com/choptastic/simple_bridge/commit/07b082dc915e7c5b84c1f7fa6ba9baaf00f868d5

I think that way will probably be safer than worrying about which version of the .hrl simple_bridge was compiled with.

What do you think?

jadahl commented 12 years ago

Sounds like a great idea.