On Sat, May 12, 2018 at 11:33:11AM +0200, Klemens Nanni wrote:
> On Sat, May 12, 2018 at 10:27:20AM +0200, Ingo Feinerer wrote:
> > please find attached a port for qrcode, a pure python QR Code generator:
> > https://github.com/lincolnloop/python-qrcode
> Looks good, but some tests fail, test.log attached.

Yes. pymaging is missing (not in ports tree as Ljuba Nedeljkovic
mentioned). I have no immediate plans to port it but think activated
(partly failing) tests are better than NO_TEST.

> I'd patch setup.py, `sed -i' always seems dirty to me except for bulk
> cases where you want to avoid all the patch churn for the same
> substitution across lots of files.

Done.

> With COMMENT, DISTNAME and CATEGORIES column aligned, it's a tad easier to
> read, imho.

Done. (The indentation comes from
infrastructure/templates/Makefile.template which I try to adhere to
typically.)

> On 2018/05/12 08:50, Daniel Jakots wrote:
> > Also the README shows it can be used as a library, wouldn't it make
> > more sense to name the port py-qrcode and provide both python flavours?

Done. But there is one problem now which I am unsure about:
Both flavors install bin/qr and man/man1/qr.1; isn't that a conflict?
If yes, what is the best strategy to solve this?

Thanks all for the input.

Best regards,
Ingo

Attachment: py-qrcode.tar.gz
Description: application/tar-gz

Reply via email to