Skip to content

Commit

Permalink
[IMP] Users can overwrite the package compiler
Browse files Browse the repository at this point in the history
  • Loading branch information
frahikLV committed Oct 17, 2019
1 parent ad36430 commit 7a75ff9
Show file tree
Hide file tree
Showing 7 changed files with 58 additions and 48 deletions.
2 changes: 1 addition & 1 deletion DESCRIPTION
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
Package: BMTME
Title: Bayesian Multi-Trait Multi-Environment for Genomic Selection Analysis
Version: 1.0.11
Version: 1.0.12
Date: 2019-9-30
Language: en-US
Authors@R: c(person("Francisco Javier", "Luna-Vazquez", email = "[email protected]", role = c("aut", "cre"), comment = c(ORCID = "0000-0002-5370-7152")),
Expand Down
7 changes: 7 additions & 0 deletions NEWS.md
Original file line number Diff line number Diff line change
@@ -1,3 +1,10 @@
BMTME v1.0.12
==============

Changes:

* [FIX] Users can overwrite the package compiler.

BMTME v1.0.11
==============

Expand Down
44 changes: 29 additions & 15 deletions README.Rmd
Original file line number Diff line number Diff line change
Expand Up @@ -10,22 +10,36 @@ output: github_document
</h4>

<p align="center">
<a href="https://www.tidyverse.org/lifecycle/#maturing">
<img src="https://img.shields.io/badge/lifecycle-maturing-blue.svg" alt="Maturing"> </a>
<a href="https://travis-ci.org/frahik/BMTME">
<img src="https://travis-ci.org/frahik/BMTME.svg?branch=master" alt="Travis build status"> </a>
<a href='https://coveralls.io/github/frahik/BMTME'>
<a href="https://www.tidyverse.org/lifecycle/#maturing">
<img src="https://img.shields.io/badge/lifecycle-maturing-blue.svg" alt="Maturing">
</a>
<a href="https://www.gnu.org/licenses/lgpl-3.0">
<img src="https://img.shields.io/badge/License-LGPL%20v3-blue.svg" alt="LGPL, Version 3.0">
</a>
<a href="http://www.repostatus.org/#active">
<img src="https://www.repostatus.org/badges/latest/active.svg" alt="The project has reached a stable, usable state and is being actively developed.">
</a>
<a href="https://cran.r-project.org/package=BMTME">
<img src="http://cranlogs.r-pkg.org/badges/BMTME" alt="Dowloads from the CRAN">
</a>
</p>

<p align="center">
<a href="https://travis-ci.org/frahik/BMTME">
<img src="https://travis-ci.org/frahik/BMTME.svg?branch=master" alt="Travis build status">
</a>
<a href="https://ci.appveyor.com/project/frahik/bmtme">
<img src="https://ci.appveyor.com/api/projects/status/r2vog9sa30yutl5n?svg=true" alt="Appveyor build status"> </a>
<img src='https://coveralls.io/repos/github/frahik/BMTME/badge.svg?branch=master' alt='Coverage Status'/> </a>
<a href="https://www.gnu.org/licenses/lgpl-3.0">
<img src="https://img.shields.io/badge/License-LGPL%20v3-blue.svg" alt="LGPL, Version 3.0"> </a>
<a href="http://www.repostatus.org/#active">
<img src="https://www.repostatus.org/badges/latest/active.svg" alt="The project has reached a stable, usable state and is being actively developed."> </a>
<a href="">
<img src="http://cranlogs.r-pkg.org/badges/BMTME" alt="Dowloads from the CRAN"> </a>
<a href="https://cran.r-project.org/package=BMTME">
<img src="http://www.r-pkg.org/badges/version-ago/BMTME" alt="CRAN"> </a>
<img src="https://ci.appveyor.com/api/projects/status/r2vog9sa30yutl5n?svg=true" alt="Appveyor build status">
</a>
<a href='https://coveralls.io/github/frahik/BMTME'>
<img src='https://coveralls.io/repos/github/frahik/BMTME/badge.svg?branch=master' alt='Coverage Status'/>
</a>
<a href="https://cran.r-project.org/package=BMTME">
<img src="http://www.r-pkg.org/badges/version-ago/BMTME" alt="CRAN">
</a>
<a href="https://cran.r-project.org/package=BMTME">
<img src="https://cranchecks.info/badges/summary/BMTME" alt="CRAN">
</a>
</p>

<h4 align="center">
Expand Down
34 changes: 20 additions & 14 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -6,36 +6,42 @@

<h3 align="center">

Bayesian Multi-Trait Multi-Environment | Development version 1.0.10
Bayesian Multi-Trait Multi-Environment | Development version 1.0.12

</h4>

<p align="center">

<a href="https://www.tidyverse.org/lifecycle/#maturing">
<img src="https://img.shields.io/badge/lifecycle-maturing-blue.svg" alt="Maturing">
</a> <a href="https://travis-ci.org/frahik/BMTME">
<img src="https://travis-ci.org/frahik/BMTME.svg?branch=master" alt="Travis build status">
</a> <a href='https://coveralls.io/github/frahik/BMTME'>
<a href="https://ci.appveyor.com/project/frahik/bmtme">
<img src="https://ci.appveyor.com/api/projects/status/r2vog9sa30yutl5n?svg=true" alt="Appveyor build status">
</a>
<img src='https://coveralls.io/repos/github/frahik/BMTME/badge.svg?branch=master' alt='Coverage Status'/>
</a> <a href="https://www.gnu.org/licenses/lgpl-3.0">
<img src="https://img.shields.io/badge/License-LGPL%20v3-blue.svg" alt="LGPL, Version 3.0">
</a> <a href="http://www.repostatus.org/#active">
<img src="https://www.repostatus.org/badges/latest/active.svg" alt="The project has reached a stable, usable state and is being actively developed.">
</a> <a href="">
</a> <a href="https://cran.r-project.org/package=BMTME">
<img src="http://cranlogs.r-pkg.org/badges/BMTME" alt="Dowloads from the CRAN">
</a>

</p>

<p align="center">

<a href="https://travis-ci.org/frahik/BMTME">
<img src="https://travis-ci.org/frahik/BMTME.svg?branch=master" alt="Travis build status">
</a> <a href="https://ci.appveyor.com/project/frahik/bmtme">
<img src="https://ci.appveyor.com/api/projects/status/r2vog9sa30yutl5n?svg=true" alt="Appveyor build status">
</a> <a href='https://coveralls.io/github/frahik/BMTME'>
<img src='https://coveralls.io/repos/github/frahik/BMTME/badge.svg?branch=master' alt='Coverage Status'/>
</a> <a href="https://cran.r-project.org/package=BMTME">
<img src="http://www.r-pkg.org/badges/version-ago/BMTME" alt="CRAN">
</a>
</a> <a href="https://cran.r-project.org/package=BMTME">
<img src="https://cranchecks.info/badges/summary/BMTME" alt="CRAN"> </a>

</p>

<h4 align="center">

\[Last README update: 2019-09-30\]
\[Last README update: 2019-10-17\]

</h4>

Expand All @@ -55,7 +61,7 @@ Bayesian Multi-Trait Multi-Environment | Development version 1.0.10

<h2 id="news">

News of this version (1.0.10)
News of this version (1.0.12)

</h2>

Expand Down Expand Up @@ -139,7 +145,7 @@ citation('BMTME')
## Francisco Javier Luna-Vazquez, Fernando H. Toledo, Osval Antonio
## Montesinos-Lopez, Abelardo Montesinos-Lopez and Jose Crossa
## (2019). BMTME: Bayesian Multi-Trait Multi-Environment for
## Genomic Selection Analysis. R package version 1.0.10.
## Genomic Selection Analysis. R package version 1.0.12.
## https://github.com/frahik/BMTME
##
## A BibTeX entry for LaTeX users is
Expand All @@ -148,7 +154,7 @@ citation('BMTME')
## title = {BMTME: Bayesian Multi-Trait Multi-Environment for Genomic Selection Analysis},
## author = {Francisco Javier Luna-Vazquez and Fernando H. Toledo and Osval Antonio Montesinos-Lopez and Abelardo Montesinos-Lopez and Jose Crossa},
## year = {2019},
## note = {R package version 1.0.10},
## note = {R package version 1.0.12},
## url = {https://github.com/frahik/BMTME},
## }

Expand Down
12 changes: 0 additions & 12 deletions cran-comments.md
Original file line number Diff line number Diff line change
Expand Up @@ -20,21 +20,9 @@ This does

clang++ -std=gnu++11 -shared -L/usr/local/clang/lib64 -L/usr/local/lib64
-o BMTME.so RcppExports.o Sample_IW.o -lstdc++
-L/data/gannet/ripley/R/R-clang/lib -lRlapack
-L/data/gannet/ripley/R/R-clang/lib -lRblas -lgfortran -lm -lquadmath

The hard-coded inclusion of g++'s C++ library is wrong and harmful.

I don't know what you think

CC=ccache clang -Qunused-arguments
CXX=ccache clang++ -Qunused-arguments

does, but users cannot override these in src/Makevars so this should be
cleaned up.

Please correct ASAP and before Oct 12 to retain the package on CRAN.

## Special comments

This version implements the corrections mentioned in the last round of comments
Expand Down
2 changes: 1 addition & 1 deletion src/Makevars
Original file line number Diff line number Diff line change
@@ -1 +1 @@
PKG_LIBS = -lstdc++ $(LAPACK_LIBS) $(BLAS_LIBS) $(FLIBS)
PKG_LIBS = $(LAPACK_LIBS) $(BLAS_LIBS) $(FLIBS)
5 changes: 0 additions & 5 deletions src/Makevars.win

This file was deleted.

0 comments on commit 7a75ff9

Please sign in to comment.