mcfm 10.2.2

[davidchall]

action-homebrew-bump-formula

---

Created with brew bump-formula-pr.

[andriish]

Hi @davidchall ,

in addition to the binaries it is nice to install the headers, i.e. patch the CMakeLists.txt with this line:

install(DIRECTORY "${CMAKE_SOURCE_DIR}/src/Inc/MCFM" DESTINATION ${CMAKE_INSTALL_INCLUDEDIR})

As those headers are needed by sherpa.

Best regards,

Andrii

[davidchall]

Error encountered during installation of bundled LHAPDF library:

A linker snapshot was created at:
[1034](https://github.com/davidchall/homebrew-hep/actions/runs/3406828591/jobs/5665864289#step:8:1036)
  	/tmp/libLHAPDF.dylib-2022-11-07-012102.ld-snapshot
[1035](https://github.com/davidchall/homebrew-hep/actions/runs/3406828591/jobs/5665864289#step:8:1037)
  ld: Assertion failed: (_file->_atomsArrayCount == computedAtomCount && "more atoms allocated than expected"), function parse, file macho_relocatable_file.cpp, line 2061.

This error was previously encountered in #285, and is related to a bug in Xcode 14. It looks like GitHub's macos-12 runner image recently upgraded to Xcode 14: actions/runner-images#6225.

[davidchall]

in addition to the binaries it is nice to install the headers, i.e. patch the CMakeLists.txt with this line:

install(DIRECTORY "${CMAKE_SOURCE_DIR}/src/Inc/MCFM" DESTINATION ${CMAKE_INSTALL_INCLUDEDIR})

As those headers are needed by sherpa.

Hi @andriish, I decided not to do it in this PR to keep it dedicated to the version bump. If we want to make MCFM compatible with Sherpa, I think this should have a dedicated PR so we can test it works correctly.

P.S. The Sherpa manual describes a script that edits the MCFM source code before installing, so I don't think we can just link Sherpa to the regular MCFM in Homebrew. Instead, we'd need the Sherpa formula to have a dedicated MCFM resource. This is also how the Rivet formula handles its fjcontrib dependency.

[davidchall]

The change to the GitHub Actions needs to be done separately. Closing this PR and will reopen.