> On Nov 5, 2016, at 5:25 PM, Adam Mercer <sky...@users.noreply.github.com> > wrote: > > Adam Mercer (skymoo) pushed a commit to branch master > in repository macports-ports. > > > https://github.com/macports/macports-ports/commit/597e3967bfbacc3a673e34e64d21e8dd51569389 > > The following commit(s) were added to refs/heads/master by this push: > > new 597e396 lalinference: update version for deactivate-hack > > 597e396 is described below > > > commit 597e3967bfbacc3a673e34e64d21e8dd51569389 > > Author: Adam Mercer <ramer...@gmail.com> > AuthorDate: Sat Nov 5 17:23:58 2016 -0500 > > > lalinference: update version for deactivate-hack > > > > this fixes upgrading the port when the previous version of lalapps is > > installed > > --- > science/lalinference/Portfile | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > > diff --git a/science/lalinference/Portfile b/science/lalinference/Portfile > > index 525a4b1..302976f 100644 > > --- a/science/lalinference/Portfile > > +++ b/science/lalinference/Portfile > > @@ -43,8 +43,8 @@ pre-activate { > > # lalinference now contains files that used to be provided by lalapps > if {![catch {set installed [lindex [registry_active lalapps] 0]}]} { > set _version [lindex $installed 1] > > - if {[vercmp $_version 6.19.0] < 0} { > > - # lalapps used to install some files now provided by lalinference in > versions < 6.19.0 > > + if {[vercmp $_version 6.20.0] < 0} { > > + # lalapps used to install some files now provided by lalinference in > versions < 6.20.0 > > registry_deactivate_composite lalapps "" [list ports_nodepcheck 1] > } > }
Being inside a pre-activate block, this code will run from the copy of the portfile installed with the port, not the copy of the portfile in the ports tree. Thus, since the goal of this change is to prevent activation failures, you have to increase the port's revision so that the new pre-activate code gets into the active copy of the port.
