Bug 365181 – gnome-extra/nautilus-dropbox-0.6.7 does not build against >=dev-python/docutils-0.8

• Status: RESOLVED FIXED
• Product: Gentoo Linux

[Description Nils Andresen 2011-04-28 11:01:26 UTC]

The Error is
> python rst2man.py dropbox.txt > dropbox.1
> TypeError: get_language() takes exactly 2 arguments (1 given)
> Exiting due to error.  Use "--traceback" to diagnose.
> Please report errors to <[e-mail]>.
> Include "--traceback" output, Docutils version (0.8 [repository]),
> Python version (2.7.1), your OS type & version, and the
> command line used.

Reproducible: Always




IMO the rst2man.py provided by nautilus-dropbox does not work with (currently unstable) >=dev-python/docutils-0.8 (compiles fine with dev-python/docutils-0.7)
I _think_ the ebuild should exchange the provided rst2man.py with the one which is provided by docutils...

[Comment 1 Justin Lecher (RETIRED) 2011-04-28 11:08:18 UTC]

Please attach full build log

[Comment 2 Nils Andresen 2011-04-28 12:02:40 UTC]

Created attachment 271441 [details]
full log

added full log

[Comment 3 Justin Lecher (RETIRED) 2011-04-28 12:33:18 UTC]

python serializeimages.py 0.6.7 < dropbox.in > dropbox
No protocol specified
/usr/lib64/python2.7/site-packages/gtk-2.0/gtk/__init__.py:57: GtkWarning: could not open display
  warnings.warn(str(e), _gtk.Warning)
chmod +x dropbox
python docgen.py 0.6.7 < dropbox.txt.in > dropbox.txt
python rst2man.py dropbox.txt > dropbox.1
TypeError: get_language() takes exactly 2 arguments (1 given)
Exiting due to error.  Use "--traceback" to diagnose.
Please report errors to <[e-mail]>.
Include "--traceback" output, Docutils version (0.8 [repository]),
Python version (2.7.1), your OS type & version, and the
command line used.
make[2]: *** [dropbox.1] Error 1

[Comment 4 Nils Andresen 2011-04-28 15:30:38 UTC]

adding this (not-so-beauty) src_unpack to the ebuild fixes the problem:
src_unpack () {
    unpack ${A}
    einfo "replacing rst2man with the one provided by docutils..."
    #TODO: get the correct ABI-Versioned rst2man instead of all incl. wrapper
    cp /usr/bin/rst2man.py* "${S}"/ || die
}

[Comment 5 Nils Andresen 2011-04-28 18:04:03 UTC]

Created attachment 271475 [details, diff]
Patch to fix the problem

I attached a patch to the ebuild to fix the problem.
I am unsure as to the quality of it - I'd appreciate every tip.

[Comment 6 Arfrever Frehtes Taifersar Arahesis (RETIRED) 2011-05-06 22:01:11 UTC]

Created attachment 272319 [details, diff]
Use system rst2man.py

It's better to directly use system rst2man.py. Please test this patch.

[Comment 7 Nils Andresen 2011-05-06 22:32:17 UTC]

(In reply to comment #6)
> Please test this patch.
I was unable to apply the patch - are you sure you diff'ed against the original?
Anyway - I applied your changes manually and the work fine.

[Comment 8 Arfrever Frehtes Taifersar Arahesis (RETIRED) 2011-05-07 17:48:26 UTC]

(In reply to comment #7)
> (In reply to comment #6)
> > Please test this patch.
> I was unable to apply the patch - are you sure you diff'ed against the
> original?

Yes. You would need to synchronize the tree and the patch would apply cleanly.

[Comment 9 Nils Andresen 2011-05-07 18:08:27 UTC]

Sorry - i did not see that.

[Comment 10 Martin von Gagern 2011-05-11 15:29:24 UTC]

(In reply to comment #6)
> It's better to directly use system rst2man.py. Please test this patch.

Did the trick for me as well. Thanks a lot. Would love to see this included in the official portage tree.

[Comment 11 Markos Chandras (RETIRED) 2011-05-11 15:34:15 UTC]

Fixed. Thanks for the patch