SConstruct on OSX

[stefan kersten-2]

i've fixed some stuff in SConstruct so it builds again on OSX. could  
a linux dev please apply the attached patch and check SC still compiles?

most prominent changes:

* convert tabs to spaces, use 4 spaces for indent
   (add corresponding note to top of file)
* add FFTW option
   * build with FFTW by default on linux
   * build with vecLib by default on osx

thx,
<sk>






_______________________________________________
Sc-devel mailing list
Sc-devel@...
http://lists.create.ucsb.edu/mailman/listinfo/sc-devel

[John Glover-2]

Hello Stefan,

Compiles fine for me, UbuntuStudio 7.10.


John

On 21/04/2008, stefan kersten <sk@...> wrote:

> i've fixed some stuff in SConstruct so it builds again on OSX. could a linux
> dev please apply the attached patch and check SC still compiles?
>
>  most prominent changes:
>
>  * convert tabs to spaces, use 4 spaces for indent
>   (add corresponding note to top of file)
>  * add FFTW option
>   * build with FFTW by default on linux
>   * build with vecLib by default on osx
>
>  thx,
>  <sk>
>
>
>
>
>
> _______________________________________________
>  Sc-devel mailing list
>  Sc-devel@...
>  http://lists.create.ucsb.edu/mailman/listinfo/sc-devel
>
>
>

_______________________________________________
Sc-devel mailing list
Sc-devel@...
http://lists.create.ucsb.edu/mailman/listinfo/sc-devel

[nescivi]

On Monday 21 April 2008 13:14:14 stefan kersten wrote:

> i've fixed some stuff in SConstruct so it builds again on OSX. could
> a linux dev please apply the attached patch and check SC still compiles?
>
> most prominent changes:
>
> * convert tabs to spaces, use 4 spaces for indent
>    (add corresponding note to top of file)
> * add FFTW option
>    * build with FFTW by default on linux
>    * build with vecLib by default on osx

wondering about this change:

@@ -849,11 +872,9 @@
 mlEnv = pluginEnv.Copy()
 mlSources = Split('Source/plugins/ML.cpp Source/plugins/Loudness.cpp
Source/plugins/BeatTrack.cpp Source/plugins/Onsets.cpp
Source/plugins/onsetsds.c Source/plugins/KeyTrack.cpp Source/plugins/MFCC.cpp
Source/plugins/SCComplex.o Source/plugins/BeatTrack2.cpp
Source/plugins/ML_SpecStats.cpp')
 plugins.append(
-    mlEnv.SharedLibrary(
+    pluginEnv.SharedLibrary(
     make_plugin_target('ML_UGens'), mlSources))

why not mlEnv ?
I know mlEnv does not really get changed, but then we could leave out the copy
as well...

sincerely,
Marije
_______________________________________________
Sc-devel mailing list
Sc-devel@...
http://lists.create.ucsb.edu/mailman/listinfo/sc-devel

[stefan kersten-2]

On 21.04.2008, at 23:54, nescivi wrote:

> wondering about this change:
>
> @@ -849,11 +872,9 @@
>  mlEnv = pluginEnv.Copy()
>  mlSources = Split('Source/plugins/ML.cpp Source/plugins/Loudness.cpp
> Source/plugins/BeatTrack.cpp Source/plugins/Onsets.cpp
> Source/plugins/onsetsds.c Source/plugins/KeyTrack.cpp Source/
> plugins/MFCC.cpp
> Source/plugins/SCComplex.o Source/plugins/BeatTrack2.cpp
> Source/plugins/ML_SpecStats.cpp')
>  plugins.append(
> -    mlEnv.SharedLibrary(
> +    pluginEnv.SharedLibrary(
>      make_plugin_target('ML_UGens'), mlSources))
>
> why not mlEnv ?
> I know mlEnv does not really get changed, but then we could leave  
> out the copy
> as well...

'twas a slip, mlEnv is the way to go. fixed and committed.

<sk>

_______________________________________________
Sc-devel mailing list
Sc-devel@...
http://lists.create.ucsb.edu/mailman/listinfo/sc-devel