[MacPorts] #29676: Fix mtree violation for Puppet and update to 2.6.8

MacPorts noreply at macports.org
Wed Jun 1 22:27:53 PDT 2011


#29676: Fix mtree violation for Puppet and update to 2.6.8
--------------------------------------+-------------------------------------
 Reporter:  nigel@…                   |       Owner:  macports-tickets@…                   
     Type:  update                    |      Status:  new                                  
 Priority:  Normal                    |   Milestone:                                       
Component:  ports                     |     Version:                                       
 Keywords:  haspatch maintainer       |        Port:  puppet                               
--------------------------------------+-------------------------------------

Comment(by ryandesign@…):

 Using the patch from #15514, I get:
 {{{
 Warning: reinplace s|/etc/puppet|/opt/local/etc/puppet| didn't change
 anything in
 /opt/local/var/macports/build/_Users_rschmidt_macports_dports_sysutils_puppet/work/puppet-2.6.8/lib/puppet/defaults.rb
 Warning: reinplace s|$confdir/ssl|/opt/local/var/puppet/ssl| didn't change
 anything in
 /opt/local/var/macports/build/_Users_rschmidt_macports_dports_sysutils_puppet/work/puppet-2.6.8/lib/puppet/defaults.rb
 }}}
 So I think some of those reinplaces aren't necessary, or need to be
 different.

 There also doesn't seem to be a reason to use the `-E` flag for reinplace.
 In fact, if I remove it, only one warning is printed:
 {{{
 Warning: reinplace s|/etc/puppet|/opt/local/etc/puppet| didn't change
 anything in
 /opt/local/var/macports/build/_Users_rschmidt_macports_dports_sysutils_puppet/work/puppet-2.6.8/lib/puppet/defaults.rb
 }}}

 These reinplaces are a bit prone to breaking in the future and leaving it
 unclear to the next person to try to update the port how to do so. It
 might be clearer if you write patchfiles that first insert the correct
 paths into the files, with the @PREFIX@ placeholder, and then just use
 reinplace to replace @PREFIX@ with ${prefix}.

 Finally, the post-destroot block doesn't do anything. It creates a
 directory, but since nothing is in that directory, MacPorts will delete it
 before the port install finishes. The solution, if you want the directory
 kept, is to add a `destroot.keepdirs` line.

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


More information about the macports-tickets mailing list