simias / rustation

Playstation emulator in the Rust programing language
Other
552 stars 21 forks source link

Issues building libretro core on Debian 8 amd64 #33

Open mdeguzis opened 7 years ago

mdeguzis commented 7 years ago
  Compiling gl v0.5.2
   Compiling rustation-retro v0.1.0 (file:///tmp/buildd/libretro-rustation-0.0.0+git20161116.562d367~1)
src/libretro.rs:785:33: 785:56 error: unresolved name `panic::AssertUnwindSafe` [E0425]
src/libretro.rs:785     let r = panic::catch_unwind(panic::AssertUnwindSafe(|| {
                                                    ^~~~~~~~~~~~~~~~~~~~~~~
src/libretro.rs:785:33: 785:56 help: run `rustc --explain E0425` to see a detailed explanation
src/libretro.rs:785:13: 785:32 error: unresolved name `panic::catch_unwind` [E0425]
src/libretro.rs:785     let r = panic::catch_unwind(panic::AssertUnwindSafe(|| {
                                ^~~~~~~~~~~~~~~~~~~
src/libretro.rs:785:13: 785:32 help: run `rustc --explain E0425` to see a detailed explanation
src/libretro.rs:795:8: 795:18 error: the type of this value must be known in this context
src/libretro.rs:795     if r.is_err() {
                           ^~~~~~~~~~
error: aborting due to previous error
Could not compile `rustation-retro`.

To learn more, run the command again with --verbose.
debian/rules:14: recipe for target 'override_dh_auto_build' failed
simias commented 7 years ago

It seems that the version of rustc is too old for this:

Selecting previously unselected package rustc.
Preparing to unpack .../rustc_1.8.0+bsos-1_amd64.deb ...
Unpacking rustc (1.8.0+bsos-1) ...

catch_unwind was introduced in 1.9.0. It's possible to comment that code and not catch the panic here however, it'll just prevent some cleanup before the emulator crashes but nothing critical.

mdeguzis commented 7 years ago

Got it, thanks for catching that. I'll boost my package build of rustc. I'll close once I retry here.