Bug 604438 - Review Request: rubygem-scaffold - Scaffold is a templating tool for Puppet
Summary: Review Request: rubygem-scaffold - Scaffold is a templating tool for Puppet
Keywords:
Status: CLOSED UPSTREAM
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Nobody's working on this, feel free to take it
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2010-06-15 21:42 UTC by Michael Stahnke
Modified: 2010-07-06 21:35 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2010-07-06 21:35:09 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)

Description Michael Stahnke 2010-06-15 21:42:21 UTC
Spec URL: http://stahnma.fedorapeople.org/reviews/rubygem-scaffold.spec
SRPM URL: http://stahnma.fedorapeople.org/reviews/rubygem-scaffold-0.0.3-1.fc14.src.rpm
Description: 
Scaffold allows you to create basic Puppet configuration, modules, nodes,
classes, functions and types.

Comment 1 Mamoru TASAKA 2010-06-25 18:27:07 UTC
Some notes:

* %define -> %global
  - Now we prefer to use %global rather than %define
    https://fedoraproject.org/wiki/Packaging/Guidelines#.25global_preferred_over_.25define

* ruby(abi) dependency
  - Writing ruby(abi) dependency is a must
    https://fedoraproject.org/wiki/Packaging/Ruby#Ruby_Packaging_Guidelines

* Explcitly defined macros
  - Explicitly defined %ruby_sitelib is used nowhere.
  - Please use explicitly defined %geminstdir also in %files

* Explicit version dependency
  - As rubygem-templater in Fedora on all supported branches have higher version
    than 0.5.0, the explicit ">= 0.5.0" part is not needed, ref:
    https://fedoraproject.org/wiki/Packaging/Guidelines#Explicit_Requires

* BuildRoot tag
  - BuildRoot tag is no longer used so you can remove this (although rpmlint may
    complain) on Fedora (not on EPEL)
    https://fedoraproject.org/wiki/Packaging/Guidelines#BuildRoot_tag

! %changelog
  - Please also write your name.

Comment 2 Michael Stahnke 2010-06-28 20:23:41 UTC
I must have submitted the gem2rpm spec and not the final one I cleaned up. :)  I'm out of town until the end of the week, so I will look at it then.

Comment 3 Michael Stahnke 2010-07-06 21:35:09 UTC
I'd like to withdraw this request. Upstream is going to replace it with another tool shortly.


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