[macports-ports] branch master updated: py-clang: python bindings to libclang
Joshua Root
jmr at macports.org
Sat Apr 7 19:22:48 UTC 2018
On 2018-4-8 03:44 , Ryan Schmidt wrote:
>
> On Apr 7, 2018, at 12:05, Eric A. Borisch wrote:
>
>> + eval [subst {
>
> This usage of "eval" was recently removed from other ports. Please look up what changes were done, and make them here as well:
>
> https://github.com/macports/macports-ports/commit/d41a0388cfdf7e6dbf3353d89939eaff876945ec
>
> Use of "eval" should be avoided where possible.
This is an antipattern not because it uses eval (though eval is almost
never needed), but because it's only using it to undo an extra level of
quoting that could simply not be added in the first place.
- Josh
More information about the macports-dev
mailing list