Note: This is a public test instance of Red Hat Bugzilla. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback at bugzilla.redhat.com.
Bug 177235 - Review Request: sysconftool - Macros for aclocal to install configuration files
Summary: Review Request: sysconftool - Macros for aclocal to install configuration files
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Tom "spot" Callaway
QA Contact: Fedora Package Reviews List
URL:
Whiteboard:
Depends On:
Blocks: FE-ACCEPT
TreeView+ depends on / blocked
 
Reported: 2006-01-08 00:57 UTC by Dominik 'Rathann' Mierzejewski
Modified: 2014-09-10 16:46 UTC (History)
1 user (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2006-07-31 17:18:23 UTC
Type: ---
Embargoed:
gwync: fedora-cvs+


Attachments (Terms of Use)

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).


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