Bug 177235

Summary: Review Request: sysconftool - Macros for aclocal to install configuration files
Product: [Fedora] Fedora Reporter: Dominik 'Rathann' Mierzejewski <dominik>
Component: Package ReviewAssignee: Tom "spot" Callaway <tcallawa>
Status: CLOSED NEXTRELEASE QA Contact: Fedora Package Reviews List <fedora-package-review>
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideCC: steve
Target Milestone: ---Flags: gwync: fedora-cvs+
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2006-07-31 17:18:23 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: 163779    

Description Dominik 'Rathann' Mierzejewski 2006-01-08 00:57:11 UTC
Spec Name or Url: http://rpm.greysector.net/extras/sysconftool.spec
SRPM Name or Url: http://rpm.greysector.net/extras/sysconftool-0.15-1.src.rpm
Description: 
sysconftool is a development utility that helps to install application
configuration files. sysconftool allows an existing application to be
upgraded without losing the older version's configuration settings.

Comment 1 Tom "spot" Callaway 2006-01-13 16:04:20 UTC
Two minor issues that you should resolve before building:

- Development/Building is a non-standard Group setting. I know that Group is
pretty worthless right now, but please try to keep to the Fedora Standards found
here: http://fedoraproject.org/wiki/RPMGroups

- The autotools that you're using (autoconf/automake) should be listed as
BuildRequires.

Also, you should consider using %{?dist} at the end of the Release, if you plan
to build this for multiple branches (FC-3,FC-4,FC-5). See:
http://fedoraproject.org/wiki/DistTag

Review:

- rpmlint checks return:
W: sysconftool non-standard-group Development/Building (noted above)
W: sysconftool symlink-should-be-relative /usr/bin/sysconftoolize
/usr/share/sysconftool/sysconftoolize.pl
W: sysconftool symlink-should-be-relative /usr/bin/sysconftoolcheck
/usr/share/sysconftool/sysconftoolcheck

Safe to ignore these symlinks.

- package meets naming guidelines
- package meets packaging guidelines
- license (GPL) OK, text in %doc, matches source
- spec file legible, in am. english
- source matches upstream
- package compiles on devel (x86)
- no missing BR
- no unnecessary BR
- spec localized for en_US, pl, ok
- not relocatable
- owns all directories that it creates
- no duplicate files
- permissions ok
- %clean ok
- macro use consistent
- code, not content
- no need for -docs
- nothing in %doc affects runtime
- no need for .desktop file

This package is approved, with the caveat that you resolve the two minor issues
before building. Also, I'll sponsor you based on this package (and your Aurora
related work). Go ahead and do your paperwork.

Comment 2 Dominik 'Rathann' Mierzejewski 2006-01-13 17:53:50 UTC
Fixed all issues, including symlinks.

Comment 3 Christian Iseli 2006-03-28 15:39:53 UTC
Any particular reason why this package is not yet imported and built ?

Comment 4 Dominik 'Rathann' Mierzejewski 2006-03-28 19:40:11 UTC
Lack od time, I guess. And I still haven't figured out how to get a CVS account. :(

Comment 5 Tom "spot" Callaway 2006-04-07 18:33:47 UTC
Dominik, please go ahead and jump through the hoops to get this in. I'm trying
to clear out my outstanding bugs. :)

Comment 6 Michael Schwendt 2006-05-29 11:14:52 UTC
Hereby I'm withdrawing my offer to sponsor Dominik (bug 177105,
177104), because I haven't heard back from him for almost two
months and because Tom has offered to sponsor him nearly three
months earlier here.

With complaints that the package review queue takes too long to
process, I find it impolite not to react in a timely manner.

Also, the NEED-SPONSOR dependencies ought to be removed from all
of a packager's tickets as soon as somebody offers sponsorship.
I'm going to do that now, since the system does NOT work if this
is not done.


Comment 7 Michael Schwendt 2006-05-29 11:20:06 UTC
Dropping FE-NEEDSPONSOR. Tom Callaway offered sponsorship in bug 177235
(and bugzilla change-several-bugs-at-once feature requires me to add/edit
something).

Comment 8 Michael Schwendt 2006-05-29 11:29:26 UTC
uhm, bugzilla is broken :(

Comment 9 Dominik 'Rathann' Mierzejewski 2006-07-30 18:49:14 UTC
Source checked in, devel build requested and succeeded, waiting for fc5 and fc4
branches to be created.

Comment 10 Dominik 'Rathann' Mierzejewski 2006-07-31 17:18:23 UTC
FC-4 and FC-5 created and built successfully, closing as NEXTRELEASE.

Comment 11 Dominik 'Rathann' Mierzejewski 2014-09-10 16:05:23 UTC
Package Change Request
======================
Package Name: sysconftool
New Branches: el6 epel7
Owners: rathann

Comment 12 Gwyn Ciesla 2014-09-10 16:46:30 UTC
Git done (by process-git-requests).