[MacPorts] #47064: Update port:rkward to version 0.6.3
MacPorts
noreply at macports.org
Sun Mar 8 01:25:54 PST 2015
#47064: Update port:rkward to version 0.6.3
---------------------------------------+------------------
Reporter: thomas.friedrichsmeier@… | Owner: mk@…
Type: submission | Status: new
Priority: Normal | Milestone:
Component: ports | Version:
Resolution: | Keywords:
Port: rkward |
---------------------------------------+------------------
Comment (by mk@…):
Hi Thomas,
Replying to [comment:4 thomas.friedrichsmeier@…]:
> Come a little closer: "lanuchctl". See? ;-)
Isn't indeed amazing how the human brain's perception is making it
sometimes impossible to see TME difference?!
In fact I have read/compared the patch umpteen times, wondering what it
was doing. I WAS comparing letter by letter, but I did - for the heck of
it - not spot the swapped 'u' and 'n'! :-/
> Very sorry about the master_sites-problem. Clearly shows the value of
testing... I will try to actually test this, but not sure, I can make it
today.
I have reworked your submission a little and committed it in r133661
(after I had messed up in r133660).
Currently the [https://build.macports.org/waterfall buildbots] are stuck
which is why I don't know whether removing the pre-activate stuff is okay,
but it should be.
When testing it I noticed that RKWard now places only 1 app icon in the
dock, which is nice! ;-)
Re the patch content I wonder why you actually call launchctl. dbus should
be running anyway on MacPorts, once you installed KDE through it. OTOH
this seems to be a nice way for the app itself to make sure that dbus is
running, which obsoletes user intervention if you install the application
including MacPorts/KDE on a new machine... Cool! I think this is something
which could/should be built into KApplication, what do you think!!?
Thanks for your submission,
Marko
--
Ticket URL: <https://trac.macports.org/ticket/47064#comment:5>
MacPorts <https://www.macports.org/>
Ports system for OS X
More information about the macports-tickets
mailing list