jeremyletang / rgtk

GTK+ bindings and wrappers for Rust (DEPRECATED SEE https://github.com/rust-gnome )
GNU Lesser General Public License v3.0
121 stars 22 forks source link

Fixes for cairo::Context::set_dash and get_dash #190

Closed SimonSapin closed 9 years ago

SimonSapin commented 9 years ago

Please see individual commit messages, and confirm that the test fails on the second commit and is fixed on the third. I was not able to run tests locally.

SimonSapin commented 9 years ago

Uh, it looks like cairotest didn’t fail on Travis at the second commit like I expected. Either this test is not actually run, or I don’t understand what’s going on.

https://travis-ci.org/jeremyletang/rgtk/builds/49418200

GuillaumeGomez commented 9 years ago

Thanks for your contribution ! We're just compiling on travis, not testing. We test on our computers. Le 4 févr. 2015 07:20, "Simon Sapin" notifications@github.com a écrit :

Uh, it looks like cairotest didn’t fail on Travis at the second commit like I expected. Either this test is not actually run, or I don’t understand what’s going on.

https://travis-ci.org/jeremyletang/rgtk/builds/49418200

— Reply to this email directly or view it on GitHub https://github.com/jeremyletang/rgtk/pull/190#issuecomment-72797014.

SimonSapin commented 9 years ago

Oh, I see. Travis runs cd ../cairotest && cargo build, not && cargo run

GuillaumeGomez commented 9 years ago

It would be difficult since it's a graphic software...

2015-02-04 7:36 GMT+01:00 Simon Sapin notifications@github.com:

Oh, I see. Travis runs cd ../cairotest && cargo build, not && cargo run

— Reply to this email directly or view it on GitHub https://github.com/jeremyletang/rgtk/pull/190#issuecomment-72798330.

web CV : www.guillaume-gomez.fr http://www.guillaume-gomez.fr portable: +33 6 21 92 31 88

SimonSapin commented 9 years ago

Yeah, I understand. I just made an incorrect assumption when first looking at the results.

GuillaumeGomez commented 9 years ago

No problem. I'll take a look at your PR this evening.