Bug 490354 - Review Request: emacs-elib - The Emacs Lisp Library
Summary: Review Request: emacs-elib - The Emacs Lisp Library
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
low
medium
Target Milestone: ---
Assignee: Jochen Schmitt
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: 490355
TreeView+ depends on / blocked
 
Reported: 2009-03-15 17:11 UTC by Dan Atmakin
Modified: 2013-10-19 14:42 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2009-10-12 11:04:28 UTC
Type: ---
Embargoed:
jochen: fedora-review?


Attachments (Terms of Use)

Description Dan Atmakin 2009-03-15 17:11:38 UTC
Spec URL: http://dev.sgu.ru/b/SPECS/emacs-elib.spec
SRPM URL: http://dev.sgu.ru/b/SRPMS/emacs-elib-1.0-1.fc10.src.rpm
Description: Elib, the GNU Emacs lisp library, is a collection of elisp functions which you can use as parts of your own elisp programs. Each file contains functions which have something in common, e.g. they handle a certain data type.

Comment 1 Jochen Schmitt 2009-05-17 19:33:36 UTC
Good:
+ Package name matches naming guildlines
+ Buildroot will be cleaned at the beginning of %clean and %install
* Specification of the Buildroot is ok.

Bad:
- Basename of the SPEC file doesn't machtes package name
- URL doesn't shows on proper project homepage
- Source doesn't contains full qualified URL
- Please change BuildArchitecures into BuildArch:
- Source are not in separe el subpackage
  (/Packaging/Emacs#Packaging_of_source_elisp_files)
- Specification of the installation destination on fullfill Emace
  Packaging guulldline
  (/Packaging/Emacs#GNU_Emacs)
- Package doesn't contains correct Changelog
  you have to specified version-release on it

 
Your package seems to need a lot of love. I want to suggest that you should
read the packaging guildlines and the guildlines for emacs packages carefully.

If you can demonstrate in your next approach, that you have understand this 
guildlines, I may be willing to sponsor you.

Comment 2 Jochen Schmitt 2009-06-03 17:29:09 UTC
Ping Dan.

Comment 3 Dan Atmakin 2009-06-03 17:48:01 UTC
Thanks for review, Jochen. Sorry for delay. Your remarks is really important but i can't rebuild srpm right now. I'll try to rebuild rpm according with gnuemacs packaging gudelines in the near future.


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