UpstandingHackers / hammer

Parser combinators for binary formats, in C. Yes, in C. What? Don't look at me like that.
GNU General Public License v2.0
430 stars 40 forks source link

fix LIBPATH order for building test_suite #156

Closed pesco closed 8 years ago

pesco commented 8 years ago

Building on a fresh OpenBSD system, 'scons test' failed trying to install libhammer.a to /usr/local/lib. However, scons prefix=/usr test works fine.

Prepending '.' to LIBPATH instead of appending it makes it work so that's what this patch changes.

A guess at what's happening is that on a typical Linux everything lives in /usr and LIBPATH will be empty, so Append and Prepend do the same. This BSD has libglib-2.0.a in /usr/local/lib so it pulls that into LIBPATH.

The above doesn't really explain why prefix=/usr avoids the issue, though...