Note: This is a beta release of Red Hat Bugzilla 5.0. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Also email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback here.
Bug 578269 - Review Request: xgospel - An X11 client for Internet Go Server
Summary: Review Request: xgospel - An X11 client for Internet Go Server
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
Target Milestone: ---
Assignee: Mamoru TASAKA
QA Contact: Fedora Extras Quality Assurance
Depends On:
TreeView+ depends on / blocked
Reported: 2010-03-30 18:13 UTC by jjwei
Modified: 2010-07-25 05:08 UTC (History)
6 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Last Closed: 2010-07-25 05:08:05 UTC

Attachments (Terms of Use)

Description jjwei 2010-03-30 18:13:47 UTC
Spec URL:
Description: Xgospel is an X11 client for Internet Go Server, it provides a graphical interface with a lot of features to play go(weiqi) on the Internet using IGS

Comment 1 jjwei 2010-03-30 19:04:58 UTC
This is my first package, and I am seeking a sponsor.

Comment 2 Germán Racca 2010-03-31 01:14:42 UTC
Hello jjwei:

Just a few comments on your package. I'm looking for sponsorship also.

It's useful to put the output of rpmlint here. I'm doing it for you:

[german@skytux Downloads]$ rpmlint ../rpmbuild/RPMS/i686/xgospel-1.12d-1.fc12.i686.rpm 
xgospel.i686: W: spelling-error %description -l en_US weiqi -> weird, Weiss, Weill
xgospel.i686: W: non-standard-group Games
xgospel.i686: W: incoherent-version-in-changelog ['1.12d-1.fc12', '1.12d-1']
xgospel.i686: W: invalid-license GPL
xgospel.i686: W: unstripped-binary-or-object /usr/bin/rport
xgospel.i686: W: unstripped-binary-or-object /usr/bin/xgospel
1 packages and 0 specfiles checked; 0 errors, 6 warnings.

OK, the comments now:

** non-standard-group: the correct group should be "Amusements/Games" in this case

** incoherent-version-in-changelog: the version should be "1.12d-1", and I also think that one changelog entry with the list of all changes is enough in this case, because all of them belong to version 1.12d-1

** invalid-license GPL: please read carefully the file "my/COPYRIGHTS" to choose the proper value for this tag

Also, you don't need to put "gcc" in BuildRequires. See here:

Good luck!

Comment 3 Howard Ning 2010-04-11 18:10:12 UTC
This package is interesting. But I haven't found any proper license in the source file. You have said that it is GPL but which version?
Also, the code is quite outdated. Is the upstream still developing it?
For your spec files, it is better for you to specify BuildRequires one each line and exclude gcc from it.

Comment 4 jjwei 2010-04-11 20:22:09 UTC
Spec URL:

do not have a ftp site, I unlinked the files on the description and here is the new links. with fix for most of output of rpmlint. thanks for the comments.

There is one paragraph on README file says it is copyleft. not sure what I should put on the spec file.
The development has stopped for a long time. but the software is good. quick response and beautiful interface.


Comment 5 Howard Ning 2010-04-11 22:50:04 UTC
I have tried your package in koji against f12 but it fails to build. Have a look at the build log:

Comment 6 jjwei 2010-04-15 14:53:43 UTC
I cannot see the log.

Got following error when I try to access   

An error occurred during a connection to

SSL peer was unable to negotiate an acceptable set of security parameters.

(Error code: ssl_error_handshake_failure_alert)

not sure if I need to install koji in my system? but I think it is just a https. 

When I do rpmbuild -ba xgospel.spec on my f12 system, sometimes I will have error, but run the same command again may get through, that is how src rpm get produced.

Comment 7 Howard Ning 2010-04-15 17:12:29 UTC

rpmbuild -bs xgospel.spec will get the srpm files.

Comment 8 Howard Ning 2010-04-15 19:26:12 UTC
Try to add 
BuildRequires:  desktop-file-utils
and change "make %{?_smp_mflags}" to "make"

It should be built by koji.

Comment 9 jjwei 2010-04-18 13:42:06 UTC

Spec URL:
(In reply to comment #8)
> Try to add 
> BuildRequires:  desktop-file-utils
> and change "make %{?_smp_mflags}" to "make"
> It should be built by koji.

Comment 11 Mamoru TASAKA 2010-05-02 17:01:02 UTC
To spot:

README says:
   Gnu copyleft. You can essentially do anything you want with the program, as
long as you  make sure  the source is  available.   Take a  look  in the  file
my/COPYRIGHTS  for  info about the   parts that are not  written  by me and of
course retain their original copyrights.

Can I assume that "Gnu copyleft" means GPL+?

Comment 12 Mamoru TASAKA 2010-05-02 17:04:08 UTC
To jjwei:
By the way, is this 10-years-old package still be useful?

Comment 13 jjwei 2010-05-09 13:51:05 UTC
(In reply to comment #12)
> To jjwei:
> By the way, is this 10-years-old package still be useful?    

The server and client are quite stable for a long time, so not too much development activity, but the software is still useful, I use it everyday -).

I will change the license to GPL+ next time.


Comment 14 Mamoru TASAKA 2010-05-19 18:31:29 UTC
Well, for now assuming "GNU copyleft = GPL+",

* License
  - For this package the license tag should be "GPLv2+".

* %{version} in SourceURL
  - It is recommended that you use %{version} tag in SourceURL:

? About Patch0
  - By the way this 20K xgospel-1.12d-friend.patch written by yourself
    or you've taken from somewhere? Also it is recommended to write
    some notes about what is this patch for:

* BuildRoot
  - BuildRoot is no longer needed on Fedora (although rpmlint may complain
    if you remove this)
    ! Note: If you want to import this package also into EPEL, BuildRoot is
      still needed.

* Configure option
  - build.log shows:
    99  checking for -lICE... no
    However as you have "BR: libICE-devel", this shows something
    is wrong (actually configure is broken at least for this part)

* Parallel make
  - Support parallel make if possible. If parallel make fails, please
    write a note in the spec file about it:

* compiler warning:
  - Well, usually I don't warn about compiler warning during
    review process, however
   441  In file included from /usr/include/string.h:642,
   442                   from /usr/include/X11/Intrinsic.h:64,
   443                   from games.c:14:
   444  In function 'memset',
   445      inlined from 'InitHash' at games.c:185:
   446  /usr/include/bits/string3.h:83: warning: call to '__warn_memset_zero_len' declared with attribute warning: memset used with constant zero length parameter; this could be due to transposed parameters
   447  In function 'memset',
   448      inlined from 'InitHash' at games.c:186:
   449  /usr/include/bits/string3.h:83: warning: call to '__warn_memset_zero_len' declared with attribute warning: memset used with constant zero length parameter; this could be due to transposed parameters
    - These are arguably bugs, please fix these.

* Icon entry in desktop file
  - It is recommended that extension (.xpm) is removed:

* Categories for Desktop files
  - Catogories "X-Fedora", "Application" are no longer used and should
    be removed.

* Document files
  - Usually "INSTALL" file is for people trying to build a software
    by themselves and not needed for people trying to install one
    using rpm.

By the way it is recommended that you upload your srpm to some place
from which we can directly download it by "wget -N", for example.

Comment 15 Mamoru TASAKA 2010-05-30 05:36:10 UTC

Comment 16 jjwei 2010-05-31 17:44:41 UTC
the link is stable now:
Spec URL:

License changed to GPLv2+

Changed to %{version} in SourceURL

patch is written by me, no upstream, explain is at the end of spec file,

libICE-devel is not required, removed(only libICE is required, and libXaw and libXpm depend on it)

make with smp in spec file will cause problem, I did see final xgospel is depends on libmy.a and libregex.a in Makefile, but seems rpmbuild will start to build final xgospel before all of the lib finish built. so no smp flag on make on this built. someone give me a hint what the Makefile should be to resolve this problem?

remove .xpm on desktop file,

remove Application on Categories,

will not put INSTALL file on docs directory,

Will try to fix compilation warning a little bit later.

Comment 17 Mamoru TASAKA 2010-05-31 18:01:46 UTC
Please change the release number every time you modify your
spec file to avoid confusion, even during review process.

Comment 18 Tom "spot" Callaway 2010-06-02 15:43:41 UTC
According to reliable sources, "GNU copyleft" was in ancient times a common way of referring to the GPL (or its predecessors).

So, I'm going to say that it is safe to assume that text means "GPL+". Lifting FE-Legal.

Comment 19 Mamoru TASAKA 2010-06-03 19:37:22 UTC
Thank you for clarifying, spot.

Comment 20 jjwei 2010-06-04 16:52:23 UTC
The license for this package should be GPL+ then?

Comment 21 Mamoru TASAKA 2010-06-04 17:08:05 UTC
No, should be GPLv2+, because some codes are under GPLv2+.
Would you change release number and upload new srpm?

Comment 22 Mamoru TASAKA 2010-06-06 16:56:17 UTC
jjwei, please bump release number.

Comment 23 jjwei 2010-06-06 17:32:53 UTC
Here is the new version:

Spec URL:

Comment 24 Mamoru TASAKA 2010-06-07 17:47:56 UTC

* BuildRoot / %clean
  - As I said above, unless you want to this package into EPEL
    BuildRoot: line is no longer needed so please remove this
    (although rpmlint may complain about it).

* Comments for Patch0
  - So, please write some comments about what Patch0 is for
    on the above of Patch0: line like:
# Fix issues when XXXXXXXXXX, fix compilation warning, etc
Patch0: xgospel-1.12d-friend.patch

* parallel make
  - As I said above, please write some comments if parallel
    make fails in the spec file.
    ( Note that I have not examined how to make parallel make
      succeed for this package for now. )

* Categories for Desktop files
  - Please also remove "X-Fedora" category from desktop file.

Comment 25 Mamoru TASAKA 2010-06-21 14:29:13 UTC

Comment 26 Mamoru TASAKA 2010-07-04 14:42:41 UTC
ping again?

Comment 27 Mamoru TASAKA 2010-07-12 15:18:09 UTC
Again ping?

Comment 28 Mamoru TASAKA 2010-07-22 16:51:56 UTC
I will close this bug as NOTABUG if no response is received from the
reported within ONE WEEK.

Comment 29 jjwei 2010-07-24 21:10:04 UTC
I was thinking put up the fixes so the program will run smoothly on fedora, really do not have that much time to do all the rest. close it please.

Comment 30 Mamoru TASAKA 2010-07-25 05:08:05 UTC
Thank you for reply. Once closing.

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