Skip to content

Add libretro Makefile #1048

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Apr 15, 2020
Merged

Add libretro Makefile #1048

merged 1 commit into from
Apr 15, 2020

Conversation

gouchi
Copy link
Contributor

@gouchi gouchi commented Apr 15, 2020 •
edited
Loading

  • We will maintain it if needed.

@RobLoach
Copy link
Contributor

Thanks for the addition! Most of the properties are now handled in the CMakeList file:
https://github.com/nesbox/TIC-80/blob/master/CMakeLists.txt#L440-L483

But it's good to have this when cmake isn't available. Thanks!

nesbox
nesbox approved these changes Apr 15, 2020
View reviewed changes
Copy link
Owner

@nesbox nesbox left a comment

Choose a reason for hiding this comment

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

@gouchi Could you please keep the makefile updated when CMakeLists.txt changes?
Thank you.

@nesbox nesbox merged commit 8d86c0f into nesbox:master Apr 15, 2020
@gouchi
Copy link
Contributor Author

gouchi commented Apr 15, 2020

  • I will try thank you.

@nesbox nesbox added devops enhancement Improvement of existing feature or adding something new labels Apr 17, 2020
@nesbox nesbox added this to the 0.80.0 milestone Apr 17, 2020
@nesbox nesbox mentioned this pull request May 2, 2020
Merged
@nesbox nesbox removed this from the 0.80.0 milestone Jul 15, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@nesbox nesbox nesbox approved these changes

Assignees

@gouchi gouchi

Labels
devops enhancement Improvement of existing feature or adding something new
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@gouchi @RobLoach @nesbox