Bug 207154 - [patch] Mk/Uses/cran.mk: fix do-test target
Summary: [patch] Mk/Uses/cran.mk: fix do-test target
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Ports Framework (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Some People
Assignee: David Naylor
URL:
Keywords: patch
Depends on:
Blocks:
 
Reported: 2016-02-13 14:09 UTC by David Naylor
Modified: 2016-02-13 15:41 UTC (History)
1 user (show)

See Also:


Attachments
Fix R_POSTCMD_CHECK_OPTIONS for do-test target. (390 bytes, text/x-makefile)
2016-02-13 14:09 UTC, David Naylor
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description David Naylor freebsd_committer freebsd_triage 2016-02-13 14:09:08 UTC
Created attachment 166931 [details]
Fix R_POSTCMD_CHECK_OPTIONS for do-test target.

R no longer accepts the --no-rebuild-vignettes and thus fails if pdflatex is not installed.  The attached patch uses the correct --no-build-vignettes option.
Comment 1 Wen Heping freebsd_committer freebsd_triage 2016-02-13 14:24:22 UTC
Please go ahead ! Thanks !

wen
Comment 2 commit-hook freebsd_committer freebsd_triage 2016-02-13 14:42:35 UTC
A commit references this bug:

Author: dbn
Date: Sat Feb 13 14:41:46 UTC 2016
New revision: 408802
URL: https://svnweb.freebsd.org/changeset/ports/408802

Log:
  Mk/Uses/cran.mk: fix test target.

  R no longer accepts the --no-rebuild-vignettes argument, instead requiring
  the --no-build-vignettes (only used if pdflatex is not installed).

  PR:		207154
  Approved by:	wen (maintainer)
  MFH:		2016Q1

Changes:
  head/Mk/Uses/cran.mk
Comment 3 commit-hook freebsd_committer freebsd_triage 2016-02-13 15:40:44 UTC
A commit references this bug:

Author: dbn
Date: Sat Feb 13 15:40:13 UTC 2016
New revision: 408805
URL: https://svnweb.freebsd.org/changeset/ports/408805

Log:
  MFH: r408802

  Mk/Uses/cran.mk: fix test target.

  R no longer accepts the --no-rebuild-vignettes argument, instead requiring
  the --no-build-vignettes (only used if pdflatex is not installed).

  PR:		207154
  Approved by:	wen (maintainer)
  Approved by:	ports-secteam (feld)

Changes:
_U  branches/2016Q1/
  branches/2016Q1/Mk/Uses/cran.mk