[Bug 1541566] Review Request: whipper - Python CD-DA ripper

https://bugzilla.redhat.com/show_bug.cgi?id=1541566 Fedora Update System changed: What|Removed |Added Status|ON_QA

[Bug 1541566] Review Request: whipper - Python CD-DA ripper

https://bugzilla.redhat.com/show_bug.cgi?id=1541566 Fedora Update System changed: What|Removed |Added Status|MODIFIED|ON_QA

[Bug 1541566] Review Request: whipper - Python CD-DA ripper

https://bugzilla.redhat.com/show_bug.cgi?id=1541566 Fedora Update System changed: What|Removed |Added Status|NEW

[Bug 1541566] Review Request: whipper - Python CD-DA ripper

https://bugzilla.redhat.com/show_bug.cgi?id=1541566 --- Comment #17 from Fedora Update System --- whipper-0.6.0-6.fc27 has been submitted as an update to Fedora 27. https://bodhi.fedoraproject.org/updates/FEDORA-2018-55d9cc8fb4 -- You are receiving this mail

[Bug 1541566] Review Request: whipper - Python CD-DA ripper

https://bugzilla.redhat.com/show_bug.cgi?id=1541566 --- Comment #16 from William Moreno --- See the dos about setup.py you must add a setup.cfg with: [build_ext] inplace=1 To build c extensions in place, a good patch to send upstream. -- You are receiving this

[Bug 1541566] Review Request: whipper - Python CD-DA ripper

https://bugzilla.redhat.com/show_bug.cgi?id=1541566 --- Comment #15 from Gwyn Ciesla --- (fedrepo-req-admin): The Pagure repository was created at https://src.fedoraproject.org/rpms/whipper -- You are receiving this mail because: You are on the CC list for the bug. You

[Bug 1541566] Review Request: whipper - Python CD-DA ripper

https://bugzilla.redhat.com/show_bug.cgi?id=1541566 William Moreno changed: What|Removed |Added Blocks|177841 (FE-NEEDSPONSOR) |

[Bug 1541566] Review Request: whipper - Python CD-DA ripper

https://bugzilla.redhat.com/show_bug.cgi?id=1541566 William Moreno changed: What|Removed |Added Flags|fedora-review?

[Bug 1541566] Review Request: whipper - Python CD-DA ripper

https://bugzilla.redhat.com/show_bug.cgi?id=1541566 --- Comment #13 from William Moreno --- You are rigth, this package build some c code. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this

[Bug 1541566] Review Request: whipper - Python CD-DA ripper

https://bugzilla.redhat.com/show_bug.cgi?id=1541566 --- Comment #12 from Matthew Ruszczyk --- (In reply to William Moreno from comment #10) > From the build log > > %make_build -C src > > + /usr/bin/make -O -j6 -C src > make: Entering directory

[Bug 1541566] Review Request: whipper - Python CD-DA ripper

https://bugzilla.redhat.com/show_bug.cgi?id=1541566 --- Comment #11 from William Moreno --- Also note that for Fedora 29 you will need to add BuildRequires: gcc https://fedoraproject.org/wiki/Changes/Remove_GCC_from_BuildRoot -- You are receiving this mail

[Bug 1541566] Review Request: whipper - Python CD-DA ripper

https://bugzilla.redhat.com/show_bug.cgi?id=1541566 --- Comment #10 from William Moreno --- From the build log %make_build -C src + /usr/bin/make -O -j6 -C src make: Entering directory '/builddir/build/BUILD/whipper-0.6.0/src' accuraterip-checksum build options:

[Bug 1541566] Review Request: whipper - Python CD-DA ripper

https://bugzilla.redhat.com/show_bug.cgi?id=1541566 --- Comment #9 from Matthew --- Spec URL: https://copr-be.cloud.fedoraproject.org/results/mruszczyk/whipper/fedora-rawhide-x86_64/00714828-whipper/whipper.spec SRPM URL:

[Bug 1541566] Review Request: whipper - Python CD-DA ripper

https://bugzilla.redhat.com/show_bug.cgi?id=1541566 --- Comment #8 from Matthew --- Spec URL: https://copr-be.cloud.fedoraproject.org/results/mruszczyk/whipper/fedora-27-x86_64/00714229-whipper/whipper.spec SRPM URL:

[Bug 1541566] Review Request: whipper - Python CD-DA ripper

https://bugzilla.redhat.com/show_bug.cgi?id=1541566 --- Comment #7 from William Moreno --- Package Review == You are requering gobjects, is whipper a command line tool or provides a grafical user interface? If so work with upstream to include a desktop

[Bug 1541566] Review Request: whipper - Python CD-DA ripper

https://bugzilla.redhat.com/show_bug.cgi?id=1541566 --- Comment #6 from William Moreno --- > > - Is it really necessary to split the python part into a separate library > > package? It doesn't seem to be used as a library, only as a binary. > > I did this after

[Bug 1541566] Review Request: whipper - Python CD-DA ripper

https://bugzilla.redhat.com/show_bug.cgi?id=1541566 --- Comment #5 from Matthew --- Spec URL: https://copr-be.cloud.fedoraproject.org/results/mruszczyk/whipper/fedora-rawhide-x86_64/00711948-whipper/whipper.spec SRPM URL:

[Bug 1541566] Review Request: whipper - Python CD-DA ripper

https://bugzilla.redhat.com/show_bug.cgi?id=1541566 William Moreno changed: What|Removed |Added CC|

[Bug 1541566] Review Request: whipper - Python CD-DA ripper

https://bugzilla.redhat.com/show_bug.cgi?id=1541566 --- Comment #3 from Matthew --- Spec URL: https://copr-be.cloud.fedoraproject.org/results/mruszczyk/whipper/fedora-rawhide-x86_64/00708655-whipper/whipper.spec SRPM URL:

[Bug 1541566] Review Request: whipper - Python CD-DA ripper

https://bugzilla.redhat.com/show_bug.cgi?id=1541566 --- Comment #2 from Matthew --- (In reply to Robert-André Mauchin from comment #1) Thank you very much for the comments. > Just preliminary comments: > > - Split your Requires on multiple lines Done. > - Is it

[Bug 1541566] Review Request: whipper - Python CD-DA ripper

https://bugzilla.redhat.com/show_bug.cgi?id=1541566 Robert-André Mauchin changed: What|Removed |Added CC|

[Bug 1541566] Review Request: whipper - Python CD-DA ripper

https://bugzilla.redhat.com/show_bug.cgi?id=1541566 Matthew changed: What|Removed |Added Blocks||177841