https://bugzilla.redhat.com/show_bug.cgi?id=851641
Bug ID: 851641 QA Contact: extras-qa@fedoraproject.org Severity: medium Version: rawhide Priority: unspecified CC: notting@redhat.com, package-review@lists.fedoraproject.org Assignee: nobody@fedoraproject.org Summary: Review Request: lprof - An open source color profiler that creates ICC compliant profiles Regression: --- Story Points: --- Classification: Fedora OS: Linux Reporter: bugs@dyroff.org Type: --- Documentation: --- Hardware: All Mount Type: --- Status: NEW Component: Package Review Product: Fedora
Spec URL: http://git.dyroff.org/?p=sdyroff-rpms.git;a=blob_plain;f=SPECS/lprof.spec;h=...
SRPM URL: http://www.dyroff.org/lprof-1.11.4.1-1.src.rpm
Description: LProf is an open source color profiler that creates ICC compliant profiles for devices such as cameras, scanners and monitors.
Fedora Account System Username:sdyroff
This is my first package, so it needs a sponsor.
https://bugzilla.redhat.com/show_bug.cgi?id=851641
--- Comment #1 from Sebastian Dyroff bugs@dyroff.org --- Created attachment 606894 --> https://bugzilla.redhat.com/attachment.cgi?id=606894&action=edit rpmlint output
https://bugzilla.redhat.com/show_bug.cgi?id=851641
Sebastian Dyroff bugs@dyroff.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Blocks| |177841 (FE-NEEDSPONSOR)
https://bugzilla.redhat.com/show_bug.cgi?id=851641
--- Comment #2 from Sebastian Dyroff bugs@dyroff.org --- Created attachment 606944 --> https://bugzilla.redhat.com/attachment.cgi?id=606944&action=edit koji output
I can't reach the koji webinteface(page loads forever). I decided to attach the console output instead.
https://bugzilla.redhat.com/show_bug.cgi?id=851641
Volker Fröhlich volker27@gmx.at changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |volker27@gmx.at
--- Comment #3 from Volker Fröhlich volker27@gmx.at --- Drop the Requires line. rpmbuild is able to figure that out. qt3-devel would have been wrong anyway.
As far as I can see, this package bundles code from at least Argyll and wxwidgets which is not allowed. Some snapshot of SCons is included as well.
What is Patch0 good for? Please put a comment in the spec file and refer to an upstream ticket, if possible.
Add sRGB_profile_License and KNOWN_BUGS to the doc macro.
""" %{_datadir}/%{name}/data/pics/* %{_datadir}/%{name}/data/profiles/* %{_datadir}/%{name}/data/template/* """
Using this syntax, you don't own {_datadir}/%{name}. I'd just replace those three lines with:
{_datadir}/%{name}
Please use the name macro for the patch names. I also suggest to use it for the icon and desktop file.
I personally consider "open source" not worth mentioning and only bloating description and summary, because everything in Fedora is Free Software.
Please inform upstream about the wrong postal address of the FSF. The actual license is GPLv2+, by the way. You can find out easily using "licensecheck".
The build doesn't use Fedora's compiler flags. Using them will also produce a useful debuginfo package.
https://bugzilla.redhat.com/show_bug.cgi?id=851641
--- Comment #4 from Sebastian Dyroff bugs@dyroff.org --- (In reply to comment #3)
First of all thanks for the feedback.
Drop the Requires line. rpmbuild is able to figure that out. qt3-devel would have been wrong anyway.
If I drop that line, I will get a runtime error complaining about the missing assistant executable(qt-assistant). Yum provides lists qt3-devel as providing package for this file. But there should be a comment in the spec-file.
As far as I can see, this package bundles code from at least Argyll and wxwidgets which is not allowed. Some snapshot of SCons is included as well.
What is a common way to fix this? Best fix would be if upstream could clean the release directory. Is it also a valid fix to remove the components and create a new release archive? Removing the SCons part is trivial, it is not used anyway. Removing wxwidgets and Argyll might be harder.
What is Patch0 good for? Please put a comment in the spec file and refer to an upstream ticket, if possible.
I will file a bug upstream for that. The Patch0 is also included in the Debian package. Without this patch it won't compile, with the error: 'BOOL' does not name a type.
""" %{_datadir}/%{name}/data/pics/* %{_datadir}/%{name}/data/profiles/* %{_datadir}/%{name}/data/template/* """
Using this syntax, you don't own {_datadir}/%{name}. I'd just replace those three lines with:
{_datadir}/%{name}
I did this in the first version, the problem is that %{_datadir}/%{name}/data/translations/ must be excluded because %find_lang takes care of these files. Is it OK to change the line %files -f %{name}.lang to just %files?
Please use the name macro for the patch names. I also suggest to use it for the icon and desktop file.
I personally consider "open source" not worth mentioning and only bloating description and summary, because everything in Fedora is Free Software.
That idea came to me too, after filing this bug. It is the project description on sourceforge :-).
https://bugzilla.redhat.com/show_bug.cgi?id=851641
--- Comment #5 from Sebastian Dyroff bugs@dyroff.org --- I fixed some of the issues, see http://git.dyroff.org/?p=sdyroff-rpms.git;a=blob;f=SPECS/lprof.spec;h=e1bf78...
I think this package won't make it in fedora any time soon. In order to remove the argyll part, which is linked statically, there needs to be a argyllcms-devel package. I think argyll doesn't even build a shared library for the used code.
I will wait until upstream reacts on the filed bug (bool-patch) and then try to discuss this issue with them.
https://bugzilla.redhat.com/show_bug.cgi?id=851641
Volker Fröhlich volker27@gmx.at changed:
What |Removed |Added ---------------------------------------------------------------------------- Whiteboard| |NotReady
--- Comment #6 from Volker Fröhlich volker27@gmx.at --- Therefore labelling it "not ready for review"
https://bugzilla.redhat.com/show_bug.cgi?id=851641
Mario Blättermann mario.blaettermann@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |mario.blaettermann@gmail.co | |m Blocks|177841 (FE-NEEDSPONSOR) |
--- Comment #7 from Mario Blättermann mario.blaettermann@gmail.com --- Lifting FE-NEEDSPONSOR, I'm sponsoring Sebastian.
https://bugzilla.redhat.com/show_bug.cgi?id=851641
--- Comment #8 from Volker Fröhlich volker27@gmx.at --- The lprof project seems to be dead as wood. Maybe we should close this for good.
https://bugzilla.redhat.com/show_bug.cgi?id=851641
--- Comment #9 from Sebastian Dyroff fedora@dyroff.org --- Go on and close it. We should also remove it from the package whishlist in the wiki.
https://bugzilla.redhat.com/show_bug.cgi?id=851641
--- Comment #10 from Upstream Release Monitoring upstream-release-monitoring@fedoraproject.org --- awd123's scratch build of color-0.0.8-1.src.rpm for f22 failed http://koji.fedoraproject.org/koji/taskinfo?taskID=11324994
https://bugzilla.redhat.com/show_bug.cgi?id=851641
--- Comment #11 from Upstream Release Monitoring upstream-release-monitoring@fedoraproject.org --- awd123's scratch build of color-0.0.8-1.src.rpm for f22 failed http://koji.fedoraproject.org/koji/taskinfo?taskID=11324998
https://bugzilla.redhat.com/show_bug.cgi?id=851641
--- Comment #12 from Upstream Release Monitoring upstream-release-monitoring@fedoraproject.org --- awd123's scratch build of color-0.0.8-1.src.rpm for f21 failed http://koji.fedoraproject.org/koji/taskinfo?taskID=11325006
https://bugzilla.redhat.com/show_bug.cgi?id=851641
--- Comment #13 from Upstream Release Monitoring upstream-release-monitoring@fedoraproject.org --- awd123's scratch build of color-0.0.8-1.src.rpm for f22 failed http://koji.fedoraproject.org/koji/taskinfo?taskID=11325010
https://bugzilla.redhat.com/show_bug.cgi?id=851641
--- Comment #14 from Upstream Release Monitoring upstream-release-monitoring@fedoraproject.org --- awd123's scratch build of color-0.0.8-1.src.rpm for f22 failed http://koji.fedoraproject.org/koji/taskinfo?taskID=11325018
package-review@lists.fedoraproject.org