[MacPorts] #49316: R: compilers portgroup, two fixes
#49316: R: compilers portgroup, two fixes -------------------------+--------------------------- Reporter: dstrubbe@… | Owner: kjell.konis@… Type: enhancement | Status: new Priority: Normal | Milestone: Component: ports | Version: 2.3.4 Keywords: haspatch | Port: R -------------------------+--------------------------- Patch to use compilers PortGroup to manage Fortran variants (which adds support for gcc5, gcc6, g95, and dragonegg, and sets gcc5 as default -- passes test phase on my machine). Comments on necessity of clang for the C compiler (at least on Yosemite). Two small fixes: comment in configure.args was ending up in the middle of the configure line since more args are appended to this line, and therefore were probably ignored since the fall after the '#'. The require_active_variants line for tk should not use the path: dependency since tk is set as a port: dependency. -- Ticket URL: <https://trac.macports.org/ticket/49316> MacPorts <https://www.macports.org/> Ports system for OS X
#49316: R: compilers portgroup, two fixes --------------------------+--------------------------- Reporter: dstrubbe@… | Owner: kjell.konis@… Type: enhancement | Status: new Priority: Normal | Milestone: Component: ports | Version: 2.3.4 Resolution: | Keywords: haspatch Port: R | --------------------------+--------------------------- Description changed by dstrubbe@…: Old description:
Patch to use compilers PortGroup to manage Fortran variants (which adds support for gcc5, gcc6, g95, and dragonegg, and sets gcc5 as default -- passes test phase on my machine). Comments on necessity of clang for the C compiler (at least on Yosemite). Two small fixes: comment in configure.args was ending up in the middle of the configure line since more args are appended to this line, and therefore were probably ignored since the fall after the '#'. The require_active_variants line for tk should not use the path: dependency since tk is set as a port: dependency.
New description: Patch to use compilers PortGroup to manage Fortran variants (which adds support for gcc5, gcc6, g95, and dragonegg, and sets gcc5 as default -- passes test phase on my machine). Comments on necessity of clang for the C compiler (at least on Yosemite). Two small fixes: comment in configure.args was ending up in the middle of the configure line since more args are appended to this line, and therefore were probably ignored since they fall after the '#'. The require_active_variants line for tk should not use the path: dependency since tk is set as a port: dependency. -- -- Ticket URL: <https://trac.macports.org/ticket/49316#comment:1> MacPorts <https://www.macports.org/> Ports system for OS X
#49316: R: compilers portgroup, two fixes --------------------------+--------------------------- Reporter: dstrubbe@… | Owner: kjell.konis@… Type: enhancement | Status: new Priority: Normal | Milestone: Component: ports | Version: 2.3.4 Resolution: | Keywords: haspatch Port: R | --------------------------+--------------------------- Comment (by kjell.konis@…): Maintainer here. Duplicate of #49266. The default fortran compiler should be that provided by gcc48 to match the official CRAN binary of R, unless there's a good MacPorts reason not to. Otherwise good to go (built and tested with gcc48 and gcc5). Just out of curiosity, is this a situation where the port revision should be bumped? -- Ticket URL: <https://trac.macports.org/ticket/49316#comment:2> MacPorts <https://www.macports.org/> Ports system for OS X
#49316: R: compilers portgroup, two fixes --------------------------+--------------------------- Reporter: dstrubbe@… | Owner: kjell.konis@… Type: enhancement | Status: new Priority: Normal | Milestone: Component: ports | Version: 2.3.4 Resolution: | Keywords: haspatch Port: R | --------------------------+--------------------------- Comment (by dstrubbe@…): Ha, funny that Sean was looking at this at the same time. Unfortunately given how our "Tickets filed against a specific port" search works with a regex, trying to search for "R" is pretty hopeless. I don't think a revbump is needed, since what was already installed shouldn't change (except conceivably regarding the configure.args comment, if the trailing args are important). Why is there a need for matching the GCC version used in some other binary? I don't see any mention of GCC versions at https://cran.r-project.org/bin/macosx. Anyway, gcc5 is the default Fortran compiler across MacPorts; it is helpful for ports to use the default unless there is a good reason otherwise, to minimize the need for users to install multiple versions of GCC. -- Ticket URL: <https://trac.macports.org/ticket/49316#comment:3> MacPorts <https://www.macports.org/> Ports system for OS X
#49316: R: compilers portgroup, two fixes --------------------------+--------------------------- Reporter: dstrubbe@… | Owner: kjell.konis@… Type: enhancement | Status: new Priority: Normal | Milestone: Component: ports | Version: 2.3.4 Resolution: | Keywords: haspatch Port: R | --------------------------+--------------------------- Comment (by kjell.konis@…): The discussion is in the appendix (C.3.1) of the R Installation and Administration manual (https://cran.r-project.org/doc/manuals/r-release/R-admin.html#Mavericks- and-later). Following the manual is my only real motivation for gcc48 as default. If MacPorts has a default fortran compiler then, for me, that's a good enough reason to use it for the R port as well. -- Ticket URL: <https://trac.macports.org/ticket/49316#comment:4> MacPorts <https://www.macports.org/> Ports system for OS X
#49316: R: compilers portgroup, two fixes --------------------------+--------------------------- Reporter: dstrubbe@… | Owner: kjell.konis@… Type: enhancement | Status: closed Priority: Normal | Milestone: Component: ports | Version: 2.3.4 Resolution: fixed | Keywords: haspatch Port: R | --------------------------+--------------------------- Changes (by dstrubbe@…): * status: new => closed * resolution: => fixed Comment: My guess is that they used gcc 4.8 for Mavericks because that is what the latest stable gcc version was at the time, same as why they would use 4.2.3 for Snow Leopard, rather than because they thought 4.9 or 5 were not good. Anyway, committed in r141379. -- Ticket URL: <https://trac.macports.org/ticket/49316#comment:5> MacPorts <https://www.macports.org/> Ports system for OS X
participants (1)
-
MacPorts