Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[WIP] Amiga support #26

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

[WIP] Amiga support #26

wants to merge 5 commits into from

Conversation

tehKaiN
Copy link

@tehKaiN tehKaiN commented Mar 2, 2018

Fixes #25

@fragglet
Copy link
Owner

fragglet commented Mar 3, 2018

I'm fine in theory with the change to an integer calculation, but it looks like it's probably broken the tests. If that is the cause then I suggest you back it out.

@@ -0,0 +1,65 @@
# Usage:
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please add the name of this file to the EXTRA_DIST = list in Makefile.am.

Copy link
Author

@tehKaiN tehKaiN Mar 3, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done! This file is temporary until I find a way to force autotools to not use libtool on Amiga build.

@@ -0,0 +1,121 @@
/*
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think if you add the name of this file to the SRC = list in lib/Makefile.am that will probably get you ~most of the way to a working autotools compile.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done!

@tehKaiN
Copy link
Author

tehKaiN commented Mar 3, 2018

I've totally forgot about rounding in permilles calc - I've fixed it so now there shouldn't be any problems. Code gets a bit more nasty, but I hope you can bear with it.

If everything looks good plz don't merge it yet - I'll look for autotools solution a bit more.

@JL2210
Copy link

JL2210 commented Jan 25, 2019

How is this going? I've heard that Amiga OS has a moderately large user-base, and I imagine this would please some users.

@tehKaiN
Copy link
Author

tehKaiN commented Jan 25, 2019

In the meantime Amiga C toolchain has become more civilised and I'm a bit smarter now, so I'll try again to finish this stuff ;)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants