Bug 2074651 - Review Request: python-sphinx-autobuild - Autobuild a Sphinx directory when a change is detected
Summary: Review Request: python-sphinx-autobuild - Autobuild a Sphinx directory when a...
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Dominik 'Rathann' Mierzejewski
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: 2074652
TreeView+ depends on / blocked
 
Reported: 2022-04-12 17:54 UTC by Jerry James
Modified: 2022-05-07 04:12 UTC (History)
1 user (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2022-05-07 04:12:48 UTC
Type: ---
Embargoed:
dominik: fedora-review+


Attachments (Terms of Use)

Description Jerry James 2022-04-12 17:54:40 UTC
Spec URL: https://jjames.fedorapeople.org/python-sphinx-autobuild/python-sphinx-autobuild.spec
SRPM URL: https://jjames.fedorapeople.org/python-sphinx-autobuild/python-sphinx-autobuild-2021.3.14-1.fc37.src.rpm
Fedora Account System Username: jjames
Description: Rebuild Sphinx documentation on changes, with live-reload in the browser.

This package was previously in Fedora, so this is an unretirement review.

Comment 1 Dominik 'Rathann' Mierzejewski 2022-04-12 23:01:08 UTC
Taking review.

Comment 2 Dominik 'Rathann' Mierzejewski 2022-04-13 10:56:46 UTC
Licensing is fine (MIT).
Source matches upstream:
$ sha512sum sphinx-autobuild-2021.3.14.tar.gz sphinx-autobuild-2021.3.14.tar.gz.orig 
8e3bcd2466343b95baa991ff8d7da9bca8978f54c4c0720390724e5ada32c5dfa25b9ad2564d675ce5489c7627bc437bde7d98b8efb6bdfe4ea5ace152e147a7  sphinx-autobuild-2021.3.14.tar.gz
8e3bcd2466343b95baa991ff8d7da9bca8978f54c4c0720390724e5ada32c5dfa25b9ad2564d675ce5489c7627bc437bde7d98b8efb6bdfe4ea5ace152e147a7  sphinx-autobuild-2021.3.14.tar.gz.orig

Builds fine in mock.
Internal testsuite passes.
Spec is clean and legible.
rpmlint clean. Requires and provides are sane:

$ rpm -qpR python3-sphinx-autobuild-2021.3.14-1.fc37.noarch.rpm 
/usr/bin/python3
python(abi) = 3.10
python3.10dist(colorama)
python3.10dist(livereload)
python3.10dist(sphinx)

$ rpm -qp --provides python3-sphinx-autobuild-2021.3.14-1.fc37.noarch.rpm 
python-sphinx-autobuild = 2021.3.14-1.fc37
python3-sphinx-autobuild = 2021.3.14-1.fc37
python3.10-sphinx-autobuild = 2021.3.14-1.fc37
python3.10dist(sphinx-autobuild) = 2021.3.14
python3dist(sphinx-autobuild) = 2021.3.14

jquery 3.5.1 and underscore 1.13.1 are bundled in the doc subpackage, you might want to declare that in Provides: bundled().

This is of course coming from sphinx and was discussed as recently as February: https://lists.fedoraproject.org/archives/list/devel@lists.fedoraproject.org/message/ZVILBEZB6MIZXP3JVQSVNCP7Q6MK4G7D/ .

You can fix this while importing as the package is good otherwise. Approved.

Comment 3 Jerry James 2022-04-14 03:14:00 UTC
(In reply to Dominik 'Rathann' Mierzejewski from comment #2)
> jquery 3.5.1 and underscore 1.13.1 are bundled in the doc subpackage, you
> might want to declare that in Provides: bundled().

I will fix that before importing.  Thank you for the review!

Comment 4 Fedora Update System 2022-04-20 20:59:35 UTC
FEDORA-2022-9161e75532 has been submitted as an update to Fedora 36. https://bodhi.fedoraproject.org/updates/FEDORA-2022-9161e75532

Comment 5 Fedora Update System 2022-04-20 23:05:31 UTC
FEDORA-2022-9161e75532 has been pushed to the Fedora 36 testing repository.
Soon you'll be able to install the update with the following command:
`sudo dnf upgrade --enablerepo=updates-testing --advisory=FEDORA-2022-9161e75532`
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2022-9161e75532

See also https://fedoraproject.org/wiki/QA:Updates_Testing for more information on how to test updates.

Comment 6 Fedora Update System 2022-05-07 04:12:48 UTC
FEDORA-2022-9161e75532 has been pushed to the Fedora 36 stable repository.
If problem still persists, please make note of it in this bug report.


Note You need to log in before you can comment on or make changes to this bug.