On 11/29/2011 05:05 PM, Richard Su wrote:
On 11/23/2011 10:44 AM, Francesco Vollero wrote:
> Hi,
>
> I just added/changed the license for conductor moving the code from
> GPL to ASL and, including the ASL license on files that does not
> contain it.
> The patch can be found here:
>
http://razorinc.org/0001-Added-changed-license-from-GPL-to-ASL.patch
>
> Cheers,
> Francesco Vollero
Francesco,
The patch didn't apply on the latest code on master. I think it is the
same version Tomas has already looked at. I can take another look,
after you have updated and rebased.
- Richard
So after looking at the rebased version
(
http://razorinc.org/0001-Moving-from-GPL-to-ASL.patch), here's a
conditional ACK
There are a few files missed:
dbomatic/dbomatic: missing license change
app/util/deployable-rng.xml, script/rails, script/cucumber: for these 3,
remove the old RH GPL header but don't replace it with anything
Once you fix these files, go ahead and push
Scott