[Bug 475018] Review Request: xtvd - A client java library for easy access to the tv data from schedulesdirect.org

2009-01-26 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=475018


Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE
   Flag|needinfo?(r...@fedoraproject |
   |.org)   |




--- Comment #14 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp  2009-01-26 
10:26:27 EDT ---
Now closing, thank you.

-- 
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 475018] Review Request: xtvd - A client java library for easy access to the tv data from schedulesdirect.org

2009-01-26 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=475018





--- Comment #15 from Fedora Update System upda...@fedoraproject.org  
2009-01-26 20:46:18 EDT ---
xtvd-2.0.1-2.fc9 has been pushed to the Fedora 9 stable repository.  If
problems still persist, please make note of it in this bug report.

-- 
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 475018] Review Request: xtvd - A client java library for easy access to the tv data from schedulesdirect.org

2009-01-26 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=475018





--- Comment #16 from Fedora Update System upda...@fedoraproject.org  
2009-01-26 20:51:10 EDT ---
xtvd-2.0.1-2.fc10 has been pushed to the Fedora 10 stable repository.  If
problems still persist, please make note of it in this bug report.

-- 
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 475018] Review Request: xtvd - A client java library for easy access to the tv data from schedulesdirect.org

2009-01-24 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=475018


Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp changed:

   What|Removed |Added

   Flag||needinfo?(r...@fedoraproject
   ||.org)




--- Comment #13 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp  2009-01-25 
02:55:15 EDT ---
Please rebuild this package on koji and for F-10/9
submit requests to push the rebuilt packages into repositories.

-- 
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 475018] Review Request: xtvd - A client java library for easy access to the tv data from schedulesdirect.org

2009-01-16 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=475018


Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 AssignedTo|nob...@fedoraproject.org|mtas...@ioa.s.u-tokyo.ac.jp
   Flag||fedora-review+




--- Comment #10 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp  2009-01-16 
11:43:57 EDT ---
Well,
- Divide a bit long %changelog line into two or so.

-
This package (xtvd) is APPROVED by mtasaka
-

-- 
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 475018] Review Request: xtvd - A client java library for easy access to the tv data from schedulesdirect.org

2009-01-16 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=475018


Sandro Mathys s...@sandro-mathys.ch changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Comment #11 from Sandro Mathys s...@sandro-mathys.ch  2009-01-16 15:31:37 
EDT ---
New Package CVS Request
===
Package Name: xtvd
Short Description: A client java library for easy access to the tv data from
schedulesdirect.org
Owners: red
Branches: F-9 F-10
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 475018] Review Request: xtvd - A client java library for easy access to the tv data from schedulesdirect.org

2009-01-16 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=475018


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

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Comment #12 from Kevin Fenzi ke...@tummy.com  2009-01-16 22:32:34 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 475018] Review Request: xtvd - A client java library for easy access to the tv data from schedulesdirect.org

2009-01-15 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=475018





--- Comment #9 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp  2009-01-15 
13:40:04 EDT ---
Sorry for not responding.. I will check this package tomorrow or so.

-- 
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 475018] Review Request: xtvd - A client java library for easy access to the tv data from schedulesdirect.org

2009-01-09 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=475018





--- Comment #8 from Sandro Mathys s...@sandro-mathys.ch  2009-01-09 08:23:08 
EDT ---
Spec URL: http://red.fedorapeople.org/SRPMS/xtvd-2.0.1-2.fc11.src.rpm
SRPM URL: http://red.fedorapeople.org/SRPMS/xtvd.spec

The source archive is now created manually after a check out with svn. Nothing
else changed.

The naming is still the same - this is not before or after the 2.0.1 release,
this _is_ the 2.0.1 release. With the exception that no archive is provided via
download.sf.net but only a release tag on the svn was created (because only
that license text was added as I requested). The previous release was 2.0 and
the next will probably be 2.1 (if there will be a next version at all).

I hope we can agree on this. :)

-- 
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 475018] Review Request: xtvd - A client java library for easy access to the tv data from schedulesdirect.org

2008-12-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=475018





--- Comment #7 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp  2008-12-19 
12:44:51 EDT ---
(In reply to comment #6)
 2.0.1 is an official release (actually the same as 2.0(.0) but with additional
 license text in the lib part as I requested from upstream). Because of only
 this little change no tarball was created by upstream. But the used tarball is
 generated by upstream's system and consists of the release tag (not a certain
 revision or trunk or something).

- But as the tarball is created from svn system, please follow
  revision control tarball creation method
  (Using Download GNU tarball on SCM viewvc is not preferred)

  If this is after 2.0.1 release, you can follow
 
https://fedoraproject.org/wiki/Packaging/NamingGuidelines#Post-Release_packages

 Yes, I could modify the tarball but isn't it part of the guidelines that the
 upstream's distribution shouldn't be modified unless there's some problems 
 with
 the contents?

- Please see above (i.e. please create tarball by using svn co)

-- 
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 475018] Review Request: xtvd - A client java library for easy access to the tv data from schedulesdirect.org

2008-12-18 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=475018





--- Comment #6 from Sandro Mathys s...@sandro-mathys.ch  2008-12-18 16:51:53 
EDT ---
Spec URL: http://red.fedorapeople.org/SRPMS/xtvd-2.0.1-2.fc11.src.rpm
SRPM URL: http://red.fedorapeople.org/SRPMS/xtvd.spec

Actually, I don't think the pre-release/snapshot guidelines or the ones for
using revision control apply here.

2.0.1 is an official release (actually the same as 2.0(.0) but with additional
license text in the lib part as I requested from upstream). Because of only
this little change no tarball was created by upstream. But the used tarball is
generated by upstream's system and consists of the release tag (not a certain
revision or trunk or something).

The license was changed.

Yes, it could be built with maven. Actually, maven needs artifacts that are not
provided in Fedora. And packaging
other java software that could be built using maven also shows that maven is
Fedora lacks lots of important artifacts and that maven is really outdated. And
I really don't understand maven enough to change any of this.

Yes, I could modify the tarball but isn't it part of the guidelines that the
upstream's distribution shouldn't be modified unless there's some problems with
the contents?

Well, xtvd-lib and xtvd-gui are upstream's naming, which should be preserved
according to the guidelines. The common name (and the name of upstream's
distribution) is xtvd. That's why.

build-classpath is now being used.

Correct me if I'm wrong in any point :)

-- 
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 475018] Review Request: xtvd - A client java library for easy access to the tv data from schedulesdirect.org

2008-12-16 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=475018


Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp changed:

   What|Removed |Added

 CC||mtas...@ioa.s.u-tokyo.ac.jp




--- Comment #5 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp  2008-12-16 
13:36:56 EDT ---
Before I can review this package:

- For svn based tarball, please follow
  * For creating source tarball
https://fedoraproject.org/wiki/Packaging/SourceURL#Using_Revision_Control
  * For versioning
   
https://fedoraproject.org/wiki/Packaging/NamingGuidelines#PreReleasePackages
https://fedoraproject.org/wiki/Packaging/NamingGuidelines#SnapshotPackages
For this package the EVR should be
2.0.1-0.2.svn35%{?dist} (as 2.0.1 does not seem to be released yet)

- As far as I checked the codes, the license tag should be GPL+ (no
  version specified)

- Can this package be built by using maven?
  https://fedoraproject.org/wiki/Packaging/Java#maven
  Note that I am not familiar with maven.

- For %setup, would you modify your tarball so that the tarball
  expands codes under the directory of which the name contains %{name}?

- Would you explain why %{name} binary rpm itself is not created?

- Please consider to use build-classpath.

-- 
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 475018] Review Request: xtvd - A client java library for easy access to the tv data from schedulesdirect.org

2008-12-15 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=475018


Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp changed:

   What|Removed |Added

 Blocks|177841  |




--- Comment #3 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp  2008-12-15 
10:01:03 EDT ---
(Removing NEEDSPONSOR)

-- 
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 475018] Review Request: xtvd - A client java library for easy access to the tv data from schedulesdirect.org

2008-12-15 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=475018


Sandro Mathys s...@sandro-mathys.ch changed:

   What|Removed |Added

 CC||s...@sandro-mathys.ch




--- Comment #4 from Sandro Mathys s...@sandro-mathys.ch  2008-12-15 23:30:57 
EDT ---
Spec URL: http://red.fedorapeople.org/SRPMS/xtvd-2.0.1-1.fc10.src.rpm
SRPM URL: http://red.fedorapeople.org/SRPMS/xtvd.spec

New version including much of the experience I got when my jcalendar pkg was
reviewed.

-- 
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 475018] Review Request: xtvd - A client java library for easy access to the tv data from schedulesdirect.org

2008-12-11 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=475018





--- Comment #2 from Sandro Mathys r...@fedoraproject.org  2008-12-11 20:01:39 
EDT ---
Successfully created a mock-(re)build of this version of this pkg.

-- 
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 475018] Review Request: xtvd - A client java library for easy access to the tv data from schedulesdirect.org

2008-12-06 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=475018





--- Comment #1 from Sandro Mathys [EMAIL PROTECTED]  2008-12-06 13:58:46 EDT 
---
Please have a close look at the license text. I didn't really know what to make
of it.

-- 
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 475018] Review Request: xtvd - A client java library for easy access to the tv data from schedulesdirect.org

2008-12-06 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=475018


Sandro Mathys [EMAIL PROTECTED] changed:

   What|Removed |Added

 Blocks||177841, 472144




-- 
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