Divmod

Merge lp://qastaging/~divmod-dev/divmod.org/1208304-cssutils-api into lp://qastaging/divmod.org

  1. 1208304-cssutils-api
  2. Merge into trunk
Proposed by Tristan Seligmann
Status: Needs review
Proposed branch: lp://qastaging/~divmod-dev/divmod.org/1208304-cssutils-api
Merge into: lp://qastaging/divmod.org
Diff against target: 20 lines (+2/-2)
1 file modified
Mantissa/xmantissa/website.py (+2/-2)
To merge this branch: bzr merge lp://qastaging/~divmod-dev/divmod.org/1208304-cssutils-api
Reviewer Review Type Date Requested Status
Jonathan Jacobs Needs Information
Review via email: mp+178488@code.qastaging.launchpad.net
To post a comment you must log in.
Revision history for this message
Jonathan Jacobs (jjacobs) wrote :

Doesn't this break compatibility with the previous version of cssutils? I think you might want to write your own replaceUrls in the case where replaceUrls cannot be imported from cssutils.

review: Needs Information
Reply
Revision history for this message
Tristan Seligmann (mithrandi) wrote :

The change in question was made in cssutils 0.9.5b1, released in 2008. This is older than the version in Debian squeeze, so it is unlikely that anyone is using Mantissa with a version of cssutils that still has the old API; thus I don't think there's much point in introducing fallback logic.

Reply

Unmerged revisions

2712. By Tristan Seligmann

Use the new API.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches

to all changes: