diff options
| author | Nat Goodspeed <nat@lindenlab.com> | 2016-12-21 20:51:18 -0500 |
|---|---|---|
| committer | Nat Goodspeed <nat@lindenlab.com> | 2016-12-21 20:51:18 -0500 |
| commit | aa2aa5e2da896a4ab96da41b878eb39e0dde8f35 (patch) | |
| tree | 982859841c4df924a27d00f4902c9f38778d496c | |
| parent | 7a83f26323b192b09b4017b5f4f986511345bd36 (diff) | |
DRTVWR-418: Migrate HAVOK suppression from autobuild.xml to build.sh.
Turns out that without HAVOK, we can't build the PhysicsExtensions_TPV; but
the viewer's build.sh is unaware of CMake switches set in autobuild.xml.
Passing those CMake overrides in build.sh allows us to test that setting
elsewhere in build.sh to skip the PhysicsExtensions_TPV step -- instead of
failing the build.
| -rw-r--r-- | autobuild.xml | 2 | ||||
| -rwxr-xr-x | build.sh | 11 |
2 files changed, 8 insertions, 5 deletions
diff --git a/autobuild.xml b/autobuild.xml index f05a665a01..2bf12fcba0 100644 --- a/autobuild.xml +++ b/autobuild.xml @@ -3344,7 +3344,6 @@ <array> <string>-G</string> <string>Xcode</string> - <string>-DHAVOK:BOOL=FALSE</string> </array> </map> <key>default</key> @@ -3407,7 +3406,6 @@ <array> <string>-G</string> <string>Xcode</string> - <string>-DHAVOK:BOOL=FALSE</string> </array> </map> <key>name</key> @@ -95,16 +95,20 @@ pre_build() && [ -r "$master_message_template_checkout/message_template.msg" ] \ && template_verifier_master_url="-DTEMPLATE_VERIFIER_MASTER_URL=file://$master_message_template_checkout/message_template.msg" + # nat 2016-12-20: disable HAVOK on Mac until we get a 64-bit Mac build. # nat 2016-12-21: disable generate_breakpad_symbols.py on Mac until we # figure out why it's breaking. if [ "$arch" == "Darwin" ] - then RELEASE_CRASH_REPORTING=OFF - else RELEASE_CRASH_REPORTING=ON + then HAVOK=OFF + RELEASE_CRASH_REPORTING=OFF + else HAVOK=ON + RELEASE_CRASH_REPORTING=ON fi "$autobuild" configure --quiet -c $variant -- \ -DPACKAGE:BOOL=ON \ -DUNATTENDED:BOOL=ON \ + -DHAVOK:BOOL="$HAVOK" \ -DRELEASE_CRASH_REPORTING:BOOL="$RELEASE_CRASH_REPORTING" \ -DVIEWER_CHANNEL:STRING="\"$viewer_channel\"" \ -DGRID:STRING="\"$viewer_grid\"" \ @@ -119,7 +123,8 @@ package_llphysicsextensions_tpv() { begin_section "PhysicsExtensions_TPV" tpv_status=0 - if [ "$variant" = "Release" ] + # nat 2016-12-21: without HAVOK, can't build PhysicsExtensions_TPV. + if [ "$variant" = "Release" -a "${HAVOK:-}" != "OFF" ] then test -r "$build_dir/packages/llphysicsextensions/autobuild-tpv.xml" || fatal "No llphysicsextensions_tpv autobuild configuration found" tpvconfig=$(native_path "$build_dir/packages/llphysicsextensions/autobuild-tpv.xml") |