Re: [NEW] x11/screenkey

2020-08-11 Thread Brian Callahan


On Tuesday, August 11, 2020 3:43 PM, Stuart Henderson  
wrote:

> On 2020/08/11 14:38, Laurence Tratt wrote:
>
> > On Tue, Aug 11, 2020 at 01:40:59PM +0200, Stefan Hagen wrote:
> > Hello Stefan,
> >
> > > Sweet! I tried to port screenkey myself but got stuck on the translation 
> > > files
> > > as well. Instead of deleting the "po" directory, you can also set i18n = 
> > > False
> > > in config.cfg.
> >
> > Aha, that's a good idea -- it's definitely clearer that way!
> > Updated port attached.
> > Laurie
>
> Small tweaks in the version attached, otherwise OK sthen@ to import

Committed. Thanks all.

~Brian



Re: [NEW] x11/screenkey

2020-08-11 Thread Stuart Henderson
On 2020/08/11 14:38, Laurence Tratt wrote:
> On Tue, Aug 11, 2020 at 01:40:59PM +0200, Stefan Hagen wrote:
> 
> Hello Stefan,
> 
> > Sweet! I tried to port screenkey myself but got stuck on the translation 
> > files
> > as well. Instead of deleting the "po" directory, you can also set i18n = 
> > False
> > in config.cfg.
> 
> Aha, that's a good idea -- it's definitely clearer that way!
> 
> Updated port attached.
> 
> 
> Laurie

Small tweaks in the version attached, otherwise OK sthen@ to import



screenkey.tgz
Description: application/tar-gz


Re: [NEW] x11/screenkey

2020-08-11 Thread Florian Viehweger
Hi,

> On Tue, Aug 11, 2020 at 01:40:59PM +0200, Stefan Hagen wrote:
> 
> Hello Stefan,
> 
> > Sweet! I tried to port screenkey myself but got stuck on the
> > translation files as well. Instead of deleting the "po" directory,
> > you can also set i18n = False in config.cfg.
> 
> Aha, that's a good idea -- it's definitely clearer that way!
> 
> Updated port attached.
> 
> 
> Laurie

works on my ports-testing VM. Thank you!

-- 
greetings,

Florian Viehweger



Re: [NEW] x11/screenkey

2020-08-11 Thread Laurence Tratt
On Tue, Aug 11, 2020 at 01:40:59PM +0200, Stefan Hagen wrote:

Hello Stefan,

> Sweet! I tried to port screenkey myself but got stuck on the translation files
> as well. Instead of deleting the "po" directory, you can also set i18n = False
> in config.cfg.

Aha, that's a good idea -- it's definitely clearer that way!

Updated port attached.


Laurie


screenkey.tgz
Description: application/tar-gz


Re: [NEW] x11/screenkey

2020-08-11 Thread Stefan Hagen
Laurence Tratt wrote:
> This is a port of screenkey-1.2. One obvious problem is that there are some
> localisation files included, but they cause the setup.py installation to
> fail. Maybe there's an obvious Python setup fix, but I couldn't find it --
> the best I could manage is to delete the initialisation files before
> building... Suggestions welcome!

Sweet! I tried to port screenkey myself but got stuck on the translation files
as well. Instead of deleting the "po" directory, you can also set i18n = False
in config.cfg.

I can't commit, but I tested it and it works well (amd64).

Best Regards,
Stefan



Re: NEW: x11/screenkey

2017-08-16 Thread Kaashif Hymabaccus
Thanks for the feedback everyone.

Updated tarball is attached. I now use MODPY_EGG_VERSION, I replaced
"0.9" in PLIST with MODPY_EGG_VERSION, added textproc/intltool to
BUILD_DEPENDS.

-- 
Kaashif Hymabaccus
GPG: 3E810B04


screenkey.tgz
Description: application/tar-gz


Re: NEW: x11/screenkey

2017-08-16 Thread Stuart Henderson
On 2017/08/16 03:59, Klemens Nanni wrote:
> You should use MODPY_EGG_VERSION and MODPY_SETUPTOOLS,

MODPY_SETUPTOOLS is already there.

If using MODPY_EGG_VERSION here, the PLIST will need to be
manually edited to use it, "make plist" will do the wrong thing.

> SEPARATE_BUILD should be set.

Nope, it creates files in the source directory. SEPARATE_BUILD is for
things which can do a proper out-of-source-tree build.

> textproc/intltool is missing as BUILD_DEPENDS.

Correct, this needs adding.

> I think DESCR's first line would read better if it was a proper sentence.

Reads fine as-is to me.

I'd be OK with importing this with just the intltool BUILD_DEPENDS fixed.
I'd also be OK with MODPY_EGG_VERSION as long as the PLIST is correct.



Re: NEW: x11/screenkey

2017-08-15 Thread Klemens Nanni
On Wed, Aug 16, 2017 at 12:00:38AM +0100, Kaashif Hymabaccus wrote:
> Hello ports@,
> 
> Please find attached a port for x11/screenkey.
> 
> pkg/DESCR:
> Screencast tool to display your keypresses, inspired by Screenflick.
> Supports several keyboard translation methods, multiple monitors,
> dynamic recording control, repeat keypress compression and more.
> 
> portcheck gives me a warning:
> 
> > translation file(s) found without devel/gettext dependency in RUN_DEPENDS
> 
> But this is a Python program, Python has a built in module for
> gettext, so there is no need to depend on devel/gettext.
> 
> Thoughts? I don't have much experience with Python, so I probably got
> some of the PLIST stuff wrong.
You should use MODPY_EGG_VERSION and MODPY_SETUPTOOLS,
see port-modules(5).

textproc/intltool is missing as BUILD_DEPENDS.

SEPARATE_BUILD should be set.

I think DESCR's first line would read better if it was a proper sentence.

This port looks useful, especially when it comes to screen recording.
Thanks for porting it!