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

[TASK] Make changes to allow the project to be build-able on Mac #109

Open
jmakhack opened this issue Jun 21, 2023 · 0 comments
Open

[TASK] Make changes to allow the project to be build-able on Mac #109

jmakhack opened this issue Jun 21, 2023 · 0 comments
Labels
C Requires a C code change help wanted Extra attention is needed research Requires research to complete up for grabs This issue is available to take

Comments

@jmakhack
Copy link
Owner

Task Context

At the moment, the project is structured in a way to be compiled and built on a Linux machine. It would be nice to allow for the project to be easily build-able on Mac machines as well.

Acceptance Criteria

This completion of this task should allow for Macs to build this project without many complications. It'll be nice to have some code in place that checks the current operating system and changes the control flow of what libraries to pull from accordingly.

This will likely come in the form of a mixture between updating the cmake and src file. It will also be good to have a step by step instruction guide on how to build this project on Mac somewhere in the docs.

Two issues I've noticed on Macs include the inclusion of argp and bsd in this project. This may be solved by using argp-standalone and string.h instead, but other solutions may be better/valid as well.

Additional Context

This Github thread may be useful for how to get argp-standalone properly linked on Mac: AltraMayor/f3#42 (comment)

A follow up issue to this later on would be to also add a Github action that builds the project on a Mac machine. However, this is not required for this current task.

Please read through the Contributing to the Project document before working on this project.

@jmakhack jmakhack added help wanted Extra attention is needed up for grabs This issue is available to take C Requires a C code change research Requires research to complete labels Jun 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C Requires a C code change help wanted Extra attention is needed research Requires research to complete up for grabs This issue is available to take
Projects
None yet
Development

No branches or pull requests

1 participant