Date: Sat, 14 Sep 2019 03:52:55 +0000 From: bugzilla-noreply@freebsd.org To: python@FreeBSD.org Subject: [Bug 240476] security/py-fido2: Add FreeBSD support Message-ID: <bug-240476-21822-GboaAtRVQt@https.bugs.freebsd.org/bugzilla/> In-Reply-To: <bug-240476-21822@https.bugs.freebsd.org/bugzilla/> References: <bug-240476-21822@https.bugs.freebsd.org/bugzilla/>
next in thread | previous in thread | raw e-mail | index | archive | help
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=3D240476 Kubilay Kocak <koobs@FreeBSD.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Keywords| |needs-patch --- Comment #15 from Kubilay Kocak <koobs@FreeBSD.org> --- Thanks Michael, nice work upstreaming.=20 A couple of leftovers: - RUN_DEPENDS don't also need to be in TEST_DEPENDS (uhid-freebsd) - setup.py says uhid-freebsd>=3D1.2.1, RUN_DEPENDS says >=3D1.0. *_DEPENDS = should match upstream requirements as closely as possible modulo any syntax limitations we have (we cant foo!=3Dx.y right now for example) On the test suite with OS specific tests, upstreams should always 'skip' th= ese tests if the environment isn't appropriate for the test. pytest and nose ha= ve very easy decorators upstreams can use to do this with various condition support) When test suites 'dont' skip inappropriate tests, one can usually (with most unittest frameworks), skip individual tests explicitly in the test invocati= on. For example with pytest you can -k 'not <expression>' and with nose you can -e{xclude} <regex> With unittest (in Python > 3.1) you can skipIf [1], i cant recall if there'= s a command line arg for skipping tests. Also, if the tests use 'unittest', then the Python package should probably tests_require on "unittest2; python_vers= ion < 3" [1] https://docs.python.org/3/library/unittest.html#unittest.skipIf --=20 You are receiving this mail because: You are the assignee for the bug.=
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?bug-240476-21822-GboaAtRVQt>