Skip to content

r-htmltable: package update#7767

Merged
adamjstewart merged 4 commits intospack:developfrom
epfl-scitas:package/r-htmltable
Apr 17, 2018
Merged

r-htmltable: package update#7767
adamjstewart merged 4 commits intospack:developfrom
epfl-scitas:package/r-htmltable

Conversation

@matteoguglielmi
Copy link
Copy Markdown
Contributor

No description provided.

depends_on('r-magrittr', type=('build', 'run'))
depends_on('r-checkmate', type=('build', 'run'))
depends_on('r-htmlwidgets', type=('build', 'run'))
depends_on('r-rstudioapi', type=('build', 'run'), when="@1.11.2:")
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Can you add the dependency version constraints? For example, [email protected]:

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

done (as specified in htmlTable/DESCRIPTION source file).

version('1.9', '08c62c19e1ffe570e7d8fa57db5094b9')

depends_on('r-stringr', type=('build', 'run'))
depends_on('r-knitr', type=('build', 'run'))
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

depends_on('r-knitr', type=('build', 'run'))
depends_on('r-magrittr', type=('build', 'run'))
depends_on('r-knitr@1.6:', type=('build', 'run'), when="@1.11.2:")
depends_on('r-magrittr@1.5:', type=('build', 'run'), when="@1.11.2:")
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

These dependencies are the same for version 1.9 as well, no need to add the when here.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Ok, 'whens' removed.

depends_on('r-magrittr@1.5:', type=('build', 'run'), when="@1.11.2:")
depends_on('r-checkmate', type=('build', 'run'))
depends_on('r-htmlwidgets', type=('build', 'run'))
depends_on('[email protected]:', type=('build', 'run'), when="@1.11.2:")
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This dependency was added in version 1.11.0, not 1.11.2.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

done.

will try to be more precise, this is my first set of packages for spack, I appreciate your guidelines.

@adamjstewart adamjstewart merged commit 1c8df39 into spack:develop Apr 17, 2018
@alalazo alalazo deleted the package/r-htmltable branch April 17, 2018 16:58
baberlevi pushed a commit to ResearchIT/spack that referenced this pull request Apr 18, 2018
* r-htmltable: package update

* r-htmltable: dependency fix

* r-htmltable: dependency fix #2

* r-htmltable: dependency fix #3
ch741 pushed a commit to ch741/spack that referenced this pull request Apr 20, 2018
* r-htmltable: package update

* r-htmltable: dependency fix

* r-htmltable: dependency fix spack#2

* r-htmltable: dependency fix spack#3
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