Bug 2077128 - Review Request: jigdo - Ease distribution of large files over the Internet
Summary: Review Request: jigdo - Ease distribution of large files over the Internet
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Neal Gompa
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2022-04-20 17:30 UTC by Michel Lind
Modified: 2022-05-07 04:30 UTC (History)
2 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2022-05-01 19:23:15 UTC
Type: ---
Embargoed:
ngompa13: fedora-review+


Attachments (Terms of Use)

Description Michel Lind 2022-04-20 17:30:10 UTC
Spec URL: https://download.copr.fedorainfracloud.org/results/salimma/misc-wip/fedora-rawhide-x86_64/04294494-jigdo/jigdo.spec
SRPM URL: https://download.copr.fedorainfracloud.org/results/salimma/misc-wip/fedora-rawhide-x86_64/04294494-jigdo/jigdo-0.8.1-1.fc37.src.rpm
Description: 
Jigsaw Download (for short jigdo) is a scheme developed primarily to make it
easy to distribute huge filesystem images (e.g. CD (ISO9660) or DVD (UDF)
images) over the internet, but it could also be used for other data which is
awkward to handle due to its size, like audio/video files or large software
packages.

jigdo tries to ensure that the large file is downloaded in small parts which can
be stored on different servers. People who want to download the image do so by
telling the jigdo download tool to process one ".jigdo" file; using it, jigdo
downloads the parts and reassembles the image. jigdo-file is used to prepare the
files for download.

Fedora Account System Username: salimma

COPR builds and review templates: https://copr.fedorainfracloud.org/coprs/salimma/misc-wip/build/4294494/

NOTE: This is part of the package unretirement process

Comment 1 Neal Gompa 2022-04-20 18:49:36 UTC
Taking this review.

Comment 2 Neal Gompa 2022-04-20 18:52:36 UTC
> # debugsourcefiles.list is empty (though debugfiles.list has content)
> %global debug_package %{nil}

This sounds like we've got something goofy with how compiler flags are propagated into the code. Looking at the build logs, I notice that "-g" doesn't actually propagate into the build itself. You might want to check to see if there's a way to keep things from stripping build flags...

Comment 3 Michel Lind 2022-04-20 20:07:18 UTC
(In reply to Neal Gompa from comment #2)
> > # debugsourcefiles.list is empty (though debugfiles.list has content)
> > %global debug_package %{nil}
> 
> This sounds like we've got something goofy with how compiler flags are
> propagated into the code. Looking at the build logs, I notice that "-g"
> doesn't actually propagate into the build itself. You might want to check to
> see if there's a way to keep things from stripping build flags...

yeah, I'll take another look. When checking the build directories though - debugfiles.list is not empty, just debugsourcefiles. Which was... weird. But maybe debugfiles.list != the files are compiled with debugging info.

Comment 4 Michel Lind 2022-04-20 20:18:16 UTC
So I had to read configure.ac, since it's easier than reading the generated configure, and... sigh

CFLAGS=`echo "$CFLAGS" | sed 's/\(^\| \)-g\( \|$\)/ /'` ... nonono.

Comment 6 Neal Gompa 2022-04-20 20:53:05 UTC
Review notes:

* Packaging complies with the guidelines
* Package builds and installs
* No serious issues from rpmlint
* Licensing is correct and license files are correctly installed

PACKAGE APPROVED.

Comment 7 Michel Lind 2022-04-20 20:59:39 UTC
Thanks! Unretirement request: https://pagure.io/releng/issue/10754

Comment 8 Michel Lind 2022-04-21 20:47:08 UTC
Built for Rawhide. Requesting the other branches now

jigdo on  rawhide [?]
❯ fedpkg request-branch --all-releases
https://pagure.io/releng/fedora-scm-requests/issue/43808
https://pagure.io/releng/fedora-scm-requests/issue/43809
https://pagure.io/releng/fedora-scm-requests/issue/43810

jigdo on  rawhide [?] took 7s
❯ fedpkg request-branch epel9
https://pagure.io/releng/fedora-scm-requests/issue/43811

jigdo on  rawhide [?] took 5s
❯ fedpkg request-branch epel8
https://pagure.io/releng/fedora-scm-requests/issue/43812

Comment 9 Fedora Update System 2022-04-22 18:59:08 UTC
FEDORA-EPEL-2022-44213d365d has been submitted as an update to Fedora EPEL 9. https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2022-44213d365d

Comment 10 Fedora Update System 2022-04-22 18:59:09 UTC
FEDORA-2022-4dd42b644f has been submitted as an update to Fedora 36. https://bodhi.fedoraproject.org/updates/FEDORA-2022-4dd42b644f

Comment 11 Fedora Update System 2022-04-22 18:59:10 UTC
FEDORA-EPEL-2022-ff4479fc04 has been submitted as an update to Fedora EPEL 8. https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2022-ff4479fc04

Comment 12 Fedora Update System 2022-04-22 18:59:10 UTC
FEDORA-2022-72a3a5e07f has been submitted as an update to Fedora 35. https://bodhi.fedoraproject.org/updates/FEDORA-2022-72a3a5e07f

Comment 13 Fedora Update System 2022-04-22 18:59:11 UTC
FEDORA-2022-6c722cfd6a has been submitted as an update to Fedora 34. https://bodhi.fedoraproject.org/updates/FEDORA-2022-6c722cfd6a

Comment 14 Fedora Update System 2022-04-23 17:13:55 UTC
FEDORA-2022-4dd42b644f has been pushed to the Fedora 36 testing repository.
Soon you'll be able to install the update with the following command:
`sudo dnf install --enablerepo=updates-testing --advisory=FEDORA-2022-4dd42b644f \*`
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2022-4dd42b644f

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

Comment 15 Fedora Update System 2022-04-23 18:42:26 UTC
FEDORA-2022-6c722cfd6a has been pushed to the Fedora 34 testing repository.
Soon you'll be able to install the update with the following command:
`sudo dnf install --enablerepo=updates-testing --advisory=FEDORA-2022-6c722cfd6a \*`
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2022-6c722cfd6a

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

Comment 16 Fedora Update System 2022-04-23 19:06:55 UTC
FEDORA-2022-72a3a5e07f has been pushed to the Fedora 35 testing repository.
Soon you'll be able to install the update with the following command:
`sudo dnf install --enablerepo=updates-testing --advisory=FEDORA-2022-72a3a5e07f \*`
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2022-72a3a5e07f

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

Comment 17 Fedora Update System 2022-04-23 19:51:04 UTC
FEDORA-EPEL-2022-ff4479fc04 has been pushed to the Fedora EPEL 8 testing repository.

You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2022-ff4479fc04

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

Comment 18 Fedora Update System 2022-04-23 19:51:09 UTC
FEDORA-EPEL-2022-44213d365d has been pushed to the Fedora EPEL 9 testing repository.

You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2022-44213d365d

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

Comment 19 Fedora Update System 2022-05-01 19:23:15 UTC
FEDORA-2022-6c722cfd6a has been pushed to the Fedora 34 stable repository.
If problem still persists, please make note of it in this bug report.

Comment 20 Fedora Update System 2022-05-01 19:39:20 UTC
FEDORA-2022-72a3a5e07f has been pushed to the Fedora 35 stable repository.
If problem still persists, please make note of it in this bug report.

Comment 21 Fedora Update System 2022-05-01 20:24:09 UTC
FEDORA-EPEL-2022-44213d365d has been pushed to the Fedora EPEL 9 stable repository.
If problem still persists, please make note of it in this bug report.

Comment 22 Fedora Update System 2022-05-01 20:37:53 UTC
FEDORA-EPEL-2022-ff4479fc04 has been pushed to the Fedora EPEL 8 stable repository.
If problem still persists, please make note of it in this bug report.

Comment 23 Fedora Update System 2022-05-07 04:30:05 UTC
FEDORA-2022-4dd42b644f 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.