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 211242 (scribes-templates) - Review Request: scribes-templates - Templates ("Snippets") for the Scribes text editor
Summary: Review Request: scribes-templates - Templates ("Snippets") for the Scribes te...
Keywords:
Status: CLOSED NEXTRELEASE
Alias: scribes-templates
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Mamoru TASAKA
QA Contact: Fedora Package Reviews List
URL:
Whiteboard:
Depends On:
Blocks: FE-ACCEPT
TreeView+ depends on / blocked
 
Reported: 2006-10-18 01:29 UTC by Peter Gordon
Modified: 2007-11-30 22:11 UTC (History)
1 user (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2006-10-20 01:03:14 UTC
Type: ---
Embargoed:

Attachments (Terms of Use)

Description Peter Gordon 2006-10-18 01:29:14 UTC 
Spec URL: http://thecodergeek.com/downloads/fedora/scribes-templates.spec
SRPM URL: http://thecodergeek.com/downloads/fedora/scribes-templates-20061017-1.src.rpm

Scribes is a simple and sleek text editor for the the GNOME desktop. It focuses
on keeping the working environment free from distractions, and providing strong
automation of mundane, repetitive tasks. One of its greatest features is a
dynamic templating system (popularly called Snippets) to automate the creation
of various types of files following the same general format. This package
contains a collection of various pre-made templates for Scribes.

rpmlint is silent on both the noarch binary RPM and the source RPM.
Comment 1 Mamoru TASAKA 2006-10-18 15:43:30 UTC 
1.
Well, while writing  %build section seems preferable even if
it is empty, don't write %pre %preun.. etc if they are empty because
writing "%post" (even if its content is empty) calls a shell 
(in this case, it is /bin/bash) without autual need 
and adds redundant "Requires: /bin/bash" to the rpm package.
2.
The line: install -m 0644 %{SOURCE1} .
Please keep timestamps, i.e. use "install -p" (or "cp -p").
3.
By the way, does upstream have a plan to change the numbering of
"VERSION"? Usually writing date to version is not preferable.
Comment 2 Peter Gordon 2006-10-18 19:39:24 UTC 
Spec: http://thecodergeek.com/downloads/fedora/scribes-templates.spec
SRPM: http://thecodergeek.com/downloads/fedora/scribes-templates-20061017-2.src.rpm

The first two issues I've fixed in release 2, linked above.

> By the way, does upstream have a plan to change the numbering of
> "VERSION"? Usually writing date to version is not preferable.
Datestamps like this are the current plan (since they will feasibly not be
updated quite as often as Scribes itself). Why are datestamps like this not
preferred? (There really is no other specific versioning to it...)

Thanks for your suggestions!
Comment 3 Mamoru TASAKA 2006-10-19 16:19:45 UTC 
(In reply to comment #2)

>  Why are datestamps like this not
> preferred? (There really is no other specific versioning to it...)

There are many cases that a source numbered by date means it is a pre-version
and in that case the VERSION in spec file should have something like
0.%{date}. However, if it is not for this package, it is okay.

---------------------------------------------------------------------
   This package (scribes-templates) is APPROVED by me.
Comment 4 Peter Gordon 2006-10-20 01:03:14 UTC 
Imported and built for devel. Thanks for the review! :)
Note You need to log in before you can comment on or make changes to this bug.