Crystal Space 3D SDK

Hmm. Are you sure that's not working? The PD lighting effects in
flarge (i.e. the flashing in the street
and so on) appeared to be working?

Greetings,

On Thu, Sep 25, 2008 at 8:04 PM, res <resqu@...> wrote:
> On 25.09.2008 19:34, Jorrit Tyberghein wrote:
>> um, what does ptpdlight do actually?
>
> Apply PD lights to lightmaps.
>
> -f.r.
>
> -------------------------------------------------------------------------
> This SF.Net email is sponsored by the Moblin Your Move Developer's challenge
> Build the coolest Linux based applications with Moblin SDK & win great prizes
> Grand prize is a trip for two to an Open Source event anywhere in the world
> http://moblin-contest.org/redirect.php?banner_id=100&url=/
> _______________________________________________
> Crystal-develop mailing list
> Crystal-develop@...
> https://lists.sourceforge.net/lists/listinfo/crystal-develop
>



-- 
Project Manager of Crystal Space (http://www.crystalspace3d.org)
and CEL (http://cel.crystalspace3d.org)
Support Crystal Space. Donate at
https://sourceforge.net/donate/index.php?group_id=649
Visit my town at http://waldir.myminicity.com/

On 25.09.2008 19:34, Jorrit Tyberghein wrote:
> um, what does ptpdlight do actually?

Apply PD lights to lightmaps.

-f.r.

um, what does ptpdlight do actually?

Greetings,

On Thu, Sep 25, 2008 at 5:00 PM, res <resqu@...> wrote:
> On 25.09.2008 15:12, jorrit@... wrote:
>> - Removed iLightingInfo interface.
>
> I think at least the use of that in ptpdlight has to be reimplemented
> using some other mechanism. (Light change callback perhaps?)
>
> -f.r.
>
> -------------------------------------------------------------------------
> This SF.Net email is sponsored by the Moblin Your Move Developer's challenge
> Build the coolest Linux based applications with Moblin SDK & win great prizes
> Grand prize is a trip for two to an Open Source event anywhere in the world
> http://moblin-contest.org/redirect.php?banner_id=100&url=/
> _______________________________________________
> Crystal-develop mailing list
> Crystal-develop@...
> https://lists.sourceforge.net/lists/listinfo/crystal-develop
>



-- 
Project Manager of Crystal Space (http://www.crystalspace3d.org)
and CEL (http://cel.crystalspace3d.org)
Support Crystal Space. Donate at
https://sourceforge.net/donate/index.php?group_id=649
Visit my town at http://waldir.myminicity.com/

On 25.09.2008 15:12, jorrit@... wrote:
> - Removed iLightingInfo interface.

I think at least the use of that in ptpdlight has to be reimplemented 
using some other mechanism. (Light change callback perhaps?)

-f.r.

jorrit@... wrote:
> Revision: 31259
> Author:   jorrit
> Modified: CS/trunk/apps/tutorial/Jamfile
>  SubInclude TOP apps tutorial simple2 ;
> -#SubInclude TOP apps tutorial simplept ;
> +;SubInclude TOP apps tutorial simplept ;
> +SubInclude TOP apps tutorial shadertut ;
>  SubInclude TOP apps tutorial simpmap ;

This Jamfile modification does not look correct. Semicolon should not 
appear before SubInclude like that.

-- ES

On 23.09.2008 15:53, mgist@... wrote:
>  template<typename Tag>
> -class StringHash
> +class CS_CRYSTALSPACE_EXPORT StringHash

This is a templated class. Templated and exported generally don't mix.
(Specifically, iirc DLL import breaks on MSVC b/c the linker tries to
import an instantiation which isn't actually exported from anywhere.)

Why is the EXPORT needed? If there are non-inlined methods they should
be inlined.

-f.r.

So perhaps this is a stupid question, but why is it called shagnetron?

~Scott

Thus Spoke res2002@...:
> Revision: 31240
>           http://crystal.svn.sourceforge.net/crystal/?rev=31240&view=rev
> Author:   res2002
> Date:     2008-09-21 16:50:06 +0000 (Sun, 21 Sep 2008)
>
> Log Message:
> -----------
> Make shagnetron also support precaching specific shader files; Add cache clearing; Add cache directory controls
>
> Modified Paths:
> --------------
>     CS/trunk/apps/tools/shagnetron/shagnetron.cpp
>
> Modified: CS/trunk/apps/tools/shagnetron/shagnetron.cpp
> ===================================================================
> --- CS/trunk/apps/tools/shagnetron/shagnetron.cpp	2008-09-21 16:49:07 UTC (rev 31239)
> +++ CS/trunk/apps/tools/shagnetron/shagnetron.cpp	2008-09-21 16:50:06 UTC (rev 31240)
> @@ -135,17 +135,15 @@
>  {
>    csRef<iCommandLineParser> cmdline = csQueryRegistry<iCommandLineParser> (object_reg);
>    
> -  csFIFO<csString> dirsToScan;
> +  csFIFO<csString> toScan;
>    for (size_t i = 0; ; i++)
>    {
>      csString dir (cmdline->GetName (i));
>      if (dir.IsEmpty()) break;
>      
> -    if (dir[dir.Length()-1] != '/')
> -      dir += "/";
> -    dirsToScan.Push (dir);
> +    toScan.Push (dir);
>    }
> -  if (dirsToScan.GetSize() == 0)
> +  if (toScan.GetSize() == 0)
>    {
>      PrintHelp ();
>      return false;
> @@ -154,22 +152,47 @@
>    docsys = csQueryRegistry<iDocumentSystem> (object_reg);
>    vfs = csQueryRegistry<iVFS> (object_reg);
>    shaderMgr = csQueryRegistry<iShaderManager> (object_reg);
> -  while (dirsToScan.GetSize() > 0)
> +  iHierarchicalCache* shaderCache = shaderMgr->GetShaderCache();
> +  if (!shaderCache)
>    {
> -    csString dir (dirsToScan.PopTop());
> +    csReport (object_reg, CS_REPORTER_SEVERITY_ERROR,
> +    	"crystalspace.application.shagnetron",
> +	"Shader cache is disabled. Please enable to use this tool");
> +    // @@@ It might be sensible to forcibly enable shader caching
> +    return false;
> +  }
> +  
> +  if (cmdline->GetBoolOption ("nodefaultcaches", false))
> +    shaderMgr->RemoveAllSubShaderCaches();
> +  for (size_t i = 0; ; i++)
> +  {
> +    const char* opt = cmdline->GetOption ("cachedir", i);
> +    if (!opt || !*opt) break;
>      
> -    csRef<iStringArray> vfsFiles (vfs->FindFiles (dir));
> -    for (size_t i = 0; i < vfsFiles->GetSize(); i++)
> +    shaderMgr->AddSubCacheDirectory (opt,
> +      iShaderManager::cachePriorityHighest);
> +  }
> +  if (cmdline->GetBoolOption ("cacheclear", false))
> +    shaderCache->ClearCache ("/");
> +  
> +  while (toScan.GetSize() > 0)
> +  {
> +    csString obj (toScan.PopTop());
> +    
> +    if (obj[obj.Length()-1] == '/')
>      {
> -      const char* file = vfsFiles->Get (i);
> -      
> -      if (FileBlacklisted (file)) continue;
> -      
> -      if (file[strlen(file)-1] == '/')
> -        dirsToScan.Push (file);
> -      else
> -        PrecacheShaderFile (file);
> +      csRef<iStringArray> vfsFiles (vfs->FindFiles (obj));
> +      for (size_t i = 0; i < vfsFiles->GetSize(); i++)
> +      {
> +	const char* file = vfsFiles->Get (i);
> +	
> +	if (FileBlacklisted (file)) continue;
> +	
> +	toScan.Push (file);
> +      }
>      }
> +    else
> +      PrecacheShaderFile (obj);
>    }
>  
>    return true;
> @@ -179,13 +202,19 @@
>  {
>    const char* appname = "shagnetron";
>  
> -  csPrintf ("Usage: %s <VFS directory> <VFS directory> <...>\n", appname);
> +  csPrintf ("Usage: %s [options] <VFS directory | shader file name> <...>\n", appname);
>    csPrintf ("\n");
>    csPrintf ("Pre-warms a shader cache with all the shaders from the given directories.\n");
>    csPrintf ("\n");
>    csPrintf ("The name is a portmanteau of 'shader' and 'magnetron'.\n");
>    csPrintf ("\n");
> -  //csPrintf ("Options:\n");
> +  csPrintf ("Available options:\n");
> +  csPrintf (" -cachedir         Specify additional cache directories.\n");
> +  csPrintf ("                   The later a directory is specified the lower its priority.\n");
> +  csPrintf ("                   The first directory is written to.\n");
> +  csPrintf (" -nodefaultcaches  Do not use default caches set in config\n");
> +  csPrintf (" -cacheclear       Clear used caches\n");
> +  csPrintf ("\n");
>  }
>  
>  /*---------------------------------------------------------------------*
>
>
> This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.
>
> -------------------------------------------------------------------------
> This SF.Net email is sponsored by the Moblin Your Move Developer's challenge
> Build the coolest Linux based applications with Moblin SDK & win great prizes
> Grand prize is a trip for two to an Open Source event anywhere in the world
> http://moblin-contest.org/redirect.php?banner_id=100&url=/
> _______________________________________________
> Crystal-cvs mailing list
> Crystal-cvs@...
> https://lists.sourceforge.net/lists/listinfo/crystal-cvs
>

I should add that all of the mentioned standard roots are handled 
automatically by CS_CHECK_LIB_WITH(), which is why it makes sense to 
write the test in a way that allows CS_CHECK_LIB_WITH() to work properly 
rather than employing it incorrectly and tacking a single case on top of 
it, which works for only one possible configuration.

-- ES

Eric Sunshine wrote:
> It is a special case because it assumes a root of /usr/local/include. It 
> does not take into account any of the other standard roots, such as 
> /usr/include, /opt/include, /opt/local/include, /sw/include, 
> /sw/local/include, etc.
> 
> -- ES
> 
> Mike Gist wrote:
>>  From what I can gather this changes nothing except *adding* a extra 
>> directory to look in.
>> That dir is also how bullet installs itself, so I don't see how it's a 
>> special case.
>>
>> 2008/9/18 Eric Sunshine <sunshine@... 
>> <mailto:sunshine@...>>
>>
>>     If that is the case, then the configure test should have worked
>>     without any change. The change to which I objected actually assumes
>>     that headers are in a "bullet" subdirectory.
>>
>>     If the real issue is that the headers are "randomly" installed
>>     sometimes in a "bullet" subdirectory and sometimes not, then that is a
>>     more difficult issue to solve, but the committed patch is still a bad
>>     special-purpose hack. A more generalized solution is warranted.
>>
>>     -- ES
>>
>>     On Thu, Sep 18, 2008 at 7:26 AM, Jorrit Tyberghein
>>     <jorrit.tyberghein@... <mailto:jorrit.tyberghein@...>>
>>     wrote:
>>      > I don't believe that's possible as in some situations the bullet
>>      > includes are not installed
>>      > in a directory relative to 'bullet'. i.e. it is not always
>>     'bullet/bla.h'
>>      >
>>      > Greetings,
>>      >
>>      > On Thu, Sep 18, 2008 at 1:24 PM, Eric Sunshine
>>     <sunshine@... <mailto:sunshine@...>> wrote:
>>      >> This is the incorrect approach. It is essentially a special-case
>>     hack
>>      >> which will work for only one configuration. Please revert. To
>>     fix this
>>      >> properly, the "bullet" prefix should be mentioned in the #include
>>      >> statements (also, it is more proper to use #include <...> rather
>>     than
>>      >> #include "..."). Finally, source code in CS itself should also
>>     use the
>>      >> "bullet" prefix in #include <...> statements.
>>      >>
>>      >> -- ES
>>      >>
>>      >>
>>      >> mgist@... <mailto:mgist@...>
>>     wrote:
>>      >>> Revision: 31196
>>      >>>          
>>     http://crystal.svn.sourceforge.net/crystal/?rev=31196&view=rev
>>     <http://crystal.svn.sourceforge.net/crystal/?rev=31196&view=rev>
>>      >>> Author:   mgist
>>      >>> Date:     2008-09-18 09:27:24 +0000 (Thu, 18 Sep 2008)
>>      >>>
>>      >>> Log Message:
>>      >>> -----------
>>      >>> - Added an extra search path for libbullet.
>>      >>>
>>      >>> Modified Paths:
>>      >>> --------------
>>      >>>     CS/trunk/configure
>>      >>>     CS/trunk/configure.ac <http://configure.ac>
>>      >>>
>>      >>> Modified: CS/trunk/configure
>>      >>> ===================================================================
>>      >>> --- CS/trunk/configure        2008-09-18 08:12:22 UTC (rev 31195)
>>      >>> +++ CS/trunk/configure        2008-09-18 09:27:24 UTC (rev 31196)
>>      >>> @@ -53037,7 +53037,7 @@
>>      >>>    cs_check_lib_paths="| /usr/local/include|/usr/local/lib \
>>      >>>      /sw/include|/sw/lib \
>>      >>>      /opt/local/include|/opt/local/lib \
>>      >>> -    /opt/include|/opt/lib "
>>      >>> +    /opt/include|/opt/lib /usr/local/include/bullet/"
>>      >>>  fi
>>      >>>
>>      >>>       for cs_lib_item in $cs_check_lib_paths
>>      >>>
>>      >>> Modified: CS/trunk/configure.ac <http://configure.ac>
>>      >>> ===================================================================
>>      >>> --- CS/trunk/configure.ac <http://configure.ac>     2008-09-18
>>     08:12:22 UTC (rev 31195)
>>      >>> +++ CS/trunk/configure.ac <http://configure.ac>     2008-09-18
>>     09:27:24 UTC (rev 31196)
>>      >>> @@ -1347,7 +1347,7 @@
>>      >>>        btSequentialImpulseConstraintSolver* s;
>>      >>>        btDefaultCollisionConfiguration* c;
>>      >>>        btDynamicsWorld* world = new btDiscreteDynamicsWorld (d,
>>     b, s, c)])],
>>      >>> -    [], [C++], [], [],
>>      >>> +    [/usr/local/include/bullet/], [C++], [], [],
>>      >>>      [], [], [-lbulletdynamics -lbulletcollision -lbulletmath],
>>      >>>      [bulletdynamics])
>>      >>
>>      >>
>>     -------------------------------------------------------------------------
>>      >> This SF.Net email is sponsored by the Moblin Your Move
>>     Developer's challenge
>>      >> Build the coolest Linux based applications with Moblin SDK & win
>>     great prizes
>>      >> Grand prize is a trip for two to an Open Source event anywhere
>>     in the world
>>      >> http://moblin-contest.org/redirect.php?banner_id=100&url=/
>>     <http://moblin-contest.org/redirect.php?banner_id=100&url=/>
>>      >> _______________________________________________
>>      >> Crystal-develop mailing list
>>      >> Crystal-develop@...
>>     <mailto:Crystal-develop@...>
>>      >> https://lists.sourceforge.net/lists/listinfo/crystal-develop
>>      >>
>>      >
>>      >
>>      >
>>      > --
>>      > Project Manager of Crystal Space (http://www.crystalspace3d.org)
>>      > and CEL (http://cel.crystalspace3d.org)
>>      > Support Crystal Space. Donate at
>>      > https://sourceforge.net/donate/index.php?group_id=649
>>      > Visit my town at http://waldir.myminicity.com/
>>      >
>>      >
>>     -------------------------------------------------------------------------
>>      > This SF.Net email is sponsored by the Moblin Your Move
>>     Developer's challenge
>>      > Build the coolest Linux based applications with Moblin SDK & win
>>     great prizes
>>      > Grand prize is a trip for two to an Open Source event anywhere in
>>     the world
>>      > http://moblin-contest.org/redirect.php?banner_id=100&url=/
>>     <http://moblin-contest.org/redirect.php?banner_id=100&url=/>
>>      > _______________________________________________
>>      > Crystal-develop mailing list
>>      > Crystal-develop@...
>>     <mailto:Crystal-develop@...>
>>      > https://lists.sourceforge.net/lists/listinfo/crystal-develop
>>      >
>>
>>     -------------------------------------------------------------------------
>>     This SF.Net email is sponsored by the Moblin Your Move Developer's
>>     challenge
>>     Build the coolest Linux based applications with Moblin SDK & win
>>     great prizes
>>     Grand prize is a trip for two to an Open Source event anywhere in
>>     the world
>>     http://moblin-contest.org/redirect.php?banner_id=100&url=/
>>     <http://moblin-contest.org/redirect.php?banner_id=100&url=/>
>>     _______________________________________________
>>     Crystal-develop mailing list
>>     Crystal-develop@...
>>     <mailto:Crystal-develop@...>
>>     https://lists.sourceforge.net/lists/listinfo/crystal-develop
>>
>>
>>
>>
>> -- 
>> -Mike
>>
>>
>> ------------------------------------------------------------------------
>>
>> -------------------------------------------------------------------------
>> This SF.Net email is sponsored by the Moblin Your Move Developer's challenge
>> Build the coolest Linux based applications with Moblin SDK & win great prizes
>> Grand prize is a trip for two to an Open Source event anywhere in the world
>> http://moblin-contest.org/redirect.php?banner_id=100&url=/
>>
>>
>> ------------------------------------------------------------------------
>>
>> _______________________________________________
>> Crystal-develop mailing list
>> Crystal-develop@...
>> https://lists.sourceforge.net/lists/listinfo/crystal-develop
> 
> 
> -------------------------------------------------------------------------
> This SF.Net email is sponsored by the Moblin Your Move Developer's challenge
> Build the coolest Linux based applications with Moblin SDK & win great prizes
> Grand prize is a trip for two to an Open Source event anywhere in the world
> http://moblin-contest.org/redirect.php?banner_id=100&url=/
> _______________________________________________
> Crystal-develop mailing list
> Crystal-develop@...
> https://lists.sourceforge.net/lists/listinfo/crystal-develop
>

Okay yes, that makes sense. I'll revert it now.

2008/9/18 Eric Sunshine <sunshine@...>

> It is a special case because it assumes a root of /usr/local/include. It
> does not take into account any of the other standard roots, such as
> /usr/include, /opt/include, /opt/local/include, /sw/include,
> /sw/local/include, etc.
>
> -- ES
>
> Mike Gist wrote:
> >  From what I can gather this changes nothing except *adding* a extra
> > directory to look in.
> > That dir is also how bullet installs itself, so I don't see how it's a
> > special case.
> >
> > 2008/9/18 Eric Sunshine <sunshine@...
> > <mailto:sunshine@...>>
> >
> >     If that is the case, then the configure test should have worked
> >     without any change. The change to which I objected actually assumes
> >     that headers are in a "bullet" subdirectory.
> >
> >     If the real issue is that the headers are "randomly" installed
> >     sometimes in a "bullet" subdirectory and sometimes not, then that is
> a
> >     more difficult issue to solve, but the committed patch is still a bad
> >     special-purpose hack. A more generalized solution is warranted.
> >
> >     -- ES
> >
> >     On Thu, Sep 18, 2008 at 7:26 AM, Jorrit Tyberghein
> >     <jorrit.tyberghein@... <mailto:jorrit.tyberghein@...>>
> >     wrote:
> >      > I don't believe that's possible as in some situations the bullet
> >      > includes are not installed
> >      > in a directory relative to 'bullet'. i.e. it is not always
> >     'bullet/bla.h'
> >      >
> >      > Greetings,
> >      >
> >      > On Thu, Sep 18, 2008 at 1:24 PM, Eric Sunshine
> >     <sunshine@... <mailto:sunshine@...>> wrote:
> >      >> This is the incorrect approach. It is essentially a special-case
> >     hack
> >      >> which will work for only one configuration. Please revert. To
> >     fix this
> >      >> properly, the "bullet" prefix should be mentioned in the #include
> >      >> statements (also, it is more proper to use #include <...> rather
> >     than
> >      >> #include "..."). Finally, source code in CS itself should also
> >     use the
> >      >> "bullet" prefix in #include <...> statements.
> >      >>
> >      >> -- ES
> >      >>
> >      >>
> >      >> mgist@... <mailto:mgist@...>
> >     wrote:
> >      >>> Revision: 31196
> >      >>>
> >     http://crystal.svn.sourceforge.net/crystal/?rev=31196&view=rev
> >     <http://crystal.svn.sourceforge.net/crystal/?rev=31196&view=rev>
> >      >>> Author:   mgist
> >      >>> Date:     2008-09-18 09:27:24 +0000 (Thu, 18 Sep 2008)
> >      >>>
> >      >>> Log Message:
> >      >>> -----------
> >      >>> - Added an extra search path for libbullet.
> >      >>>
> >      >>> Modified Paths:
> >      >>> --------------
> >      >>>     CS/trunk/configure
> >      >>>     CS/trunk/configure.ac <http://configure.ac>
> >      >>>
> >      >>> Modified: CS/trunk/configure
> >      >>>
> ===================================================================
> >      >>> --- CS/trunk/configure        2008-09-18 08:12:22 UTC (rev
> 31195)
> >      >>> +++ CS/trunk/configure        2008-09-18 09:27:24 UTC (rev
> 31196)
> >      >>> @@ -53037,7 +53037,7 @@
> >      >>>    cs_check_lib_paths="| /usr/local/include|/usr/local/lib \
> >      >>>      /sw/include|/sw/lib \
> >      >>>      /opt/local/include|/opt/local/lib \
> >      >>> -    /opt/include|/opt/lib "
> >      >>> +    /opt/include|/opt/lib /usr/local/include/bullet/"
> >      >>>  fi
> >      >>>
> >      >>>       for cs_lib_item in $cs_check_lib_paths
> >      >>>
> >      >>> Modified: CS/trunk/configure.ac <http://configure.ac>
> >      >>>
> ===================================================================
> >      >>> --- CS/trunk/configure.ac <http://configure.ac>     2008-09-18
> >     08:12:22 UTC (rev 31195)
> >      >>> +++ CS/trunk/configure.ac <http://configure.ac>     2008-09-18
> >     09:27:24 UTC (rev 31196)
> >      >>> @@ -1347,7 +1347,7 @@
> >      >>>        btSequentialImpulseConstraintSolver* s;
> >      >>>        btDefaultCollisionConfiguration* c;
> >      >>>        btDynamicsWorld* world = new btDiscreteDynamicsWorld (d,
> >     b, s, c)])],
> >      >>> -    [], [C++], [], [],
> >      >>> +    [/usr/local/include/bullet/], [C++], [], [],
> >      >>>      [], [], [-lbulletdynamics -lbulletcollision -lbulletmath],
> >      >>>      [bulletdynamics])
> >      >>
> >      >>
> >
> -------------------------------------------------------------------------
> >      >> This SF.Net email is sponsored by the Moblin Your Move
> >     Developer's challenge
> >      >> Build the coolest Linux based applications with Moblin SDK & win
> >     great prizes
> >      >> Grand prize is a trip for two to an Open Source event anywhere
> >     in the world
> >      >> http://moblin-contest.org/redirect.php?banner_id=100&url=/
> >     <http://moblin-contest.org/redirect.php?banner_id=100&url=/>
> >      >> _______________________________________________
> >      >> Crystal-develop mailing list
> >      >> Crystal-develop@...
> >     <mailto:Crystal-develop@...>
> >      >> https://lists.sourceforge.net/lists/listinfo/crystal-develop
> >      >>
> >      >
> >      >
> >      >
> >      > --
> >      > Project Manager of Crystal Space (http://www.crystalspace3d.org)
> >      > and CEL (http://cel.crystalspace3d.org)
> >      > Support Crystal Space. Donate at
> >      > https://sourceforge.net/donate/index.php?group_id=649
> >      > Visit my town at http://waldir.myminicity.com/
> >      >
> >      >
> >
> -------------------------------------------------------------------------
> >      > This SF.Net email is sponsored by the Moblin Your Move
> >     Developer's challenge
> >      > Build the coolest Linux based applications with Moblin SDK & win
> >     great prizes
> >      > Grand prize is a trip for two to an Open Source event anywhere in
> >     the world
> >      > http://moblin-contest.org/redirect.php?banner_id=100&url=/
> >     <http://moblin-contest.org/redirect.php?banner_id=100&url=/>
> >      > _______________________________________________
> >      > Crystal-develop mailing list
> >      > Crystal-develop@...
> >     <mailto:Crystal-develop@...>
> >      > https://lists.sourceforge.net/lists/listinfo/crystal-develop
> >      >
> >
> >
> -------------------------------------------------------------------------
> >     This SF.Net email is sponsored by the Moblin Your Move Developer's
> >     challenge
> >     Build the coolest Linux based applications with Moblin SDK & win
> >     great prizes
> >     Grand prize is a trip for two to an Open Source event anywhere in
> >     the world
> >     http://moblin-contest.org/redirect.php?banner_id=100&url=/
> >     <http://moblin-contest.org/redirect.php?banner_id=100&url=/>
> >     _______________________________________________
> >     Crystal-develop mailing list
> >     Crystal-develop@...
> >     <mailto:Crystal-develop@...>
> >     https://lists.sourceforge.net/lists/listinfo/crystal-develop
> >
> >
> >
> >
> > --
> > -Mike
> >
> >
> > ------------------------------------------------------------------------
> >
> > -------------------------------------------------------------------------
> > This SF.Net email is sponsored by the Moblin Your Move Developer's
> challenge
> > Build the coolest Linux based applications with Moblin SDK & win great
> prizes
> > Grand prize is a trip for two to an Open Source event anywhere in the
> world
> > http://moblin-contest.org/redirect.php?banner_id=100&url=/
> >
> >
> > ------------------------------------------------------------------------
> >
> > _______________________________________________
> > Crystal-develop mailing list
> > Crystal-develop@...
> > https://lists.sourceforge.net/lists/listinfo/crystal-develop
>
>
> -------------------------------------------------------------------------
> This SF.Net email is sponsored by the Moblin Your Move Developer's
> challenge
> Build the coolest Linux based applications with Moblin SDK & win great
> prizes
> Grand prize is a trip for two to an Open Source event anywhere in the world
> http://moblin-contest.org/redirect.php?banner_id=100&url=/
> _______________________________________________
> Crystal-develop mailing list
> Crystal-develop@...
> https://lists.sourceforge.net/lists/listinfo/crystal-develop
>



-- 
-Mike

It is a special case because it assumes a root of /usr/local/include. It 
does not take into account any of the other standard roots, such as 
/usr/include, /opt/include, /opt/local/include, /sw/include, 
/sw/local/include, etc.

-- ES

Mike Gist wrote:
>  From what I can gather this changes nothing except *adding* a extra 
> directory to look in.
> That dir is also how bullet installs itself, so I don't see how it's a 
> special case.
> 
> 2008/9/18 Eric Sunshine <sunshine@... 
> <mailto:sunshine@...>>
> 
>     If that is the case, then the configure test should have worked
>     without any change. The change to which I objected actually assumes
>     that headers are in a "bullet" subdirectory.
> 
>     If the real issue is that the headers are "randomly" installed
>     sometimes in a "bullet" subdirectory and sometimes not, then that is a
>     more difficult issue to solve, but the committed patch is still a bad
>     special-purpose hack. A more generalized solution is warranted.
> 
>     -- ES
> 
>     On Thu, Sep 18, 2008 at 7:26 AM, Jorrit Tyberghein
>     <jorrit.tyberghein@... <mailto:jorrit.tyberghein@...>>
>     wrote:
>      > I don't believe that's possible as in some situations the bullet
>      > includes are not installed
>      > in a directory relative to 'bullet'. i.e. it is not always
>     'bullet/bla.h'
>      >
>      > Greetings,
>      >
>      > On Thu, Sep 18, 2008 at 1:24 PM, Eric Sunshine
>     <sunshine@... <mailto:sunshine@...>> wrote:
>      >> This is the incorrect approach. It is essentially a special-case
>     hack
>      >> which will work for only one configuration. Please revert. To
>     fix this
>      >> properly, the "bullet" prefix should be mentioned in the #include
>      >> statements (also, it is more proper to use #include <...> rather
>     than
>      >> #include "..."). Finally, source code in CS itself should also
>     use the
>      >> "bullet" prefix in #include <...> statements.
>      >>
>      >> -- ES
>      >>
>      >>
>      >> mgist@... <mailto:mgist@...>
>     wrote:
>      >>> Revision: 31196
>      >>>          
>     http://crystal.svn.sourceforge.net/crystal/?rev=31196&view=rev
>     <http://crystal.svn.sourceforge.net/crystal/?rev=31196&view=rev>
>      >>> Author:   mgist
>      >>> Date:     2008-09-18 09:27:24 +0000 (Thu, 18 Sep 2008)
>      >>>
>      >>> Log Message:
>      >>> -----------
>      >>> - Added an extra search path for libbullet.
>      >>>
>      >>> Modified Paths:
>      >>> --------------
>      >>>     CS/trunk/configure
>      >>>     CS/trunk/configure.ac <http://configure.ac>
>      >>>
>      >>> Modified: CS/trunk/configure
>      >>> ===================================================================
>      >>> --- CS/trunk/configure        2008-09-18 08:12:22 UTC (rev 31195)
>      >>> +++ CS/trunk/configure        2008-09-18 09:27:24 UTC (rev 31196)
>      >>> @@ -53037,7 +53037,7 @@
>      >>>    cs_check_lib_paths="| /usr/local/include|/usr/local/lib \
>      >>>      /sw/include|/sw/lib \
>      >>>      /opt/local/include|/opt/local/lib \
>      >>> -    /opt/include|/opt/lib "
>      >>> +    /opt/include|/opt/lib /usr/local/include/bullet/"
>      >>>  fi
>      >>>
>      >>>       for cs_lib_item in $cs_check_lib_paths
>      >>>
>      >>> Modified: CS/trunk/configure.ac <http://configure.ac>
>      >>> ===================================================================
>      >>> --- CS/trunk/configure.ac <http://configure.ac>     2008-09-18
>     08:12:22 UTC (rev 31195)
>      >>> +++ CS/trunk/configure.ac <http://configure.ac>     2008-09-18
>     09:27:24 UTC (rev 31196)
>      >>> @@ -1347,7 +1347,7 @@
>      >>>        btSequentialImpulseConstraintSolver* s;
>      >>>        btDefaultCollisionConfiguration* c;
>      >>>        btDynamicsWorld* world = new btDiscreteDynamicsWorld (d,
>     b, s, c)])],
>      >>> -    [], [C++], [], [],
>      >>> +    [/usr/local/include/bullet/], [C++], [], [],
>      >>>      [], [], [-lbulletdynamics -lbulletcollision -lbulletmath],
>      >>>      [bulletdynamics])
>      >>
>      >>
>     -------------------------------------------------------------------------
>      >> This SF.Net email is sponsored by the Moblin Your Move
>     Developer's challenge
>      >> Build the coolest Linux based applications with Moblin SDK & win
>     great prizes
>      >> Grand prize is a trip for two to an Open Source event anywhere
>     in the world
>      >> http://moblin-contest.org/redirect.php?banner_id=100&url=/
>     <http://moblin-contest.org/redirect.php?banner_id=100&url=/>
>      >> _______________________________________________
>      >> Crystal-develop mailing list
>      >> Crystal-develop@...
>     <mailto:Crystal-develop@...>
>      >> https://lists.sourceforge.net/lists/listinfo/crystal-develop
>      >>
>      >
>      >
>      >
>      > --
>      > Project Manager of Crystal Space (http://www.crystalspace3d.org)
>      > and CEL (http://cel.crystalspace3d.org)
>      > Support Crystal Space. Donate at
>      > https://sourceforge.net/donate/index.php?group_id=649
>      > Visit my town at http://waldir.myminicity.com/
>      >
>      >
>     -------------------------------------------------------------------------
>      > This SF.Net email is sponsored by the Moblin Your Move
>     Developer's challenge
>      > Build the coolest Linux based applications with Moblin SDK & win
>     great prizes
>      > Grand prize is a trip for two to an Open Source event anywhere in
>     the world
>      > http://moblin-contest.org/redirect.php?banner_id=100&url=/
>     <http://moblin-contest.org/redirect.php?banner_id=100&url=/>
>      > _______________________________________________
>      > Crystal-develop mailing list
>      > Crystal-develop@...
>     <mailto:Crystal-develop@...>
>      > https://lists.sourceforge.net/lists/listinfo/crystal-develop
>      >
> 
>     -------------------------------------------------------------------------
>     This SF.Net email is sponsored by the Moblin Your Move Developer's
>     challenge
>     Build the coolest Linux based applications with Moblin SDK & win
>     great prizes
>     Grand prize is a trip for two to an Open Source event anywhere in
>     the world
>     http://moblin-contest.org/redirect.php?banner_id=100&url=/
>     <http://moblin-contest.org/redirect.php?banner_id=100&url=/>
>     _______________________________________________
>     Crystal-develop mailing list
>     Crystal-develop@...
>     <mailto:Crystal-develop@...>
>     https://lists.sourceforge.net/lists/listinfo/crystal-develop
> 
> 
> 
> 
> -- 
> -Mike
> 
> 
> ------------------------------------------------------------------------
> 
> -------------------------------------------------------------------------
> This SF.Net email is sponsored by the Moblin Your Move Developer's challenge
> Build the coolest Linux based applications with Moblin SDK & win great prizes
> Grand prize is a trip for two to an Open Source event anywhere in the world
> http://moblin-contest.org/redirect.php?banner_id=100&url=/
> 
> 
> ------------------------------------------------------------------------
> 
> _______________________________________________
> Crystal-develop mailing list
> Crystal-develop@...
> https://lists.sourceforge.net/lists/listinfo/crystal-develop

>From what I can gather this changes nothing except *adding* a extra
directory to look in.
That dir is also how bullet installs itself, so I don't see how it's a
special case.

2008/9/18 Eric Sunshine <sunshine@...>

> If that is the case, then the configure test should have worked
> without any change. The change to which I objected actually assumes
> that headers are in a "bullet" subdirectory.
>
> If the real issue is that the headers are "randomly" installed
> sometimes in a "bullet" subdirectory and sometimes not, then that is a
> more difficult issue to solve, but the committed patch is still a bad
> special-purpose hack. A more generalized solution is warranted.
>
> -- ES
>
> On Thu, Sep 18, 2008 at 7:26 AM, Jorrit Tyberghein
> <jorrit.tyberghein@...> wrote:
> > I don't believe that's possible as in some situations the bullet
> > includes are not installed
> > in a directory relative to 'bullet'. i.e. it is not always 'bullet/bla.h'
> >
> > Greetings,
> >
> > On Thu, Sep 18, 2008 at 1:24 PM, Eric Sunshine <sunshine@...>
> wrote:
> >> This is the incorrect approach. It is essentially a special-case hack
> >> which will work for only one configuration. Please revert. To fix this
> >> properly, the "bullet" prefix should be mentioned in the #include
> >> statements (also, it is more proper to use #include <...> rather than
> >> #include "..."). Finally, source code in CS itself should also use the
> >> "bullet" prefix in #include <...> statements.
> >>
> >> -- ES
> >>
> >>
> >> mgist@... wrote:
> >>> Revision: 31196
> >>>
> http://crystal.svn.sourceforge.net/crystal/?rev=31196&view=rev
> >>> Author:   mgist
> >>> Date:     2008-09-18 09:27:24 +0000 (Thu, 18 Sep 2008)
> >>>
> >>> Log Message:
> >>> -----------
> >>> - Added an extra search path for libbullet.
> >>>
> >>> Modified Paths:
> >>> --------------
> >>>     CS/trunk/configure
> >>>     CS/trunk/configure.ac
> >>>
> >>> Modified: CS/trunk/configure
> >>> ===================================================================
> >>> --- CS/trunk/configure        2008-09-18 08:12:22 UTC (rev 31195)
> >>> +++ CS/trunk/configure        2008-09-18 09:27:24 UTC (rev 31196)
> >>> @@ -53037,7 +53037,7 @@
> >>>    cs_check_lib_paths="| /usr/local/include|/usr/local/lib \
> >>>      /sw/include|/sw/lib \
> >>>      /opt/local/include|/opt/local/lib \
> >>> -    /opt/include|/opt/lib "
> >>> +    /opt/include|/opt/lib /usr/local/include/bullet/"
> >>>  fi
> >>>
> >>>       for cs_lib_item in $cs_check_lib_paths
> >>>
> >>> Modified: CS/trunk/configure.ac
> >>> ===================================================================
> >>> --- CS/trunk/configure.ac     2008-09-18 08:12:22 UTC (rev 31195)
> >>> +++ CS/trunk/configure.ac     2008-09-18 09:27:24 UTC (rev 31196)
> >>> @@ -1347,7 +1347,7 @@
> >>>        btSequentialImpulseConstraintSolver* s;
> >>>        btDefaultCollisionConfiguration* c;
> >>>        btDynamicsWorld* world = new btDiscreteDynamicsWorld (d, b, s,
> c)])],
> >>> -    [], [C++], [], [],
> >>> +    [/usr/local/include/bullet/], [C++], [], [],
> >>>      [], [], [-lbulletdynamics -lbulletcollision -lbulletmath],
> >>>      [bulletdynamics])
> >>
> >>
> -------------------------------------------------------------------------
> >> This SF.Net email is sponsored by the Moblin Your Move Developer's
> challenge
> >> Build the coolest Linux based applications with Moblin SDK & win great
> prizes
> >> Grand prize is a trip for two to an Open Source event anywhere in the
> world
> >> http://moblin-contest.org/redirect.php?banner_id=100&url=/
> >> _______________________________________________
> >> Crystal-develop mailing list
> >> Crystal-develop@...
> >> https://lists.sourceforge.net/lists/listinfo/crystal-develop
> >>
> >
> >
> >
> > --
> > Project Manager of Crystal Space (http://www.crystalspace3d.org)
> > and CEL (http://cel.crystalspace3d.org)
> > Support Crystal Space. Donate at
> > https://sourceforge.net/donate/index.php?group_id=649
> > Visit my town at http://waldir.myminicity.com/
> >
> > -------------------------------------------------------------------------
> > This SF.Net email is sponsored by the Moblin Your Move Developer's
> challenge
> > Build the coolest Linux based applications with Moblin SDK & win great
> prizes
> > Grand prize is a trip for two to an Open Source event anywhere in the
> world
> > http://moblin-contest.org/redirect.php?banner_id=100&url=/
> > _______________________________________________
> > Crystal-develop mailing list
> > Crystal-develop@...
> > https://lists.sourceforge.net/lists/listinfo/crystal-develop
> >
>
> -------------------------------------------------------------------------
> This SF.Net email is sponsored by the Moblin Your Move Developer's
> challenge
> Build the coolest Linux based applications with Moblin SDK & win great
> prizes
> Grand prize is a trip for two to an Open Source event anywhere in the world
> http://moblin-contest.org/redirect.php?banner_id=100&url=/
> _______________________________________________
> Crystal-develop mailing list
> Crystal-develop@...
> https://lists.sourceforge.net/lists/listinfo/crystal-develop
>



-- 
-Mike

If that is the case, then the configure test should have worked
without any change. The change to which I objected actually assumes
that headers are in a "bullet" subdirectory.

If the real issue is that the headers are "randomly" installed
sometimes in a "bullet" subdirectory and sometimes not, then that is a
more difficult issue to solve, but the committed patch is still a bad
special-purpose hack. A more generalized solution is warranted.

-- ES

On Thu, Sep 18, 2008 at 7:26 AM, Jorrit Tyberghein
<jorrit.tyberghein@...> wrote:
> I don't believe that's possible as in some situations the bullet
> includes are not installed
> in a directory relative to 'bullet'. i.e. it is not always 'bullet/bla.h'
>
> Greetings,
>
> On Thu, Sep 18, 2008 at 1:24 PM, Eric Sunshine <sunshine@...> wrote:
>> This is the incorrect approach. It is essentially a special-case hack
>> which will work for only one configuration. Please revert. To fix this
>> properly, the "bullet" prefix should be mentioned in the #include
>> statements (also, it is more proper to use #include <...> rather than
>> #include "..."). Finally, source code in CS itself should also use the
>> "bullet" prefix in #include <...> statements.
>>
>> -- ES
>>
>>
>> mgist@... wrote:
>>> Revision: 31196
>>>           http://crystal.svn.sourceforge.net/crystal/?rev=31196&view=rev
>>> Author:   mgist
>>> Date:     2008-09-18 09:27:24 +0000 (Thu, 18 Sep 2008)
>>>
>>> Log Message:
>>> -----------
>>> - Added an extra search path for libbullet.
>>>
>>> Modified Paths:
>>> --------------
>>>     CS/trunk/configure
>>>     CS/trunk/configure.ac
>>>
>>> Modified: CS/trunk/configure
>>> ===================================================================
>>> --- CS/trunk/configure        2008-09-18 08:12:22 UTC (rev 31195)
>>> +++ CS/trunk/configure        2008-09-18 09:27:24 UTC (rev 31196)
>>> @@ -53037,7 +53037,7 @@
>>>    cs_check_lib_paths="| /usr/local/include|/usr/local/lib \
>>>      /sw/include|/sw/lib \
>>>      /opt/local/include|/opt/local/lib \
>>> -    /opt/include|/opt/lib "
>>> +    /opt/include|/opt/lib /usr/local/include/bullet/"
>>>  fi
>>>
>>>       for cs_lib_item in $cs_check_lib_paths
>>>
>>> Modified: CS/trunk/configure.ac
>>> ===================================================================
>>> --- CS/trunk/configure.ac     2008-09-18 08:12:22 UTC (rev 31195)
>>> +++ CS/trunk/configure.ac     2008-09-18 09:27:24 UTC (rev 31196)
>>> @@ -1347,7 +1347,7 @@
>>>        btSequentialImpulseConstraintSolver* s;
>>>        btDefaultCollisionConfiguration* c;
>>>        btDynamicsWorld* world = new btDiscreteDynamicsWorld (d, b, s, c)])],
>>> -    [], [C++], [], [],
>>> +    [/usr/local/include/bullet/], [C++], [], [],
>>>      [], [], [-lbulletdynamics -lbulletcollision -lbulletmath],
>>>      [bulletdynamics])
>>
>> -------------------------------------------------------------------------
>> This SF.Net email is sponsored by the Moblin Your Move Developer's challenge
>> Build the coolest Linux based applications with Moblin SDK & win great prizes
>> Grand prize is a trip for two to an Open Source event anywhere in the world
>> http://moblin-contest.org/redirect.php?banner_id=100&url=/
>> _______________________________________________
>> Crystal-develop mailing list
>> Crystal-develop@...
>> https://lists.sourceforge.net/lists/listinfo/crystal-develop
>>
>
>
>
> --
> Project Manager of Crystal Space (http://www.crystalspace3d.org)
> and CEL (http://cel.crystalspace3d.org)
> Support Crystal Space. Donate at
> https://sourceforge.net/donate/index.php?group_id=649
> Visit my town at http://waldir.myminicity.com/
>
> -------------------------------------------------------------------------
> This SF.Net email is sponsored by the Moblin Your Move Developer's challenge
> Build the coolest Linux based applications with Moblin SDK & win great prizes
> Grand prize is a trip for two to an Open Source event anywhere in the world
> http://moblin-contest.org/redirect.php?banner_id=100&url=/
> _______________________________________________
> Crystal-develop mailing list
> Crystal-develop@...
> https://lists.sourceforge.net/lists/listinfo/crystal-develop
>

I don't believe that's possible as in some situations the bullet
includes are not installed
in a directory relative to 'bullet'. i.e. it is not always 'bullet/bla.h'

Greetings,

On Thu, Sep 18, 2008 at 1:24 PM, Eric Sunshine <sunshine@...> wrote:
> This is the incorrect approach. It is essentially a special-case hack
> which will work for only one configuration. Please revert. To fix this
> properly, the "bullet" prefix should be mentioned in the #include
> statements (also, it is more proper to use #include <...> rather than
> #include "..."). Finally, source code in CS itself should also use the
> "bullet" prefix in #include <...> statements.
>
> -- ES
>
>
> mgist@... wrote:
>> Revision: 31196
>>           http://crystal.svn.sourceforge.net/crystal/?rev=31196&view=rev
>> Author:   mgist
>> Date:     2008-09-18 09:27:24 +0000 (Thu, 18 Sep 2008)
>>
>> Log Message:
>> -----------
>> - Added an extra search path for libbullet.
>>
>> Modified Paths:
>> --------------
>>     CS/trunk/configure
>>     CS/trunk/configure.ac
>>
>> Modified: CS/trunk/configure
>> ===================================================================
>> --- CS/trunk/configure        2008-09-18 08:12:22 UTC (rev 31195)
>> +++ CS/trunk/configure        2008-09-18 09:27:24 UTC (rev 31196)
>> @@ -53037,7 +53037,7 @@
>>    cs_check_lib_paths="| /usr/local/include|/usr/local/lib \
>>      /sw/include|/sw/lib \
>>      /opt/local/include|/opt/local/lib \
>> -    /opt/include|/opt/lib "
>> +    /opt/include|/opt/lib /usr/local/include/bullet/"
>>  fi
>>
>>       for cs_lib_item in $cs_check_lib_paths
>>
>> Modified: CS/trunk/configure.ac
>> ===================================================================
>> --- CS/trunk/configure.ac     2008-09-18 08:12:22 UTC (rev 31195)
>> +++ CS/trunk/configure.ac     2008-09-18 09:27:24 UTC (rev 31196)
>> @@ -1347,7 +1347,7 @@
>>        btSequentialImpulseConstraintSolver* s;
>>        btDefaultCollisionConfiguration* c;
>>        btDynamicsWorld* world = new btDiscreteDynamicsWorld (d, b, s, c)])],
>> -    [], [C++], [], [],
>> +    [/usr/local/include/bullet/], [C++], [], [],
>>      [], [], [-lbulletdynamics -lbulletcollision -lbulletmath],
>>      [bulletdynamics])
>
> -------------------------------------------------------------------------
> This SF.Net email is sponsored by the Moblin Your Move Developer's challenge
> Build the coolest Linux based applications with Moblin SDK & win great prizes
> Grand prize is a trip for two to an Open Source event anywhere in the world
> http://moblin-contest.org/redirect.php?banner_id=100&url=/
> _______________________________________________
> Crystal-develop mailing list
> Crystal-develop@...
> https://lists.sourceforge.net/lists/listinfo/crystal-develop
>



-- 
Project Manager of Crystal Space (http://www.crystalspace3d.org)
and CEL (http://cel.crystalspace3d.org)
Support Crystal Space. Donate at
https://sourceforge.net/donate/index.php?group_id=649
Visit my town at http://waldir.myminicity.com/

This is the incorrect approach. It is essentially a special-case hack 
which will work for only one configuration. Please revert. To fix this 
properly, the "bullet" prefix should be mentioned in the #include 
statements (also, it is more proper to use #include <...> rather than 
#include "..."). Finally, source code in CS itself should also use the 
"bullet" prefix in #include <...> statements.

-- ES


mgist@... wrote:
> Revision: 31196
>           http://crystal.svn.sourceforge.net/crystal/?rev=31196&view=rev
> Author:   mgist
> Date:     2008-09-18 09:27:24 +0000 (Thu, 18 Sep 2008)
> 
> Log Message:
> -----------
> - Added an extra search path for libbullet.
> 
> Modified Paths:
> --------------
>     CS/trunk/configure
>     CS/trunk/configure.ac
> 
> Modified: CS/trunk/configure
> ===================================================================
> --- CS/trunk/configure	2008-09-18 08:12:22 UTC (rev 31195)
> +++ CS/trunk/configure	2008-09-18 09:27:24 UTC (rev 31196)
> @@ -53037,7 +53037,7 @@
>    cs_check_lib_paths="| /usr/local/include|/usr/local/lib \
>      /sw/include|/sw/lib \
>      /opt/local/include|/opt/local/lib \
> -    /opt/include|/opt/lib "
> +    /opt/include|/opt/lib /usr/local/include/bullet/"
>  fi
>  
>  	for cs_lib_item in $cs_check_lib_paths
> 
> Modified: CS/trunk/configure.ac
> ===================================================================
> --- CS/trunk/configure.ac	2008-09-18 08:12:22 UTC (rev 31195)
> +++ CS/trunk/configure.ac	2008-09-18 09:27:24 UTC (rev 31196)
> @@ -1347,7 +1347,7 @@
>        btSequentialImpulseConstraintSolver* s;
>        btDefaultCollisionConfiguration* c;
>        btDynamicsWorld* world = new btDiscreteDynamicsWorld (d, b, s, c)])],
> -    [], [C++], [], [],
> +    [/usr/local/include/bullet/], [C++], [], [],
>      [], [], [-lbulletdynamics -lbulletcollision -lbulletmath],
>      [bulletdynamics])

For CS 2.0 we don't have to keep source code compatibility. While we 
shouldn't break things just for the heck of it there is the chance to 
make some changes/rectify stupid things which would otherwise break 
source compatibility.

For a start, check
http://trac.crystalspace3d.org/trac/CS/wiki/Things%20We%20Want%20To%20Break%20But%20Can%27t%20(Yet) 
- and of course feel free to comment or add your own little pet issues.

-f.r.

On 19.08.2008 20:24, Eric Sunshine wrote:
 > Maintenance
> also becomes problematic since .csplugin files must be combined (unless 
> this can be automated by the build system).

Since .csplugin files are XML writing a combiner tool wasn't that hard.

> Most significantly, it seems like a bad idea to make such a significant 
> policy decision based only upon speculation that the loaders might be to 
> blame. There may, in fact, be actual bugs or misuses on the CS side from 
> which the problems arise.

Well, there are also (slight) economic benefits: linking multiple 
plugins into one shared lib allows sharing of certain code (notably 
template instantiations). Not the the extent a static build does, but still.

Providing merged plugins is a bit of a balance between total modularity 
of one shared lib per plugin and monolithic static builds: there is 
still a choice, we just "pre-bundle" some of the choices. Also note that 
this is not the exclusive way to distribute plugins: it could be the 
default mode for binaries we create or distro installs; however, 3rd 
parties are still free to choose anything between total modularity and 
linking everything into one binary, in both cases allowing tailoring the 
set of plugins/features to an exact selection.

-f.r.

On 06.09.2008 00:52, Mike Gist wrote:
> lol, well you don't append strings to iFiles in any special way afaik. 
> So there's nothing to implement.

Well read some code next time. All that print stuff is to get some (XML) 
string data into some sort of data output. In the case of files that 
would obviously be writing to a file. Just doing nothing is not the 
right way to achieve that.

-f.r.