[MacPorts] #28276: arb: Upgrade to 5.2

MacPorts noreply at macports.org
Mon Feb 28 01:48:22 PST 2011


#28276: arb: Upgrade to 5.2
----------------------------------+-----------------------------------------
 Reporter:  matt.cottrell@…       |       Owner:  macports-tickets@…                   
     Type:  update                |      Status:  new                                  
 Priority:  Normal                |   Milestone:                                       
Component:  ports                 |     Version:  1.9.2                                
 Keywords:  libpng                |        Port:  arb                                  
----------------------------------+-----------------------------------------

Comment(by ryandesign@…):

 Replying to [comment:31 matt.cottrell@…]:
 > I've got a suggestion for the arb port using post-activate to avoid
 clobbering the config file.  Could you take a look at my Portfile.5.diff?
 I'd appreciate your input.

  * r76562: Committed it with these changes:
   * No need to add a pre-destroot block; I just added these lines to the
 beginning of the existing destroot block
   * In the post-activate block, you were copying the defaults from
 worksrcdir, but by post-activate time, the worksrcdir might have already
 been deleted; copy from the right place in ${prefix} instead
   * In post-activate, I removed the foreach loop over arb_tcp.dat macros,
 since files and directories need different handling. The file can be
 simply installed with the right permissions using "xinstall -m 777",
 whereas for the directory, I presumed that your intention was that the
 user be able to modify the files in it (not just modify the directory
 itself) so I added an fs-traverse block to traverse every item in the
 directory and make it writable too.
   * In post-activate, the several lines that deal with creating the pts
 directory can be much more simply accomplished using "xinstall -d" and
 "destroot.keepdirs", so I kept the "destroot.keepdirs" that was already
 there and added the "xinstall" to install it with the desired permissions
   * Changed the comment in post-activate to more accurately reflect what
 it's doing.

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


More information about the macports-tickets mailing list