search

mmp / pbrt-v3

Source code for pbrt, the renderer described in the third edition of "Physically Based Rendering: From Theory To Implementation", by Matt Pharr, Wenzel Jakob, and Greg Humphreys.
http://pbrt.org
BSD 2-Clause "Simplified" License
4.87k stars 1.18k forks source link

PBRT api header file ? #203

Closed nyue closed 4 years ago

nyue commented 5 years ago

Hi,

In the book, there is a mentioned of core/api.h

I build and install the current master but there was no api.h in the destination include directory.

Is the api.h meant to be install via the standard build+install approach ? Are there other PBRT header file that should be installed for client code to build ?

Cheers

mmp commented 5 years ago

That stuff isn't super well specified unfortunately (and nothing is installed currently; see also #2 ).

e.g. api.h includes pbrt.h, which, in addition to regular system headers, includes core/error.h and glog/logging.h (even though neither is needed in this particular case.) However, to use api.h one almost certainly needs core/paramset.h, and that in turn includes a bunch of stuff, etc.

The most expedient thing is to probably install just about all the headers in core/ , or to just point at them directly. :-p