gentoo icon indicating copy to clipboard operation
gentoo copied to clipboard

Switch to lextudio forks in pysnmp ecosystem

Open parona-source opened this issue 1 year ago • 11 comments

Lots of dropped keywords due to adding dev-python/pysnmpcrypto.

Documentations disabled due to new sphinx plugin dependencies.


Please check all the boxes that apply:

  • [x] I can submit this contribution in agreement with the Copyright Policy.
  • [x] I have certified the above via adding a Signed-off-by line to every commit in the pull request.
  • [x] This contribution has not been created with the assistance of Natural Language Processing artificial intelligence tools, in accordance with the AI policy.
  • [x] I have run pkgcheck scan --commits --net to check for issues with my commits.

Please note that all boxes must be checked for the pull request to be merged.

parona-source avatar Jun 01 '24 15:06 parona-source

The problem I see, since the new package have new package names, install_requires from other packages won't be satisfied by them.

To my knowledge the only package in tree that would be affected is dev-python/pysnmp-mibs, but thats old and is tree cleaning material.

https://github.com/etingof/pysnmp-mibs?tab=readme-ov-file#pre-compiled-pysnmp-mibs

WARNING: since version 4.3, PySNMP performs MIB parsing automatically by calling pysmi library. You may still need pysnmp-mibs package if you are running older version of PySNMP.

$ qdepends -Qqqt  net-analyzer/snmpclitools dev-python/pysmi dev-python/pysnmp 
dev-python/pysmi-0.3.4-r1
dev-python/pysnmp-4.4.12-r2
dev-python/pysnmp-mibs-0.1.6
net-analyzer/snmpclitools-0.6.4-r1

parona-source avatar Jun 01 '24 15:06 parona-source

Well, in that case I'd prefer that we renamed the packages to match the PyPI name.

mgorny avatar Jun 01 '24 16:06 mgorny

Pull Request assignment

Submitter: @parona-source Areas affected: ebuilds Packages affected: dev-python/pyasn1, dev-python/pyasn1-modules, dev-python/pysmi-lextudio, dev-python/pysnmp-lextudio, dev-python/pysnmpcrypto...

dev-python/pyasn1: @gentoo/python dev-python/pyasn1-modules: @gentoo/python dev-python/pysmi-lextudio: @gentoo/proxy-maint (new package) dev-python/pysnmp-lextudio: @gentoo/proxy-maint (new package) dev-python/pysnmpcrypto: @gentoo/proxy-maint (new package) net-analyzer/snmpclitools-lextudio: @gentoo/proxy-maint (new package)

Linked bugs

No bugs to link found. If your pull request references any of the Gentoo bug reports, please add appropriate GLEP 66 tags to the commit message and request reassignment.

If you do not receive any reply to this pull request, please open or link a bug to attract the attention of maintainers.


In order to force reassignment and/or bug reference scan, please append [please reassign] to the pull request title.

Docs: Code of ConductCopyright policy (expl.) ● DevmanualGitHub PRsProxy-maint guide

gentoo-bot avatar Jun 06 '24 14:06 gentoo-bot

Pull request CI report

Report generated at: 2024-06-06 15:16 UTC Newest commit scanned: 6bb512bb3e00d321d2aeeca43afc630c29dc7380 Status: :white_check_mark: good

There are existing issues already. Please look into the report to make sure none of them affect the packages in question: https://qa-reports.gentoo.org/output/gentoo-ci/6af0372dbd/output.html

gentoo-repo-qa-bot avatar Jun 06 '24 15:06 gentoo-repo-qa-bot

Should the new forked packages be assigned back to netmon or should the maintainer ship be reconsidered. All packages except snmpclitools have test suites and pysmi and pysnmp should definetly be under the maintainer instead of being split.

So im thinking snmpclitools-lextudio could stay with netmon and the rest goes to python.

parona-source avatar Jun 06 '24 20:06 parona-source

If tests pass, move them to python@. netmon@ is roughly equivalent to maintainer-needed@ these days.

mgorny avatar Jun 07 '24 18:06 mgorny

I'm finishing this off, thanks!

thesamesam avatar Jun 17 '24 09:06 thesamesam

Good news! Looks like upstream managed to claim the projects on PyPI, so my name concerns are no longer a thing.

mgorny avatar Jul 17 '24 04:07 mgorny

Ah, great! I'll rebase my branch.

thesamesam avatar Jul 17 '24 04:07 thesamesam

https://pypi.org/project/pysnmp/ https://pypi.org/project/pysmi/ It works well.

Could anyone please update dev-python/pysnmp and dev-python/pysmi in order to support Python 3.12?

wwwdyne avatar Aug 19 '24 13:08 wwwdyne

That's that the whole PR is for. I know it needs doing, I just need to finish it. You can just ping instead ;)

I will try to look at it tonight.

thesamesam avatar Aug 19 '24 20:08 thesamesam