All submissions to this site are governed by the Second Life Viewer Contribution Agreement. By submitting patches and other information using this site, you acknowledge that you have read, understood, and agreed to those terms.

Review Board 1.6.11

Welcome to the Second Life Viewer Code Review tool.
See the documentation on our wiki for how to use this site.

Cleaning up testing cmake macros to prepare for refactoring them to improve build time.

Review Request #301 - Created May 9, 2011 and submitted

Brad Kittenbrink Reviewers
http://bitbucket.org/brad_linden/viewer-mustbuildfaster-1 viewer
CHOP-609
None viewer-development 
Consolidating all references to the LL_TESTS config var in one file to make the logic surrounding it more cohesive.
I've tested with LL_TESTS=ON, but I'd like some help from people who use this feature testing with LL_TESTS=OFF to make sure I haven't missed anything.
Boroondas Gupte
Posted (May 10, 2011, 4:37 a.m.)
Review Board can't apply your patch. Please upload a diff file that works. Some tips:
* Generate the diff file from hg rather than git, so that it contains revision IDs that RB knows about.
* If the change you want to review isn't based on a revision already in the repository you selected for this RB review request (lindenlab/viewer-development in this case), you must provide the diff between a known revision (i.e. one in lindenlab/viewer-development) and the base revision of your changeset as "Parent Diff".
* When creating a new review request or uploading a new diff file to an existing one, you can preview the diff (and see whether RB can apply it) before submitting the request/diff update by clicking the "View Diff" button.
** WARNING: Any changes to description, summary, tests done etc. that are being edited (i.e. have an "OK" and "Cancel" button visible) will be lost when you click "View Diff". To 'save' them (without submitting the request, yet), use the "OK" buttons associated with the respective input fields, and they will be kept while (p)reviewing your Diff.