textproc/py-qrcode: Update to 6.0
ClosedPublic
Actions

Authored by leres on Jun 20 2018, 5:53 PM.

Details

Reviewers

ler
matthew

Commits

rP472941: Update to 6.0:

Summary

Proposed commit message:

Update to 6.0:

 - Fix optimize length being ignored in QRCode.add_data.

 - Better calculation of the best mask pattern and related
   optimizations.

Reviewed by: ? (mentor)
Approved by: ? (mentor)
Differential Revision: ?

Diff Detail

Repository

rP FreeBSD ports repository

Lint

No Lint Coverage

Unit

No Test Coverage

Build Status

Buildable 17501
Build 17328: arc lint + arc unit

Event Timeline

leres created this revision.Jun 20 2018, 5:53 PM

Herald added a subscriber: mat. · View Herald TranscriptJun 20 2018, 5:53 PM

Harbormaster completed remote builds in B17486: Diff 44168.Jun 20 2018, 5:53 PM

mat added inline comments.Jun 20 2018, 6:28 PM

textproc/py-qrcode/Makefile
16–17	Why change PY_FLAVOR to FLAVOR?

What mat said. Always use PY_FLAVOR for python dependencies.

Although mat did just update the original py-qrcode port amongst many others in r472884
to enforce the rule about always using PY_FLAVOR only about 50 minutes before you posted this review.
I guess you hadn't noticed that update while working on this port?

Anyhow, given the addition of the PY_FLAVOR fix, this looks fine.

This revision now requires changes to proceed.Jun 20 2018, 8:52 PM

Update diff to not step on r472884 PY_FLAVOR change.

Harbormaster completed remote builds in B17501: Diff 44189.Jun 20 2018, 9:30 PM

I tested my change in a ports tree that had not been updated in a few hours then copied to the ports tree I use for committer work; I lost the race with r472884. I'll try to be more careful moving forward.

lgtm