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

Update qcdnum installation #323

Closed
wants to merge 6 commits into from
Closed

Conversation

amsimone
Copy link
Contributor

@amsimone amsimone commented Sep 2, 2023

I have updated the qcdnum formula to install the qcdnum-config executable and not only the libraries.

Have you:

  • [ X] Run brew audit --strict --online <formula> (where <formula> is the name of the formula you're submitting) and corrected all errors?

  • [X ] Built your formula locally prior to submission with brew install <formula>?

Updates to existing formula: have you

  • [ X] Checked that the tests still pass?

Adjust qcdnum installer to compile qcdnum-config
Add Formula for xFitter
fix installation of qcdnum
added qcdnum to audit exceptions
bump fastjet formula to 3.4.2
@davidchall
Copy link
Owner

Hi @amsimone - thanks for your contribution! The GHA builds require each PR only edits a single formula, so I've recreated these changes in #333. I also kept the previous test (builds an example executable), which was more robust.

@davidchall davidchall closed this Oct 11, 2023
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