bugCompact Disc Input and Control Library - Bugs: bug #38185, pycdio 0.18 is broken when built...

 
 

bug #38185: pycdio 0.18 is broken when built against libcdio 0.90

Submitted by:  Thomas Vander Stichele <thomasvs>
Submitted on:  Mon 28 Jan 2013 08:01:06 PM UTC  
Votes:  10  
 
Category: PythonSeverity: 5 - Average
Item Group: NoneStatus: Works For Me
Privacy: PublicAssigned to: Rocky Bernstein <rocky>
Open/Closed: Closed

Add a New Comment (Rich MarkupRich Markup):
   

You are not logged in

Please log in, so followups can be emailed to you.

 

Sat 16 Feb 2013 08:50:32 AM UTC, comment #4:

Haven't heard from you so I'll assume this is fixed.

If this is still of interest to you then try with release pycdio 0.19.

Rocky Bernstein <rocky>
Project AdministratorIn charge of this item.
Thu 31 Jan 2013 02:03:00 AM UTC, comment #3:

Does commit b8b1d4d01367d5d095c9ed6fe755ab5b8d043e95 address the CDTEXT_ARRANGER versus CDTEXT_FIELD_ARRANGER problem?

As for MIN_DRIVER, I don't see that anymore in the pycdio code. And let's split out these two things since they are separate bugs.

Rocky Bernstein <rocky>
Project AdministratorIn charge of this item.
Mon 28 Jan 2013 08:07:33 PM UTC, comment #2:

Suggest changing the title to 'pycdio 0.18 is broken against libcdio 0.83 and higher' - savannah doesn't let me edit the title myself.

Thomas Vander Stichele <thomasvs>
Mon 28 Jan 2013 08:06:38 PM UTC, comment #1:

Actually, from reading the code again after I was trying to work around MIN_DRIVER having gone away, I realized that it's also pycdio.py taking MIN_DRIVER from _pycdio.so

From this I conclude that pycdio 0.18 is also broken with older libcdio versions, since 0.82 (first release after 8e8b18e1970bd16f46d629a56096ef33d1cac6a9)

Thomas Vander Stichele <thomasvs>
Mon 28 Jan 2013 08:01:06 PM UTC, original submission:

When importing pycdio on e.g. Fedora Rawhide:

File "/usr/lib64/python2.7/site-packages/pycdio.py", line 411, in <module>
CDTEXT_ARRANGER = _pycdio.CDTEXT_ARRANGER
AttributeError: 'module' object has no attribute 'CDTEXT_ARRANGER'

- pycdio.py in 0.18 always does
CDTEXT_ARRANGER = _pycdio.CDTEXT_ARRANGER
no matter what the version of libcdio

- _pycdio.so contains CDTEXT_ARRANGER if libcdio is pre-0.90 and
CDTEXT_FIELD_ARRANGER if libcdio is 0.90

- hence, pycdio.py 0.18 simply cannot be imported if it was built
against libcdio 0.90

I originally thought it might have been my fault or Fedora's (because I
prefer to blame from inwards out, starting with me, and because I was
under the impression that it worked on Ubuntu based on reports).

But in fact:
- this doesn't work in fedora rawhide
- according to my user's reports, the same problem exists in ArchLinux
- Debian and Ubuntu don't even package pycdio at all; the users that
had reported the issue of MIN_DRIVER and had built pycdio from source
against a 0.83 libcdio package (so they had an importable pycdio.py)

Feel free to point out if I misread anything in the code; from my
current reading it looks like pycdio 0.18 simply cannot be used with
libcdio 0.90

Thomas Vander Stichele <thomasvs>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach File(s):
   
   
Comment:
   

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by rocky (Posted a comment)
  • -unavailable- added by plasmasheep (Voted in favor of this item)
  • -unavailable- added by thomasvs (Submitted the item)
  •  

    Do you think this task is very important?
    If so, you can click here to add your encouragement to it.
    This task has 10 encouragements so far.

    Only logged-in users can vote.

     

    Please enter the title of George Orwell's famous dystopian book (it's a date):

     

     

    Follow 4 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Sat 16 Feb 2013 08:50:32 AM UTCrockyStatusNone=>Works For Me
      Assigned toNone=>rocky
      Open/ClosedOpen=>Closed
    Wed 30 Jan 2013 11:42:12 PM UTCplasmasheepCarbon-Copy-=>Added plasmasheep

    Back to the top


    Powered by Savane 3.1-cleanup