[Bug 229476] Review Request: xblast - Lay bombs and Blast the other players of the field (SDL version)

bugzilla at redhat.com bugzilla at redhat.com
Sat Feb 24 17:11:51 UTC 2007

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: xblast - Lay bombs and Blast the other players of the field (SDL version)


------- Additional Comments From mtasaka at ioa.s.u-tokyo.ac.jp  2007-02-24 12:11 EST -------

A. First for xblast-2.10.4-1:

* File dependency
  - Writing the package which provides the file is recommended
    expect you have somewhat strong reason to write file dependency
    (for vera font). Please check:
    (This is a draft)

* Source URL
  - Please use http://downloads.sourceforge.net/<package_name>/XXXX.tar.gz
    if it is possible. Please check:
    (This is a draft).
  - Please specify the URL of xblast.png if possible.

* Timestamps
install -m 755 %{SOURCE3} $RPM_BUILD_ROOT%{_bindir}/%{name}
  - This is only a wrapper script and keeping timestamp
   (i.e. install -p) is recommended.

* Documentation
  - Perhaps the following files can be used.

* Functionality
  - xblast-x11 cannot be launched for me.
[tasaka1 at localhost xblast]$ xblast-x11 
could not load font 24
could not load font 18
could not load font 14
X Error of failed request:  BadFont (invalid Font parameter)
  Major opcode of failed request:  56 (X_ChangeGC)
  Resource id in failed request:  0x800010
  Serial number of failed request:  519
  Current serial number in output stream:  541

* Directory/file ownership
  - Well as the build log says:
    I think that %{_datadir}/xblast should be owned by
    xblast-common, not by xblast-data because xblast requires
    that the files are installed under %{_datadir}/xblast.

  - And currently the location of gettext mo files are
    not correct because build log says:
    This should be moved to %{_datadir}/locale (well, some
    messages are corrupted on both fr_FR and de_DE, perhaps
    due to ISO-8859 style vs UTF-8 style).

Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug, or are watching the QA contact.

More information about the package-review mailing list