Bug 246843 - New port: devel/R-cran-roxygen2: In-Line Documentation for R
Summary: New port: devel/R-cran-roxygen2: In-Line Documentation for R
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Only Me
Assignee: Guangyuan Yang
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2020-05-29 13:45 UTC by Jean-Paul Beconne
Modified: 2021-06-21 02:16 UTC (History)
1 user (show)

See Also:

Attachments
SHAR file (2.20 KB, text/plain)
2020-05-29 13:45 UTC, Jean-Paul Beconne 		no flags Details
SHAR file for version 7.1.1 (1.73 KB, patch)
2021-06-16 18:43 UTC, Jean-Paul Beconne 		no flags Details | Diff
Modified SHAR file for version 7.1.1 (1.68 KB, patch)
2021-06-17 19:02 UTC, Jean-Paul Beconne 		no flags Details | Diff
Modified SHAR with dependencies fix (1.68 KB, patch)
2021-06-18 16:55 UTC, Jean-Paul Beconne 		no flags Details | Diff
Modified port in GIT diff format (1.92 KB, patch)
2021-06-19 08:44 UTC, Jean-Paul Beconne 		no flags Details | Diff
Show Obsolete (4) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Jean-Paul Beconne 2020-05-29 13:45:28 UTC 
Created attachment 215022 [details]
SHAR file

R package
Must be evaluated after bug 246841
Comment 1 Guangyuan Yang freebsd_committer freebsd_triage 2021-06-03 12:21:42 UTC 
7.1.1 is the latest version, please update the diff, and I can take it from here, thanks!
Comment 2 Jean-Paul Beconne 2021-06-16 18:43:17 UTC 
Created attachment 225868 [details]
SHAR file for version 7.1.1
Comment 3 Guangyuan Yang freebsd_committer freebsd_triage 2021-06-16 22:44:59 UTC 
- PORTREVISION should be removed for new ports.
- LICENSE seems wrong.
- It appears that BUILD_DEPENDS only need knitr, please verify.
- TEST_DEPENDS?
- WWW should be https://roxygen2.r-lib.org/.
Comment 4 Guangyuan Yang freebsd_committer freebsd_triage 2021-06-16 23:00:00 UTC 
Please also consider submitting git format-patch in the future.
Comment 5 Jean-Paul Beconne 2021-06-17 19:01:43 UTC 
(In reply to Guangyuan Yang from comment #4)
Dependencies are detailed in https://cloud.r-project.org/web/packages/roxygen2/index.html, Imports section
I don't find TEST_DEPENDS
All others items modified
Comment 6 Jean-Paul Beconne 2021-06-17 19:02:41 UTC 
Created attachment 225898 [details]
Modified SHAR file for version 7.1.1
Comment 7 Guangyuan Yang freebsd_committer freebsd_triage 2021-06-18 03:12:10 UTC 
Build test failed for me - please correct. And please consider using git format-patch for future submissions, thanks.

[00:00:03] Error: devel/R-cran-roxygen2 depends on nonexistent origin 'devel/R-cran-commonmark'; Please contact maintainer of the port to fix this.
[00:00:03] Error: devel/R-cran-roxygen2 depends on nonexistent origin 'devel/R-cran-xml2'; Please contact maintainer of the port to fix this.
Comment 8 Jean-Paul Beconne 2021-06-18 16:54:07 UTC 
(In reply to Guangyuan Yang from comment #7)
Sorry again, I started to port it 1 year ago and the dependencies were in devel and not in textproc. The dependency packages stayed in devel on my machine.
Comment 9 Jean-Paul Beconne 2021-06-18 16:55:18 UTC 
Created attachment 225910 [details]
Modified SHAR with dependencies fix
Comment 10 Guangyuan Yang freebsd_committer freebsd_triage 2021-06-19 08:04:07 UTC 
(In reply to Jean-Paul Beconne from comment #9)

Still builds fail for me - please consider using Poudriere for testing, and have the QA result attached to your future patches.

ERROR: dependency 'knitr' is not available for package 'roxygen2'
* removing '/wrkdirs/usr/ports/devel/R-cran-roxygen2/work/stage/usr/local/lib/R/library/roxygen2'
*** Error code 1
Comment 11 Jean-Paul Beconne 2021-06-19 08:43:43 UTC 
(In reply to Guangyuan Yang from comment #10)
Sorry, it is a missing dependency. Dependencies have changed with the new version and I forgot this one.
I am testing the git diff for the new version
Comment 12 Jean-Paul Beconne 2021-06-19 08:44:36 UTC 
Created attachment 225920 [details]
Modified port in GIT diff format
Comment 13 commit-hook freebsd_committer freebsd_triage 2021-06-21 02:15:38 UTC 
A commit in branch main references this bug:

URL: https://cgit.FreeBSD.org/ports/commit/?id=483db0ebe3ffd77e5d57aa111ccd0bcbd1764db8

commit 483db0ebe3ffd77e5d57aa111ccd0bcbd1764db8
Author:     Jean-Paul Beconne <jpbeconne@free.fr>
AuthorDate: 2021-06-21 01:56:10 +0000
Commit:     Guangyuan Yang <ygy@FreeBSD.org>
CommitDate: 2021-06-21 01:56:10 +0000

    devel/R-cran-roxygen2: New port: In-Line Documentation for R

    PR:             246843
    Approved by:    lwhsu (mentor, implicit)

 devel/Makefile                        |  1 +
 devel/R-cran-roxygen2/Makefile (new)  | 29 +++++++++++++++++++++++++++++
 devel/R-cran-roxygen2/distinfo (new)  |  3 +++
 devel/R-cran-roxygen2/pkg-descr (new) |  6 ++++++
 4 files changed, 39 insertions(+)
Comment 14 Guangyuan Yang freebsd_committer freebsd_triage 2021-06-21 02:16:43 UTC 
Committed, thanks!