Opened 13 years ago

Closed 13 years ago

Last modified 12 years ago

#30211 closed defect (fixed)

gmp @5.0.2 did not detect Core i7

Reported by: M.Hami@… Owned by: MarcusCalhoun-Lopez (Marcus Calhoun-Lopez)
Priority: Normal Milestone:
Component: ports Version: 1.9.2
Keywords: gmp corei i7 Cc: ryandesign (Ryan Carsten Schmidt)
Port: gmp

Description (last modified by ryandesign (Ryan Carsten Schmidt))

When I install gmp on my iMac Core i7 it says:

Warning: No processor dependent assembly code being used. gmp might be slower.

I found an older patch ( patch-config.guess.i7.diff ) here which fix the problem. I could not find anything why it was removed. #28892

Please put it back in.

Change History (4)

comment:1 Changed 13 years ago by ryandesign (Ryan Carsten Schmidt)

Cc: mcalhoun@… removed
Description: modified (diff)
Owner: changed from macports-tickets@… to mcalhoun@…

So the patch was added to gmp 5.0.1 in r77717, and removed when gmp was updated to 5.0.2 in r79373. The comment that had been added to the port in r77717 that the patch could be safely removed with gmp 5.0.2 appears to have been erroneous.

comment:2 Changed 13 years ago by ryandesign (Ryan Carsten Schmidt)

Description: modified (diff)

comment:3 Changed 13 years ago by ryandesign (Ryan Carsten Schmidt)

Cc: ryandesign@… added
Resolution: fixed
Status: newclosed

I see upstream's main (unstable) development branch has this fix, but the 5.0 branch does not. So I guess we'll need this patch until the next version (5.1? 6.0?) comes out. Re-added patch in r80766. Revbump so i7 users rebuild to get the faster code.

comment:4 Changed 12 years ago by lrenaud (Luke)

It looks like the patch was correctly removed. The method that the patch used in now corrected, but this warning is still produced because the build_cpu name scheme has changed slightly since the Portfile was written. Currently the extra check that is performed should just check if $build_cpu is prefixed with corei, not that the two names are the same.

Note: See TracTickets for help on using tickets.