Bug 1905277

Summary:	Request for Package Review & Inclusion: Diffuse Now Has an Active Python 3 Fork
Product:	[Fedora] Fedora	Reporter:	niohiani <notinsideofhereiamnotinside>
Component:	Package Review	Assignee:	Nobody's working on this, feel free to take it <nobody>
Status:	CLOSED WONTFIX	QA Contact:	Fedora Extras Quality Assurance <extras-qa>
Severity:	unspecified	Docs Contact:
Priority:	unspecified
Version:	rawhide	CC:	package-review, sanjay.ankur
Target Milestone:	---
Target Release:	---
Hardware:	All
OS:	Linux
Whiteboard:
Fixed In Version:		Doc Type:	If docs needed, set a value
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2020-12-08 09:58:09 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:

Description niohiani 2020-12-07 22:30:31 UTC

Description: The Diffuse diff utility now has an active Python 3 fork, which should be included in Fedora repositories.

Additional info:
I've been hosting builds in a copr repo for now, but this has always been a reliable and quick diff utility, and seeing that it was originally available in the official repos for Fedora, I believe that it should be made available to all Fedora users again, now that it has been ported to Python 3.

Current spec file: https://download.copr.fedorainfracloud.org/results/niohiani/Diffuse-Python-3/fedora-33-x86_64/01817739-diffuse/diffuse3-meson.spec

Latest build: https://copr.fedorainfracloud.org/coprs/niohiani/Diffuse-Python-3/build/1817739/

I'm relatively new to packaging for Fedora, but I would have no problem maintaining the packaging for Diffuse.

Comment 1 niohiani 2020-12-08 01:27:08 UTC

I should also note that the nemo-extensions package contains a useful plugin for the nemo file manager, called nemo-compare. It supports numerous diff utilities (including Diffuse), yet the maintainer for said package seems to believe that Meld should be treated as a dependency for this plugin, when it is absolutely functional without it. Simply removing "Requires:    meld" from the spec file for the package in question would resolve the issue, but my bug report was immediately closed with "WONTFIX", when the maintainer doesn't even seem to understand the problem.

See: https://bugzilla.redhat.com/show_bug.cgi?id=1905281

Comment 2 Ankur Sinha (FranciscoD) 2020-12-08 09:58:09 UTC

Hiya,

If you'd like to take over maintenance of the diffuse package to bring it back into Fedora, please follow this process:

https://fedoraproject.org/wiki/Orphaned_package_that_need_new_maintainers#Claiming_Ownership_of_a_Retired_Package

https://src.fedoraproject.org/rpms/diffuse


I think the package will require a re-review, and I'll be happy to review it for you and sponsor you to the packager group if that's required.
Please feel free to contact me at ankursinha AT fedoraproject DOT org.

More information on being sponsored to the packager group here:

https://fedoraproject.org/wiki/User:Ankursinha/PackagerSponsor
https://fedoraproject.org/wiki/How_to_get_sponsored_into_the_packager_group

Please open a new ticket following the  package review process if that required. Since you already know how to build packages, you'd start from around step 2.1.10 here (but it's worth reading from the start just to not miss anything):
https://fedoraproject.org/wiki/Join_the_package_collection_maintainers

I'll close this ticket now. Automation requires our review tickets to be in a particular format (as you'll see in the above linked document).

Cheers,
Ankur

Comment 3 niohiani 2020-12-08 19:38:21 UTC

Thanks Ankur! I'll look over the documentation you've linked me to, and go through the proper channels. Will eMail you with any more questions.