Note: This is a beta release of Red Hat Bugzilla 5.0. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Also email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback here.

Bug 226433

Summary: Merge Review: stardict
Product: [Fedora] Fedora Reporter: Nobody's working on this, feel free to take it <nobody>
Component: Package ReviewAssignee: Parag AN(पराग) <panemade>
Status: CLOSED RAWHIDE QA Contact: Fedora Package Reviews List <fedora-package-review>
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideCC: zhu
Target Milestone: ---Flags: panemade: fedora-review+
Target Release: ---   
Hardware: All   
OS: Linux   
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2007-09-19 06:58:54 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:

Description Nobody's working on this, feel free to take it 2007-01-31 21:01:38 UTC
Fedora Merge Review: stardict
Initial Owner:

Comment 1 Parag AN(पराग) 2007-08-31 10:40:45 UTC
rpmlint on binary RPM gave ->

I: stardict checking
W: stardict non-conffile-in-etc /etc/gconf/schemas/stardict.schemas
A non-executable file in your package is being installed in /etc, but is not
a configuration file. All non-executable files in /etc should be configuration
files. Mark the file as %config in the spec file.

W: stardict invalid-license GPL
The value of the License tag was not recognized.  Known values are:
"Affero GPL", "AFL", "ASL 1.0", "ASL 1.0+", "ASL 1.1", "ASL 1.1+", "ASL 2.0",
"ASL 2.0+", "APSL 2.0", "APSL 2.0+", "Artistic 2.0", "Artistic clarified",
"BitTorrent", "Boost", "BSD", "BSD with advertising", "CeCILL", "CDDL", "CPL",
"Condor", "Copyright only", "Cryptix", "Crystal Stacker", "EPL", "eCos", "EFL
2.0", "EFL 2.0+", "EU Datagrid", "Giftware", "Glide", "GPL+", "GPL+ or
Artistic", "GPLv2", "GPLv2 with exceptions", "GPLv2+", "GPLv3", "GPLv3+",
"IBM", "IJG", "iMatix", "Intel ACPI", "Interbase", "ISC", "Jabber", "JasPer",
"LGPLv2", "LGPLv2 with exceptions", "LGPLv2+", "LGPLv3", "LGPLv3+", "libtiff",
"LPL", "LPPL", "mecab-ipadic", "MIT", "MPLv1.0", "MPLv1.0+", "MPLv1.1",
"MPLv1.1+", "NCSA", "NGPL", "NOSL", "Netscape", "Nokia", "OpenLDAP", "OSL
1.0", "OSL 1.0+", "OSL 2.0", "OSL 2.0+", "OSL 3.0", "OSL 3.0+", "OpenSSL",
"Phorum", "PHP", "Public Domain", "Python", "QPL", "RPSL", "Ruby",
"Sleepycat", "SISSL", "SPL", "TCL", "UCD", "Vim", "VNLSL", "VSL", "W3C",
"WTFPL", "wxWindows", "xinetd", "Zend", "ZPLv1.0", "ZPLv1.0+", "ZPLv2.0",
"ZPLv2.0+", "zlib", "CDL", "FBSDDL", "GFDL", "Open Publication", "CC-BY", "CC-
BY-SA", "DSL", "Free Art", "Arphic", "Bitstream Vera", "mplus", "OFL",
"Utopia", "Redistributable, no modification permitted", "Freely
redistributable without restriction".

Comment 2 Hu Zheng 2007-09-03 02:51:50 UTC
How to resolve the first warning? I see gedit, pidgin all have this problem.

I will change the License tag to GPLv3.

Comment 3 Parag AN(पराग) 2007-09-03 03:32:35 UTC
add %config to %{_sysconfdir}/gconf/schemas/*.schemas

%config(noreplace)  %{_sysconfdir}/gconf/schemas/*.schemas

Comment 4 Hu Zheng 2007-09-03 03:45:44 UTC
OK, cvs updated. No rpmlint warning now.

Comment 5 Mamoru TASAKA 2007-09-03 04:20:15 UTC
(In reply to comment #3)
> add %config to %{_sysconfdir}/gconf/schemas/*.schemas
> -->
> %config(noreplace)  %{_sysconfdir}/gconf/schemas/*.schemas

Well.. gconf schemas is actually not a config file and we
usually don't mark this as %config.

Comment 6 Parag AN(पराग) 2007-09-03 04:26:35 UTC
(In reply to comment #5)
> (In reply to comment #3)
> > add %config to %{_sysconfdir}/gconf/schemas/*.schemas
> > 
> > -->
> > %config(noreplace)  %{_sysconfdir}/gconf/schemas/*.schemas
> > 
> Well.. gconf schemas is actually not a config file and we
> usually don't mark this as %config.

thanks for pointing that. So still no decision has been made on this issue. Also
,rpmlint bug is waiting for some conclusion.

  Can you please revert back changes in %files section to remove %config(noreplace)?

Can't see changelog updated in SPEC?
why following files is not added to %doc?
COPYING Changelog Doc/FAQ  Doc/HACKING  doc/HowToCreateDictionary
doc/StarDictFileFormat  doc/Translation

Also, Is it good to add test directory to %doc?

Comment 7 Hu Zheng 2007-09-03 05:50:59 UTC
Done. Updated in CVS.

Comment 8 Parag AN(पराग) 2007-09-03 08:54:23 UTC
Looks I missed one last thing -> desktop files handling.
and update SPEC. Not necessary to increase release but add changelog under
3.0.0-4 version

Comment 9 Hu Zheng 2007-09-03 09:37:46 UTC
Done again:)

Comment 10 Parag AN(पराग) 2007-09-03 11:06:50 UTC
hey you forgot to change %files
mock build gave me
error: Installed (but unpackaged) file(s) found:
When using syntax to install desktop files from
desktop file name got renamed to fedora-%{desktopfilename}.desktop

Also, I think you should use following syntax for installation of desktop file
desktop-file-install --vendor fedora --delete-original  \
  --dir $RPM_BUILD_ROOT%{_datadir}/applications         \

Comment 11 Hu Zheng 2007-09-04 02:49:15 UTC
Sorry, It is fixed now.
Is it OK now?

Comment 12 Parag AN(पराग) 2007-09-04 04:16:58 UTC
+ package builds in mock (development i386).
+ rpmlint is silent for SRPM.
- rpmlint is not silent on RPM
W: stardict non-conffile-in-etc /etc/gconf/schemas/stardict.schemas
 but this can be ignored.
+ source files match upstream.
23bbfd0a75baab5d0def5596f19b80cb  stardict-3.0.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 is included in package.
+ %doc files present.
+ BuildRequires are proper.
+ %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.
+ Desktop file handled correctly.
+ scrollkeeper scriptlet used.
+ Binary RPM stardict-3.0.0-4.fc8 ->
  Requires: bonobo-activation >= 2.2.0 enchant espeak gucharmap
libbonobo >= 2.2.0 libgnome >= 2.2.0 libgnomeui >= 2.2.0 libsigc++20
>= 2.0.17 rtld(GNU_HASH)
+ GUI app.