Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
https://bugzilla.redhat.com/show_bug.cgi?id=314871
Summary: Review Request: gimp-lqr-plugin - GIMP LiquidRescale Plug-In Product: Fedora Version: devel Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: low Component: Package Review AssignedTo: nobody@fedoraproject.org ReportedBy: alex@tvtransilvania.ro QAContact: extras-qa@fedoraproject.org CC: fedora-package-review@redhat.com,notting@redhat.com
Spec URL: http://tvtransilvania.ro/ics/Fedora/8/gimp-lqr-plugin.spec SRPM URL: http://tvtransilvania.ro/ics/Fedora/8/gimp-lqr-plugin-0.2.1-1.fc8.src.rpm Description: This package is a plug-in for GIMP 2.2 and newer. It implements the algorithm described in the paper "Seam Carving for Content-Aware Image Resizing" by Shai Avidan and Ariel Shamir, which can be found at http://www.faculty.idc.ac.il/arik/imret.pdf
This is my first package and I am seeking a sponsor.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: gimp-lqr-plugin - GIMP LiquidRescale Plug-In
https://bugzilla.redhat.com/show_bug.cgi?id=314871
------- Additional Comments From fedora@marcbradshaw.co.uk 2007-10-14 22:21 EST ------- Hi Alexandru,
As I am unable to sponsor you this is an informal (and incomplete) review but does hi-light some issues which need to be fixed.
?? - Meets Packaging Guidelines. ?? - Package named correctly NA - Patches named correctly OK - Spec file named correctly to match base OK - License is valid OK - Licence field matches package OK - Licence file installed if supplied OK - Spec file in American English ?? - Source matches upstream (md5) NO * Locales use %find_lang OK - %clean is present and correct OK - Package has correct buildroot. OK - Specfile Legible NO * Builds in Mock NA - %post/%postun calls ldconfig for sh libs OK - Owns directories it creates OK - No duplicate files NO * Has %defattr and has correct permissions NO * Macros used consistently OK - %doc does not affect runtime NA - Headers/static libs in -devel NA - .pc files in -devel NA - .so files in -devel NA - -devel requires base OK - Contains no .la libtool archive files OK - Does not own others files NA - .desktop files installed correctly NO * BuildRequires correct. ?? - Package is code or permissible content. OK - Package has rm -rf %{buildroot} at top of %install. $RPM_BUILD_ROOT used instead ?? - Package compiles and builds on at least one arch. ?? - rpmlint output. NA - documentation in -doc package ?? - final provides and requires are sane. OK - should have dist tag ?? - should package latest version
Comments and Suggestions.
Summary Field should not be in camel caps, in my opinion it would be better as "Liquid rescale plug-in for the GIMP" or something more descriptive like "Content-aware resizing plug-in for the GIMP"
The description does not state what the plugin actually does.
Please use the ${name} macrom in the Source0: URL:
%defattr MUST be at top of %files section
Group: would be better as Applications/Multimedia
There is a typo in the URL field.
Package fails to build in mock. Please see http://marcbradshaw.co.uk/packages/review/gimp-lqr-plugin/ for build logs
Please see the Locale section of http://fedoraproject.org/wiki/Packaging/Guidelines
-Marc
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: gimp-lqr-plugin - GIMP LiquidRescale Plug-In
https://bugzilla.redhat.com/show_bug.cgi?id=314871
tibbs@math.uh.edu changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |NEEDINFO Flag| |needinfo?(alex@tvtransilvani | |a.ro)
------- Additional Comments From tibbs@math.uh.edu 2007-11-21 16:16 EST ------- Was there any response to this? It's been over a month, and the above commentary needs to be addressed.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: gimp-lqr-plugin - GIMP LiquidRescale Plug-In
https://bugzilla.redhat.com/show_bug.cgi?id=314871
alex@tvtransilvania.ro changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEEDINFO |NEW Flag|needinfo?(alex@tvtransilvani| |a.ro) |
------- Additional Comments From alex@tvtransilvania.ro 2007-11-22 13:21 EST ------- Hi Marc,
I've followed your pointers to resolve the issues. I'm very new when it comes to mock, so I don't have a fully buildsys right now (I'm still studying) and didn't test if it builds in mock.
Spec URL: http://tvtransilvania.ro/ics/Fedora/8/gimp-lqr-plugin.spec SRPM URL: http://tvtransilvania.ro/ics/Fedora/8/gimp-lqr-plugin-0.3.0-1.fc8.src.rpm
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: gimp-lqr-plugin - GIMP LiquidRescale Plug-In
https://bugzilla.redhat.com/show_bug.cgi?id=314871
------- Additional Comments From alex@tvtransilvania.ro 2007-11-22 14:30 EST ------- I've just successfully completed a mock build too.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: gimp-lqr-plugin - GIMP LiquidRescale Plug-In
https://bugzilla.redhat.com/show_bug.cgi?id=314871
mtasaka@ioa.s.u-tokyo.ac.jp changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |mtasaka@ioa.s.u-tokyo.ac.jp
------- Additional Comments From mtasaka@ioa.s.u-tokyo.ac.jp 2007-11-29 06:30 EST ------- Rebuild failed at least on dist-f9, i386. http://koji.fedoraproject.org/koji/taskinfo?taskID=265339 http://koji.fedoraproject.org/koji/getfile?taskID=265354&name=build.log
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: gimp-lqr-plugin - GIMP LiquidRescale Plug-In
https://bugzilla.redhat.com/show_bug.cgi?id=314871
bugzilla@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Version|devel |rawhide
mtasaka@ioa.s.u-tokyo.ac.jp changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |NEEDINFO Flag| |needinfo?(alex@tvtransilvani | |a.ro)
------- Additional Comments From mtasaka@ioa.s.u-tokyo.ac.jp 2007-12-09 08:33 EST ------- ping?
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: gimp-lqr-plugin - GIMP LiquidRescale Plug-In
https://bugzilla.redhat.com/show_bug.cgi?id=314871
alex@tvtransilvania.ro changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEEDINFO |NEW Flag|needinfo?(alex@tvtransilvani| |a.ro) |
------- Additional Comments From alex@tvtransilvania.ro 2007-12-10 09:13 EST ------- It builds OK on my rawhide machine, i386. Am I doing something wrong?
http://tvtransilvania.ro/ics/Fedora/devel/build.log http://tvtransilvania.ro/ics/Fedora/devel/gimp-lqr-plugin-0.3.0-1.fc9.src.rp...
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: gimp-lqr-plugin - GIMP LiquidRescale Plug-In
https://bugzilla.redhat.com/show_bug.cgi?id=314871
------- Additional Comments From mtasaka@ioa.s.u-tokyo.ac.jp 2007-12-10 11:22 EST ------- (In reply to comment #7)
It builds OK on my rawhide machine, i386. Am I doing something wrong?
You actually modified your spec file :) --------------------------------------------------- @@ -11,7 +8,7 @@ Source0: http://liquidrescale.wikidot.com/local--files/en:download-page/%%7Bname%7D-%... BuildRoot: %{_tmppath}/%{name}-%{version}-%{release}-%(%{__id_u} -n)
-BuildRequires: gimp, gimp-devel >= 2.2.13, perl-XML-Parser +BuildRequires: gimp, gimp-devel >= 2.2.13, perl-XML-Parser, gettext Requires: gimp >= 2.2.13, perl-XML-Parser
%description --------------------------------------------------- Now your srpm builds. http://koji.fedoraproject.org/koji/taskinfo?taskID=285397
! Note - From next time please change the release number of your spec file every time you modify your spec file.
For 0.3.0-1.fc9: * Perl modules dependency c.f. http://fedoraproject.org/wiki/Packaging/Perl - When writing perl modules dependency, please write the module name as dependency, not writing the rpm name directly, for example: --------------------------------------------------- BuildRequires: perl(XML::Parser)
* Tarball, versioning - The written Source0 tarball could not found. Instead I found
http://liquidrescale.wikidot.com/local--files/en:download-page/gimp-lqr-plug...
Umm... In this case, we regard the version as "0.3.0.6" as rpm "Version" does not accept hyphen.
* CFLAGS ---------------------------------------------------- make %{?_smp_mflags} CFLAGS="%{optflags}" ---------------------------------------------------- - Please check if this CFLAGS is really needed. %configure sets CFLAGS as environ (check what %configure actually does by `rpm --eval %configure`) and most recent Makefiles treats this properly.
* Timestamps - Please check if the following method keeps timestamps on installed files ---------------------------------------------------- make install DESTDIR=$RPM_BUILD_ROOT INSTALL="install -p" ---------------------------------------------------- While sometimes this does not work, this usually works on recent Makefiles.
* Documents - Please add the following documents to %doc. ---------------------------------------------------- AUTHORS NEWS ----------------------------------------------------
* Directory ownership issue: - Please consider the ownership issue of directories. ---------------------------------------------------- %files %{_datadir}/gimp-lqr-plugin/help/en/gimp-help.xml %{_datadir}/gimp-lqr-plugin/help/en/index.html %{_datadir}/gimp-lqr-plugin/help/images/dialog1.png %{_datadir}/gimp-lqr-plugin/help/images/dialog2.png ....... ----------------------------------------------------
This causes the following problem, for example. ---------------------------------------------------- [tasaka1@localhost ~]$ rpm -qf /usr/share/gimp-lqr-plugin/help/en/index.html gimp-lqr-plugin-0.3.0-1.fc9 [tasaka1@localhost ~]$ rpm -qf /usr/share/gimp-lqr-plugin/help/en/ file /usr/share/gimp-lqr-plugin/help/en is not owned by any package [tasaka1@localhost ~]$ rpm -qf /usr/share/gimp-lqr-plugin/help/ file /usr/share/gimp-lqr-plugin/help is not owned by any package [tasaka1@localhost ~]$ rpm -qf /usr/share/gimp-lqr-plugin/ file /usr/share/gimp-lqr-plugin is not owned by any package ---------------------------------------------------- Here - Installing gimp-lpr-plugin binary rpm creates the directories /usr/share/gimp-lpr-plugin, .../help, .../help/en to install index.html under the last directory - But these directories left unowned by any package. This autually causes some problems.
! Note - When we write ------------------------------------------------------ %files %{_datadir}/gimp-lqr-plugin/ ------------------------------------------------------ This contains the directory %_datadir/gimp-lpr-plugin itself _and_ all files/directories/etc under %_datadir/gimp-lpr-plugin, where ------------------------------------------------------ %files %dir %{_datadir}/gimp-lqr-plugin/ ------------------------------------------------------ contains the directory %_datadir/gimp-lqr-plugin only.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: gimp-lqr-plugin - GIMP LiquidRescale Plug-In
https://bugzilla.redhat.com/show_bug.cgi?id=314871
mtasaka@ioa.s.u-tokyo.ac.jp changed:
What |Removed |Added ---------------------------------------------------------------------------- OtherBugsDependingO| |177841 nThis| |
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: gimp-lqr-plugin - GIMP LiquidRescale Plug-In
https://bugzilla.redhat.com/show_bug.cgi?id=314871
mtasaka@ioa.s.u-tokyo.ac.jp changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |NEEDINFO Flag| |needinfo?(alex@tvtransilvani | |a.ro)
------- Additional Comments From mtasaka@ioa.s.u-tokyo.ac.jp 2007-12-18 11:59 EST ------- ping?
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: gimp-lqr-plugin - GIMP LiquidRescale Plug-In
https://bugzilla.redhat.com/show_bug.cgi?id=314871
------- Additional Comments From mtasaka@ioa.s.u-tokyo.ac.jp 2007-12-25 13:25 EST ------- ping again?
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: gimp-lqr-plugin - GIMP LiquidRescale Plug-In
https://bugzilla.redhat.com/show_bug.cgi?id=314871
------- Additional Comments From mtasaka@ioa.s.u-tokyo.ac.jp 2008-01-06 10:27 EST ------- I will close this bug as NOTABUG if no response from the reporter is received by 2008-01-17.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: gimp-lqr-plugin - GIMP LiquidRescale Plug-In
https://bugzilla.redhat.com/show_bug.cgi?id=314871
ewan@macmahon.me.uk changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |ewan@macmahon.me.uk
------- Additional Comments From ewan@macmahon.me.uk 2008-01-15 19:57 EST ------- I've had a completely independent (and slightly different) package of this plugin sat around for a while pending some tidy ups. I've got it into a presentable state and submitted a review request as bug #428916; if the OP of this package doesn't reappear it may be worth considering that as an alternative.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: gimp-lqr-plugin - GIMP LiquidRescale Plug-In
https://bugzilla.redhat.com/show_bug.cgi?id=314871
alex@tvtransilvania.ro changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEEDINFO |NEW Flag|needinfo?(alex@tvtransilvani| |a.ro) |
------- Additional Comments From alex@tvtransilvania.ro 2008-01-16 07:41 EST ------- The OP just finished his vacation and will build the new lqr plugin tonight. Alex
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: gimp-lqr-plugin - GIMP LiquidRescale Plug-In
https://bugzilla.redhat.com/show_bug.cgi?id=314871
------- Additional Comments From alex@tvtransilvania.ro 2008-01-17 17:02 EST ------- Spec URL: http://tvtransilvania.ro/ics/Fedora/9/gimp-lqr-plugin.spec SRPM URL: http://tvtransilvania.ro/ics/Fedora/9/gimp-lqr-plugin-0.4.0-1.fc9.src.rpm
I have also commited a review request for the lqr library: BZ #429202
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: gimp-lqr-plugin - GIMP LiquidRescale Plug-In
https://bugzilla.redhat.com/show_bug.cgi?id=314871
mtasaka@ioa.s.u-tokyo.ac.jp changed:
What |Removed |Added ---------------------------------------------------------------------------- BugsThisDependsOn| |429202
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: gimp-lqr-plugin - GIMP LiquidRescale Plug-In
https://bugzilla.redhat.com/show_bug.cgi?id=314871
mtasaka@ioa.s.u-tokyo.ac.jp changed:
What |Removed |Added ---------------------------------------------------------------------------- OtherBugsDependingO| |182235 nThis| |
------- Additional Comments From mtasaka@ioa.s.u-tokyo.ac.jp 2008-01-18 11:27 EST ------- Well, I also checked liblqr-1 and
* liblqr-1 (bug 429202) is actually licensed under GPLv3 (strict). * This package gimp-lpr-plugin is licensed under GPLv2 (strict) * gimp-lpr-plugin depends on liblqr-1 * GPLv2 (strict) and GPLv3 (strict) are incompatible....
Setting FE-Legal
It seems that the upstreams of the both softwares are same.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: gimp-lqr-plugin - GIMP LiquidRescale Plug-In
https://bugzilla.redhat.com/show_bug.cgi?id=314871
------- Additional Comments From tcallawa@redhat.com 2008-01-18 12:23 EST ------- Perhaps, given that the upstreams are identical, they could sync up the licenses to match? :)
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: gimp-lqr-plugin - GIMP LiquidRescale Plug-In
https://bugzilla.redhat.com/show_bug.cgi?id=314871
------- Additional Comments From mtasaka@ioa.s.u-tokyo.ac.jp 2008-01-31 02:39 EST ------- Are there any response from upstream?
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: gimp-lqr-plugin - GIMP LiquidRescale Plug-In
https://bugzilla.redhat.com/show_bug.cgi?id=314871
------- Additional Comments From alex@tvtransilvania.ro 2008-01-31 09:44 EST ------- Carlo Baldassi wrote:
Ok, I have changed the licensing of the plugin from "GPLv2 strict" to "GPLv2 or later", which I hope solves the issue.
I've been studying the licenses and as far as I can tell the two packages are now fully compatible. I'll post SRPMS later today.
Regards, Alex
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: gimp-lqr-plugin - GIMP LiquidRescale Plug-In
https://bugzilla.redhat.com/show_bug.cgi?id=314871
mtasaka@ioa.s.u-tokyo.ac.jp changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |NEEDINFO Flag| |needinfo?(alex@tvtransilvani | |a.ro)
------- Additional Comments From mtasaka@ioa.s.u-tokyo.ac.jp 2008-02-12 07:04 EST ------- ping again?
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: gimp-lqr-plugin - GIMP LiquidRescale Plug-In
https://bugzilla.redhat.com/show_bug.cgi?id=314871
davidf@sjsoft.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |davidf@sjsoft.com
------- Additional Comments From davidf@sjsoft.com 2008-02-19 03:24 EST ------- This spec file needs to be adjusted to reflect the 0.4.0-4 tarball versioning done by gimp-lqr-plugin. In addition it should include BuildRequires: lqr-1-devel
Adjusted spec file at http://davidf.sjsoft.com/files/gimp-lqr-plugin.spec
This builds installs and runs fine on my Fedora 8 i386 system.
These improvements are from https://bugzilla.redhat.com/show_bug.cgi?id=428916 which is a duplicate package request
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: gimp-lqr-plugin - GIMP LiquidRescale Plug-In
https://bugzilla.redhat.com/show_bug.cgi?id=314871
alex@tvtransilvania.ro changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEEDINFO |NEW Flag|needinfo?(alex@tvtransilvani| |a.ro) |
------- Additional Comments From alex@tvtransilvania.ro 2008-02-19 15:32 EST ------- New packages with all the issues fixed(hopefully). Spec URL: http://tvtransilvania.ro/ics/Fedora/devel/gimp-lqr-plugin.spec SRPM URL: http://tvtransilvania.ro/ics/Fedora/devel/gimp-lqr-plugin-0.4.0-3.fc9.src.rp...
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: gimp-lqr-plugin - GIMP LiquidRescale Plug-In
https://bugzilla.redhat.com/show_bug.cgi?id=314871
------- Additional Comments From mtasaka@ioa.s.u-tokyo.ac.jp 2008-02-20 00:41 EST ------- Still gimp-lqr-plugin is GPLv2 strict and liblqr-1 (bug 429202) is GPLv3. Note that we are not judging the license issues by the license tags in the spec files, but by actually checking the source tarballs.
Would you ask the upstream to actually release the new version of gimp-lqr-plugin tarball to fix the license conflict from liblqr-1?
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: gimp-lqr-plugin - GIMP LiquidRescale Plug-In
https://bugzilla.redhat.com/show_bug.cgi?id=314871
------- Additional Comments From mtasaka@ioa.s.u-tokyo.ac.jp 2008-02-20 00:48 EST ------- By the way the reason we are saying that gimp-lqr-plugin is actually licensed under strict GPLv2 is that the source codes under src directory actually says they are licensed under GPLv2 only.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: gimp-lqr-plugin - GIMP LiquidRescale Plug-In
https://bugzilla.redhat.com/show_bug.cgi?id=314871
------- Additional Comments From alex@tvtransilvania.ro 2008-02-20 06:17 EST -------
Spec URL: http://tvtransilvania.ro/ics/Fedora/devel/gimp-lqr-plugin.spec SRPM URL: http://tvtransilvania.ro/ics/Fedora/devel/gimp-lqr-plugin-0.4.0.4-1.fc9.src....
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: gimp-lqr-plugin - GIMP LiquidRescale Plug-In
https://bugzilla.redhat.com/show_bug.cgi?id=314871
mtasaka@ioa.s.u-tokyo.ac.jp changed:
What |Removed |Added ---------------------------------------------------------------------------- OtherBugsDependingO|182235 | nThis| |
------- Additional Comments From mtasaka@ioa.s.u-tokyo.ac.jp 2008-02-20 10:47 EST ------- Okay, now gimp-lqr-plugin is under GPLv2+. Removing FE-Legal.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: gimp-lqr-plugin - GIMP LiquidRescale Plug-In
https://bugzilla.redhat.com/show_bug.cgi?id=314871
mtasaka@ioa.s.u-tokyo.ac.jp changed:
What |Removed |Added ---------------------------------------------------------------------------- AssignedTo|nobody@fedoraproject.org |mtasaka@ioa.s.u-tokyo.ac.jp Status|NEW |ASSIGNED Flag| |fedora-review?
------- Additional Comments From mtasaka@ioa.s.u-tokyo.ac.jp 2008-02-29 11:23 EST ------- Once liblqr-1 is imported on devel branch, I will review this.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: gimp-lqr-plugin - GIMP LiquidRescale Plug-In
https://bugzilla.redhat.com/show_bug.cgi?id=314871
mtasaka@ioa.s.u-tokyo.ac.jp changed:
What |Removed |Added ---------------------------------------------------------------------------- OtherBugsDependingO|177841 | nThis| |
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: gimp-lqr-plugin - GIMP LiquidRescale Plug-In
https://bugzilla.redhat.com/show_bug.cgi?id=314871
Bug 314871 depends on bug 429202, which changed state.
Bug 429202 Summary: Review Request: liblqr-1 - LiquidRescale library https://bugzilla.redhat.com/show_bug.cgi?id=429202
What |Old Value |New Value ---------------------------------------------------------------------------- Status|ASSIGNED |CLOSED Resolution| |NEXTRELEASE
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: gimp-lqr-plugin - GIMP LiquidRescale Plug-In
https://bugzilla.redhat.com/show_bug.cgi?id=314871
------- Additional Comments From mtasaka@ioa.s.u-tokyo.ac.jp 2008-03-02 13:26 EST ------- For 0.4.0.4-1:
* Requires - Requires: liblqr-1 >= 0.1.0 is not needed. Library dependencies like these are automatically checked and added by rpmbuild itself.
- Requires: perl(XML::Parser) is not needed. This is used only for BuildRequires.
* BuildRequires - "BuildRequres: gimp" is not needed ("BuildRequires: gimp-devel" is needed)
* Directoy ownership issue - The directory %{_datadir}/gimp-lqr-plugin is not owned by any package.
* %changelog - It is preferable you write %changelog as: --------------------------------------------------------- * Wed Feb 20 2008 Alexandru Ciobanu alex@tvtransilvania.ro - 0.4.0.4-1 - Remode hyphen from versioning.
* Tue Feb 19 2008 Alexandru Ciobanu alex@tvtransilvania.ro - 0.4.0-3 - Fixed licensing issue.
* Mon Feb 4 2008 Alexandru Ciobanu alex@tvtransilvania.ro - 0.4.0-2 ---------------------------------------------------------
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: gimp-lqr-plugin - GIMP LiquidRescale Plug-In
https://bugzilla.redhat.com/show_bug.cgi?id=314871
------- Additional Comments From alex@tvtransilvania.ro 2008-03-02 16:25 EST ------- Spec URL: http://tvtransilvania.ro/ics/Fedora/devel/gimp-lqr-plugin.spec SRPM URL: http://tvtransilvania.ro/ics/Fedora/devel/gimp-lqr-plugin-0.4.0.4-2.fc9.src....
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: gimp-lqr-plugin - GIMP LiquidRescale Plug-In
https://bugzilla.redhat.com/show_bug.cgi?id=314871
mtasaka@ioa.s.u-tokyo.ac.jp changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag|fedora-review? |fedora-review+
------- Additional Comments From mtasaka@ioa.s.u-tokyo.ac.jp 2008-03-03 09:25 EST ------- Okay.
----------------------------------------------------------------------- This package (gimp-lqr-plugin) is APPROVED by me -----------------------------------------------------------------------
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: gimp-lqr-plugin - GIMP LiquidRescale Plug-In
https://bugzilla.redhat.com/show_bug.cgi?id=314871
alex@tvtransilvania.ro changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag| |fedora-cvs?
------- Additional Comments From alex@tvtransilvania.ro 2008-03-03 10:15 EST ------- New Package CVS Request ======================= Package Name: gimp-lqr-plugin Short Description: Content-aware resizing plug-in for the GIMP Owners: ics Branches: F-7 F-8 InitialCC: Cvsextras Commits: yes
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: gimp-lqr-plugin - GIMP LiquidRescale Plug-In
https://bugzilla.redhat.com/show_bug.cgi?id=314871
kevin@tummy.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag|fedora-cvs? |fedora-cvs+
------- Additional Comments From kevin@tummy.com 2008-03-03 15:00 EST ------- cvs done.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: gimp-lqr-plugin - GIMP LiquidRescale Plug-In
https://bugzilla.redhat.com/show_bug.cgi?id=314871
alex@tvtransilvania.ro changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |CLOSED Resolution| |NEXTRELEASE
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: gimp-lqr-plugin - GIMP LiquidRescale Plug-In
https://bugzilla.redhat.com/show_bug.cgi?id=314871
------- Additional Comments From mtasaka@ioa.s.u-tokyo.ac.jp 2008-03-19 13:35 EST ------- *** Bug 428916 has been marked as a duplicate of this bug. ***
package-review@lists.fedoraproject.org