Bug 431373 - Review Request: incollector - Information collector
Summary: Review Request: incollector - Information collector
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Parag AN(पराग)
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2008-02-03 19:49 UTC by Krzysztof Kurzawski
Modified: 2008-02-05 11:48 UTC (History)
2 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2008-02-05 11:48:25 UTC
Type: ---
Embargoed:
panemade: fedora-review+
kevin: fedora-cvs+


Attachments (Terms of Use)

Description Krzysztof Kurzawski 2008-02-03 19:49:12 UTC
Spec URL: http://kurzawa.nonlogic.org/rpm/incollector/incollector.spec
SRPM URL: http://kurzawa.nonlogic.org/rpm/incollector/incollector-1.0-1.fc8.src.rpm
Description: Incollector is an application to collect various kinds of information 
(like notes, conversation logs, quotes, serial numbers, source code, 
web addresses, words). All the entries can be tagged, so you can find 
them very easily. There are also search folders which allows you to 
search for entries by specified criteria. You can also export 
(and import, of course) entries to an external file.

Comment 1 Parag AN(पराग) 2008-02-04 03:45:02 UTC
build.log gave me
+ /usr/lib/rpm/find-lang.sh /var/tmp/incollector-1.0-1.fc9-root-mockbuild
incollector
No translations found for incollector in
/var/tmp/incollector-1.0-1.fc9-root-mockbuild
===> missing BR: gettext desktop-file-utils

any reason to exclude 
%exclude %{_datadir}/locale/be-latin/LC_MESSAGES/incollector.mo



Comment 2 Parag AN(पराग) 2008-02-04 03:58:38 UTC
Source URL should be
http://www.incollector.devnull.pl/download/sources/%{name}-%{version}.tar.gz

Comment 3 Krzysztof Kurzawski 2008-02-04 08:09:14 UTC
Fixed
Spec URL: http://kurzawa.nonlogic.org/rpm/incollector/incollector.spec
SRPM URL: http://kurzawa.nonlogic.org/rpm/incollector/incollector-1.0-2.fc8.src.rpm

I exclude %exclude %{_datadir}/locale/be-latin/LC_MESSAGES/incollector.mo
because rpmlint warns:

incollector.i386: E: incorrect-locale-subdir
/usr/share/locale/be-latin/LC_MESSAGES/incollector.mo

Thanks for review!

Comment 4 Parag AN(पराग) 2008-02-04 09:17:12 UTC
I think you can try to use sed to replace be-latin.* to be@latin.* in build
directory and also you need to have following %prep

pushd po
mv be-latin.po be
mv be-latin.gmo be
popd


Comment 5 Parag AN(पराग) 2008-02-04 09:23:04 UTC
you just need to use following in %prep
sed -i 's|be-latin|be@latin|' configure
pushd po
mv be-latin.po be
mv be-latin.gmo be
popd

and this will solve your problem of rpmlint error.

Comment 7 Parag AN(पराग) 2008-02-04 10:23:38 UTC
koji build failed
http://koji.fedoraproject.org/koji/taskinfo?taskID=394153

SHOULD:
   Not use macro for install command "%{__install} -p". Just use "install -p"
   Because you have not used macros for commands rm , make in spec.



Comment 8 Parag AN(पराग) 2008-02-04 10:34:04 UTC
This worked for me 
ExclusiveArch: %ix86 x86_64 ppc 


Comment 9 Parag AN(पराग) 2008-02-04 10:45:30 UTC
Actually it should be
ExclusiveArch: %ix86 x86_64 ppc ia64 armv4l sparc alpha
----from gtk-sharp2.spec

Comment 11 Krzysztof Kurzawski 2008-02-04 11:07:07 UTC
Whooops I wrote bad link do SRPMS:

SRPM URL: http://kurzawa.nonlogic.org/rpm/incollector/incollector-1.0-3.fc8.src.rpm

Comment 12 Krzysztof Kurzawski 2008-02-04 11:07:48 UTC
Here is correct link:

SRPM URL: http://kurzawa.nonlogic.org/rpm/incollector/incollector-1.0-4.fc8.src.rpm

Comment 13 Parag AN(पराग) 2008-02-04 12:19:25 UTC
oops you add "----from gtk-sharp2.spec"
remove this text

Comment 15 Parag AN(पराग) 2008-02-04 12:45:04 UTC
Review:
+ package builds in mock (rawhide i386).
koji build=>http://koji.fedoraproject.org/koji/taskinfo?taskID=394396
+ rpmlint is silent for SRPM
- rpmlint complains for RPM as
incollector.i386: E: no-binary
incollector.i386: E: only-non-binary-in-usr-lib
But these can be ignored for mono packages.
+ source files match upstream.
da510a279ebb5e8ebf9dbdcb2fc1cf18  incollector-1.0.tar.gz
+ 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 is included in package.
+ %doc files present.
+ BuildRequires are proper.
+ Compiler flags are honoured correctly.
+ defattr usage is correct.
+ %clean is present.
+ package installed properly.
+ Macro use appears rather consistent.
+ Package contains code.
+ no static libraries.
+ no .pc file present.
+ no -devel subpackage exists.
+ no .la files.
+ translations are available.
+ Does owns the directories it creates.
+ no duplicates in %files.
+ file permissions are appropriate.
+ no scriptlets are used.
+ Desktop file handled correctly.
+ GUI app.

APPROVED.


Comment 16 Krzysztof Kurzawski 2008-02-04 12:51:19 UTC
New Package CVS Request
=======================
Package Name: incollector
Short Description: Information collector
Owners: kurzawa
Branches: F-7 F-8
InitialCC:
Cvsextras Commits: yes

Comment 17 Kevin Fenzi 2008-02-04 19:43:38 UTC
cvs done.

Comment 18 Krzysztof Kurzawski 2008-02-05 11:48:25 UTC
Imported and built. Thanks again for review!


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