[Bug 1169898] Review Request: kbrickbuster - Destroy bricks with a ball

2015-02-12 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1169898

Ngo Than t...@redhat.com changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution|--- |RAWHIDE
Last Closed||2015-02-12 10:11:34



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1169898] Review Request: kbrickbuster - Destroy bricks with a ball

2015-02-12 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1169898



--- Comment #11 from Ngo Than t...@redhat.com ---
it's imported to fedora git

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1169898] Review Request: kbrickbuster - Destroy bricks with a ball

2015-02-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1169898

Ngo Than t...@redhat.com changed:

   What|Removed |Added

  Flags||fedora-cvs?



--- Comment #9 from Ngo Than t...@redhat.com ---
New Package SCM Request
===
Package Name: kbrickbuster
Short Description: Destroy bricks with a ball
Upstream URL: https://projects.kde.org/projects/kde/kdegames/kbrickbuster
Owners: than rdieter jgrulich dvratil kkofler ltinkl group::kde-sig
Branches: f20 f21 f22
InitialCC: group::kde-sig

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1169898] Review Request: kbrickbuster - Destroy bricks with a ball

2015-02-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1169898



--- Comment #10 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.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1169898] Review Request: kbrickbuster - Destroy bricks with a ball

2015-02-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1169898

Jon Ciesla limburg...@gmail.com changed:

   What|Removed |Added

  Flags|fedora-cvs? |fedora-cvs+



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1169898] Review Request: kbrickbuster - Destroy bricks with a ball

2015-02-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1169898

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

   What|Removed |Added

  Flags|fedora-review?  |fedora-review+



--- Comment #7 from Rex Dieter rdie...@math.unl.edu ---
FYI, rawhide (and f21/f20) are now up to,
kbreakout-14.12.1-1

but I would assume that adapting this for it would be easy (just rebase to
higher Version and respin the package).


Otherwise, package looks good, thanks: APPROVED

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1169898] Review Request: kbrickbuster - Destroy bricks with a ball

2015-02-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1169898



--- Comment #8 from Ngo Than t...@redhat.com ---
the script works fine for new version 14.12.1-1.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1169898] Review Request: kbrickbuster - Destroy bricks with a ball

2015-02-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1169898



--- Comment #6 from Ngo Than t...@redhat.com ---
i uploaded new spec and srpm, it fixes 1 and 2.

Spec URL: https://than.fedorapeople.org/f21/kbrickbuster.spec
SRPM URL: https://than.fedorapeople.org/f21/kbrickbuster-4.14.3-1.fc21.src.rpm

Rex, could you please review it?

Thanks

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1169898] Review Request: kbrickbuster - Destroy bricks with a ball

2015-01-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1169898

Ngo Than t...@redhat.com changed:

   What|Removed |Added

  Flags|needinfo?(t...@redhat.com)  |



--- Comment #5 from Ngo Than t...@redhat.com ---
sadly i have been sick this week. It's my high prio task for next week.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1169898] Review Request: kbrickbuster - Destroy bricks with a ball

2015-01-12 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1169898

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

   What|Removed |Added

 CC||t...@redhat.com
  Flags||needinfo?(t...@redhat.com)



--- Comment #4 from Rex Dieter rdie...@math.unl.edu ---
I guess we're still waiting for the promised fixes per comment #2

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1169898] Review Request: kbrickbuster - Destroy bricks with a ball

2015-01-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1169898

mejiko kame55-itasenpara...@y2.dion.ne.jp changed:

   What|Removed |Added

 CC||kame55-itasenpara...@y2.dio
   ||n.ne.jp



--- Comment #3 from mejiko kame55-itasenpara...@y2.dion.ne.jp ---
Hello.

How it this bug current status ?

Thanks.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1169898] Review Request: kbrickbuster - Destroy bricks with a ball

2014-12-09 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1169898



--- Comment #2 from Ngo Than t...@redhat.com ---
i'm fixing 1 and 2, will upload new srpm for review. Thanks

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1169898] Review Request: kbrickbuster - Destroy bricks with a ball

2014-12-02 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1169898

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

   What|Removed |Added

 Blocks||1167062




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1167062
[Bug 1167062] non-free Trademark problem
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1169898] Review Request: kbrickbuster - Destroy bricks with a ball

2014-12-02 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1169898

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

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||rdie...@math.unl.edu
   Assignee|nob...@fedoraproject.org|rdie...@math.unl.edu
  Flags||fedora-review?



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1169898] Review Request: kbrickbuster - Destroy bricks with a ball

2014-12-02 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1169898



--- Comment #1 from Rex Dieter rdie...@math.unl.edu ---
Looks good, but I'll echo what I mentioned in bug #1167062 

1.  the Obsoletes/Provides MUST to be versioned, options include:
1a.  start kbrickbuster at some higher Release, like:
Release: 10
combined with:
Obsoletes: kbreakout  4.14.3-10
Provides:  kbreakout = %{version}-%{release}

1b. alternatively, to ensure it kbrickbuster always wins, could even introduce
an Epoch in the Obsoletes/Provides,

Obsoletes: kbreakout  1:4.14.3-1
Provides:  kbreakout = 1:%{version}-%{release}


2.  Sources: Not ok, cannot verify.  .spec currently includes:
#Source0:
http://download.kde.org/%{stable}/%{version}/src/%{name}-%{version}.tar.xz
# patched source, rename the game
Source0: kbrickbuster-4.14.3-rename.tar.xz 

SHOULD include a script to automate this renaming process.
(doing this manually for every release would be tedious)

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review