Bug 225793 - Merge Review: gftp
Summary: Merge Review: gftp
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review (Show other bugs)
(Show other bugs)
Version: rawhide
Hardware: All Linux
Target Milestone: ---
Assignee: Tyler Owen
QA Contact: Fedora Package Reviews List
Depends On:
TreeView+ depends on / blocked
Reported: 2007-01-31 18:43 UTC by Nobody's working on this, feel free to take it
Modified: 2008-01-31 14:41 UTC (History)
1 user (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Last Closed: 2008-01-31 14:41:06 UTC
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---
tyler.l.owen: fedora-review+

Attachments (Terms of Use)

Description Nobody's working on this, feel free to take it 2007-01-31 18:43:12 UTC
Fedora Merge Review: gftp

Initial Owner: caillon@redhat.com

Comment 1 Tyler Owen 2008-01-13 14:53:18 UTC
OK - Package meets naming and packaging guidelines
OK - Spec file matches base package name.
FIX - Spec has consistant macro usage.
        Don't use %makeinstall macro
OK - Meets Packaging Guidelines.
OK - License
OK - License field in spec matches
OK - License file included in package
OK - Spec in American English
OK - Spec is legible.
OK - Sources match upstream md5sum:
44b19442d06f6dc68c7211d66029d826  gftp-2.0.18.tar.bz2
44b19442d06f6dc68c7211d66029d826  gftp-2.0.18.tar.bz2.1

OK - BuildRequires correct
OK - Spec handles locales/find_lang
OK - Package has %defattr and permissions on files is good.
OK - Package has a correct %clean section.
OK - Package has correct buildroot
      %{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n)
OK - Package is code or permissible content.

OK - Package is a GUI app and has a .desktop file
        should change vendor to fedora

OK - Package compiles and builds on at least one arch.
OK - Package has no duplicate files in %files.
OK - Package doesn't own any directories other packages own.
OK - Package owns all the directories it creates.
FIX - No rpmlint output.
        see issue #1


OK - Should build in mock.
OK - Should build on all supported archs
OK - Should function as described.
OK - Should have sane scriptlets.
OK - Should have dist tag
OK - Should package latest version


1. SRPM: 
        gftp.src: E: non-utf8-spec-file
        - The character encoding of the spec file is not UTF-8.  Convert it for
          example using iconv(1).

        gftp.src:142: W: macro-in-%changelog prefix
         - in changelog entries use %% instead of %

        gftp.src: W: mixed-use-of-spaces-and-tabs (spaces: line 55, tab: line 57)
        - Fix if you like

        gftp.src: E: tag-not-utf8 %changelog
        - The character encoding of the value of this tag is not UTF-8.

        gftp.i386: W: incoherent-version-in-changelog 1:2.0.18-1 2:2.0.18-1.fc9
        - The last entry in %changelog contains a version identifier that is not
          coherent with the epoch:version-release tuple of the package. 
        - Since this package has the history it does it is not possible to fix this

        gftp.i386: E: tag-not-utf8 %changelog
        - The character encoding of the value of this tag is not UTF-8.

2. Do not use %makeinstall macro

Comment 2 Christopher Aillon 2008-01-30 22:20:48 UTC
ok, try again using the cvs copy.

Comment 3 Tyler Owen 2008-01-31 14:41:06 UTC
Looks good.


All issues fixed, Closing out this Merge Review

Note You need to log in before you can comment on or make changes to this bug.