Bug 188661

Summary: Review Request: gcalctool
Product: [Fedora] Fedora Reporter: Matthias Clasen <mclasen>
Component: Package ReviewAssignee: David Cantrell <dcantrell>
Status: CLOSED RAWHIDE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideCC: fedora-package-review, wtogami
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2006-04-19 17:44:25 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 188268    

Description Matthias Clasen 2006-04-12 00:57:56 UTC
Spec URL: http://people.redhat.com/mclasen/review/gcalctool.spec
SRPM URL: http://people.redhat.com/mclasen/review/gcalctool-5.7.32-2.src.rpm
Description: This is part of the effort to split the current hard to maintain
multitarball gnome-utils packages into separate packages.

Comment 1 Matthias Clasen 2006-04-12 01:32:03 UTC
I notice that I missed the gconf schema installation in %post

Comment 2 Warren Togami 2006-04-12 18:47:22 UTC
If you make changes to the package, please bump the Release and post the new
URL.  Please do not replace the existing filename with new content.

Comment 3 Jesse Keating 2006-04-13 16:22:22 UTC
Requires may automatically get picked up from %post, but not sure, will test later.

.desktop file is included, but not installed.  Please use desktop-file-install
within %install.

Will try building package next.

Comment 4 Jesse Keating 2006-04-13 16:59:05 UTC
Also, before anybody else notices, the spec file linked is not the same one in
the srpm, the linked one is newer.

Comment 5 Jesse Keating 2006-04-13 18:45:53 UTC
During the build of i386, there are lots of errors about not being able to
install schema files.  Are these expected?

WARNING: failed to install schema `/schemas/apps/gcalctool/accuracy' locale `C':
Unable to store a value at key '/schemas/apps/gcalctool/accuracy', as the
configuration server has no writable databases. There are some common causes of
this problem: 1) your configuration path file /etc/gconf/2/path doesn't contain
any databases or wasn't found 2) somehow we mistakenly created two gconfd
processes 3) your operating system is misconfigured so NFS file locking doesn't
work in your home directory or 4) your NFS client machine crashed and didn't
properly notify the server on reboot that file locks should be dropped. If you
have two gconfd processes (or had two at the time the second was launched),
logging out, killing all copies of gconfd, and logging back in may help. If you
have stale locks, remove ~/.gconf*/*lock. Perhaps the problem is that you
attempted to use GConf from two machines at once, and ORBit still has its
default configuration that prevents remote CORBA connections - put
"ORBIIOPIPv4=1" in /etc/orbitrc. As always, check the user.* syslog for details
on problems gconfd encountered. There can only be one gconfd per home directory,
and it must own a lockfile in ~/.gconfd and also lockfiles in individual storage
locations such as ~/.gconf

Same for /schemas/apps/gcalctool/base, /schemas/apps/gcalctool/display,
/schemas/apps/gcalctool/mode, /schemas/apps/gcalctool/showzeroes,
/schemas/apps/gcalctool/showhelp, /schemas/apps/gcalctool/showregisters,
/schemas/apps/gcalctool/righthanded, /schemas/apps/gcalctool/trigtype,
/schemas/apps/gcalctool/beep.

Also there was this:

Resolved address "xml:merged:/etc/gconf/gconf.xml.defaults" to a read-only
configuration source at position 0
None of the resolved addresses are writable; saving configuration settings will
not be possible


Comment 6 Matthias Clasen 2006-04-13 18:54:41 UTC
Hmm, I will investigate this after the spring break.

Comment 7 Jesse Keating 2006-04-13 19:03:46 UTC
[builder@dhcp83-49 SPECS]$ rpmlint
/var/lib/mock/fedora-development-x86_64-core/result/gcalctool-5.7.32-2.x86_64.rpm
W: gcalctool non-conffile-in-etc /etc/gconf/schemas/gcalctool.schemas
W: gcalctool one-line-command-in-%post scrollkeeper-update
W: gcalctool one-line-command-in-%postun scrollkeeper-update

[builder@dhcp83-49 SPECS]$ rpmlint
/var/lib/mock/fedora-development-i386-core/result/gcalctool-5.7.32-2.i386.rpm
W: gcalctool non-conffile-in-etc /etc/gconf/schemas/gcalctool.schemas
W: gcalctool one-line-command-in-%post scrollkeeper-update
W: gcalctool one-line-command-in-%postun scrollkeeper-update

Package builds w/ mock.

Comment 8 Matthias Clasen 2006-04-14 00:12:14 UTC
I have to admit that I am somewhat sceptic regarding some of the rpmlint warnings.
I do not think .schemas files should be conffiles, and I have no idea whats wrong
about a one-line %post script.

Comment 9 Jesse Keating 2006-04-14 02:35:58 UTC
OH right, I forgot to note that those are PASSES of rpmlint, not failures (;

So just the other issues should be looked into.

Comment 10 Matthias Clasen 2006-04-17 18:58:09 UTC
New spec: http://people.redhat.com/mclasen/review/gcalctool.spec
New srpm: http://people.redhat.com/mclasen/review/gcalctool-5.7.32-3.src.rpm

changes:
- disable schema installation during %install
- install schema in %post
- added Requires for gconf
- I did not try yo fix the (harmless) scrollkeeper error messages
  during %install, since the configure does not understand --disable-scrollkeeper

Comment 11 Jesse Keating 2006-04-17 20:40:04 UTC
Changes look sane, rpmlint much quieter, package builds w/ mock.  Approving.

Comment 12 Matthias Clasen 2006-04-19 17:44:25 UTC
Package is in rawhide

Comment 13 Michael Schwendt 2006-04-20 11:25:27 UTC
It's missing the --makefile-uninstall rule in %postun

Comment 14 Matthias Clasen 2006-04-20 14:15:37 UTC
None of the core packages uninstalls schemas...

Comment 15 Jesse Keating 2006-04-20 14:18:44 UTC
Doesn't mean we shouldn't.  I let it go in w/out the removal, but you should add
it to CVS and next time the package gets bumped it will have the code there.

Comment 16 Matthias Clasen 2006-04-20 14:28:16 UTC
This bug is closed; and I'm not going to accept a "permanently under review"
position, just because I pushed the package through the initial review...

Comment 17 Michael Schwendt 2006-04-20 20:01:53 UTC
Be nice, be friendly, fix your package, and lead by example.


Comment 18 Jesse Keating 2006-04-20 20:08:14 UTC
This has already been fixed in cvs.  Any further issues should be a new bug, not
more comments on a closed bug.