Skip to content

add new package : httperf@master#13875

Merged
alalazo merged 3 commits intospack:developfrom
open-estuary:httperf
Nov 26, 2019
Merged

add new package : httperf@master#13875
alalazo merged 3 commits intospack:developfrom
open-estuary:httperf

Conversation

@darmac
Copy link
Copy Markdown
Contributor

@darmac darmac commented Nov 25, 2019

No description provided.

Copy link
Copy Markdown
Member

@alalazo alalazo left a comment

Choose a reason for hiding this comment

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

One minor request before merging

Copy link
Copy Markdown
Member

@alalazo alalazo left a comment

Choose a reason for hiding this comment

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

Apologies, I missed on a first read that we have no released versions. Since this package is only using VCS we can't rely on https://github.com/httperf/httperf/archive/master.zip staying the same as time passes. Can you specify the git repository with a git attribute and either pin a specific commit hash or use:

version('master', branch='master'`)

?

@darmac
Copy link
Copy Markdown
Contributor Author

darmac commented Nov 26, 2019

@alalazo
Thanks for your tips, I have already changed the code source.

@alalazo
Copy link
Copy Markdown
Member

alalazo commented Nov 26, 2019

Thanks for the quick reply @darmac . Waiting for Travis to go green and then merging.

@alalazo alalazo merged commit 577391e into spack:develop Nov 26, 2019
@darmac darmac deleted the httperf branch November 27, 2019 02:03
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.

3 participants