Fix a couple of VS2012 build issues

mmp / pbrt-v2

Source code for the version of pbrt described in the second edition of "Physically Based Rendering"

http://pbrt.org

989 stars 343 forks source link

Fix a couple of VS2012 build issues #27

Closed Reedbeta closed 10 years ago

Reedbeta commented 10 years ago

I fixed a couple of minor hygiene issues with the VS2012 build.

Correct version number in solution file so it opens in VS2012 when double-clicked.
Add *.user to ignore list.
Ignore and remove files generated by flex/bison during the build.

About the last, I'm not sure if this might screw up some other build configurations or something. I see these files were commented out of .gitignore before; not sure why that was done. If it's an issue, that commit can be left out.

mmp commented 10 years ago

I think we just want the second commit, that fixes the version number and ignore list. The outputs of pbrtparse and pbrtlex are checked in so that people can build the system without installing flex and bison if they don't want to modify the input files. Can you make another pull request with jut the second one? (Or if you know how I can just grab that one commit through github, let me know!)

Thanks!

Reedbeta commented 10 years ago

Sure. I've submitted a new pull request, #28...still getting the hang of how Git works :)