[Bug 842296] Review Request: kde-plasma-akonadi-calendars - Akonadi calendars plasmoid

2012-08-12 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=842296

Fedora Update System upda...@fedoraproject.org changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
 Resolution|--- |ERRATA
Last Closed||2012-08-12 22:24:01

--- Comment #15 from Fedora Update System upda...@fedoraproject.org ---
kde-plasma-akonadi-calendars-0.1.1-1.fc17 has been pushed to the Fedora 17
stable repository.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 842296] Review Request: kde-plasma-akonadi-calendars - Akonadi calendars plasmoid

2012-07-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=842296

--- Comment #11 from Jan Grulich jgrul...@redhat.com ---
New Package SCM Request
===
Package Name: kde-plasma-akonadi-calendars
Short Description: Calendars plasmoid for Akonadi resources
Owners: jgrulich
Branches: f17
InitialCC: jgrulich ltinkl

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 842296] Review Request: kde-plasma-akonadi-calendars - Akonadi calendars plasmoid

2012-07-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=842296

Jan Grulich jgrul...@redhat.com changed:

   What|Removed |Added

  Flags||fedora-cvs?

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 842296] Review Request: kde-plasma-akonadi-calendars - Akonadi calendars plasmoid

2012-07-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=842296

--- Comment #12 from Jon Ciesla limburg...@gmail.com ---
Git done (by process-git-requests).

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 842296] Review Request: kde-plasma-akonadi-calendars - Akonadi calendars plasmoid

2012-07-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=842296

--- Comment #13 from Fedora Update System upda...@fedoraproject.org ---
kde-plasma-akonadi-calendars-0.1.1-1.fc17 has been submitted as an update for
Fedora 17.
https://admin.fedoraproject.org/updates/kde-plasma-akonadi-calendars-0.1.1-1.fc17

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 842296] Review Request: kde-plasma-akonadi-calendars - Akonadi calendars plasmoid

2012-07-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=842296

Fedora Update System upda...@fedoraproject.org changed:

   What|Removed |Added

 Status|ASSIGNED|MODIFIED

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 842296] Review Request: kde-plasma-akonadi-calendars - Akonadi calendars plasmoid

2012-07-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=842296

Fedora Update System upda...@fedoraproject.org changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA

--- Comment #14 from Fedora Update System upda...@fedoraproject.org ---
kde-plasma-akonadi-calendars-0.1.1-1.fc17 has been pushed to the Fedora 17
testing repository.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 842296] Review Request: kde-plasma-akonadi-calendars - Akonadi calendars plasmoid

2012-07-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=842296

Lukáš Tinkl lti...@redhat.com changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
   Assignee|nob...@fedoraproject.org|lti...@redhat.com
  Flags||fedora-review?

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 842296] Review Request: kde-plasma-akonadi-calendars - Akonadi calendars plasmoid

2012-07-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=842296

--- Comment #4 from Lukáš Tinkl lti...@redhat.com ---
[ltinkl@goblin 842296]$ rpmlint
kde-plasma-akonadi-calendars-0.1.1-1.fc17.src.rpm
kde-plasma-akonadi-calendars.src: W: spelling-error Summary(en_US) plasmoid -
plasma
kde-plasma-akonadi-calendars.src: W: spelling-error %description -l en_US
plasmoid - plasma
kde-plasma-akonadi-calendars.src: W: invalid-url Source0:
http://kde-look.org/CONTENT/content-files/150034-10534-akonadi-calendars.tar.bz2
HTTP Error 404: Not Found
1 packages and 0 specfiles checked; 0 errors, 3 warnings.

- please provide a working Source URL

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 842296] Review Request: kde-plasma-akonadi-calendars - Akonadi calendars plasmoid

2012-07-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=842296

--- Comment #5 from Lukáš Tinkl lti...@redhat.com ---
- the correct URL seems to be:
http://kde-look.org/CONTENT/content-files/150034-150034-akonadi-calendars.tar.bz2

- no newline at the end of specfile

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 842296] Review Request: kde-plasma-akonadi-calendars - Akonadi calendars plasmoid

2012-07-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=842296

--- Comment #6 from Jan Grulich jgrul...@redhat.com ---
New:
Spec URL:
http://phoenix.inf.upol.cz/~grulichj/rpmbuild/SPECS/kde-plasma-akonadi-calendars.spec
SRPM URL:
http://phoenix.inf.upol.cz/~grulichj/rpmbuild/SRPMS/kde-plasma-akonadi-calendars-0.1.1-1.fc17.src.rpm

- link and also tarball name fixed
- others are not problems

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 842296] Review Request: kde-plasma-akonadi-calendars - Akonadi calendars plasmoid

2012-07-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=842296

--- Comment #7 from Lukáš Tinkl lti...@redhat.com ---
License tag from .spec file (GPLv2+) doesn't match the actual source code
license (GPLv3+)

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 842296] Review Request: kde-plasma-akonadi-calendars - Akonadi calendars plasmoid

2012-07-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=842296

--- Comment #8 from Jan Grulich jgrul...@redhat.com ---
New:
Spec URL:
http://phoenix.inf.upol.cz/~grulichj/rpmbuild/SPECS/kde-plasma-akonadi-calendars.spec
SRPM URL:
http://phoenix.inf.upol.cz/~grulichj/rpmbuild/SRPMS/kde-plasma-akonadi-calendars-0.1.1-1.fc17.src.rpm

- fix licence tag

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 842296] Review Request: kde-plasma-akonadi-calendars - Akonadi calendars plasmoid

2012-07-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=842296

--- Comment #9 from Jan Grulich jgrul...@redhat.com ---
New:
Spec URL:
http://phoenix.inf.upol.cz/~grulichj/rpmbuild/SPECS/kde-plasma-akonadi-calendars.spec
SRPM URL:
http://phoenix.inf.upol.cz/~grulichj/rpmbuild/SRPMS/kde-plasma-akonadi-calendars-0.1.1-1.fc17.src.rpm

- add missing newline at the end of specfile

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 842296] Review Request: kde-plasma-akonadi-calendars - Akonadi calendars plasmoid

2012-07-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=842296

Lukáš Tinkl lti...@redhat.com changed:

   What|Removed |Added

  Flags|fedora-review?  |fedora-review+

--- Comment #10 from Lukáš Tinkl lti...@redhat.com ---
Package builds ok, meets all the MUST criteria, license correct too.

RPMlint:

[ltinkl@goblin 842296]$ rpmlint
kde-plasma-akonadi-calendars-0.1.1-1.fc17.src.rpm 
kde-plasma-akonadi-calendars.src: W: spelling-error Summary(en_US) plasmoid -
plasma
kde-plasma-akonadi-calendars.src: W: spelling-error %description -l en_US
plasmoid - plasma
1 packages and 0 specfiles checked; 0 errors, 2 warnings.


Builds:
http://koji.fedoraproject.org/koji/taskinfo?taskID=4322576
http://koji.fedoraproject.org/koji/taskinfo?taskID=4322839

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 842296] Review Request: kde-plasma-akonadi-calendars - Akonadi calendars plasmoid

2012-07-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=842296

Jan Grulich jgrul...@redhat.com changed:

   What|Removed |Added

 Blocks||177841 (FE-NEEDSPONSOR)

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 842296] Review Request: kde-plasma-akonadi-calendars - Akonadi calendars plasmoid

2012-07-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=842296

Lukáš Tinkl lti...@redhat.com changed:

   What|Removed |Added

 CC||lti...@redhat.com

--- Comment #1 from Lukáš Tinkl lti...@redhat.com ---
- BuildRoot and Group tags are redundant
- Source should point to the canonical source on the internet
- the preferred tarball format is tar.bz2 or even tar.xz

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 842296] Review Request: kde-plasma-akonadi-calendars - Akonadi calendars plasmoid

2012-07-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=842296

--- Comment #2 from Jan Grulich jgrul...@redhat.com ---
New

Spec URL:
http://phoenix.inf.upol.cz/~grulichj/rpmbuild/SPECS/kde-plasma-akonadi-calendars.spec
SRPM URL:
http://phoenix.inf.upol.cz/~grulichj/rpmbuild/SRPMS/kde-plasma-akonadi-calendars-0.1.1-1.fc17.src.rpm

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 842296] Review Request: kde-plasma-akonadi-calendars - Akonadi calendars plasmoid

2012-07-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=842296

Rex Dieter rdie...@math.unl.edu changed:

   What|Removed |Added

 CC||rdie...@math.unl.edu
 Blocks|177841 (FE-NEEDSPONSOR) |656997 (kde-reviews)

--- Comment #3 from Rex Dieter rdie...@math.unl.edu ---
I've sponsored Jan into packager group, largely due to other mentoring tasks by
Lukáš and other fellow kde-sig'ers.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review