Bug 427342 - Review Request: jday - A simple command to convert calendar dates to julian dates
Summary: Review Request: jday - A simple command to convert calendar dates to julian d...
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review   
(Show other bugs)
Version: rawhide
Hardware: All Linux
Target Milestone: ---
Assignee: Parag AN(पराग)
QA Contact: Fedora Extras Quality Assurance
Depends On:
TreeView+ depends on / blocked
Reported: 2008-01-03 09:31 UTC by Huzaifa S. Sidhpurwala
Modified: 2008-01-12 15:07 UTC (History)
4 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Last Closed: 2008-01-12 15:07:47 UTC
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---
panemade: fedora-review+
kevin: fedora-cvs+

Attachments (Terms of Use)

Description Huzaifa S. Sidhpurwala 2008-01-03 09:31:58 UTC
Spec URL: http://www.geocities.com/fatema_mhs/jday.spec.txt
SRPM URL: http://www.geocities.com/fatema_mhs/jday-2.4-0.fc8.src.rpm
A simple command to convert calendar dates to julian dates.
Quite useful in timing situations where you need elapsed time between dates.
Also useful for astronomy applications.
I want to include this in the astronomy spin of fedora.

Comment 1 Parag AN(पराग) 2008-01-03 09:47:50 UTC
You need some SPEC cleanups and follow packaging guidelines as per given at

1) change source URL from
  check more on it http://fedoraproject.org/wiki/Packaging/SourceURL

2) SHOULD follow macros. READ more on it

3) ldconfig usage should be as per given

4) you need to add -devel subpackage which includes header files and .so files

provide updated package links, updating release tag and adding correct changelog
of what change you did in SPEC.

Comment 2 Parag AN(पराग) 2008-01-03 09:52:13 UTC
5) Don't include .a and .la files in %files. Fedora packaging guidelines

6) usage of make should be as per given at

7) This package does not honor compiler flags look at 

Comment 3 Huzaifa S. Sidhpurwala 2008-01-03 11:07:57 UTC
I have made the required changed to the spec please review the new spec at:
and SRPM at:

Comment 4 Parag AN(पराग) 2008-01-04 04:50:35 UTC
1) You need to follow Source URL (macro using) as
2) Don't use like this
   use clean and not confusing and accepted way of writing in spec 

3)Also, I see your spec file ends with .txt in SPEC url you provides. any reason?
SPEC file should end with .spec only.

Comment 5 Huzaifa S. Sidhpurwala 2008-01-04 05:37:54 UTC
Made the necessary changes as requested.
The reason why i am calling the spec file as spec.txt because i am using free
yahoo web hosting to upload my files and it does not accept .spec when uploading
the files.

SPEC File: http://www.geocities.com/fatema_mhs/jday.spec.txt
SRPM: http://www.geocities.com/fatema_mhs/jday-2.4-1.f8.srpm

Comment 6 Parag AN(पराग) 2008-01-04 06:55:18 UTC
Thanks for updates.
Will sponsor you after I see enough reviews from you to other people's packages.
and then review this officially also.

Comment 7 Huzaifa S. Sidhpurwala 2008-01-07 10:57:36 UTC

Comment 8 Parag AN(पराग) 2008-01-07 11:18:37 UTC
+ package builds in mock (development i386).
+ rpmlint is silent for RPM But NOT for RPM.
jday.src: W: mixed-use-of-spaces-and-tabs (spaces: line 2, tab: line 1)
==>cosmetic message. 
+ source files match upstream url
817af56005d1346677271ed2fab63ef5  jday-2.4.tar.gz
+ package meets naming and packaging guidelines.
+ specfile is properly named, is cleanly written
+ Spec file is written in American English.
+ Spec file is legible.
+ dist tag is present.
+ build root is correct.
+ license is open source-compatible.
+ License text is included in package.
+ BuildRequires are proper.
+ Compiler flags used correctly.
+ defattr usage is correct.
+ %clean is present.
+ package installed properly.
+ Macro use appears rather consistent.
+ Package contains code, not content.
+ no static libraries.
+ jday.pc file present.
+ -devel subpackage exists.
+ no .la files.
+ no translations are available.
+ Does owns the directories it creates.
+ ldconfig scriptlets present.
+ no duplicates in %files.
+ file permissions are appropriate.
+ Package jday-2.4-1.fc9 ->
  Provides: libjday.so.2
  Requires: libc.so.6 libc.so.6(GLIBC_2.0) libc.so.6(GLIBC_2.1.3)
libc.so.6(GLIBC_2.3.4) libjday.so.2 rtld(GNU_HASH)
+ Package jday-devel-2.4-1.fc9 ->
  Requires: jday = 2.4-1.fc9 libjday.so.2
+ Not a GUI App.

Comment 9 Huzaifa S. Sidhpurwala 2008-01-08 01:47:20 UTC
New Package CVS Request
Package Name: jday
Short Description: A Simple command to convert calendar dates to Julian dates
Owners: huzaifas
Branches:  F-8
InitialCC: huzaifsa
Cvsextras Commits: no


Comment 10 Kevin Fenzi 2008-01-08 03:18:15 UTC
cvs done. 

Any reason to now allow cvsextras commits? 

Comment 11 Huzaifa S. Sidhpurwala 2008-01-08 03:26:10 UTC
Sorry i copy pasted this from somewhere else and this is an oversight
Can you do this please.

Comment 12 Kevin Fenzi 2008-01-08 03:37:24 UTC
cvsextras added. Thanks. 

Comment 13 Parag AN(पराग) 2008-01-12 15:07:47 UTC
Closing this Review ticket as this package is successfully built on koji for
requested branches.

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