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

libretro-buildbot-recipe.sh: Build all three mame2014 cores at once. #652

Merged
merged 3 commits into from
Dec 22, 2017

Conversation

orbea
Copy link
Contributor

@orbea orbea commented Dec 21, 2017

This adds three commits, the first two are minor and are included to avoid merge conflicts.

The last commit should allow to have only one mame2014 recipe per file and it will build all three libretro cores, mame2014, mess2014 and ume2014.

Some side effects:

  1. recipes/blackberry/cores-qnx-generic was previously missing the mess2014 and ume2014 cores.
  2. In recipes/linux/cores-linux-armhf-generic only the ume2014 sets PTR64=0 and PARTIAL=1 while neither mame2014 or mess2014 does. I'm not sure if this is intended, but assuming it is I think fixing this in the mame2014 Makefile would be the best solution. Alternatively I can accommodate in this script by adding a hack. The buildbot should be able to test this either way.

orbea added 3 commits December 21, 2017 08:56
It is not required for any of them and some of these names are no
longer used anyways.
This is not useful yet, but its better behavior.
@orbea
Copy link
Contributor Author

orbea commented Dec 21, 2017

Also, it is no longer required to build the buildtools separately so I am removing that step.

@inactive123 inactive123 merged commit b0b4cf0 into libretro:master Dec 22, 2017
@orbea orbea deleted the cores branch December 22, 2017 07:59
@orbea orbea restored the cores branch December 23, 2017 19:06
@orbea orbea deleted the cores branch December 23, 2017 19:07
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

2 participants