chiark / gitweb /
Merge branch 'two-fixes' into 'master'
authorDaniel Martí <mvdan@mvdan.cc>
Thu, 20 Aug 2015 16:39:38 +0000 (16:39 +0000)
committerDaniel Martí <mvdan@mvdan.cc>
Thu, 20 Aug 2015 16:39:38 +0000 (16:39 +0000)
commitbca798660070bc73d68dc407c0afde3a4a4e5ce6
tree8745945fcc9f1aae8e47967f7a278dc593a80c65
parent46b1f09de39a1bff9b674430a1fd37fee33f8343
parentd53a5af715c66e6faf6af187bb7b13f204fc03bd
Merge branch 'two-fixes' into 'master'

Two fixes

I pulled two commits out of !57 since it would be good to have them included, and specifically the `buildToolsVersion` issue was just discussed.

The current state of the `buildToolsVersion` replacement is not good.  It will replace this:

```
buildToolsVersion '21.0.1'
```

with this:

```
buildToolsVersion = '21.0.1'
```

That is incorrect, since the first form is more like a function call than a variable assignment.  So most likely that is causing the `buildToolsVersion` to be unset.  The commit here includes a fix that works for both the rare, non-standard variable assignment way, i.e. `buildToolsVersion = '21.0.1'` as well as the standard way: `buildToolsVersion '21.0.1'`.  It also includes tests for both cases.

See merge request !65