Bug 1308681

Summary: rename livereload package to meet python guidelines.
Product: [Fedora] Fedora Reporter: Matthias Runge <mrunge>
Component: livereloadAssignee: William Moreno <williamjmorenor>
Status: CLOSED RAWHIDE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: 24CC: panemade, suanand, williamjmorenor
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2017-01-12 16:54:42 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:
Bug Depends On: 1342688    
Bug Blocks:    

Description Matthias Runge 2016-02-15 17:50:21 UTC
Description of problem:

livereload is an executable, included in the package; but it's more.
Even upstream names it python-livereload.

https://github.com/lepture/python-livereload

This package named livereload caused already duplicate work

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

Comment 1 Sundeep Anand 2016-02-19 07:15:01 UTC
Package python-sphinx-autobuild is a consumer of livereload with support for both py2 and py3.

William, it would be of much help if you can package it for python2 as well.

Comment 2 William Moreno 2016-02-19 16:09:39 UTC
python-livereload was retired for rawhide, so this will require a package review to unretire. 

Can you do the package review? Any way this package was reviewed al tess 3 times but I want to run the test with python2 and this will requiere some adiotional libraries.

Comment 3 Matthias Runge 2016-02-22 07:52:20 UTC
(In reply to William Moreno from comment #2)
> python-livereload was retired for rawhide, so this will require a package
> review to unretire. 
> 
> Can you do the package review? Any way this package was reviewed al tess 3
> times but I want to run the test with python2 and this will requiere some
> adiotional libraries.

It would require a package review for renaming a package, yes.
My reasoning is, this package is named wrong, it should have been python-livereload from the beginning.
Please propose a package rename review and add it as dependency to this ticket. I'd be happily review it, if I find the time. You may also ask the (former) maintainer of python-livereload or the reviewer of that package review.

Comment 4 Parag AN(पराग) 2016-02-22 12:15:42 UTC
William,

I agree with Matthias. Please file a rename review request for livereload to python-livereload. Create subpackages python2 and python3 as given in sample spec file in https://fedoraproject.org/wiki/Packaging:Python#Example_common_spec_file

If both subpackages need to provide same livereload binary then I suggest to install python2 version as /usr/bin/livereload and python3 version as /usr/bin/livereload-3

Comment 5 Jan Kurik 2016-02-24 15:21:57 UTC
This bug appears to have been reported against 'rawhide' during the Fedora 24 development cycle.
Changing version to '24'.

More information and reason for this action is here:
https://fedoraproject.org/wiki/Fedora_Program_Management/HouseKeeping/Fedora24#Rawhide_Rebase