Bug 464151

Summary: About 1.0.0 build failure
Product: [Fedora] Fedora Reporter: Mamoru TASAKA <mtasaka>
Component: delugeAssignee: Peter Gordon <peter>
Status: CLOSED RAWHIDE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideCC: peter
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2008-09-28 05:34:19 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Attachments:
Description Flags
patch candidate none

Description Mamoru TASAKA 2008-09-26 14:32:04 UTC
Created attachment 317799 [details]
patch candidate

Description of problem:
(Well, Is this issue being discussed on the upstream?)
http://koji.fedoraproject.org/koji/taskinfo?taskID=838268

The problem is:
* libboost_filesystem.so surely exists as well as libboost_filesystem-mt.so.
  libboost_filesystem.so is single thread version of boost filesystem
  library (they say) and actually differs (in size) from libboost_filesystem-mt.so.

  Only libboost_thread.so does not exist, because there is no "single thread"
  variant of boost "thread" library (see bug 250802). So current -mt check
  by deluge setup.py:
--------------------------------------------------------------
   147      # Modify the libs if necessary for systems with only -mt boost libs    
   148      if not os.path.exists(
   149          os.path.join(sysconfig.PREFIX, "lib", "libboost_filesystem.so")):
   150          # It's likely that this system doesn't have symlinks setup
   151          # So add '-mt' to the libraries
   152          for lib in _libraries:
   153              if lib[:6] == "boost_":
   154                  _libraries[_libraries.index(lib)] = lib + "-mt"
--------------------------------------------------------------
  fails.

  Note that this perhaps succeeds on x86_64, because "/usr/lib/libboost_filesystem.so"
  is not found on x86_64, however this is not what these line expects.

  So: a possible fix might be:
  * to change the file to be checked from "libboost_filesystem.so" to
    "libboost_thread.so"
  * and to support libdir (or LIBDIR) argument (or environment)

Version-Release number of selected component (if applicable):
deluge-1.0.0-1.fc10

Additional info:
Possible patch attached. Changing to
----------------------------------------------------------------
CFLAGS="%{optflags}" LIBDIR=%{_libdir} %{__python} setup.py build
----------------------------------------------------------------
with the attached patch will make the build succeed:
http://koji.fedoraproject.org/koji/taskinfo?taskID=845299

Comment 1 Peter Gordon 2008-09-27 20:46:19 UTC
I tried to commit and build it a few days ago but it failed with this issue of mt libs; and I didn't have time due to schoolwork and job stuff to look at it until the weekend. 

I'll take care of this after some food. Thank you very much for the patch! :)

Comment 2 Peter Gordon 2008-09-28 05:34:19 UTC
Committed and built to rawhide. Thanks again for the patch!