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

add compilation notes #94

Closed
wants to merge 3 commits into from
Closed

add compilation notes #94

wants to merge 3 commits into from

Conversation

retokromer
Copy link
Contributor

This is a first draft regarding macOS. I will add Windows 10, as well as Debian 9.5 or Ubuntu 18.04.1 LTS as soon as… I have the time ;-)

@ianfixes
Copy link
Member

ianfixes commented Mar 4, 2019

We should look at #100 first just in case this has changed

@retokromer
Copy link
Contributor Author

retokromer commented Mar 4, 2019

When will this be the case? (I will stop working on the code, and merge my own patches only once this has been done.)

@retokromer
Copy link
Contributor Author

retokromer commented Mar 5, 2019

@ianfixes FYI: today I also compiled and tested on Debian 9.5 and Ubuntu 16.04.5 LTS (also on Windows), as well as on Slackware 14.2.

@ianfixes
Copy link
Member

ianfixes commented Mar 5, 2019

I don't have an estimate, it depends on what comes from the discussion at kanjitalk755/macemu#14

When I first brought this code over, I didn't realize that @kanjitalk755's fork had become the unofficial master -- many other branches from personal forks have been merged into that fork. So it may be that we need to rebase all our changes on top of that one. Time will tell, and I don't think it will be too terrible for me to sort it all out.

@ianfixes ianfixes added the BasiliskII Affects BasiliskII executable label Mar 28, 2019
@retokromer
Copy link
Contributor Author

Is this attempt to revitalise and coordinate the project still moving forward?

@ianfixes
Copy link
Member

Yes, although I admit I'm going down a bit of a rabbit hole regarding CI. Travis CI's future looks tenuous, and I haven't had much luck getting stuff to compile on Appveyor. That led me to consider Azure, given Microsoft's commitment to stand behind it. But no matter what, my intended contribution to this project was going to be the test/release process and I haven't been able to get that working with any modern Windows tech.

@ianfixes
Copy link
Member

What do you think of these merge conflicts btw?

@retokromer
Copy link
Contributor Author

What do you think of these merge conflicts btw?

I have no idea at all why there are conflicts when I wish to add a document without changing any of the already existing ones.

@retokromer
Copy link
Contributor Author

I have put this information on Gist.

@retokromer retokromer closed this May 27, 2019
@ianfixes
Copy link
Member

Many warnings are generated which I am investigating.

Did you reach any conclusions on these?

@retokromer
Copy link
Contributor Author

I could silence the majority of them by cleaning the code.

@ianfixes
Copy link
Member

I had hoped to delay merging this until I got the CI working (i.e. I wanted the compilation notes and the CI configuration to agree, as a way of verifying their correctness). Unfortunately (as of right now) there haven't been any replies to my help forum question. I'm stuck on Windows builds specifically.

@retokromer
Copy link
Contributor Author

No problem, once it’s a good time, I can open a new PR.

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

Successfully merging this pull request may close these issues.

2 participants