Re: [opensource-dev] autobuild & VS 2010 support merged to viewer-development

[Oz Linden (Scott Lawrence)]

On 2011-04-03 0:18, Nicky Perian wrote:

Development viewer installed.
1. crashed at startup because no settings_development.xml delivered 
from installer.

2. copied settings_lindendeveloper.xml to settings_development.xml
3. logged to aditi. login process took more than a minute maybe close 
to 2 minutes.

4. 10 minutes into online and still a ghost.
5. attempted to tp home to resolve ghost of different sim and crashed.


You didn't specify which platform you installed, or which version (there 
have been 4 since I posted my note).


___
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] autobuild & VS 2010 support merged to viewer-development

[Mike Chase]

On Sat, 2011-04-02 at 23:17 -0400, Oz Linden (Scott Lawrence) wrote: 
> On 2011-04-01 20:56, Oz Linden (Scott Lawrence) wrote:
> > On 2011-04-01 18:08, Oz Linden (Scott Lawrence) wrote:
> >> The autobuild and Visual Studio 2010 branch has been merged to 
> >> viewer-development
> >
> > This is a big change in the tools and many of the libraries that go 
> > into the viewer, so please download a Development viewer for your 
> > weekend fun and let us know how it works for you.
> 
> 
> As much as I want everyone to be able to build this new version (and as 
> I said, we'll be making that easier for open developers shortly), I'd 
> love to see some reports that people are just downloading the 
> Development viewer and running our build to let us know what's working 
> and what isn't.

Quick pass through on Linux and all seemed ok. Profiles worked fine and
the annoying pause that previouly made the viewer unusable seems fixed.
I'll use it this evening for an event I'm hosting so it will get a
workout and I'll post more experiences then.

Mike

> https://wiki.secondlife.com/wiki/Snowstorm_Team
> 
> or
> 
> https://wiki.secondlife.com/wiki/Downloading_test_builds#Development_Integration
> ___
> 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] autobuild & VS 2010 support merged to viewer-development

[Oz Linden (Scott Lawrence)]

On 2011-04-03 9:27, Mike Chase wrote:
> Quick pass through on Linux and all seemed ok. Profiles worked fine and
> the annoying pause that previouly made the viewer unusable seems fixed.

Glad to hear that the problem there was in how some of the libraries 
were built.  Fortunately, in our new build structure it will be much 
easier for open developers to do library builds the same we that we do, 
and therefor to contribute improvements and upgrades.

> I'll use it this evening for an event I'm hosting so it will get a
> workout and I'll post more experiences then

Thanks!

___
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] autobuild & VS 2010 support merged to viewer-development

[Nicky Perian]

From: Oz Linden (Scott Lawrence) 
To: Nicky Perian 
Cc: opensource-dev 
Sent: Sun, April 3, 2011 8:27:02 AM
Subject: Re: [opensource-dev] autobuild & VS 2010 support merged to 
viewer-development

On 2011-04-03 0:18, Nicky Perian wrote: 
Development viewer installed.
>1. crashed at startup because no settings_development.xml 
>delivered 
>from installer.
>2. copied settings_lindendeveloper.xml to settings_development.xml
>3. logged to aditi. login process took more than a minute maybe 
>close to 2 minutes.
>4. 10 minutes into online and still a ghost.
>
>5. attempted to tp home to resolve ghost of different sim and 
>crashed.
>
You didn't specify which platform you installed, or which version (there 
have been 4 since I posted my note).
sorry,
225662 win7/64bit
___
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] autobuild & VS 2010 support merged to viewer-development

[Kiptic ‌]

> > Building from the VS command prompt? Not so lucky. I get this:

> > devenv.com is not an internal command

> What were you trying to build. viewer-development shouldn't cause that
> error.  You will run into that error trying to build the 3p-* libraries,
> as it uses a couple functions still coded to use devenv :-( This is only
> a problem for Express users as the Pro and higher version include
> devenv.com.

It was viewer-development, but the problem was on my end not the code. After 
running the VS command prompt as Admin and fiddling with my PATH it went away. 
Now everything builds straight out of the box (well, save for that script I 
needed to put in the Python Scripts dir)!

> On #opensl, we've been using autobuild for several weeks now. Please
> feel free to come ask for help and use our experience with it ;-)

Great, thanks :)

/Kip

  ___
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: STORM-1122 Linux viewer sucks up file descriptors, stops loading content and crashes

[Merov Linden]

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


I'm thinking the fix should go deeper and, possibly, get rid of 
getDynamicFallbackFontList() entirely (see the equivalent code for Mac and 
Windows). Would be good to have some other Linux devs chiming in.


indra/llwindow/llwindow.cpp


Hmmm, this is basically doing for Linux what is already done for Mac and 
Win32. If that's the case, I'd rather have your hack move to llwindowsdl.cpp. 
It might be time to retire that getDynamicFallbackFontList() code entirely then.


- Merov


On March 30, 2011, 11:49 a.m., Discrete Dreamscape wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> http://codereview.secondlife.com/r/245/
> ---
> 
> (Updated March 30, 2011, 11:49 a.m.)
> 
> 
> Review request for Viewer.
> 
> 
> Summary
> ---
> 
> Resolved Linux file descriptor greediness by removing obsolete fallback font 
> searching (call to LLWindowSDL::getDynamicFallbackFontList()), as this seems 
> to be obsolete unless your skin's configuration references font files that 
> are not packaged with the viewer, which is not the default case. Would like 
> to know if this solves the instability described in STORM-1122 for Linux 
> users, particularly those with lower than average file descriptor limits set 
> (find out by running `ulimit -a`, it's the value 'open files', mine is 1024 
> on Ubuntu 10.10 and I'd have extreme problems prior to the patch).
> 
> 
> This addresses bug STORM-1122.
> http://jira.secondlife.com/browse/STORM-1122
> 
> 
> Diffs
> -
> 
>   doc/contributions.txt a8f868007986 
>   indra/llwindow/llwindow.cpp a8f868007986 
> 
> Diff: http://codereview.secondlife.com/r/245/diff
> 
> 
> Testing
> ---
> 
> Useful ways to examine file descriptor usage for the viewer
> 
> lsof -c do-not | less
> 
> This should be much, much less than 1024
> lsof -c do-not | wc -l
> 
> This shows all descriptors containing the word 'font' along with the number 
> of each (there are tons of duplicates)
> lsof -c do-not | egrep -o '[^ ]*font[^ ]*' | sort | uniq -c | less
> 
> 
> Thanks,
> 
> Discrete
> 
>

___
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: STORM-1095 Chat preferences > font size should increase size of input text in the chat box

[Merov Linden]

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


Boroondas comments should be addressed, especially the one here under...


indra/newview/llviewerchat.cpp


Should be initialized to NULL.


- Merov


On March 30, 2011, 5:43 a.m., Jonathan Yap wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> http://codereview.secondlife.com/r/244/
> ---
> 
> (Updated March 30, 2011, 5:43 a.m.)
> 
> 
> Review request for Viewer.
> 
> 
> Summary
> ---
> 
>  Chat preferences > font size should increase size of input text in the chat 
> box
> 
> 
> This addresses bug STORM-1095.
> http://jira.secondlife.com/browse/STORM-1095
> 
> 
> Diffs
> -
> 
>   doc/contributions.txt 65ff7415f171 
>   indra/llui/lllineeditor.h 65ff7415f171 
>   indra/llui/lllineeditor.cpp 65ff7415f171 
>   indra/newview/llbottomtray.cpp 65ff7415f171 
>   indra/newview/llfloaterpreference.cpp 65ff7415f171 
>   indra/newview/llnearbychatbar.h 65ff7415f171 
>   indra/newview/llnearbychatbar.cpp 65ff7415f171 
>   indra/newview/llviewerchat.h 65ff7415f171 
>   indra/newview/llviewerchat.cpp 65ff7415f171 
> 
> Diff: http://codereview.secondlife.com/r/244/diff
> 
> 
> Testing
> ---
> 
> Change font size in preferences and see
> 1) Font size in chat input box changes to new size immediately
> 2) Font size is set to selected size when viewer is restarted
> 
> 
> Thanks,
> 
> Jonathan
> 
>

___
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: STORM-1122 Linux viewer sucks up file descriptors, stops loading content and crashes

[Discrete Dreamscape]

> On April 3, 2011, 8:29 p.m., Merov Linden wrote:
> > indra/llwindow/llwindow.cpp, line 210
> > 
> >
> > Hmmm, this is basically doing for Linux what is already done for Mac 
> > and Win32. If that's the case, I'd rather have your hack move to 
> > llwindowsdl.cpp. It might be time to retire that 
> > getDynamicFallbackFontList() code entirely then.

After speaking to a friend and looking a little deeper, it appears that the 
function does have some reasonable purpose.. the font files included in the 
viewer do not have substantial Unicode support, which I discovered when trying 
to input Japanese. On Windows/Mac, there are sections clearly defining fonts 
that provide this support, but no such fonts exist for Linux, perhaps because 
it's not certain that they're on the system. I recall from the past that Arial 
Unicode isn't a completely free font and can't be included by default.. So I 
guess the proper solution would actually first be to either add Linux (or 
universal) fonts that provide proper support, or dig deeper into the fallback 
font function to figure out why it's going descriptor-crazy. It's a pretty 
hairy-looking thing..


- Discrete


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


On March 30, 2011, 11:49 a.m., Discrete Dreamscape wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> http://codereview.secondlife.com/r/245/
> ---
> 
> (Updated March 30, 2011, 11:49 a.m.)
> 
> 
> Review request for Viewer.
> 
> 
> Summary
> ---
> 
> Resolved Linux file descriptor greediness by removing obsolete fallback font 
> searching (call to LLWindowSDL::getDynamicFallbackFontList()), as this seems 
> to be obsolete unless your skin's configuration references font files that 
> are not packaged with the viewer, which is not the default case. Would like 
> to know if this solves the instability described in STORM-1122 for Linux 
> users, particularly those with lower than average file descriptor limits set 
> (find out by running `ulimit -a`, it's the value 'open files', mine is 1024 
> on Ubuntu 10.10 and I'd have extreme problems prior to the patch).
> 
> 
> This addresses bug STORM-1122.
> http://jira.secondlife.com/browse/STORM-1122
> 
> 
> Diffs
> -
> 
>   doc/contributions.txt a8f868007986 
>   indra/llwindow/llwindow.cpp a8f868007986 
> 
> Diff: http://codereview.secondlife.com/r/245/diff
> 
> 
> Testing
> ---
> 
> Useful ways to examine file descriptor usage for the viewer
> 
> lsof -c do-not | less
> 
> This should be much, much less than 1024
> lsof -c do-not | wc -l
> 
> This shows all descriptors containing the word 'font' along with the number 
> of each (there are tons of duplicates)
> lsof -c do-not | egrep -o '[^ ]*font[^ ]*' | sort | uniq -c | less
> 
> 
> Thanks,
> 
> Discrete
> 
>

___
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