Can you switch it to python3 please. (MODPY_VERSION=${MODPY_DEFAULT_VERSION_3}
and sprinkle ${MODPY_FLAVOR} in the deps).
Would have preferred not to have wholesale whitespace changes mixed in with an
update (unless it's an update where essentially the whole thing is a new
Makefile), might as well keep it now, but it's really in the interests of
diff submitters to make it as quick and easy to review as possible and
the extra noise from whitespace diffs doesn't help this :)
On 2020/01/07 23:34, clematis wrote:
> Hi,
> Here's an update for devel/legit from 0.2.0 to 1.2.0.
> Note a couple of new requirements, devel/py-click and devel/crayons
> (that last one wasn't in ports/ submitted earlier).
> No test, no customers. It runs on amd64.
>
> Comments? OK?
> Thanks,
> --
> clematis (0x7e96fd2400fe7b59)
> ? diff_devel_legit-1.2.0
> Index: Makefile
> ===================================================================
> RCS file: /cvs/ports/devel/legit/Makefile,v
> retrieving revision 1.6
> diff -u -p -r1.6 Makefile
> --- Makefile 12 Jul 2019 20:44:37 -0000 1.6
> +++ Makefile 7 Jan 2020 22:24:13 -0000
> @@ -1,27 +1,30 @@
> # $OpenBSD: Makefile,v 1.6 2019/07/12 20:44:37 sthen Exp $
>
> -COMMENT= complementary command-line interface for Git
> +COMMENT = complementary command-line interface for Git
>
> -MODPY_EGG_VERSION= 0.2.0
> -DISTNAME= legit-${MODPY_EGG_VERSION}
> -CATEGORIES= devel
> -EXTRACT_SUFX= .zip
> +MODPY_EGG_VERSION = 1.2.0
> +DISTNAME = legit-${MODPY_EGG_VERSION}
> +CATEGORIES = devel
>
> -HOMEPAGE= http://www.git-legit.org/
> +HOMEPAGE = https://github.com/frostming/legit
>
> # BSD
> -PERMIT_PACKAGE= Yes
> +PERMIT_PACKAGE = Yes
>
>
> -MODULES= lang/python
> +MODULES = lang/python
>
> -MODPY_PI= Yes
> -MODPY_SETUPTOOLS= Yes
> +MODPY_PI = Yes
> +MODPY_SETUPTOOLS = Yes
>
> -RUN_DEPENDS= devel/py-clint>=0.5.1 \
> +RUN_DEPENDS = devel/py-clint>=0.5.1 \
> devel/py-gitpython>=1.0.2 \
> - devel/py-six
> + devel/py-six \
> + devel/py-click \
> + devel/py-crayons
>
> -NO_TEST= Yes
> +
> +# tests aren't available
> +NO_TEST = Yes
>
> .include <bsd.port.mk>
> Index: distinfo
> ===================================================================
> RCS file: /cvs/ports/devel/legit/distinfo,v
> retrieving revision 1.2
> diff -u -p -r1.2 distinfo
> --- distinfo 25 Apr 2016 09:36:39 -0000 1.2
> +++ distinfo 7 Jan 2020 22:24:13 -0000
> @@ -1,2 +1,2 @@
> -SHA256 (legit-0.2.0.zip) = myoAjU7gJtgmkO38lwAoxf242ymrsPxefWJ1UQMGFzU=
> -SIZE (legit-0.2.0.zip) = 18856
> +SHA256 (legit-1.2.0.tar.gz) = LHi5e1lH0D1VFgs2JV2v08PzsZlBSFGcCaWEbbIa8Fk=
> +SIZE (legit-1.2.0.tar.gz) = 16302
> Index: pkg/PLIST
> ===================================================================
> RCS file: /cvs/ports/devel/legit/pkg/PLIST,v
> retrieving revision 1.1.1.1
> diff -u -p -r1.1.1.1 PLIST
> --- pkg/PLIST 21 Jan 2013 14:31:53 -0000 1.1.1.1
> +++ pkg/PLIST 7 Jan 2020 22:24:13 -0000
> @@ -16,9 +16,9 @@ lib/python${MODPY_VERSION}/site-packages
> lib/python${MODPY_VERSION}/site-packages/legit/cli.pyc
> lib/python${MODPY_VERSION}/site-packages/legit/core.py
> lib/python${MODPY_VERSION}/site-packages/legit/core.pyc
> -lib/python${MODPY_VERSION}/site-packages/legit/helpers.py
> -lib/python${MODPY_VERSION}/site-packages/legit/helpers.pyc
> lib/python${MODPY_VERSION}/site-packages/legit/scm.py
> lib/python${MODPY_VERSION}/site-packages/legit/scm.pyc
> lib/python${MODPY_VERSION}/site-packages/legit/settings.py
> lib/python${MODPY_VERSION}/site-packages/legit/settings.pyc
> +lib/python${MODPY_VERSION}/site-packages/legit/utils.py
> +lib/python${MODPY_VERSION}/site-packages/legit/utils.pyc
