[opensource-dev] Daily Scrum Summary - Monday, May 16

[Anya Kanevsky]

 Sprint 15, ends 05.16.11 Monday, May 16 Team Status
--
 Oz Linden
--

*PAST*

   - Merge Monkeying - assorted late arrivals
   - Open Dev User Group
   - GPU table testing (built test viewer)

*FUTURE*

   - Integrate mesh?
   - Integrate GPU table updates
   - Sprint Planning

*IMPEDIMENTS*

   - none

Wolf Linden
--

*PAST*

   - Organizational meetings: Rhett, Oz

*FUTURE*

   - Check in with Grumpity
   - WLRS status review

*IMPEDIMENTS*

   - none

Grumpity ProductEngine
--

*PAST*

   - VWR triage
   - Sprint retrospective
   - jira workflow changes for VWR triage
   - sprint 15 cleanup

*FUTURE*

   - sprint planning
   - meet with wolf

*IMPEDIMENTS*

   - none

Paul ProductEngine
--

*PAST*

   - TASK STORM-1202 (Time spin control)
  - Made some changes according to Vadim's comments in review
   - BUG STORM-951 (Accidental Self-Muting/Self-Blocking)
  - WIP. Investigating possibilities to refactor fix for ticket
  according to Vadim's comments in review

*FUTURE*

   - STORM-1205 (Add "Environment" tab to the "Region/Estate" floater)

*IMPEDIMENTS*

   - none

Seth ProductEngine
--

*PAST*

   - TASK (STORM-1244) Implement Environment Settings floater
  - WIP. Integrating with Vadim's new way of storing/applying
  environment settings. 90% complete.

*FUTURE*

   - TASK (STORM-1244) Implement Environment Settings floater
   - TASK (STORM-1256) Region environment settings editor

*IMPEDIMENTS*

   - none

Vadim ProductEngine
--

*PAST*

   - Task STORM-1245 (Implement new way of storing/applying environment
   settings):
  - Completed by adding region settings support (read-only).
  - After Seth finishes with the new Environment Settings Floater
  (hopefully today) we'll be ready to demonstrate the new basic WLRS UI.
   - Created JIRA tasks for the remaining WLRS work.

*FUTURE*

   - Help Seth with an assertion problem he's experiencing.
   - Proceed with WLRS tasks (probably skies management).

*IMPEDIMENTS*

   - none

Andrey ProductEngine
--

*PAST*

   - picked up the latest v-d build r229908
   - verified 8 integrated tickets
   - started regression pass:
  - Notifications & Alerts
  - Busy Mode
  - MiniMap
  - World Map
  - About Land
  - ToS

*FUTURE*

   - pick up next beta build?

*IMPEDIMENTS*

   - none

Wolfpup Lowenhar
--

*PAST*

   - STORM-859 : created local and online repositories and also created
   issue on RB.
   - STORM-1100 : DLed test viewer which crashes on start on linux laptop so
   posted crash report log to issue.
   - Attended Open Development User Group meeting.
   - Working inworld(this is part of testing localy built viewers).
  - Checking on horses and bundles, plus work on note cards for them.
  - Desiding on what type of build to work on next.

*FUTURE*

   - Working inworld(this is part of testing localy built viewers).

*IMPEDIMENTS*

   - Getting Autobuild system and seceurity software to play nicely on my
   system.
___
Policies and (un)subscribe information available here:
http://wiki.secondlife.com/wiki/OpenSource-Dev
Please read the policies before posting to keep unmoderated posting privileges

[opensource-dev] Daily Scrum Summary - Friday, May 13

[Anya Kanevsky]

 Sprint 15, ends 05.16.11 Friday, May 13 Team Status
--
 Oz Linden
--

*PAST*

   - Merge Monkeying
   - Viewer Evolution User Group
   - GPU table testing
   - windlight project review

*FUTURE*

   - Check in GPU table & manual test script

*IMPEDIMENTS*

   - none

Grumpity ProductEngine
--

*PAST*

   - storm triage and sprint cleaning
   - QA roundup

*FUTURE*

   - VWR triage
   - Sprint retrospective
   - Discuss with Oz:
  - add "verify" fix version?
  - add "unactionable" transition to needs more info state.
  - other jira transitions specified in VWR triage doc?
  - find (push to create?) list of currently supported versions.
  - is it possible to limit (or is it already limited?) who can
  "acknowledge" vwr issues?
  - is it possible to automatically uncheck approvals when issue is
  reopened by QA?

*IMPEDIMENTS*

   - none


Paul ProductEngine
--

*PAST*

   - TASK STORM-1202 (Time spin control)
  - Implemented. Sent for review.
   - STORM-1205 (Add "Environment" tab to the "Region/Estate" floater)
  - WIP. Investigating. Not providing the estimate until i've not
  investigated the task.

*FUTURE*

   - STORM-1205 (Add "Environment" tab to the "Region/Estate" floater)

*IMPEDIMENTS*

   - none

Seth ProductEngine
--

*PAST*

   - TASK (STORM-1244) Implement Environment Settings floater
  - XUI cleanup.
  - Investigating sky and water preset managers.

*FUTURE*

   - TASK (STORM-1244) Implement Environment Settings floater
  - Estimated: 4 - 6 hours

*IMPEDIMENTS*

   - none

Vadim ProductEngine
--

*PAST*

   - Task STORM-1200 (Create fixed sun presets):
  - Submitted.
   - Task STORM-1245 (Implement new way of storing/applying environment
   settings):
  - WIP. Finished with local presets. Adding region settings support
  (r/o).

*FUTURE*

   - Complete STORM-1245, including integration with Seth's Environment
   Settings floater.
   - Create JIRA tickets for remaining tasks.

*IMPEDIMENTS*

   - none

Andrey ProductEngine
--

*PAST*

   - picked up 2.6.6 release build r229662
   - smoke test has been failed by EXP-749 (the issue with voice indicator)
   - tested CHOP-663 according to the test plan on on wiki on all platforms
   - continued with STORM-1027 testing
  - testing on OSX and Windows - done
  - on Linux - in progress

*FUTURE*

   - pickup the latest v-d build and verify integrated issues

*IMPEDIMENTS*

   - none

Wolfpup Lowenhar
--

*PAST*

   - Replaced Video card in Windows system.
   - Attended Viewer Evolution User Group meeting.
   - Working inworld (this is part of testing localy built viewers).
  - Checking on horses and bundles, plus work on note cards for them.
  - Desiding on what type of build to work on next.
   - STORM-941 : Acceptance Criteria added.
   - STORM-859 : Tested Ardy's patch by manualy applying it to local
   repository.

*FUTURE*

   - STORM-859 : Create local and online repositories and RB issue so this
   can go through the steps needed for intergration.

*IMPEDIMENTS*

   - Getting Autobuild system and security software to play nicely on my
   system.
___
Policies and (un)subscribe information available here:
http://wiki.secondlife.com/wiki/OpenSource-Dev
Please read the policies before posting to keep unmoderated posting privileges

[opensource-dev] Daily Scrum Summary - Thursday, May 12

[Anya Kanevsky]

(last one for today)

Sprint 15, ends 05.16.11 Thursday, May 12 Team Status
--
 Oz Linden
--

*PAST*

   - Merge Monkeying
   - Sent query to the Moles about making Clouds
   - Open Development User Group
   - GPU table testing
   - Updated windlight project branch from viewer-development
  - NOTE: project viewer build changed
 - from "Project Windlight Estate"
 - to "Project Windlight Region"

*FUTURE*

   - More GPU table testing
   - Code reviews

*IMPEDIMENTS*

   - none

Grumpity ProductEngine
--

*PAST*

   - VWR triage
   - PO build
   - reports
   - chop-662 followup

*FUTURE*

   - STORM triage
   - add "verify" fix version?
   - add "unactionable" transition to needs more info state.
   - other jira transitions specified in VWR triage doc?
   - find (push to create?) list of currently supported versions.
   - is it possible to limit (or is it already limited?) who can
   "acknowledge" vwr issues?

*IMPEDIMENTS*

   - QA team needs L$

Paul ProductEngine
--

*PAST*

   - TASK STORM-1202 (Time spin control)
  - Still working on it. Estimate ~ 5 hours.

*FUTURE*

   - TASK STORM-1202 (Time spin control)
   - STORM-1205 (Add "Environment" tab to the "Region/Estate" floater)

*IMPEDIMENTS*

   - none

Seth ProductEngine
--

*PAST*

   - Replacing the old Environment Settings dialog with the new one.
  - Going to create a jira for this task.

*FUTURE*

   - Work on Environment Settings dialog.

*IMPEDIMENTS*

   - none

Vadim ProductEngine
--

*PAST*

   - WLRS implementation: new way of saving/applying water/sky presets and
   day cycles to support the new Environment Settings floater.

*FUTURE*

   - We'll continue working on WLRS, aiming to deliver the new basic UI on
   Monday.

*IMPEDIMENTS*

   - none

Andrey ProductEngine
--

*PAST*

   - tested CHOP-663 on all platforms using the build referenced in the
   ticket
   - finished testing in scope of IQA-162
  - also has been run additional regression testing on r229289

*FUTURE*

   - run smoke & integrity on 2.6.8 build#2
   - pickup v-d latest build and verify integrated tickets

*IMPEDIMENTS*

   - none

Cron Stardust
--

*PAST*

   - I finally got my edit UI suggestion JIRA'd Now to make patches at
   my next breather... :P ** VWR-25739 Reversing the single-axis grid-based
   drag, scale, and rotate layout

*FUTURE*

   - Get patches and Repository created for the above

*IMPEDIMENTS*

   - School, work, and other RL... :(
___
Policies and (un)subscribe information available here:
http://wiki.secondlife.com/wiki/OpenSource-Dev
Please read the policies before posting to keep unmoderated posting privileges

[opensource-dev] svn.secondlife.com

[Jonathan Welch]

Someone on IRC just pointed out that the old source url,
http://svn.secondlife.com/ now puts up web pages selling surveillance
cameras/systems.  Has your web site been hacked or are you now
shopping out part of your domain name to third parties?
___
Policies and (un)subscribe information available here:
http://wiki.secondlife.com/wiki/OpenSource-Dev
Please read the policies before posting to keep unmoderated posting privileges

Re: [opensource-dev] Review Request: Update autobuilds default VC version to 2010

[Boroondas Gupte]

---
This is an automatically generated e-mail. To reply, visit:
http://codereview.secondlife.com/r/305/#review703
---

Ship it!


This change looks good.

It should be investigated whether other places need changing too. (Either to 
VC100, or to not hardcode the compiler name at all ... ideally we'd only set it 
at a single place.) If this change builds fine on TeamCity, that investigation 
should IMHO not block integration of this change, though, so ship it!

Here's a little search I did. Files where multiple compiler versions occur are 
probably fine, but those where only VC80 gets mentioned should be looked at 
more closely:

$> ack -a --ignore-case "vc[[:digit:]][[:digit:]]+"
scripts/automated_build_scripts/opensrc-build.sh
110:  echo build-vc80
178:  cmake_generator="vc80"

autobuild.xml
2034:  build-vc100

build.sh
30:  echo build-vc100

.hgignore
16:build-vc80/
17:build-vc100/

indra/llcommon/llerror.cpp
1191:   // VC80 was optimizing the error away.

indra/llcommon/tests/llallocator_heap_profile_test.cpp
109:"0040-02681000 r-xp  00:00 0   
c:\\proj\\tcmalloc-eval-9\\indra\\build-vc80\\newview\\RelWithDebInfo\\secondlife-bin.exe\n"
129:"0040-004d5000 r-xp  00:00 0   
c:\\code\\linden\\tcmalloc\\indra\\build-vc80\\llcommon\\RelWithDebInfo\\llallocator_test.exe\n"
140:"1000-10041000 r-xp  00:00 0   
c:\\code\\linden\\tcmalloc\\indra\\build-vc80\\llcommon\\RelWithDebInfo\\libtcmalloc_minimal.dll\n"
141:"7c42-7c4a7000 r-xp  00:00 0   
C:\\WINDOWS\\WinSxS\\x86_Microsoft.VC80.CRT_1fc8b3b9a1e18e3b_8.0.50727.1433_x-ww_5cf844d2\\MSVCP80.dll\n"
142:"7813-781cb000 r-xp  00:00 0   
C:\\WINDOWS\\WinSxS\\x86_Microsoft.VC80.CRT_1fc8b3b9a1e18e3b_8.0.50727.1433_x-ww_5cf844d2\\MSVCR80.dll\n";

indra/cmake/Copy3rdPartyLibs.cmake
67:if (MSVC80)
71: 
[HKEY_LOCAL_MACHINE\\SOFTWARE\\Microsoft\\VisualStudio\\8.0\\Setup\\VC;ProductDir]/redist/Debug_NonRedist/x86/Microsoft.VC80.DebugCRT
80:Microsoft.VC80.DebugCRT.manifest
96: 
[HKEY_LOCAL_MACHINE\\SOFTWARE\\Microsoft\\VisualStudio\\8.0\\Setup\\VC;ProductDir]/redist/x86/Microsoft.VC80.CRT
105:Microsoft.VC80.CRT.manifest
126:FIND_PATH(debug_msvc10_redist_path msvcr100d.dll
129: 
[HKEY_LOCAL_MACHINE\\SOFTWARE\\Microsoft\\VisualStudio\\10.0\\Setup\\VC;ProductDir]/redist/Debug_NonRedist/x86/Microsoft.VC100.DebugCRT
135:if(EXISTS ${debug_msvc10_redist_path})
136:set(debug_msvc10_files
142:${debug_msvc10_redist_path}
145:${debug_msvc10_files}
151:FIND_PATH(release_msvc10_redist_path msvcr100.dll
154: 
[HKEY_LOCAL_MACHINE\\SOFTWARE\\Microsoft\\VisualStudio\\10.0\\Setup\\VC;ProductDir]/redist/x86/Microsoft.VC100.CRT
160:if(EXISTS ${release_msvc10_redist_path})
161:set(release_msvc10_files
167:${release_msvc10_redist_path}
170:${release_msvc10_files}
175:${release_msvc10_redist_path}
178:${release_msvc10_files}
183:endif (MSVC80)

indra/cmake/Boost.cmake
21:if(MSVC80)
23:  optimized libboost_program_options-vc80-mt-${BOOST_VERSION}
24:  debug libboost_program_options-vc80-mt-gd-${BOOST_VERSION})
26:  optimized libboost_regex-vc80-mt-${BOOST_VERSION}
27:  debug libboost_regex-vc80-mt-gd-${BOOST_VERSION})
29:  optimized libboost_signals-vc80-mt-${BOOST_VERSION}
30:  debug libboost_signals-vc80-mt-gd-${BOOST_VERSION})
32:  optimized libboost_system-vc80-mt-${BOOST_VERSION}
33:  debug libboost_system-vc80-mt-gd-${BOOST_VERSION})
35:  optimized libboost_filesystem-vc80-mt-${BOOST_VERSION}
36:  debug libboost_filesystem-vc80-mt-gd-${BOOST_VERSION})
37:else(MSVC80)
40:  optimized libboost_program_options-vc100-mt-${BOOST_VERSION}
41:  debug libboost_program_options-vc100-mt-gd-${BOOST_VERSION})
43:  optimized libboost_regex-vc100-mt-${BOOST_VERSION}
44:  debug libboost_regex-vc100-mt-gd-${BOOST_VERSION})
46:  optimized libboost_system-vc100-mt-${BOOST_VERSION}
47:  debug libboost_system-vc100-mt-gd-${BOOST_VERSION})
49:  optimized libboost_filesystem-vc100-mt-${BOOST_VERSION}
50:  debug libboost_filesystem-vc100-mt-gd-${BOOST_VERSION})
51:endif (MSVC80)

indra/cmake/JsonCpp.cmake
14:  debug json_vc100debug_libmt.lib
15:  optimized json_vc100_libmt)

indra/newview/build_win32_appConfig.py
55:src_manifest_name = dst_dir + '/Microsoft.VC80.DebugCRT.manifest'
58:src_manifest_name = dst_dir + '/Microsoft.VC80.CRT.manifest'

indra/newview/llworldmipmap.cpp
188://imageurl = 
"file://C:\\Develop\\mapserver-distribute-3\\indra\\build-vc80\\mapserver\\relwithdebinfo\\regions\\00995\\01001\\region-995-1001-prims.jpg";

indra/newview/vie

Re: [opensource-dev] svn.secondlife.com

[Thomas Grimshaw]

The SVN server was hosted at SoftLayer, and that server (rather, the 
address) has been taken over by someone else, that's all. Just needs to 
be removed from the dns pool.

On 17/05/2011 10:20, Jonathan Welch wrote:
> Someone on IRC just pointed out that the old source url,
> http://svn.secondlife.com/ now puts up web pages selling surveillance
> cameras/systems.  Has your web site been hacked or are you now
> shopping out part of your domain name to third parties?
> ___
> Policies and (un)subscribe information available here:
> http://wiki.secondlife.com/wiki/OpenSource-Dev
> Please read the policies before posting to keep unmoderated posting privileges

___
Policies and (un)subscribe information available here:
http://wiki.secondlife.com/wiki/OpenSource-Dev
Please read the policies before posting to keep unmoderated posting privileges

Re: [opensource-dev] Review Request: CHOP-658 moving panels and floaters into a new static library project viewer_components/viewerui

[Ardy Lay]

On 5/16/2011 9:04 PM, Brad Kittenbrink wrote:

It's also availible on 
bitbucket(https://bitbucket.org/brad_linden/viewer-mustbuildfaster-1/changeset/25c39e3d8a88)
 for testing in TeamCity.

Brad, is the TeamCity build result listed anywhere yet?

I pulled from your viewer-mustbuildfaster-1 at commit 16171- 
6f777a541bf6 and built it in 19 minutes without tests.  BUT... when I 
logged in I find an empty "MY FRIENDS" list.  "No matches found".  Heh, 
that might scare somebody.  ;-)
___
Policies and (un)subscribe information available here:
http://wiki.secondlife.com/wiki/OpenSource-Dev
Please read the policies before posting to keep unmoderated posting privileges

Re: [opensource-dev] Review Request: VWR-25654: fix memory leak in LLTranslate::getTranslateUrl

[Boroondas Gupte]

---
This is an automatically generated e-mail. To reply, visit:
http://codereview.secondlife.com/r/282/
---

(Updated May 17, 2011, 9:18 a.m.)


Review request for Viewer.


Changes
---

made the temporary string const


Summary
---

curl_escape() returns a pointer to a C-string. When one is done with the 
C-string, one must curl_free() the pointer. (See 
http://curl.haxx.se/libcurl/c/curl_escape.html .) To be able to call curl_free 
on the pointer, we must keep it in the first place.

Also, a cpp file in which curl functions are called should include 
 directly, even if it already gets included indirectly, i.e. via 
other includes. So adding that, too.


Note that curl_escape() is deprecated and should be replaced by 
curl_easy_escape(). This change does not do that, as this should probably be 
done for all occurrences at once and be accompanied by some refactoring.


This addresses bug VWR-25654.
http://jira.secondlife.com/browse/VWR-25654


Diffs (updated)
-

  doc/contributions.txt 9ff8625cadec 
  indra/newview/lltranslate.cpp 9ff8625cadec 

Diff: http://codereview.secondlife.com/r/282/diff


Testing
---

* Built Viewer (Linux 64bit STANDALONE)
* Verified that translation still works (logged in, enabled translation in 
nearby chat floater, changed "translate to" language in preferences > chat and 
went to Ahern)


Thanks,

Boroondas

___
Policies and (un)subscribe information available here:
http://wiki.secondlife.com/wiki/OpenSource-Dev
Please read the policies before posting to keep unmoderated posting privileges

[opensource-dev] Review Request: Correctly identify Server 2008, Server 2008 R2, Server 2012, and Windows 8

[jaeger_Reg]

---
This is an automatically generated e-mail. To reply, visit:
http://codereview.secondlife.com/r/311/
---

Review request for Viewer.


Summary
---

The stated OS in help -> about for server 2008 and server 2008 R2 is 
inaccurate.  It shows up as "Microsoft Windows Vista Server" or "Microsoft 
Windows 7 Server" instead of "Microsoft Server 2008" or "Microsoft Server 2008 
R2" respectively.  This patch corrects the text display error and also adds 
support for "Microsoft Server 2012" and "Microsoft Windows 8" which I have 
confirmed works on those OSs.  I came to this work by mimicking the code logic 
for how server 2003 was handled.

This patch does not correct the error of XP always showing up as running in 
compatibility mode.  This also doesn’t affect the redundant display of x64 for 
Server 2008 R2 and Server 2012, which are only available in 64bit versions.

I have applied my patch to my clone of viewer-dev at 
https://bitbucket.org/Tank_Master/storm-1258/changeset/f4b293bbdedf and my 
initial firestorm commit can be found at 
http://hg.phoenixviewer.com/phoenix-firestorm-lgpl/rev/6bf2de50ea83 


This addresses bug STORM-1258.
http://jira.secondlife.com/browse/STORM-1258


Diffs
-

  indra/llcommon/llsys.cpp ddfc2ed6aee1 

Diff: http://codereview.secondlife.com/r/311/diff


Testing
---

I made this patch for Firestorm, where it was applied and tested initially and 
no errors have been found.


Thanks,

tankmaster.finesmith

___
Policies and (un)subscribe information available here:
http://wiki.secondlife.com/wiki/OpenSource-Dev
Please read the policies before posting to keep unmoderated posting privileges

[opensource-dev] Review Request: Convert autobuild library builds to use msbuild.exe instead of devenv.com.

[Nicky Perian]

---
This is an automatically generated e-mail. To reply, visit:
http://codereview.secondlife.com/r/312/
---

Review request for Viewer.


Summary
---

Convert autobuild library builds to use msbuild.exe instead of devenv.com. 
Includes Ima Mechanique change from http://codereview.secondlife.com/r/305/


This addresses bug OPEN-69.
http://jira.secondlife.com/browse/OPEN-69


Diffs
-

  autobuild/autobuild_tool_source_environment.py 2a560b1d8f95 

Diff: http://codereview.secondlife.com/r/312/diff


Testing
---

Made calling script 3p-ogvoribs/build-cmd.sh changes, which will be published 
later, to use msbuild parameters and successfully built and packaged library. 
Compared files in the package to those from LL prebuilt download and they match 
by name and size.
TODO: Build the viewer with the local package. 


Thanks,

Nicky

___
Policies and (un)subscribe information available here:
http://wiki.secondlife.com/wiki/OpenSource-Dev
Please read the policies before posting to keep unmoderated posting privileges

[opensource-dev] Mesh branch merged to viewer-development

[Oz Linden (Scott Lawrence)]

The mesh team development branch has been merged to viewer-development 
(with the actual mesh support disabled, but with many rendering and 
other fixes active).

Your next pull of the tip of viewer-development will bring you ~2250 
changesets; pay careful attention to any merges of your local changes.

Some of the changes may cause problems with using the Debug (and 
DebugOS) configurations.  If you have problems, please bring them up on 
the list.  Don't file a jira yet unless you believe you have diagnosed a 
specific issue well enough that a fix is possible.  If it turns out that 
these configurations are not working well, getting them fixed will be a 
high priority this week.

If you would like to update your repository to the latest possible 
change _before_ the mesh branch move (and let the rest of us sort out 
any problems that arise), then you can pull the tag '2.6.9-start'; the 
mesh merge is the next revision after that tag.

___
Policies and (un)subscribe information available here:
http://wiki.secondlife.com/wiki/OpenSource-Dev
Please read the policies before posting to keep unmoderated posting privileges

Re: [opensource-dev] Review Request: CHOP-658 moving panels and floaters into a new static library project viewer_components/viewerui

[WolfPup Lowenhar]

I also built this repository locally but from the tip and my build time which 
was a full standard build including all test took 2hrs and 22min this is 
actually 8 minutes longer that a viewer development build currently. Also only 
one of the side panels works and that is the inventory side panel.

 

From: opensource-dev-boun...@lists.secondlife.com 
[mailto:opensource-dev-boun...@lists.secondlife.com] On Behalf Of Ardy Lay
Sent: Tuesday, May 17, 2011 11:18 AM
To: opensource-dev@lists.secondlife.com
Subject: Re: [opensource-dev] Review Request: CHOP-658 moving panels and 
floaters into a new static library project viewer_components/viewerui

 

On 5/16/2011 9:04 PM, Brad Kittenbrink wrote: 

It's also availible on 
bitbucket(https://bitbucket.org/brad_linden/viewer-mustbuildfaster-1/changeset/25c39e3d8a88)
 for testing in TeamCity.

Brad, is the TeamCity build result listed anywhere yet?

I pulled from your viewer-mustbuildfaster-1 at commit 16171 - 6f777a541bf6 and 
built it in 19 minutes without tests.  BUT... when I logged in I find an empty 
"MY FRIENDS" list.  "No matches found".  Heh, that might scare somebody.  ;-)

  _  

No virus found in this message.
Checked by AVG - www.avg.com
Version: 10.0.1375 / Virus Database: 1509/3643 - Release Date: 05/17/11

___
Policies and (un)subscribe information available here:
http://wiki.secondlife.com/wiki/OpenSource-Dev
Please read the policies before posting to keep unmoderated posting privileges

Re: [opensource-dev] Review Request: Convert autobuild library builds to use msbuild.exe instead of devenv.com.

[Nicky Perian]

---
This is an automatically generated e-mail. To reply, visit:
http://codereview.secondlife.com/r/312/
---

(Updated May 17, 2011, 2:45 p.m.)


Review request for Viewer.


Changes
---

Incorporate Ima Mechanique suggestions for added msbuild.exe parameters.


Summary
---

Convert autobuild library builds to use msbuild.exe instead of devenv.com. 
Includes Ima Mechanique change from http://codereview.secondlife.com/r/305/


This addresses bug OPEN-69.
http://jira.secondlife.com/browse/OPEN-69


Diffs (updated)
-

  autobuild/autobuild_tool_source_environment.py 2a560b1d8f95 

Diff: http://codereview.secondlife.com/r/312/diff


Testing (updated)
---

Made calling script 3p-ogvoribs/build-cmd.sh changes, which will be published 
later, to use msbuild parameters and successfully built and packaged library. 
Compared files in the package to those from LL prebuilt download and they match 
by name and size.
TODO: Build the viewer with the local package. 

Built and packaged 3p-ogvorbis with added parameters.


Thanks,

Nicky

___
Policies and (un)subscribe information available here:
http://wiki.secondlife.com/wiki/OpenSource-Dev
Please read the policies before posting to keep unmoderated posting privileges

Re: [opensource-dev] Review Request: Convert autobuild library builds to use msbuild.exe instead of devenv.com.

[Boroondas Gupte]

---
This is an automatically generated e-mail. To reply, visit:
http://codereview.secondlife.com/r/312/#review704
---



autobuild/autobuild_tool_source_environment.py


Who is I?



autobuild/autobuild_tool_source_environment.py


"solution($1)" isn't a function call, is it? Maybe better write "solution 
(i.e. $1)".



autobuild/autobuild_tool_source_environment.py


"this it the path taken."? "is", maybe?


- Boroondas


On May 17, 2011, 2:45 p.m., Nicky Perian wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> http://codereview.secondlife.com/r/312/
> ---
> 
> (Updated May 17, 2011, 2:45 p.m.)
> 
> 
> Review request for Viewer.
> 
> 
> Summary
> ---
> 
> Convert autobuild library builds to use msbuild.exe instead of devenv.com. 
> Includes Ima Mechanique change from http://codereview.secondlife.com/r/305/
> 
> 
> This addresses bug OPEN-69.
> http://jira.secondlife.com/browse/OPEN-69
> 
> 
> Diffs
> -
> 
>   autobuild/autobuild_tool_source_environment.py 2a560b1d8f95 
> 
> Diff: http://codereview.secondlife.com/r/312/diff
> 
> 
> Testing
> ---
> 
> Made calling script 3p-ogvoribs/build-cmd.sh changes, which will be published 
> later, to use msbuild parameters and successfully built and packaged library. 
> Compared files in the package to those from LL prebuilt download and they 
> match by name and size.
> TODO: Build the viewer with the local package. 
> 
> Built and packaged 3p-ogvorbis with added parameters.
> 
> 
> Thanks,
> 
> Nicky
> 
>

___
Policies and (un)subscribe information available here:
http://wiki.secondlife.com/wiki/OpenSource-Dev
Please read the policies before posting to keep unmoderated posting privileges

Re: [opensource-dev] Mesh branch merged to viewer-development

[Sergey Litovchuk]

Running the latest viewer-development build for me causes the viewer to 
crash on start up with an error:

2011-05-17T21:41:18Z newview/llviewertexturelist.cpp(492) : error
2011-05-17T21:41:18Z ERROR: LLViewerTextureList::addImageToList: ASSERT 
(mInitialized)

It looks like there is a trouble with finding the settings in both user 
and application directory.

I'm using a RelWithDebInfoOS configuration on Linux.

Has someone experienced this kind of crash?

On 05/17/2011 09:22 PM, Oz Linden (Scott Lawrence) wrote:
>
> The mesh team development branch has been merged to viewer-development
> (with the actual mesh support disabled, but with many rendering and
> other fixes active).
>
> Your next pull of the tip of viewer-development will bring you ~2250
> changesets; pay careful attention to any merges of your local changes.
>
> Some of the changes may cause problems with using the Debug (and
> DebugOS) configurations.  If you have problems, please bring them up on
> the list.  Don't file a jira yet unless you believe you have diagnosed a
> specific issue well enough that a fix is possible.  If it turns out that
> these configurations are not working well, getting them fixed will be a
> high priority this week.
>
> If you would like to update your repository to the latest possible
> change _before_ the mesh branch move (and let the rest of us sort out
> any problems that arise), then you can pull the tag '2.6.9-start'; the
> mesh merge is the next revision after that tag.

___
Policies and (un)subscribe information available here:
http://wiki.secondlife.com/wiki/OpenSource-Dev
Please read the policies before posting to keep unmoderated posting privileges

Re: [opensource-dev] Review Request: Convert autobuild library builds to use msbuild.exe instead of devenv.com.

[Nicky Perian]

---
This is an automatically generated e-mail. To reply, visit:
http://codereview.secondlife.com/r/312/
---

(Updated May 17, 2011, 3:42 p.m.)


Review request for Viewer.


Changes
---

Incorporate Boroondas Gupte review of comments.


Summary
---

Convert autobuild library builds to use msbuild.exe instead of devenv.com. 
Includes Ima Mechanique change from http://codereview.secondlife.com/r/305/


This addresses bug OPEN-69.
http://jira.secondlife.com/browse/OPEN-69


Diffs (updated)
-

  autobuild/autobuild_tool_source_environment.py 2a560b1d8f95 

Diff: http://codereview.secondlife.com/r/312/diff


Testing
---

Made calling script 3p-ogvoribs/build-cmd.sh changes, which will be published 
later, to use msbuild parameters and successfully built and packaged library. 
Compared files in the package to those from LL prebuilt download and they match 
by name and size.
TODO: Build the viewer with the local package. 

Built and packaged 3p-ogvorbis with added parameters.


Thanks,

Nicky

___
Policies and (un)subscribe information available here:
http://wiki.secondlife.com/wiki/OpenSource-Dev
Please read the policies before posting to keep unmoderated posting privileges

Re: [opensource-dev] Mesh branch merged to viewer-development

[Trilo Byte]

It's also surprisingly difficult to do much in the way of testing against all 
those wonderful changes, when the vast majority of the JIRA's referenced aren't 
public.  Can someone maybe consider the idea that if a change is going to get 
merged into viewer development, the associated jira's need to be publicly 
accessible (except in cases of security issues, of course).

On May 17, 2011, at 11:22 AM, Oz Linden (Scott Lawrence) wrote:

> 
> The mesh team development branch has been merged to viewer-development 
> (with the actual mesh support disabled, but with many rendering and 
> other fixes active).
> 
> Your next pull of the tip of viewer-development will bring you ~2250 
> changesets; pay careful attention to any merges of your local changes.
> 
> Some of the changes may cause problems with using the Debug (and 
> DebugOS) configurations.  If you have problems, please bring them up on 
> the list.  Don't file a jira yet unless you believe you have diagnosed a 
> specific issue well enough that a fix is possible.  If it turns out that 
> these configurations are not working well, getting them fixed will be a 
> high priority this week.
> 
> If you would like to update your repository to the latest possible 
> change _before_ the mesh branch move (and let the rest of us sort out 
> any problems that arise), then you can pull the tag '2.6.9-start'; the 
> mesh merge is the next revision after that tag.
> 
> ___
> Policies and (un)subscribe information available here:
> http://wiki.secondlife.com/wiki/OpenSource-Dev
> Please read the policies before posting to keep unmoderated posting privileges

___
Policies and (un)subscribe information available here:
http://wiki.secondlife.com/wiki/OpenSource-Dev
Please read the policies before posting to keep unmoderated posting privileges

Re: [opensource-dev] Mesh branch merged to viewer-development

[Ardy Lay]

I hear viewer-development is calling itself 'second life mesh beta' 
build 2.6.9 230088.
I suspect that's relatively harmless but will get corrected.
___
Policies and (un)subscribe information available here:
http://wiki.secondlife.com/wiki/OpenSource-Dev
Please read the policies before posting to keep unmoderated posting privileges

Re: [opensource-dev] Mesh branch merged to viewer-development

[Ardy Lay]

I have been asked if the X button to clear inventory search is 
deliberately removed or if that's an "oops".
___
Policies and (un)subscribe information available here:
http://wiki.secondlife.com/wiki/OpenSource-Dev
Please read the policies before posting to keep unmoderated posting privileges

Re: [opensource-dev] Mesh branch merged to viewer-development

[Brendan Wilson]

Yes im seeing the same error in the exact same place and after doing some
investigation it looks like the crash is being cause by storm-973 as there
was a change @ that line and others in the file for that issue.

> -Original Message-
> From: opensource-dev-boun...@lists.secondlife.com [mailto:opensource-dev-
> boun...@lists.secondlife.com] On Behalf Of Sergey Litovchuk
> Sent: Tuesday, May 17, 2011 6:05 PM
> To: Oz Linden (Scott Lawrence)
> Cc: opensource-dev
> Subject: Re: [opensource-dev] Mesh branch merged to viewer-development
> 
> Running the latest viewer-development build for me causes the viewer to
> crash on start up with an error:
> 
> 2011-05-17T21:41:18Z newview/llviewertexturelist.cpp(492) : error
> 2011-05-17T21:41:18Z ERROR: LLViewerTextureList::addImageToList: ASSERT
> (mInitialized)
> 
> It looks like there is a trouble with finding the settings in both user
> and application directory.
> 
> I'm using a RelWithDebInfoOS configuration on Linux.
> 
> Has someone experienced this kind of crash?
> 
> On 05/17/2011 09:22 PM, Oz Linden (Scott Lawrence) wrote:
> >
> > The mesh team development branch has been merged to viewer-
> development
> > (with the actual mesh support disabled, but with many rendering and
> > other fixes active).
> >
> > Your next pull of the tip of viewer-development will bring you ~2250
> > changesets; pay careful attention to any merges of your local changes.
> >
> > Some of the changes may cause problems with using the Debug (and
> > DebugOS) configurations.  If you have problems, please bring them up on
> > the list.  Don't file a jira yet unless you believe you have diagnosed a
> > specific issue well enough that a fix is possible.  If it turns out that
> > these configurations are not working well, getting them fixed will be a
> > high priority this week.
> >
> > If you would like to update your repository to the latest possible
> > change _before_ the mesh branch move (and let the rest of us sort out
> > any problems that arise), then you can pull the tag '2.6.9-start'; the
> > mesh merge is the next revision after that tag.
> 
> ___
> Policies and (un)subscribe information available here:
> http://wiki.secondlife.com/wiki/OpenSource-Dev
> Please read the policies before posting to keep unmoderated posting
> privileges
> -
> No virus found in this message.
> Checked by AVG - www.avg.com
> Version: 10.0.1375 / Virus Database: 1509/3643 - Release Date: 05/17/11

___
Policies and (un)subscribe information available here:
http://wiki.secondlife.com/wiki/OpenSource-Dev
Please read the policies before posting to keep unmoderated posting privileges

Re: [opensource-dev] Mesh branch merged to viewer-development

[Ardy Lay]

Little niggle:  J2C Decoder Version display is having issues.

Second Life 2.6.9 (230165) May 17 2011 17:11:58 (Second Life Development)
Release Notes

You are at 255,719.0, 256,674.0, 59.7 in Lusk located at 
sim9054.agni.lindenlab.com (216.82.41.230:13002)
Second Life RC BlueSteel 11.05.05.228940
Release Notes

CPU: Intel(R) Xeon(R) CPU   W3550  @ 3.07GHz (3059.03 MHz)
Memory: 12280 MB
OS Version: Microsoft Windows 7 64-bit Service Pack 1 (Build 7601)
Graphics Card Vendor: NVIDIA Corporation
Graphics Card: GeForce GTX 570/PCI/SSE2

Windows Graphics Driver Version: 8.17.0012.6309
OpenGL Version: 4.1.0

libcurl Version: libcurl/7.21.1 OpenSSL/0.9.8q zlib/1.2.5 c-ares/1.7.1
J2C Decoder Version: ???

(??

Qt Webkit Version: 4.7.1 (version number hard-coded)
Voice Server Version: Vivox 3.2.0002.9361

Built with MSVC version 1600
Packets Lost: 212/68,259 (0.3%)
___
Policies and (un)subscribe information available here:
http://wiki.secondlife.com/wiki/OpenSource-Dev
Please read the policies before posting to keep unmoderated posting privileges

Re: [opensource-dev] Mesh branch merged to viewer-development

[Ardy Lay]

~~
  Loading Shaders:
~~
2011-05-18T02:32:15Z INFO: LLShaderMgr::loadShaderFile: Loading file: 
shaders/class2/windlight/atmosphericsVarsV.glsl (Want class 2)
...
2011-05-18T02:32:16Z INFO: LLShaderMgr::loadShaderFile: Loading file: 
shaders/class2/deferred/edgeF.glsl (Want class 2)
2011-05-18T02:32:16Z llrender/llrendertarget.cpp(694) : error
2011-05-18T02:32:16Z ERROR: LLMultisampleBuffer::addColorAttachment: 
WTF? 8cd6

That's nice.

People are getting this WTF? error message when attempting to activate 
lights and shadows.
___
Policies and (un)subscribe information available here:
http://wiki.secondlife.com/wiki/OpenSource-Dev
Please read the policies before posting to keep unmoderated posting privileges

Re: [opensource-dev] Mesh branch merged to viewer-development

[Trilo Byte]

It's there in 230165 (Mac client).  Once you start typing in the field, an x 
appears at the right to clear it.  

The wrong name in the application window persists.

The Ambient Occlusion & Depth Of Field checkboxes are still disabled in 
preferences on nVidia Macs, though the functionality works on nVidia 9400m and 
above GPU's.

Attempting to check the Lighting and Shadows checkbox on an ATI Mac results in 
a hard system crash.  Per comments in STORM-1210 and email discussion with 
Runitai & others, the checkbox was to have been disabled/ghosted for ATI Mac 
users until they could get the functionality working.  The checkbox is still 
enabled, and the hard crash persists.  Runitai is aware of this situation, and 
the need to get this fixed before release.

There is a significant graphical tearing  on the Mac client that persists in 
230165 (avatar standing in place, hold down left or right arrow keys to turn in 
a circle).  Turning off VBO reduces the effect, but doesn't eliminate it.

Changes to Anti-Aliasing in Mac client now require a restart to take effect.  
The change in behavior is documented, but still unwelcome.  For users on 
low-end or mid-range computers (who often have AA off while exploring or at 
events for performance, but turn it on/up for snapshots), this is a significant 
break in immersion.

On May 17, 2011, at 5:48 PM, Ardy Lay wrote:

> I have been asked if the X button to clear inventory search is 
> deliberately removed or if that's an "oops".
> ___
> Policies and (un)subscribe information available here:
> http://wiki.secondlife.com/wiki/OpenSource-Dev
> Please read the policies before posting to keep unmoderated posting privileges

___
Policies and (un)subscribe information available here:
http://wiki.secondlife.com/wiki/OpenSource-Dev
Please read the policies before posting to keep unmoderated posting privileges

Re: [opensource-dev] Mesh branch merged to viewer-development

[Robert "Exile In Paradise" Murphey]

On Tue, 2011-05-17 at 21:35 -0500, Ardy Lay wrote:
> ~~
>   Loading Shaders:
> ~~
> 2011-05-18T02:32:15Z INFO: LLShaderMgr::loadShaderFile: Loading file: 
> shaders/class2/windlight/atmosphericsVarsV.glsl (Want class 2)
> ...
> 2011-05-18T02:32:16Z INFO: LLShaderMgr::loadShaderFile: Loading file: 
> shaders/class2/deferred/edgeF.glsl (Want class 2)
> 2011-05-18T02:32:16Z llrender/llrendertarget.cpp(694) : error
> 2011-05-18T02:32:16Z ERROR: LLMultisampleBuffer::addColorAttachment: 
> WTF? 8cd6
> 
> That's nice.
> 
> People are getting this WTF? error message when attempting to activate 
> lights and shadows.

It can also happen at login if you already have deferred
render/shadows and lights enabled, and it always happens
after the login state is pretty well along, just as the
3D geometry is getting setup to start rendering.

On Linux, I have to log out of my GUI and log back into it in
order to reset the video driver/OpenSL before I can login again
once that "addColorAttachment" error starts. Otherwise
I get that error on every login attempt once it starts.

I reported this a while back to Armin on the Imprudence
viewer dev team with backtraces and crash logs, since it
happened to me for the first time  when running Imprudence
experimental.

As for developers and their "colorful" comments, that's
pretty tame. There's been some interesting reports from
SourceForge and GitHub on the statistics of how often
certain "colorful comments" appear by developers.

-- 
Robert "Exile In Paradise" Murphey
If I traveled to the end of the rainbow As Dame Fortune did
intend, Murphy would be there to tell me The pot's at the
other end. -- Bert Whitney

___
Policies and (un)subscribe information available here:
http://wiki.secondlife.com/wiki/OpenSource-Dev
Please read the policies before posting to keep unmoderated posting privileges

Re: [opensource-dev] Mesh branch merged to viewer-development

[Ardy Lay]

On 5/17/2011 10:57 PM, Robert "Exile In Paradise" Murphey wrote:
> On Tue, 2011-05-17 at 21:35 -0500, Ardy Lay wrote:
>> ~~
>>   Loading Shaders:
>> ~~
>> 2011-05-18T02:32:15Z INFO: LLShaderMgr::loadShaderFile: Loading file: 
>> shaders/class2/windlight/atmosphericsVarsV.glsl (Want class 2)
>> ...
>> 2011-05-18T02:32:16Z INFO: LLShaderMgr::loadShaderFile: Loading file: 
>> shaders/class2/deferred/edgeF.glsl (Want class 2)
>> 2011-05-18T02:32:16Z llrender/llrendertarget.cpp(694) : error
>> 2011-05-18T02:32:16Z ERROR: LLMultisampleBuffer::addColorAttachment: 
>> WTF? 8cd6
>>
>> That's nice.
>>
>> People are getting this WTF? error message when attempting to activate 
>> lights and shadows.
> It can also happen at login if you already have deferred
> render/shadows and lights enabled, and it always happens
> after the login state is pretty well along, just as the
> 3D geometry is getting setup to start rendering.
>
> On Linux, I have to log out of my GUI and log back into it in
> order to reset the video driver/OpenSL before I can login again
> once that "addColorAttachment" error starts. Otherwise
> I get that error on every login attempt once it starts.
>
> I reported this a while back to Armin on the Imprudence
> viewer dev team with backtraces and crash logs, since it
> happened to me for the first time  when running Imprudence
> experimental.
>
> As for developers and their "colorful" comments, that's
> pretty tame. There's been some interesting reports from
> SourceForge and GitHub on the statistics of how often
> certain "colorful comments" appear by developers.
>
This one is getting attention because the code is deliberately
referencing a null pointer afterward to cause a crash.  I guess this
means that mister WTF? wants the condition reported.
___
Policies and (un)subscribe information available here:
http://wiki.secondlife.com/wiki/OpenSource-Dev
Please read the policies before posting to keep unmoderated posting privileges