Re: [Bf-committers] Keep SCons relevant to CMake

2014-12-23 Thread Campbell Barton
On Mon, Dec 22, 2014 at 10:10 PM, Sergey Sharybin wrote: > On Mon, Dec 22, 2014 at 3:56 PM, Campbell Barton > wrote: > >> Apart from my response (to being made an example of :) ) >> > > Hey, don't get it personal or get insulted! :) As said, that commit just > happened to be the most recent, it d

Re: [Bf-committers] Keep SCons relevant to CMake

2014-12-22 Thread Sergey Sharybin
On Mon, Dec 22, 2014 at 3:56 PM, Campbell Barton wrote: > Apart from my response (to being made an example of :) ) > Hey, don't get it personal or get insulted! :) As said, that commit just happened to be the most recent, it doesn't mean it's your personal fault or so. We all (including myself)

Re: [Bf-committers] Keep SCons relevant to CMake

2014-12-22 Thread Campbell Barton
On Mon, Dec 22, 2014 at 9:57 AM, Sergey Sharybin wrote: > Hey, > > This rant is caused by the following change: > https://developer.blender.org/rBe67fd7a (Cambo, sorry, this commit just > happened to be the most recent which demonstrates the issue we're having). > > Well, it was fair enough change

Re: [Bf-committers] Keep SCons relevant to CMake

2014-12-22 Thread Sergey Sharybin
This isn't really special yeah. But the things are: 1. this kind of issue happened like 2-3 times in past month. 2. it might be really tricky to discover such kind of issues (could be even trickier than discovering stupidness in actual code). What i call for is: just pay a bit more attention when

Re: [Bf-committers] Keep SCons relevant to CMake

2014-12-22 Thread Campbell Barton
On Mon, Dec 22, 2014 at 9:57 AM, Sergey Sharybin wrote: > Hey, > > This rant is caused by the following change: > https://developer.blender.org/rBe67fd7a (Cambo, sorry, this commit just > happened to be the most recent which demonstrates the issue we're having). > > Well, it was fair enough change

[Bf-committers] Keep SCons relevant to CMake

2014-12-22 Thread Sergey Sharybin
Hey, This rant is caused by the following change: https://developer.blender.org/rBe67fd7a (Cambo, sorry, this commit just happened to be the most recent which demonstrates the issue we're having). Well, it was fair enough change to split appdir routines into dedicated file keeping path utils smal