Bug 682489 - Review Request: drupal6-acidfree - A full-featured media management system for Drupal
Summary: Review Request: drupal6-acidfree - A full-featured media management system fo...
Keywords:
Status: CLOSED WONTFIX
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Paul W. Frields
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On: 682486 682487 682488
Blocks:
TreeView+ depends on / blocked
 
Reported: 2011-03-06 01:26 UTC by eric
Modified: 2012-02-10 05:20 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2012-02-10 05:20:35 UTC
Type: ---
Embargoed:
stickster: fedora-review?


Attachments (Terms of Use)

Description eric 2011-03-06 01:26:12 UTC
Spec URL: http://sparks.fedorapeople.org/Packages/drupal6-acidfree.spec

SRPM URL: http://sparks.fedorapeople.org/Packages/drupal6-acidfree-6.x.1.0.beta2-1.fc14.src.rpm

Description: 
A full-featured media management system for drupal. This makes it so you can 
easily manage your photos and videos with drupal. Acidfree albums and media 
items are drupal nodes to integrate better into your system. It offers 
themeable drawing routines for albums and each media type so it can fit in 
better with the rest of your website. Items can be added individually (the 
drupal node-centric method) and also via a mass-import page that allows server-
side directories of media to be imported or zipped, bzipped or gzipped archives
to be uploaded and imported.

Comment 1 Paul W. Frields 2011-04-06 00:22:22 UTC
Hey Eric, a couple things that need fixing:

* %exclude the documentation that's currently showing up in %{_datadir}/drupal6. (LICENSE, README, UPGRADE, CHANGELOG.)

* The version you're using needs fixing. The 6.x should not appear there, that's what the '6' in 'drupal6' is for. Check the specfile template on the wiki for an example: https://fedoraproject.org/wiki/Drupal_module_specfile_template

* The release tag also needs fixing, since this is a pre-release. Read http://fedoraproject.org/wiki/Packaging/NamingGuidelines#Pre-Release_packages for more information. This should be something like: 0.1.beta2%{?dist}

All together the NVR would be: drupal6-acidfree-1.0-0.1.beta2%{?dist}

Feel free to NEEDINFO me when you put your next attempt up and I'll get right to it.

Comment 2 Paul W. Frields 2011-06-26 15:46:53 UTC
Any update for this package, Eric?

Comment 3 Paul W. Frields 2011-08-26 19:57:20 UTC
Hi Eric, hopefully the changes I made to some of your other packages show what needs to be done for this one.  If you don't have time to work on this package, let me know and I'll release the review for now.


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