Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 646516 - games-board/pysolfc-2.1.2 version bump
Summary: games-board/pysolfc-2.1.2 version bump
Status: RESOLVED OBSOLETE
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Gentoo Games
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2018-02-03 21:30 UTC by neil
Modified: 2018-09-19 17:26 UTC (History)
2 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
pysolfc-2.1.2.ebuild (bad) (pysolfc-2.1.2.ebuild,1.60 KB, text/plain)
2018-02-03 21:30 UTC, neil
Details
PysolFC-2.2.0 ebuild errors (pysolfc220errors.tar.bz,34.82 KB, application/x-bzip)
2018-05-20 17:28 UTC, Radu
Details

Note You need to log in before you can comment on or make changes to this bug.
Description neil 2018-02-03 21:30:16 UTC
Created attachment 517690 [details]
pysolfc-2.1.2.ebuild (bad)

PySolFC has been updated upstream, as mentioned here: 

http://pysolfc.sourceforge.net/

Apparently active development has restarted on github, but distfiles for the new versions are still being provided at sourceforge.

I created a version bump ebuild that I was able to install locally, which I have attached. However, it is very hackish, and is almost certainly not suitable for inclusion in the official portage tree as-is.

Its faults include:

-For the extra cardsets, I had to replace the version-numbering magic with a hard "2.0" since the extra cardsets haven't updated in sync with the app itself

-I had to remove a couple lines of documentation-related functions from the ebuild before it would build and install.

-Upstream is supposed to be compatible with Python 3 now. I added support to the ebuild, and it installed but would not run, complaining about a lack of something called "sgmllib". I gave up on python 3 and reset it as a 2.7-only ebuild.

-As a 2.7 version, it installed, but still failed to run complaining about about "ImportError: No module named random2". I changed a few instances of "random2" to "random" in the already-installed /usr/lib64/python2.7/site-packages/pysollib/pysolrandom.py, and then it *finally* worked. I could probably make a diff patch that allow emerge to make this change during installation, but that would still just be really hackish, and I suspect it might conflict with any attempts to get it actually working on python3.
Comment 1 neil 2018-02-03 21:36:20 UTC
Since I don't really know what I'm doing, it would probably make more sense for someone who knows what they're doing to version-bump this themselves, completely ignoring my attachment. But, I included it for the sake of completeness.
Comment 2 Jonas Stein gentoo-dev 2018-02-04 12:18:36 UTC
Thank you for your contribution. I had a short look on the ebuild. 
Here a few ideas:

* Please test the ebuild with repoman full -x https://wiki.gentoo.org/wiki/Repoman

* please fix the header. (see /usr/portage/skel.ebuild and https://devmanual.gentoo.org/ebuild-writing/eapi/)
# Copyright 1999-2016 Gentoo Foundation
# Copyright 1999-2018 Gentoo Foundation

* Do you need eutils?

* doesn't the package need one of the helpers to update the menu and icons?
https://wiki.gentoo.org/wiki/Notes_on_ebuilds_with_GUI

* I would avoid the definition of a variable SOL_URI="mirror://sourceforge/${PN}"
but write it in the SRC_URI line.
It makes the ebuild more difficult to read brings no improvements.
Comment 3 neil 2018-02-04 19:16:13 UTC
I'm already aware that the ebuild is severely defective, as it was a hackish edit I did of pysolfc-2.0-r5.ebuild without understanding much about ebuilds. I'm not even sure why I contributed it; I guess it just seemed better to attach *something* rather than just complaining that the thing needed updating.

Although, that weird SOL_URI variable was present in the original. Not terribly comforting to know that this ebuild was already non-standard even *before* I started messing with it...

If I use `repoman full -x`, will that give more info than `repoman --digest=y -d full`? Because that's what I used, and it actually didn't complain at all despite how faulty this ebuild is.
Comment 4 neil 2018-02-04 22:01:59 UTC
Okay, I ran `repoman full -x`, and this happened:

---------
RepoMan scours the neighborhood...

Note: use --include-dev (-d) to check dependencies for 'dev' profiles

RepoMan sez: "If everyone were like you, I'd be out of business!"
---------

So, as far as the automated checker is concerned, there isn't apparently anything wrong. Of course that's not saying it doesn't need repair, only that the problems are all outside the scope of Repoman's tests.
Comment 5 Radu 2018-05-20 17:27:19 UTC
I wish to add my own failure at compiling PysolFC-2.2.0 (which is my favorite game by far)
It fails at compile phase with:

running build_scripts
creating /var/tmp/portage/games-board/pysolfc-2.2.0/work/PySolFC-pysolfc-2.2.0-python3_6/scripts
copying and adjusting pysol.py -> /var/tmp/portage/games-board/pysolfc-2.2.0/work/PySolFC-pysolfc-2.2.0-python3_6/scripts
changing mode of /var/tmp/portage/games-board/pysolfc-2.2.0/work/PySolFC-pysolfc-2.2.0-python3_6/scripts/pysol.py from 644 to 755
 * python3_6: running distutils-r1_run_phase python_compile_all
Traceback (most recent call last):
  File "gen-html.py", line 6, in <module>
    from pysollib.mygettext import fix_gettext
ModuleNotFoundError: No module named 'pysollib'

And I attached in pysolfc220errors.tar.bz:

emerge --info '=games-board/pysolfc-2.2.0::gentoo > emerge__info
emerge -pqv '=games-board/pysolfc-2.2.0::gentoo > emerge__pqv
/var/tmp/portage/games-board/pysolfc-2.2.0/temp/build.log
/var/tmp/portage/games-board/pysolfc-2.2.0/temp/environment

as advised by the ebuild error. How else may I help?
Comment 6 Radu 2018-05-20 17:28:04 UTC
Created attachment 532342 [details]
PysolFC-2.2.0 ebuild errors
Comment 7 Bernard Cafarelli gentoo-dev 2018-09-19 17:26:49 UTC
Thanks for the work here, closing this bug as 2.2.0 was added in tree in May