cvs commit: ports/deskutils/cairo-dock-plugins Makefile distinfo ports/deskutils/cairo-dock-plugins/files patch-CDBashApplet.sh patch-Doncky-data-nvidia-config patch-GMenu-applet-menu patch-Network-Monitor-data-access-point patch-Network-Monitor-data-wifi patch-System-Monitor-data-nvidia-config patch-System-Monitor.conf.in patch-applet-appli-finder.c ...

[Gerald Pfeifer]

On Sun, 15 Apr 2012, Michael Scheidell wrote:
>> This will break when LOCALBASE != /usr/local and it's user-settable
>> variable as documented in ports(7). And two small questions:
>> - could some of this be better solved by using REINPLACE_CMD ?
>> - there's a lot bash scripts, shouldn't be there RUN_DEPENDS+= bash ?
> let me ping the previous committer gerald@ and see if he had any 
> discussions on this. his commit was about 7 days ago, and I have 
> searched the archives and don't see that any similar discussions took 
> place last week.

Yes, I did suggest to the maintainer that he approaches upstream to
find a better solution rather than hardcoding those paths in several
places.  That update of his was a clear improvement, however, so I
did commit it without waiting for perfection.

> I will also ping (without cvs-* and private groups) maintainer 
> fidaj at ukr.net (I am sure he would welcome patches and a pr.. he
> seems to be very active with updates to this port)

I found him very responsive and helpful and a pleasure to work with.
So whatever further notes you guys have, let him know and I am sure
he will try to honor them.

Gerald