On 05/23/11 17:03, Landry Breuil wrote:
On Mon, May 23, 2011 at 08:27:00AM +0200, Jochem Kossen wrote:
On 05/21/11 10:39, Remi Pointel wrote:
On Thu, 19 May 2011 17:36:03 +0200
Jochem Kossen<joc...@jkossen.nl>   wrote:
Terminator is a GTK2 terminal emulator with split-window and tabs
support. You can also use it to send commands to multiple terminals at
once a la ClusterSSH. It's written in Python using devel/py-vte.
Homepage: http://www.tenshu.net/terminator/

This is my first attempt at an OpenBSD port, so if you think something
is wrong, it probably is ;-) If you find something funny, please hit me
with a cluebat.

The port is at github as well:
https://github.com/jkossen/openbsd-ports/tree/master/x11/terminator

Stuff I'm not sure about:
* the override of MODPY_SETUP in Makefile to prevent generating icon cache
* patches/patch-doc_terminator_1 removes .TP lines before .SP. man would
otherwise fail with a syntax error. Perhaps a different solution would
be more appropriate, but it renders OK here.

Perhaps the descriptions could be a bit nicer as well.

Thanks,

Jochem

Hi,

RUN_DEPENDS is not correct, I think you must have this:

RUN_DEPENDS=           devel/vte,-python \
                        devel/py-notify

also, you must fmt -w 72 pkg/DESCR.

Thanks for looking at it. Actually, devel/py-notify is only needed
if you want the ActivityWatch plugin to work. Terminator itself runs
fine without it, so i'd consider py-notify more of a recommended
thing.

Then mention it in the DESCR file maybe if you don't want to add it as a
dep..
It lacks dependency on desktop-file-utils and the @exec/@unexec lines
for the .desktop file, otherwise it's ok for me too.

After giving it some thought, I've added py-notify to the RUN_DEPENDS since ActivityWatch is enabled by default by upstream as well.

I've also added the desktop-file-utils and @exec/@unexec update-desktop-database lines.

Regards,

Jochem

Attachment: terminator.tgz
Description: Binary data

Reply via email to