Bug 196292

Summary: math/R: Configure error: Support for MBCS locales is required
Product: Ports & Packages Reporter: Harrison Grundy <harrison.grundy>
Component: Individual Port(s)Assignee: Joseph Mingrone <jrm>
Status: Closed Overcome By Events    
Severity: Affects Only Me CC: davor.cubranic, jrm, nq1n407ba, rhurlin, riggs
Priority: --- Keywords: needs-patch, needs-qa
Version: LatestFlags: bugzilla: maintainer-feedback? (jrm)
Hardware: Any   
OS: Any   
Attachments:
Description Flags
Configure Log
none
patch math/R: update from 3.0.2 to 3.1.2
none
No Op, See PR 195783 none

Description Harrison Grundy 2014-12-27 09:03:58 UTC
Created attachment 150982 [details]
Configure Log

math/R fails on configure with:

checking for mbstate_t... (cached) yes
configure: error: Support for MBCS locales is required.

Configure log attached.
Comment 1 Bugzilla Automation freebsd_committer freebsd_triage 2014-12-27 09:03:58 UTC
Auto-assigned to maintainer bf@FreeBSD.org
Comment 2 Rainer Hurling freebsd_committer freebsd_triage 2014-12-29 20:30:47 UTC
Created attachment 151089 [details]
patch math/R: update from 3.0.2 to 3.1.2

This is a NOT READY patch to update math/R. It builds on HEAD amd64 (Poudriere), but is not tested for other OS versions and platforms.

The main purpose of this patch is to expedite the update process of math/R.

Two bigger problems were (1) the absence of a JAVA dependency and (2) that the TeX dependency was only installed, if PDF option was enabled.

What this draft brings:
- update from 3.0.2 to 3.1.2
- remove R-patched-20131112-r64207.diff
- add JAVA dependency
- add NLS option
- move TeX dependency one step upwards
- change USES=iconv into USES=iconv:wchar_t
- changes for new math functions in 10.x and HEAD
- split DOCS option from INFO_MANUALS and PDF_MANUALS
- corrections and enhancements in pkg-plist
- is able to produce PDF files again

TODO:
- INSTALL target has to strip some .so files
- finer adaption of new or better function in math libs [1]
- math/libmissing should also be updated


[1] NOT TESTED, if the following patches are only in HEAD,
    but also in 10x:
    r257770, r271651, r272845 and r275819
Comment 3 Harrison Grundy 2014-12-29 20:58:15 UTC
Created attachment 151091 [details]
No Op, See PR 195783
Comment 4 Fernando Herrero Carrón 2015-01-02 00:48:02 UTC
Dear all,

I am facing exactly this problem, and I see that it is being addressed altogether within the more ambitious target of upgrading R from 3.0.2 to 3.1.2.

I wonder if there is a quick fix to get 3.0.2 to get to compile on 10.1-RELEASE. I have tried USES+= iconv:wchar_t to no avail, but I don't see anything else in the proposed patch that would have anything to do with locales.

Please let me know if I can be of any help with this issue.

Thank you very much.
Comment 5 Davor Cubranic 2015-07-19 17:48:16 UTC
I'm curious: how was the package built with no errors, but when users try to build the port, it fails?
Comment 6 Rene Ladan freebsd_committer freebsd_triage 2016-06-27 21:49:51 UTC
Maintainer reset.
Comment 7 Thomas Zander freebsd_committer freebsd_triage 2016-08-21 12:18:29 UTC
Assign to current port maintainer
Comment 8 Joseph Mingrone freebsd_committer freebsd_triage 2016-08-21 12:40:49 UTC
Harrison and elferdo@gmail.com,

Can you confirm that this is no longer a problem with the current version of math/R in the ports tree?
Comment 9 Joseph Mingrone freebsd_committer freebsd_triage 2016-08-21 12:47:42 UTC
(In reply to Davor Cubranic from comment #5)

Davor, because the packages are built in a clean/pristine jail using poudriere, rather than a live system.  This eliminates build errors caused by users having some customization in blah.conf, or package X installed with non-default option Y, etc.
Comment 10 Fernando Herrero Carrón 2016-08-22 18:24:16 UTC
(In reply to Joseph Mingrone from comment #8)

I'm building on amd64, 10.3-RELEASE-p7 FreeBSD 10.3-RELEASE-p7 #0: Thu Aug 11 18:38:15 UTC 2016, R-3.3.1_2 and it builds fine, both with and without NLS.