Bug 241752 - emulators/fs-uae: Update to 3.0.2
Summary: emulators/fs-uae: Update to 3.0.2
Status: New
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Only Me
Assignee: freebsd-ports-bugs mailing list
URL:
Keywords: buildisok
Depends on:
Blocks:
 
Reported: 2019-11-06 07:51 UTC by sirdice
Modified: 2019-11-07 18:03 UTC (History)
1 user (show)

See Also:


Attachments
svn diff update to 3.0.2 (790 bytes, patch)
2019-11-06 07:51 UTC, sirdice
no flags Details | Diff
svn diff update to 3.0.2 (including JIT) (2.08 KB, patch)
2019-11-07 00:06 UTC, sirdice
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description sirdice 2019-11-06 07:51:51 UTC
Created attachment 208907 [details]
svn diff update to 3.0.2

Update to 3.0.1 was posted yesterday and quickly after that 3.0.2.

This will update fs-uae to 3.0.2.

https://fs-uae.net/2019/11/05/fs-uae-3-0-2-released
Comment 1 mikael.urankar 2019-11-06 10:20:29 UTC
any reason to not enable JIT?
Comment 2 sirdice 2019-11-06 10:33:28 UTC
On previous versions (<2.8.4) it broke the build with a message that stated that the option was unsupported on FreeBSD. I have read that the developers had changed the JIT code a lot for 3.0.x and it is definitely on my TODO list to give it another try.

A quick test shows that it's still an issue:
configure: error: in `/tmp/build/usr/home/dice/Sources/FreeBSD/ports/fs-uae/work/fs-uae-3.0.2':
configure: error: JIT is not supported on amd64-portbld-freebsd12.1
Comment 3 mikael.urankar 2019-11-06 10:37:58 UTC
It seems to build with this but I don't know how to test it:
--- configure.orig      2019-11-06 11:37:10.261102000 +0100
+++ configure   2019-11-06 11:37:21.930023000 +0100
@@ -13526,9 +13526,6 @@ main ()
         #if !defined(__i386__) && !defined(__x86_64__)
         #error JIT is only supported on x86/x86-64
         #endif
-        #if defined(__OpenBSD__) || defined(__FreeBSD__)
-        #error no JIT on OpenBSD/FreeBSD right now
-        #endif
 
   ;
   return 0;
Comment 4 sirdice 2019-11-06 10:49:32 UTC
Don't exactly know how to test the difference either. But I do have a large collection of Amiga stuff that's running fine on fs-uae without JIT. I'm going to try and patch out that test, if it builds properly I have enough material to see if it continues to work or not.
Comment 5 sirdice 2019-11-06 12:13:44 UTC
With the test patched out it does build but I need to do some further testing if it actually works or not. I also need to keep that option disabled for anything other than i386 or amd64. I'm probably going to make it a port option, so it can be easily turned on or off in case it causes problems. 

Created the submitted patch this morning before I went to work. Will continue testing it when I get back home and provide an updated patch. I'm pretty sure my boss doesn't like me playing Amiga games while at work ;)
Comment 6 Automation User 2019-11-06 19:22:56 UTC
Build info is available at https://gitlab.com/swills/freebsd-ports/pipelines/94102917
Comment 7 sirdice 2019-11-07 00:06:40 UTC
Created attachment 208926 [details]
svn diff update to 3.0.2 (including JIT)

I have good news and bad news. The bad news is that it didn't work for me. With JIT enabled fs-uae seemingly loops forever on something and either quits on its own or I have to kill it. I'm going to need a lot more time to figure out why. With JIT disabled everything works as before.

The good news is that I've added it as an option in the port as experimental. It's off by default. Added a patchfile for configure, couldn't figure out how to make the patch itself optional though. But it's harmless if JIT is disabled.

% svn status
M       Makefile
M       distinfo
A       files/patch-configure

svn add files/patch-configure
Comment 8 mikael.urankar 2019-11-07 07:41:02 UTC
(In reply to sirdice from comment #7)
Ok, it probably not worth the trouble
Comment 9 sirdice 2019-11-07 11:02:42 UTC
I was kind of hopeful when it built cleanly. But alas, it's going to need more work to get it actually working. 

This is my first maintainer job, I'm not yet familiar with the process. I've submitted patches before, but never as maintainer. Do I need to do anything more?
Comment 10 mikael.urankar 2019-11-07 18:03:54 UTC
(In reply to sirdice from comment #9)
Check point 3.4 3.5 3.6 of the porters handbook:
https://www.freebsd.org/doc/en/books/porters-handbook/book.html#porting-testing