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

[joss review] Update Makefiles #14

Merged
merged 1 commit into from
Apr 20, 2024
Merged

Conversation

victorapm
Copy link
Contributor

Hi @jcorbino, kudos on this cool package!

Regarding openjournals/joss-reviews#6288, In the process of going over the installation procedure, I implemented a few changes to the Makefiles to improve user experience:

  • Show an error if ARMA is not defined
  • Allow usage of an external SuperLU installation with SUPERLU=PATH_GOES_HERE make
  • Add an option for a debug build with make DEBUG=1

Hope this is useful to you

@jcorbino
Copy link
Owner

jcorbino commented Apr 20, 2024

Hello @victorapm - Sorry for the late reply. I had to relocate to Oregon this past month, and it was just now that I had the time to sit down and take care of the library. I saw your changes, thanks for proposing them. I'll proceed with the approval.

Thank you for the kind words!

@jcorbino jcorbino merged commit f03d65f into jcorbino:master Apr 20, 2024
1 of 2 checks passed
@victorapm victorapm deleted the build branch April 27, 2024 21:03
@victorapm
Copy link
Contributor Author

Glad this was helpful. Thanks!

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.

2 participants