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

dbcsr: use Accelerate as default, fix tests #17344

Merged
merged 1 commit into from
Jan 16, 2023

Conversation

barracuda156
Copy link
Contributor

Description

For the reasons behind changes, see: cp2k/dbcsr#645

Type(s)
  • bugfix
  • enhancement
  • security fix
Tested on

macOS 10.6.8 Server
Xcode 3.2.6

Verification

Have you

  • followed our Commit Message Guidelines?
  • squashed and minimized your commits?
  • checked that there aren't other open pull requests for the same change?
  • referenced existing tickets on Trac with full URL?
  • checked your Portfile with port lint --nitpick?
  • tried existing tests with sudo port test?
  • tried a full install with sudo port -vst install?
  • tested basic functionality of all binary files?
  • checked that the Portfile's most important variants haven't been broken?

@barracuda156
Copy link
Contributor Author

barracuda156 commented Jan 15, 2023

Hmm, I thought linear_algebra PG handles that, but not with Clang, apparently:

ld: library not found for -lvecLibFort
  collect2: error: ld returned 1 exit status

UPD. Nah, sorry, I should not have added vecLibFort to configure.args, PG would do that. Let us see if it builds now.

@barracuda156 barracuda156 marked this pull request as draft January 15, 2023 22:10
@barracuda156 barracuda156 marked this pull request as ready for review January 15, 2023 22:47
@barracuda156
Copy link
Contributor Author

@herbygillot Could you take a look, please?

@herbygillot herbygillot merged commit 8e1fa24 into macports:master Jan 16, 2023
@barracuda156 barracuda156 deleted the dbcsr branch January 16, 2023 08:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

Successfully merging this pull request may close these issues.

3 participants