Spec URL: http://tgmweb.at/gadllah/fotox.spec SRPM URL: http://tgmweb.at/gadllah/fotox-39-1.fc8.src.rpm Description: Easy to use image editor with features like panoramas, high dynamic range (HDR) composites, change brightness distribution, intensify colors, reduce haze, resize, rotate (any angle), crop, red-eye removal, sharpen, reduce noise, thumbnail image browser and navigator.
Well, fotox is under rapid development and at the time I am writing this comment the latest version is 40. So: Would you ask the upstream developer to change installation scripts to autotool based ones ? - Current upstream-created "build" script does not accept compiler option - desktop file is created under an unexpected directory - Current installation process is rather manual Also - No license text is included in tarball (v40)
(In reply to comment #1) Sorry for the delay. > Well, fotox is under rapid development and at the time I am > writing this comment the latest version is 40. So: Yeah now 41 is out. > Would you ask the upstream developer to change installation scripts > to autotool based ones ? > - Current upstream-created "build" script does not accept compiler > option Will ask upstream, if the don't do it I can patch it to use the flags. > - desktop file is created under an unexpected directory ? %{_datadir}/applications > - Current installation process is rather manual > Also > - No license text is included in tarball (v40) "MUST: If (and only if) the source package includes the text of the license(s) in its own file, then that file, containing the text of the license(s) for the package must be included in %doc." http://fedoraproject.org/wiki/Packaging/ReviewGuidelines So should be fine. I will build and upload an updated package in the next days.
(In reply to comment #2) > (In reply to comment #1) > > - desktop file is created under an unexpected directory > ? %{_datadir}/applications What I mean here that if you use "./build" script and execute "./build install", then this script tries to create desktop file under $HOME/Desktop. > > - Current installation process is rather manual > > Also > > - No license text is included in tarball (v40) > > "MUST: If (and only if) the source package includes the text of the license(s) > in its own file, then that file, containing the text of the license(s) for the > package must be included in %doc." > > http://fedoraproject.org/wiki/Packaging/ReviewGuidelines > > So should be fine. I know it. However I want you to contact with upstream.
*** Bug 447829 has been marked as a duplicate of this bug. ***
Any updates here?
Well I sent a patch to upstream to add an autotools based buildsys because upstream was saying it was "too complicated". Upstream has yet to release a version with this patch. Will ping him next week.
fotoxx the successor of fotox is under review now, so I will close this one.. *** This bug has been marked as a duplicate of 456517 ***