Package Details: python2-rarfile 2.7-1

Package Base: python2-rarfile
Description: Rar archive reader for Python
Upstream URL: https://pypi.python.org/pypi/rarfile
Category: lib
Licenses: custom:ISC
Provides: python-rarfile
Submitter: vadmium
Maintainer: vadmium
Last Packager: vadmium
Votes: 3
First Submitted: 2012-02-15 13:15
Last Updated: 2015-01-11 22:15

Dependencies (7)

Required by (2)

Sources

Latest Comments

Comment by crass00

2014-01-16 11:41

Even though I was clicking the PKGBUILD file I was getting an old version for some reason. If you like to install everything and anything them it's not a problem, but I prefer to not install unnecessary packages, especially when it can easily be worked around.

I don't think we will see each others point of view here so I'm just gong to say, the package would be perfect with 1 change now. A python2-docutil package does exist so makedepends should have that package in it, not python-docutils. I've no problem doing this when I need to compile, i just hope you can see why it's better to get it done on the upstream so to speak for everyones benifit.

Comment by vadmium

2014-01-15 22:15

If you really want to avoid Python 3 I suggest skipping the “make html” step, but I don’t understand why it is such a problem. Also, please check you have the 2.6-3 version of my PKGBUILD; that “package_python3-rarfile” stuff was only in the 2.6-1 version.

Comment by crass00

2014-01-15 12:53

I seem to only be giving bad news and I'm sorry about that. Still forces python3 to install before it will build. I think the makedepends line should just be simplified to makedepends=(python2). Also this line looks odd " PYTHON=python2 package_python3-rarfile "$@"" as it mentioned python3. thanks

Comment by vadmium

2014-01-13 13:29

“python-docutils” is for the “rst2html” command. No particular Python version is needed for that, and Arch happens to use Python 3.

“python-sphinx” did seem to be using Python 3 to parse the source files. But they are meant to be compatible with 2 and 3, so either version of Sphinx should work. Anyway I did change it to “python2-sphinx” for consistency.

The AUR dependency is fixed now after uploading a full “.AURINFO” file.

The “provides” bit is not required. I added it to help searching for the package. If there are conflicts you use the “conficts” variable instead.

Lastly, I think I agree now about the “python2” dependency. If there ever was a Python 2.8, it would help detect this package needs rebuilding.

Comment by crass00

2014-01-12 13:59

Thanks, I haven't had a chance to test but I can see a problem though (will test tomorrow when on dev machine). now that your building the docs your still forcing python(3) to be installed due to the python-docutils python-sphinx makedepends (they both rely on python) for this package they would need the python2- version of each package in the make depends. also in the AUR the run dependency comes up as python not python2. I can see who your setting it to python2, i just don't know if it is just the aur not picking it properly. I'm also not sure it provides is required in this package, i think you only use that if you have 2 packages that do the same thing that would conflict if installed at the same time, not 100% on reasoning for that one

For my original request about adding the dependency, thats the standard for doing this, python2-rarfile needs python2 to work. it's also used for dependancy walking with pacman. Take a look at any python2 module like python2-pillow to see how the arch devs do it https://www.archlinux.org/packages/community/i686/python2-pillow/

Cheers

Comment by vadmium

2014-01-12 02:50

Updated package; now builds documentation as well

Comment by vadmium

2014-01-11 12:27

I’ll look at removing that Python 3 dependency and update the package tomorrow. I’ll also add Python 2 to “depends” for you, though I’m curious why it would make a difference, since this is only a library.

Comment by crass00

2014-01-10 08:52

This doesn't build without python3 installed which isn't ideal for a python2 module. Can this please be changed to require python2 for the makedepends and also there should be a depends=('python2') in the PKGBUILD file as well. tks