scikit-build/scikit-build-core

RPM CI testing-farm failures

henryiii opened this issue · 2 comments

Current failure:

We are getting red x's as usual:

python3-scikit-build-core+pyproject.noarch: E: spelling-error ('Metapackage', 'Summary(en_US) Metapackage -> Meta package, Meta-package, Prepackage')
python3-scikit-build-core+pyproject.noarch: E: spelling-error ('metapackage', '%description -l en_US metapackage -> meta package, meta-package, prepackage')
python3-scikit-build-core+pyproject.noarch: W: no-documentation
python-scikit-build-core.spec: W: invalid-url Source0: scikit_build_core-0.8.2.tar.gz
python3-scikit-build-core.noarch: W: files-duplicate /usr/lib/python3.12/site-packages/scikit_build_core/setuptools/__init__.py /usr/lib/python3.12/site-packages/scikit_build_core/_compat/__init__.py:/usr/lib/python3.12/site-packages/scikit_build_core/_compat/importlib/__init__.py:/usr/lib/python3.12/site-packages/scikit_build_core/file_api/__init__.py:/usr/lib/python3.12/site-packages/scikit_build_core/file_api/model/__init__.py:/usr/lib/python3.12/site-packages/scikit_build_core/metadata/__init__.py:(and 2 more)

The duplicate file issue should ignore __init__.py's, those are often basically empty and therefore likely to be the same. I don't know where the spelling error is coming from, I don't see that in our repo. And not sure where we should add documentation.

I'll try to fix it. It should be fine just adding these to .rpmlintrc. Lack of documentation is fine because that's on optional dependency, the spelling error is from auto-generated stuff.

Oh, now I remember the issue we had before, F38 rpmlint doesn't use those spellchecks. But I think we can just disable rpmlint there, downstream uses a container with latest version and the lint checks are not blocking.