Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Rttf2pt1 is scheduled for archival on CRAN #12

Closed
emiliotorres opened this issue Jul 19, 2021 · 3 comments · Fixed by #13
Closed

Rttf2pt1 is scheduled for archival on CRAN #12

emiliotorres opened this issue Jul 19, 2021 · 3 comments · Fixed by #13

Comments

@emiliotorres
Copy link

emiliotorres commented Jul 19, 2021

Dear Mr. Chang,

I am the creator of the R package xkcd and I have received a message from Professor Brian Ripley announcing that package Rttf2pt1 is now scheduled for archival on CRAN at the end of July, 2021. He mentions this problem.

Since my package depends on extrafont, it would be very nice if you consider to update your package. Otherwise, I need to remove the dependencies of my package in just a few days.

Please let me know if I must migrate from your package to another package. Thank you very much.

Best regards
Emilio

@wch
Copy link
Owner

wch commented Jul 19, 2021

Hi, I'm attempting to fix the issues. It's proving to be difficult to get a proper build environment with gcc11.

In the meantime, I'd suggest looking at the showtext package and ragg package.

For the record, the warnings are:

* checking whether package ‘Rttf2pt1’ can be installed ... [17s/115s] WARNING
Found the following significant warnings:
  pt1.c:5219:41: warning: 'fdotsegdist2' accessing 32 bytes in a region of size 16 [-Wstringop-overflow=]
  pt1.c:5222:41: warning: 'fdotsegdist2' accessing 32 bytes in a region of size 16 [-Wstringop-overflow=]
  pt1.c:6049:29: warning: 'fcrossraysge' accessing 32 bytes in a region of size 16 [-Wstringop-overflow=]

@wch
Copy link
Owner

wch commented Jul 19, 2021

I have a workaround, but please note that this package relies on an old, unmaintained program written in C called ttf2pt1. The most recent commit in that project was made about 18 years ago, and I have basically been patching it along the way, so satisfy compiler warnings which have been getting more stringent over time. It is possible that I eventually will not be able to keep this package on CRAN.

If someone else is interested in maintaining this package, please file an issue on the repository.

@wch wch closed this as completed in #13 Jul 19, 2021
@emiliotorres
Copy link
Author

I really appreciate your effort maintaining the package based on old programs: It is so frustrating and exhausting!

I will look to your suggestions (showtext package and ragg package) to avoid you carrying this burden. But it takes me some time to integrate the new package. So, in mi case, it is very nice that you send a new version to CRAN.

Thank you very much for your work and quick response!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants