Bug 226998 - Review Request: gemdropx - Falling blocks puzzlegame
Review Request: gemdropx - Falling blocks puzzlegame
Status: CLOSED NEXTRELEASE
Product: Fedora
Classification: Fedora
Component: Package Review (Show other bugs)
rawhide
All Linux
medium Severity medium
: ---
: ---
Assigned To: Mamoru TASAKA
Fedora Package Reviews List
:
Depends On:
Blocks: FE-ACCEPT
  Show dependency treegraph
 
Reported: 2007-02-02 10:47 EST by Hans de Goede
Modified: 2007-11-30 17:11 EST (History)
0 users

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Environment:
Last Closed: 2007-02-14 03:37:13 EST
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
mtasaka: fedora‑review+
notting: fedora‑cvs+


Attachments (Terms of Use)

  None (edit)
Description Hans de Goede 2007-02-02 10:47:27 EST
Spec URL: http://people.atrpms.net/~hdegoede/gemdropx.spec
SRPM URL: http://people.atrpms.net/~hdegoede/gemdropx-0.9-1.fc7.src.rpm
Description:
Gem Drop X is a fast-paced puzzle game where it is your job to clear
the screen of gems before they squash you.
Comment 1 Mamoru TASAKA 2007-02-07 12:45:33 EST
Well,

* BuildRequires
  - Mockbuild fails on FC7 i386 (needs desktop-file-utils)

* Documentation
  - Please add other "README" files 

* Timestamps
  - keep timestamps on image files and etc, i.e. use "cp -p"
    or "install -p"

* Desktop file entry
-----------------------------------------------------
  --add-category X-Fedora                       \
-----------------------------------------------------
  is deprecated and should be removed.

* Scriptlets
-----------------------------------------------------
if [ -x %{_bindir}/gtk-update-icon-cache ]; then
-----------------------------------------------------
  is redundant because execution of non-existing file
  simply exits with 127 and is ignored by || :
Comment 2 Hans de Goede 2007-02-09 04:04:06 EST
(In reply to comment #1)
> Well,
> 
> * BuildRequires
>   - Mockbuild fails on FC7 i386 (needs desktop-file-utils)
> 
Oops, can't believe I forgot that, fixed.

> * Documentation
>   - Please add other "README" files 
> 
I assume you mean INSTALL.txt ICON.txt and TODO.txt?
INSTALL.txt and ICON.txt both contain installation instructions and thus should
not be packaged. I've added TODO.txt

> * Timestamps
>   - keep timestamps on image files and etc, i.e. use "cp -p"
>     or "install -p"
> 

Fixed

> * Desktop file entry
> -----------------------------------------------------
>   --add-category X-Fedora                       \
> -----------------------------------------------------
>   is deprecated and should be removed.
> 
> * Scriptlets
> -----------------------------------------------------
> if [ -x %{_bindir}/gtk-update-icon-cache ]; then
> -----------------------------------------------------
>   is redundant because execution of non-existing file
>   simply exits with 127 and is ignored by || :

I've been doing things like this for over a year now, I know there was some
discussion about both of these but I never followed it close enough to learn the
new guidelines, to busy with other Fedora stuff. I've read the new guidelines
now, thanks for straightening me out on this -> Both fixed.

Here is a new version with all this fixed:
Spec URL: http://people.atrpms.net/~hdegoede/gemdropx.spec
SRPM URL: http://people.atrpms.net/~hdegoede/gemdropx-0.9-2.fc7.src.rpm
Comment 3 Mamoru TASAKA 2007-02-09 04:45:38 EST
(In reply to comment #2)
> > * Documentation
> >   - Please add other "README" files 
> > 
> I assume you mean INSTALL.txt ICON.txt and TODO.txt?
> INSTALL.txt and ICON.txt both contain installation instructions and thus should
> not be packaged. I've added TODO.txt

Ah.. I meant:
----------------------------------------
./data/images/README
./data/sounds/README
./README.txt
./action/README.txt
----------------------------------------
These files explains where the image/sounds files came
from and I think adding these files is kinder.
Comment 4 Hans de Goede 2007-02-09 05:49:35 EST
(In reply to comment #3)
> (In reply to comment #2)
> > > * Documentation
> > >   - Please add other "README" files 
> > > 
> > I assume you mean INSTALL.txt ICON.txt and TODO.txt?
> > INSTALL.txt and ICON.txt both contain installation instructions and thus should
> > not be packaged. I've added TODO.txt
> 
> Ah.. I meant:
> ----------------------------------------
> ./data/images/README
> ./data/sounds/README
> ./README.txt
> ./action/README.txt
> ----------------------------------------
> These files explains where the image/sounds files came
> from and I think adding these files is kinder.

I see, Good idea, new version with these files added is here:
Spec URL: http://people.atrpms.net/~hdegoede/gemdropx.spec
SRPM URL: http://people.atrpms.net/~hdegoede/gemdropx-0.9-2.fc7.src.rpm
Comment 5 Mamoru TASAKA 2007-02-09 06:13:40 EST
Well, I will check 0.9-2 later.

By the way, would you tell me in advance to which repository
(FE-7/6/5) you want to import this package? (I currently
don't know how the new review process will become...
and.. do I have to ask the submitter in advance??)
Comment 6 Hans de Goede 2007-02-09 06:23:13 EST
Its intended for FE-6 and devel.

Comment 7 Mamoru TASAKA 2007-02-09 09:07:44 EST
Okay.
----------------------------------------------------
  This package is APPROVED by me with the following summary:

* Owner: j.w.r.degoede@hhs.nl
* Name:  gemdropx
* Summary: Falling blocks puzzlegame
* Branch: FC-devel FC-6
-----------------------------------------------------
  Requesting cvs admin for making initial directories.
  Please wait until cvs admin answers on this bug.
Comment 8 Hans de Goede 2007-02-14 03:37:13 EST
Imported and build for FC-6 and devel, closing.

p.s.

Thanks for the review!
Comment 9 Mamoru TASAKA 2007-02-14 04:00:50 EST
You are welcome!

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