Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!

Bug 365181

Summary: gnome-extra/nautilus-dropbox-0.6.7 does not build against >=dev-python/docutils-0.8
Product: Gentoo Linux Reporter: Nils Andresen <nils>
Component: Current packagesAssignee: Markos Chandras (RETIRED) <hwoarang>
Status: RESOLVED FIXED    
Severity: normal CC: arfrever, jlec, Martin.vGagern, nils
Priority: Normal    
Version: unspecified   
Hardware: All   
OS: Linux   
Whiteboard:
Package list:
Runtime testing required: ---
Attachments: full log
Patch to fix the problem
Use system rst2man.py

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 <docutils-users@lists.sf.net>.
> 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) gentoo-dev 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) gentoo-dev 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 <docutils-users@lists.sf.net>.
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) gentoo-dev 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) gentoo-dev 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) gentoo-dev 2011-05-11 15:34:15 UTC
Fixed. Thanks for the patch