svn commit: r367888 - in head: accessibility/accerciser accessibility/jovie accessibility/kaccessible accessibility/kdeaccessibility4 accessibility/kmag accessibility/kmousetool accessibility/kmout...

[Tijl Coosemans]

On Fri, 12 Sep 2014 02:24:42 +0200 (CEST) Gerald Pfeifer <gerald at pfeifer.com> wrote:
> On Thu, 11 Sep 2014, Tijl Coosemans wrote:
>> devel/kBuild wasn't bumped either (reported on emulation@).
> 
> I looked into this, and so far have not been able to see why
> Tools/scripts/bump_revision.pl did not catch this one.

I suspect it's the INDEX file.  It doesn't record a dependency on gcc
for USE_GCC=any, probably because it isn't generated on FreeBSD 10, but
even if it was it wouldn't be correct for arm and powerpc.  The only
reliable way is to grep the tree, in this case for "lang/gcc", "USE_GCC"
and "compiler:*" knobs that may select gcc.

> In some other cases, ports were a bit more creative than I'd
> usually recommend, using a pattern like the following:
> 
>     PORTREVISION?= ${XORG_REVISION}
>     :
>     .if defined(WITH_NEW_XORG)
>      XORG_REVISION= 8
>      :
>      .else
>      XORG_REVISION= 13
>      :
>      .endif
> 
> Good luck handling things like that with a simple script.
> 
> If it were me, I'd just make this port XORG_REVISION=13 for
> both branches, and thus simplify this.  I am not sure the
> complexity here is worth it.

Yeah, the script should print an error message if it cannot bump a port.