[MacPorts] #21516: dosbox 0.73 update

[MacPorts]

#21516: dosbox 0.73 update
------------------------------------+---------------------------------------
 Reporter:  sava.chankov@…          |       Owner:  macports-tickets@…                   
     Type:  update                  |      Status:  new                                  
 Priority:  Normal                  |   Milestone:                                       
Component:  ports                   |     Version:  1.8.0                                
 Keywords:  haspatch                |        Port:  dosbox                               
------------------------------------+---------------------------------------

Comment(by ryandesign@…):

 Replying to [comment:10 sava.chankov@…]:
 > I provide a patch for i386 ppc universal variant - it built on Core 2
 Duo.
 But your patch would disable the FPU and dynamic x86 stuff even in the
 Intel portion of a universal binary. We should either disable it always,
 even on non-universal builds, or presumably this code is useful, so we
 should enable it always on Intel, even in a universal binary.

 > I've got the feeling that configure arguments for platform powerpc
 should be the same as variant universal, because the FPU code is newly
 added to 0.73 - however I don't have access to an old Mac with PowerPC and
 can't test it.
 I can test it on my Power Mac G4.

 > Besides, why doesn't MacPorts consider {{{platform}}} declarations when
 building universal binary for that platform?
 Well, it just doesn't. "platform powerpc" is executed when building on
 powerpc, regardless what architectures the build is going to be for, and
 similarly with "platform i386". Universal builds happen all at once --
 there aren't separate configure and build phases that get run for each
 architecture. Unless you use the muniversal portgroup. Then they are run
 separately, and you do have the chance to pass separate arguments for each
 architecture. So maybe using muniversal is the way we should go with the
 dosbox port.

 > The other patch I attached removes zlib as dependency - it's actually
 dependency of libpng.
 Sounds good; changed in r58183.

-- 
Ticket URL: <http://trac.macports.org/ticket/21516#comment:11>
MacPorts <http://www.macports.org/>
Ports system for Mac OS