Make @deprecate_binding work with --depwarn cmdline flags

[jakebolewski]

Adds tests for --depwarn and @deprecate_binding.

cc @tkelman

[JeffBezanson]

Nice, thanks.

[tkelman]

awesome. note to self to also backport 184c56b with this

[jakebolewski]

Does Appveyor just time out after 40 minutes?

[tkelman]

redirection hang, maybe same as #12519

[tkelman]

Since we should try to get this backported for RC3, unless someone has a great idea for how to fix the redirection bug on Windows I propose making the tests unix-only for now with a FIXME comment.

[tkelman]

Ping. This is needed for RC3. Do you have time to implement my suggestion, or need someone else to do it?

[jakebolewski]

done

On Wed, Sep 23, 2015 at 6:49 PM, Tony Kelman notifications@github.com
wrote:

Ping. This is needed for RC3. Do you have time to implement my suggestion,
or need someone else to do it?

—
Reply to this email directly or view it on GitHub
#13269 (comment).

[tkelman]

You didn't add a FIXME comment and a reference to the issue/PR number, which is an important thing to put in for hacks like these otherwise we'll never remember why things are running conditionally - I had done ae5452a but your version at least runs some of the tests.

[jakebolewski]

go ahead

[KristofferC]

Will this also make deprecation's only print once for each deprecation like for functions? Right now, it feels that whatever command I run from an external package i get 10 pages of deprecation warnings.

[JeffBezanson]

Bump. Ready to merge?