#33627 closed enhancement (fixed)
pgplot: add gcc46 variant
Reported by: | Russell-Jones-OxPhys (Russell Jones) | Owned by: | ryandesign (Ryan Carsten Schmidt) |
---|---|---|---|
Priority: | Normal | Milestone: | |
Component: | ports | Version: | 2.0.4 |
Keywords: | haspatch | Cc: | MarcusCalhoun-Lopez (Marcus Calhoun-Lopez), openmaintainer@… |
Port: | pgplot |
Description
Added in attached portfile.
Attachments (1)
Change History (5)
Changed 13 years ago by Russell-Jones-OxPhys (Russell Jones)
Attachment: | Portfile-pgplot added |
---|
comment:1 Changed 13 years ago by ryandesign (Ryan Carsten Schmidt)
Keywords: | haspatch added |
---|---|
Owner: | changed from macports-tickets@… to ryandesign@… |
Status: | new → assigned |
Summary: | pgplot does not have a gcc46 variant → pgplot: add gcc46 variant |
Type: | defect → enhancement |
In the future please provide a unified diff of your changes instead of a complete new Portfile; this will make it easier for us to review your changes.
The line "build.env-append F77=${fc} F90=${fc}" that you added to the gcc43 variant does not belong; it breaks that variant.
Your've also changed the whitespace of some of the lines. While I agree with changing the port's whitespace because it is currently a mix of tabs and spaces at 2-space multiples, neither of which conforms to the modeline which says the port uses multiple-of-4-space indents, whitespace changes should be made separately from functional changes.
comment:2 follow-up: 4 Changed 13 years ago by ryandesign (Ryan Carsten Schmidt)
You also changed the description of the g95 variant from "build with g95" to "build with gfortran from f95". There are two problems with this: 1) the port is called g95, not f95; 2) the g95 port provides a program called g95, not gfortran (only gcc provides gfortran). So the previous variant description was correct, and should not have been changed.
comment:3 Changed 13 years ago by ryandesign (Ryan Carsten Schmidt)
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
comment:4 Changed 13 years ago by Russell-Jones-OxPhys (Russell Jones)
Replying to ryandesign@…:
You also changed the description of the g95 variant from "build with g95" to "build with gfortran from f95". There are two problems with this: 1) the port is called g95, not f95; 2) the g95 port provides a program called g95, not gfortran (only gcc provides gfortran). So the previous variant description was correct, and should not have been changed.
Thanks for sorting out my Portfile n00bishness. I'll take more care to get it right on my next submission.
Add gcc46 variant.