Register Guidelines E-Books Search Today's Posts Mark Forums Read

Go Back   MobileRead Forums > E-Book Software > Calibre > Development

Notices

Reply
 
Thread Tools Search this Thread
Old 01-23-2012, 03:33 AM   #1
DOMBlogger
Junior Member
DOMBlogger began at the beginning.
 
Posts: 2
Karma: 10
Join Date: Jan 2012
Device: none
Problems compiling calibre

I am attempting to compile calibre on CentOS 6.2 + EPEL fully patched.

I started with the Fedora src.rpm from Fedora 16 - all build dependencies met from CentOS and EPEL except 1 - python-cssutils - which I met by rebuilding the Fedora src.rpm (that is a platform independent python library)

When I saw that calibre-0.8.33-2.fc16.src.rpm was failing and would need some TLC to build in CentOS I started hacking at the rpm spec file, first thing I did was grab the latest source for calibre - 0.8.36.

It's not even getting past the setup, and what is choking the setup appears to be bits of python added since 0.8.33 - now I'm no expert in python, but it appears to me to just be sloppy code. For example, the following patch got me past one of the choke points:

Code:
diff -ur calibre/setup/check.py calibre.patched/setup/check.py
--- calibre/setup/check.py      2012-01-19 20:44:58.000000000 -0800
+++ calibre.patched/setup/check.py      2012-01-22 23:00:58.000000000 -0800
@@ -89,7 +89,7 @@
             self.info('\tChecking', f)
             errors = False
             ext = os.path.splitext(f)[1]
-            if ext in {'.py', '.recipe'}:
+            if ext in ('.py', '.recipe'):
                 w = check_for_python_errors(open(f, 'rb').read(), f)
                 if w:
                     errors = True
Why curly brackets were used in the original, I'm not sure - again I'm not a python guru but all my experience with python says parenthesis are what should have been used.

Current choke point is in calibre/setup/upload.py -

Code:
Traceback (most recent call last):
  File "setup.py", line 13, in <module>
    import setup.commands as commands
  File "/home/domblogger/rpm/BUILD/calibre/setup/commands.py", line 68, in <module>
    from setup.upload import (UploadUserManual, UploadDemo, UploadInstallers,
  File "/home/domblogger/rpm/BUILD/calibre/setup/upload.py", line 117
    files = {x:installer_description(x) for x in
                                          ^
SyntaxError: invalid syntax
error: Bad exit status from /home/domblogger/rpm/tmp/rpm-tmp.N9v9EI (%build)
Looking at that code -

Code:
files = {x:installer_description(x) for x in
                all_possible.intersection(available)}
Again, I'm no expert in python, but in any language that at least looks to be extremely sloppy and hard to understand and thus difficult to maintain.

Have the contents of the setup directory been run through a python lint tool?
What kind of checking is done before releases are pushed? Is the error bad python as I'm alleging, or is perhaps the python on my system (python-2.6.6) just too old?

If the issue is python version, might I suggest your coding standards specify that code needs to work on Python 2.6? A lot of enterprise Linux distributions (not just RHEL and clones) are currently running Python 2.6 and I would hate to see a project such as this excluded from those users.

Is there a project bugzilla where I can look to see if my issues are already being addressed?

Thank you, I've never actually used calibre or really any epub reader extensively, but I now have been given assignment of writing tools to generate epub content (epub 3 w/ 2 compat) and having calibre on my system would make testing the output of my code (php - not python) a lot easier, and for that I am very grateful.
DOMBlogger is offline   Reply With Quote
Old 01-23-2012, 05:20 AM   #2
chaley
"chaley", not "charley"
chaley ought to be getting tired of karma fortunes by now.chaley ought to be getting tired of karma fortunes by now.chaley ought to be getting tired of karma fortunes by now.chaley ought to be getting tired of karma fortunes by now.chaley ought to be getting tired of karma fortunes by now.chaley ought to be getting tired of karma fortunes by now.chaley ought to be getting tired of karma fortunes by now.chaley ought to be getting tired of karma fortunes by now.chaley ought to be getting tired of karma fortunes by now.chaley ought to be getting tired of karma fortunes by now.chaley ought to be getting tired of karma fortunes by now.
 
Posts: 5,814
Karma: 1216136
Join Date: Jan 2010
Location: France
Device: Many android devices
Quote:
Originally Posted by DOMBlogger View Post
Why curly brackets were used in the original, I'm not sure - again I'm not a python guru but all my experience with python says parenthesis are what should have been used.
The curly brackets are correct. It means create a set instead of a list. This syntax is valid starting in python 2.7, which is the minimum version supported by calibre.
chaley is offline   Reply With Quote
 
Advertisement
Old 01-23-2012, 05:48 AM   #3
DOMBlogger
Junior Member
DOMBlogger began at the beginning.
 
Posts: 2
Karma: 10
Join Date: Jan 2012
Device: none
Well, that's a shame.

I guess I need to look for a different solution.

Is there a technical reason why such a recent version of python is required? And if 2.6.6 isn't new enough, shouldn't that have been caught at the beginning?

I'm a die hard FOSS fanatic but I guess the constant desire to use bleeding edge and ignore stable is the one weakness many FOSS projects have that commercial software with top down management demanding sane compatability with stable platforms don't have.

Oh well.
DOMBlogger is offline   Reply With Quote
Reply

Tags
centos python

Thread Tools Search this Thread
Search this Thread:

Advanced Search

Forum Jump

Similar Threads
Thread Thread Starter Forum Replies Last Post
Compiling Calibre on NAS Yancoucou Devices 3 01-14-2012 06:11 PM
Regarding Calibre source code compiling shivas Introduce Yourself 4 12-23-2009 10:52 AM
Problems compiling unkilbeeg Sigil 1 09-13-2009 02:49 PM
calibre 0.4.136 not compiling on linux thebadtouch Calibre 2 02-10-2009 08:08 PM
Problems compiling eb1150 on Ubuntu Intrepid luqmaninbmore Fictionwise eBookwise 10 01-15-2009 08:14 PM


All times are GMT -4. The time now is 03:31 AM.


MobileRead.com is a privately owned, operated and funded community.