Skip to content

openblas: derives from MakefilePackage#2488

Merged
tgamblin merged 2 commits intospack:developfrom
epfl-scitas:packages/openblas_derives_from_makefilepackage
Dec 15, 2016
Merged

openblas: derives from MakefilePackage#2488
tgamblin merged 2 commits intospack:developfrom
epfl-scitas:packages/openblas_derives_from_makefilepackage

Conversation

@alalazo
Copy link
Copy Markdown
Member

@alalazo alalazo commented Dec 5, 2016

I tried this as an exercise, feel free to merge it if considered useful, or close it otherwise

Modifications

@davydden

Copy link
Copy Markdown
Member

@davydden davydden left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

openblas and astyle looks ok to me

@citibeth
Copy link
Copy Markdown
Member

citibeth commented Dec 7, 2016

MakefilePackage looks like it provides a lot of clean-up from the way things previously were. Unfortunately, it both uses advanced features and is completely undocumented (see https://github.com/LLNL/spack/search?utf8=%E2%9C%93&q=MakefilePackage ). The result is I have a really hard time knowing how the code in this PR works.

I would like to see documentation on MakefilePackage submitted (in a separate PR) before this PR is merged.

@tgamblin tgamblin merged commit 87c9b01 into spack:develop Dec 15, 2016
@tgamblin
Copy link
Copy Markdown
Member

@alalazo: when you get a chance, do you mind documenting the build systems? I could help with this as well but I think they're used frequently enough now it would be good to have docs soon-ish.

@alalazo alalazo deleted the packages/openblas_derives_from_makefilepackage branch December 15, 2016 18:43
@alalazo
Copy link
Copy Markdown
Member Author

alalazo commented Dec 15, 2016

when you get a chance, do you mind documenting the build systems?

@tgamblin Sure. I am currently finishing the porting to pytest of all the tests that require mocking (hopefully I will be done by the end of the week). Then I'll start working on build system docs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants