[Bug 485853] Review Request: calendar - Reminder utility

2009-02-21 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=485853


Juha Tuomala t...@iki.fi changed:

   What|Removed |Added

URL||http://www.openbsd.org:80/c
   ||gi-bin/cvsweb/src/usr.bin/c
   ||alendar/
 CC||t...@iki.fi




-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 485853] Review Request: calendar - Reminder utility

2009-02-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=485853


Kevin Fenzi ke...@tummy.com changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
   Flag|fedora-cvs? |fedora-cvs+




--- Comment #9 from Kevin Fenzi ke...@tummy.com  2009-02-20 14:44:54 EDT ---
cvs done.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 485853] Review Request: calendar - Reminder utility

2009-02-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=485853


David Cantrell dcantr...@redhat.com changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 485853] Review Request: calendar - Reminder utility

2009-02-19 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=485853


Warren Togami wtog...@redhat.com changed:

   What|Removed |Added

 AssignedTo|nob...@fedoraproject.org|wtog...@redhat.com
   Flag||fedora-review+




--- Comment #7 from Warren Togami wtog...@redhat.com  2009-02-19 21:06:01 EDT 
---
APPROVED

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 485853] Review Request: calendar - Reminder utility

2009-02-19 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=485853


David Cantrell dcantr...@redhat.com changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Comment #8 from David Cantrell dcantr...@redhat.com  2009-02-19 21:11:55 
EDT ---
New Package CVS Request
===
Package Name: calendar
Short Description: Reminder utility
Owners: dcantrel
Branches: F-9 F-10 EL-4 EL-5
InitialCC:

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 485853] Review Request: calendar - Reminder utility

2009-02-17 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=485853





--- Comment #1 from David Cantrell dcantr...@redhat.com  2009-02-17 15:37:14 
EDT ---
Correction on SRPM URL:
http://dcantrel.fedorapeople.org/calendar-1.25-1.fc11.src.rpm

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 485853] Review Request: calendar - Reminder utility

2009-02-17 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=485853


Warren Togami wtog...@redhat.com changed:

   What|Removed |Added

 CC||wtog...@redhat.com




--- Comment #2 from Warren Togami wtog...@redhat.com  2009-02-17 16:02:22 EDT 
---
calendar.src: W: strange-permission export-calendar-source.sh 0775

export-calendar-source.sh to verify the source goes boom.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 485853] Review Request: calendar - Reminder utility

2009-02-17 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=485853





--- Comment #3 from Warren Togami wtog...@redhat.com  2009-02-17 16:20:12 EDT 
---
- RPM spec looks fine.
- export-calendar-source.sh fails for me, but manual comparison passes.
- My only concern is calendar might be too generic of a name.  Are you sure
nothing else that could possibly be added to the distro has this name?  I
approve this package if you ask fedora-devel-list for opinions and nobody
objects within 24 hours of that posting.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 485853] Review Request: calendar - Reminder utility

2009-02-17 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=485853





--- Comment #4 from David Cantrell dcantr...@redhat.com  2009-02-17 16:48:20 
EDT ---
Latest spec and source RPM:

http://dcantrel.fedorapeople.org/calendar.spec
http://dcantrel.fedorapeople.org/calendar-1.25-3.fc11.src.rpm

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 485853] Review Request: calendar - Reminder utility

2009-02-17 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=485853





--- Comment #5 from David Cantrell dcantr...@redhat.com  2009-02-17 18:53:37 
EDT ---
(In reply to comment #2)
 calendar.src: W: strange-permission export-calendar-source.sh 0775
 
 export-calendar-source.sh to verify the source goes boom.

rpmlint seems to just dislike executable scripts in source RPM files.  I've
changed this to 0644.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 485853] Review Request: calendar - Reminder utility

2009-02-17 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=485853





--- Comment #6 from David Cantrell dcantr...@redhat.com  2009-02-17 18:54:32 
EDT ---
(In reply to comment #3)
 - export-calendar-source.sh fails for me, but manual comparison passes.

I've changed export-calendar-source.sh to use 'cvs status' to get the revision
number of calendar.c.  That seemed to be failing on your system, but working on
mine,

 - My only concern is calendar might be too generic of a name.  Are you sure
 nothing else that could possibly be added to the distro has this name?  I
 approve this package if you ask fedora-devel-list for opinions and nobody
 objects within 24 hours of that posting.

I will send a message to fedora-devel-list asking for comments.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review