[Bug 235825] New: Review Request: kftpgrabber - Graphical FTP client for KDE

[Red Hat Bugzilla]

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/bugzilla/show_bug.cgi?id=235825

Summary: Review Request: kftpgrabber - Graphical FTP client for
                    KDE
           Product: Fedora Extras
           Version: devel
          Platform: All
        OS/Version: Linux
            Status: NEW
          Severity: medium
          Priority: medium
         Component: Package Review
        AssignedTo: [e-mail]
        ReportedBy: [e-mail]
         QAContact: [e-mail]

Spec URL: http://odysseus.x-tnd.be/fedora/kftpgrabber/kftpgrabber-0.8.0-1.src.rpm
SRPM URL: http://odysseus.x-tnd.be/fedora/kftpgrabber/kftpgrabber.spec
Description: 
KFTPgrabber is a graphical FTP client for the K Desktop Environment.
It implements many features required for usable FTP interaction.

[Bug 235825] Review Request: kftpgrabber - Graphical FTP client for KDE

[Red Hat Bugzilla]

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: kftpgrabber - Graphical FTP client for KDE

https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=235825

------- Additional Comments From [e-mail]  2007-04-10 19:04 EST -------
Sorry for the inversion in the links :)

QT was not sourced
The icon didn't appears under gnome nor xfce
It seems a libs package was needed

New version is available : 
Spec URL: http://odysseus.x-tnd.be/fedora/kftpgrabber/kftpgrabber.spec
SRPM URL: http://odysseus.x-tnd.be/fedora/kftpgrabber/kftpgrabber-0.8.0-2.src.rpm

[Bug 235825] Review Request: kftpgrabber - Graphical FTP client for KDE

[Red Hat Bugzilla]

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: kftpgrabber - Graphical FTP client for KDE

https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=235825

------- Additional Comments From [e-mail]  2007-04-26 23:40 EST -------
unable to download SRPM

[Bug 235825] Review Request: kftpgrabber - Graphical FTP client for KDE

[Red Hat Bugzilla]

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: kftpgrabber - Graphical FTP client for KDE

https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=235825

------- Additional Comments From [e-mail]  2007-04-27 00:49 EST -------
Fixed (my apache was down...).

Thank you for poiting me out.

[Bug 235825] Review Request: kftpgrabber - Graphical FTP client for KDE

[Red Hat Bugzilla]

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: kftpgrabber - Graphical FTP client for KDE

https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=235825

------- Additional Comments From [e-mail]  2007-04-27 03:02 EST -------
the -devel package is missing to include 'Requires: pkgconfig'.

[Bug 235825] Review Request: kftpgrabber - Graphical FTP client for KDE

[Red Hat Bugzilla]

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: kftpgrabber - Graphical FTP client for KDE

https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=235825

------- Additional Comments From [e-mail]  2007-04-27 03:07 EST -------
oops my bad ignore above comment

[Bug 235825] Review Request: kftpgrabber - Graphical FTP client for KDE

[Red Hat Bugzilla]

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: kftpgrabber - Graphical FTP client for KDE

https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=235825

[e-mail] changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED
         AssignedTo|[e-mail]    |[e-mail]
               Flag|                            |fedora-review?

[Bug 235825] Review Request: kftpgrabber - Graphical FTP client for KDE

[Red Hat Bugzilla]

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: kftpgrabber - Graphical FTP client for KDE

https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=235825

[e-mail] changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
               Flag|fedora-review?              |fedora-review+

------- Additional Comments From [e-mail]  2007-04-27 03:15 EST -------
Review:
+ package builds in mock (development i386).
+ rpmlint is silent for SRPM and for RPMs.
+ source files match upstream.
dbbbca5cd4303db886a2d8dac39dd98c  kftpgrabber-0.8.0.tar.bz2
+ package meets naming and packaging guidelines.
+ specfile is properly named, is cleanly written
+ Spec file is written in American English.
+ Spec file is legible.
+ dist tag is present.
+ build root is correct.
+ license is open source-compatible.
+ License text COPYING is included in package.
+ %doc is small so no need of -doc subpackage.
+ BuildRequires are proper.
+ %clean is present.
+ package installed properly.
+ Macro use appears rather consistent.
+ Package contains code, not content.
+ no static libraries.
+ no .pc files are present.
+ -devel subpackage exists.
+ no .la files.
+ translations are available.
+ Does owns the directories it creates.
+ no duplicates in %files.
+ file permissions are appropriate.
+ Desktop file installed correctly.
+ scriptlets are used.
+ kftpgrabber Provides: kftpimportplugin_filezilla3.so kftpimportplugin_gftp.so
kftpimportplugin_kftp.so kftpimportplugin_ncftp.so
+ kftpgrabber-libs Provides: libkftpinterfaces.so.0
+ GUI app.
APPROVED.

[Bug 235825] Review Request: kftpgrabber - Graphical FTP client for KDE

[Red Hat Bugzilla]

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: kftpgrabber - Graphical FTP client for KDE

https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=235825

[e-mail] changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[e-mail]

------- Additional Comments From [e-mail]  2007-04-27 04:01 EST -------
Several comments:

* Are there any benefit for splitting -libs subpackage?
  Should this package support multilib installation, for example?
  If so, please explain why.

* Sourcing /etc/profile.d/qt.sh should not be needed now.

* Usually "INSTALL" document is needed for people who want to
  install this package by themselves and should not be needed for
  people using rpm system.

[Bug 235825] Review Request: kftpgrabber - Graphical FTP client for KDE

[Red Hat Bugzilla]

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: kftpgrabber - Graphical FTP client for KDE

https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=235825

------- Additional Comments From [e-mail]  2007-04-27 05:01 EST -------
(In reply to comment #7)
> Several comments:
> Are there any benefit for splitting -libs subpackage?

Do we always need to ask this question to submitter?
> Should this package support multilib installation, for example?
>   If so, please explain why.
> Sourcing /etc/profile.d/qt.sh should not be needed now.

ohh I have less experience on KDE package reviewing. Is that somewhere
mentioned (may be I missed to read that)?
> Usually "INSTALL" document is needed for people who want to
> install this package by themselves and should not be needed for
> people using rpm system.

Why should we so bother about INSTALL file installation?

[Bug 235825] Review Request: kftpgrabber - Graphical FTP client for KDE

[Red Hat Bugzilla]

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: kftpgrabber - Graphical FTP client for KDE

https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=235825

[e-mail] changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
               Flag|fedora-review+              |fedora-review?

------- Additional Comments From [e-mail]  2007-04-27 05:02 EST -------
mtasaka,
thanks for pointing out some missing things.
removing fedora-review+ flag for now.

[Bug 235825] Review Request: kftpgrabber - Graphical FTP client for KDE

[Red Hat Bugzilla]

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: kftpgrabber - Graphical FTP client for KDE

https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=235825

------- Additional Comments From [e-mail]  2007-04-27 05:18 EST -------
(In reply to comment #8)
> (In reply to comment #7)
> > Several comments:
> > Are there any benefit for splitting -libs subpackage?
>
> Do we always need to ask this question to submitter?

Splitting -libs package usually means that there are
_other_ software which uses only this -libs package and does
not require the "main" package of this software.

We should avoid unnecessary splitting of subpackage, which
causes only confusion.
> > Usually "INSTALL" document is needed for people who want to
> > install this package by themselves and should not be needed for
> > people using rpm system.
> Why should we so bother about INSTALL file installation?

Just because "don't install unnecessary files".
For INSTALL files, I see many people including INSTALL file without
any consideration and for most cases this file is not necessary.
The other case is that people don't include AUTHORS or COPYING
files..

We must check what files should be installed as documentation
carefully. On several (not a few) review requests, I comment
about documentation like "This file should be included as documentation"
or "This file is not necessary"

[Bug 235825] Review Request: kftpgrabber - Graphical FTP client for KDE

[Red Hat Bugzilla]

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: kftpgrabber - Graphical FTP client for KDE

https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=235825

------- Additional Comments From [e-mail]  2007-05-12 11:19 EST -------
New version is available : 
Spec URL: http://odysseus.x-tnd.be/fedora/kftpgrabber/kftpgrabber.spec
SRPM URL: http://odysseus.x-tnd.be/fedora/kftpgrabber/kftpgrabber-0.8.1-1.src.rpm

- Updated to version 0.8.1
- I've removed the libs subpackage

As for the INSTALL file, I really don't know if it should be removed or not, so
I let it for the devel package for the moment.

[Bug 235825] Review Request: kftpgrabber - Graphical FTP client for KDE

[Red Hat Bugzilla]

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: kftpgrabber - Graphical FTP client for KDE

https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=235825

------- Additional Comments From [e-mail]  2007-05-12 21:37 EST -------
Ok. So when I use "locate INSTALL" on FC6 system I got many packages using INSTALL.

Will review this on Monday.

[Bug 235825] Review Request: kftpgrabber - Graphical FTP client for KDE

[Red Hat Bugzilla]

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: kftpgrabber - Graphical FTP client for KDE

https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=235825

------- Additional Comments From [e-mail]  2007-05-12 22:12 EST -------
Ok requesting you to remove INSTALL as it really increases unnecessary files
count on system.

[Bug 235825] Review Request: kftpgrabber - Graphical FTP client for KDE

[Red Hat Bugzilla]

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: kftpgrabber - Graphical FTP client for KDE

https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=235825

------- Additional Comments From [e-mail]  2007-05-13 05:38 EST -------
Done, INSTALL is removed, I'll remove it from all other packages I will build
from now :)

Here is the last version package :
Spec URL: http://odysseus.x-tnd.be/fedora/kftpgrabber/kftpgrabber.spec
SRPM URL: http://odysseus.x-tnd.be/fedora/kftpgrabber/kftpgrabber-0.8.1-2.src.rpm

[Bug 235825] Review Request: kftpgrabber - Graphical FTP client for KDE

[Red Hat Bugzilla]

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: kftpgrabber - Graphical FTP client for KDE

https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=235825

[e-mail] changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
               Flag|fedora-review?              |fedora-review+

------- Additional Comments From [e-mail]  2007-05-13 23:47 EST -------
Review:
+ package builds in mock (development i386).
+ rpmlint is silent for SRPM and for RPMs.
+ source files match upstream.
56610fd3e2e7f092b7d8eed10d3e5d36  kftpgrabber-0.8.1.tar.bz2
+ package meets naming and packaging guidelines.
+ specfile is properly named, is cleanly written
+ Spec file is written in American English.
+ Spec file is legible.
+ dist tag is present.
+ build root is correct.
+ license is open source-compatible.
+ License text COPYING is included in package.
+ %doc is small so no need of -doc subpackage.
+ BuildRequires are proper.
+ %clean is present.
+ package installed properly.
+ Macro use appears rather consistent.
+ Package contains code, not content.
+ no static libraries.
+ no .pc files are present.
+ -devel subpackage exists.
+ no .la files.
+ translations are available.
+ Does owns the directories it creates.
+ no duplicates in %files.
+ file permissions are appropriate.
+ Desktop file installed correctly.
+ scriptlets are used.
+ Provides: kftpimportplugin_filezilla3.so kftpimportplugin_gftp.so
kftpimportplugin_kftp.so kftpimportplugin_ncftp.so libkftpinterfaces.so.0
+ GUI app.
APPROVED.

[Bug 235825] Review Request: kftpgrabber - Graphical FTP client for KDE

[Red Hat Bugzilla]

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: kftpgrabber - Graphical FTP client for KDE

https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=235825

[e-mail] changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
               Flag|                            |fedora-cvs?

------- Additional Comments From [e-mail]  2007-05-17 03:58 EST -------
New Package CVS Request
=======================
Package Name: kftpgrabber
Short Description: FTP client for K Desktop Environment
Owners: [e-mail]
Branches: FC-6
InitialCC: [e-mail]

[Bug 235825] Review Request: kftpgrabber - Graphical FTP client for KDE

[Red Hat Bugzilla]

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: kftpgrabber - Graphical FTP client for KDE

https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=235825

[e-mail] changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
               Flag|fedora-cvs?                 |fedora-cvs+

[Bug 235825] Review Request: kftpgrabber - Graphical FTP client for KDE

[Red Hat Bugzilla]

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: kftpgrabber - Graphical FTP client for KDE

https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=235825

[e-mail] changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
            Product|Fedora Extras               |Fedora

[e-mail] changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |NEEDINFO
               Flag|                            |needinfo?([e-mail])

------- Additional Comments From [e-mail]  2007-07-03 14:17 EST -------
What is the status of this bug?

[Bug 235825] Review Request: kftpgrabber - Graphical FTP client for KDE

[Red Hat Bugzilla]

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: kftpgrabber - Graphical FTP client for KDE

https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=235825

[e-mail] changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEEDINFO                    |ASSIGNED
               Flag|needinfo?([e-mail])   |

------- Additional Comments From [e-mail]  2007-07-03 15:17 EST -------
The package is now on the repositories for FC6 and F7, this bug should be closed
(I don't know if I can do this, and even, how to...)

[Bug 235825] Review Request: kftpgrabber - Graphical FTP client for KDE

[Red Hat Bugzilla]

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: kftpgrabber - Graphical FTP client for KDE

https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=235825

[e-mail] changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |CLOSED
         Resolution|                            |NEXTRELEASE

------- Additional Comments From [e-mail]  2007-07-03 23:22 EST -------
Just to change "Bug Status Change" written below.
For now I will change.

[Bug 235825] Review Request: kftpgrabber - Graphical FTP client for KDE

[Red Hat Bugzilla]

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: kftpgrabber - Graphical FTP client for KDE

https://bugzilla.redhat.com/show_bug.cgi?id=235825

[e-mail] changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
               Flag|fedora-cvs+                 |fedora-cvs?

------- Additional Comments From [e-mail]  2007-09-02 10:18 EST -------
Package Change Request
======================
Package Name: kftpgrabber
New Branches: F-7

[Bug 235825] Review Request: kftpgrabber - Graphical FTP client for KDE

[Red Hat Bugzilla]

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: kftpgrabber - Graphical FTP client for KDE

https://bugzilla.redhat.com/show_bug.cgi?id=235825

[e-mail] changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
               Flag|fedora-cvs?                 |fedora-cvs+

------- Additional Comments From [e-mail]  2007-09-03 14:38 EST -------
cvs done.