Bug 1169898 - Review Request: kbrickbuster - Destroy bricks with a ball
Summary: Review Request: kbrickbuster - Destroy bricks with a ball
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Rex Dieter
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: 1167062
TreeView+ depends on / blocked
 
Reported: 2014-12-02 16:23 UTC by Than Ngo
Modified: 2015-02-12 15:11 UTC (History)
5 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2015-02-12 15:11:34 UTC
Type: ---
Embargoed:
rdieter: fedora-review+
gwync: fedora-cvs+


Attachments (Terms of Use)

Description Than Ngo 2014-12-02 16:23:28 UTC
Spec URL: https://than.fedorapeople.org/f21/kbrickbuster.spec
SRPM URL: https://than.fedorapeople.org/f21/kbrickbuster-4.14.3-1.fc20.src.rpm
Description:
The objective KBrickbuster game is to destroy as many bricks as possible
without losing the ball.

Fedora Account System Username: than

Comment 1 Rex Dieter 2014-12-02 16:47:26 UTC
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)

Comment 2 Than Ngo 2014-12-09 14:59:51 UTC
i'm fixing 1 and 2, will upload new srpm for review. Thanks

Comment 3 mejiko 2015-01-12 06:44:53 UTC
Hello.

How it this bug current status ?

Thanks.

Comment 4 Rex Dieter 2015-01-12 16:48:37 UTC
I guess we're still waiting for the promised fixes per comment #2

Comment 5 Than Ngo 2015-01-14 11:45:11 UTC
sadly i have been sick this week. It's my high prio task for next week.

Comment 6 Than Ngo 2015-02-10 18:39:12 UTC
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

Comment 7 Rex Dieter 2015-02-11 15:46:09 UTC
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

Comment 8 Than Ngo 2015-02-11 15:51:50 UTC
the script works fine for new version 14.12.1-1.

Comment 9 Than Ngo 2015-02-11 17:08:56 UTC
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

Comment 10 Gwyn Ciesla 2015-02-11 19:27:20 UTC
Git done (by process-git-requests).

Comment 11 Than Ngo 2015-02-12 15:06:30 UTC
it's imported to fedora git


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