#59063 closed defect (fixed)
gdal: change perf variant to native
Reported by: | ryandesign (Ryan Carsten Schmidt) | Owned by: | Veence (Vincent) |
---|---|---|---|
Priority: | Normal | Milestone: | |
Component: | ports | Version: | |
Keywords: | Cc: | ||
Port: | gdal |
Description
Joshua mentioned on the list that we decided to standardize on the variant "native" for compiling code optimized to the native CPU, but gdal's variant for that purpose is named "perf" so it should be renamed (leaving a compatibility variant for one year enabling users to seamlessly upgrade to the new variant name).
Change History (3)
comment:1 Changed 5 years ago by Veence (Vincent)
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
comment:2 Changed 4 years ago by ryandesign (Ryan Carsten Schmidt)
You added a new native variant but kept the existing perf variant so now there are two variants that do the same thing which isn't what we want. I've submitted a PR to help get things migrated over to only using a native variant. The migration code can be removed after a year.
comment:3 Changed 4 years ago by ryandesign (Ryan Carsten Schmidt)
Note: See
TracTickets for help on using
tickets.
Ok, I've pushed a modified port file with that new variant added. Didn't bump the revision, but it should be fine.