On 2026/02/02 12:52, Kirill A. Korinsky wrote:
> ports@,
> 
> I'd like to pdate audio/beets to 2.6.1 which makes one dependency unused:
> audio/py-musicbrainzngs, and requires adding www/py-requests-ratelimiter.
> The latter needs devel/py-pyrate-limiter <3, which is new, so I used the
> last supported version.
> 
> Our www/py-requests-cache is old. It was imported in 2020 with
> www/py-metallum and never updated. www/py-requests-cache is a dependency of
> two ports: www/py-metallum and databases/py-shillelagh. The first has no
> tests, and the second has broken tests.
> 
> Anyway, www/py-requests-cache's tests require two new ports:
> devel/py-timeout-decorator and www/py-url-normalize.
> 
> Ok? to:
> 
> update:
>   - audio/beets to 2.6.1

i didn't look at beets yet.

>   - www/py-requests-cache to 1.2.1

i started looking at updating py-requests-cache last week, that one
isn't ready yet. you are missing some RDEPs, and some of those TDEPs
need to be RDEPs, also some other bits to get tests working a bit more
sensibly (still some failures but it's no longer thousands of lines of
error output).

RUN_DEPENDS =           www/py-requests \
                        www/py-urllib3 \
                        devel/py-attrs \
                        devel/py-cattrs \
                        sysutils/py-platformdirs \
                        www/py-url-normalize

# tests require network and nothing listening on localhost:8080
TEST_DEPENDS =          databases/py-mongo \
                        devel/py-mock \
                        devel/py-rich \
                        devel/py-tenacity \
                        devel/py-test-httpbin \
                        devel/py-timeout-decorator \
                        www/py-responses

PORTHOME =              ${WRKDIR}
# move from default of commonly used localhost:8080 to avoid conflicts
TEST_ENV =              HTTPBIN_URL=http://localhost:6823/ \
                        USE_PYTEST_HTTPBIN=true
# backends need setup (and boto3, for dynamodb)
MODPY_PYTEST_ARGS =     --ignore tests/integration/test_dynamodb.py \
                        --ignore tests/integration/test_mongodb.py



> 
> import:
>  - www/py-requests-ratelimiter

ok once requests-cache has been updated


the others are ok after tweaks:

>  - devel/py-pyrate-limiter

typo in COMMENT
needs TDEP on py-test-asyncio and some tests disabling (unported
modules django_redis and fakeredis, test collection fails if not found
so no other tests are run)
MODPY_PYTEST_ARGS= --ignore tests/test_with_django.py \
                   --ignore tests/test_02.py

with those fixed, ok


>  - devel/py-timeout-decorator

no tests in sdist, either use NO_TEST, or GH_ACCOUNT=pnpnpn
GH_PROJECT=timeout-decorator GH_TAGNAME=${MODPY_DISTV} and drop
MODPY_PI (and regen distinfo).

I'd slightly prefer fetching from gh, there is a - in the name so
if there ever is an update, portroach won't find it on pypi because
they normalized filenames for newer files (which will change - to
_).


>  - www/py-url-normalize

OK as-is


> and to make it easy, I attached diffs for update and .tgz for import.

btw, as long as there aren't a silly number of new ports (this is fine),
a single .tgz containing all the new deps is easier to review, rather
than having to untar each of them separately..

Reply via email to