Ubuntu Terminal App

Merge lp://staging/~xnox/ubuntu-terminal-app/drop-lxml-dep into lp://staging/~dpm/ubuntu-terminal-app/merge-plugin

  1. drop-lxml-dep
  2. Merge into merge-plugin
Proposed by Dimitri John Ledkov
Status: Rejected
Rejected by: David Planella
Proposed branch: lp://staging/~xnox/ubuntu-terminal-app/drop-lxml-dep
Merge into: lp://staging/~dpm/ubuntu-terminal-app/merge-plugin
Diff against target: 38 lines (+1/-8)
3 files modified
README.md (+0/-6)
debian/control (+0/-1)
tests/autopilot/ubuntu_terminal_app/CMakePluginParser.py (+1/-1)
To merge this branch: bzr merge lp://staging/~xnox/ubuntu-terminal-app/drop-lxml-dep
Reviewer Review Type Date Requested Status
David Planella Needs Fixing
Nicholas Skaggs (community) Approve
Review via email: mp+218482@code.staging.launchpad.net
To post a comment you must log in.
Revision history for this message
Dimitri John Ledkov (xnox) wrote :

Please test. Note that autopilot package still shipps modules in python2 location, instead of python3. I'll make another merge proposal to fix that.

Revision history for this message
Nicholas Skaggs (nskaggs) :
review: Approve
Revision history for this message
David Planella (dpm) wrote :

Note that it's not a matter of just replacing the import, as it seems that the standard module is not a drop-in replacement for lxml.

So further code changes will be required in CMakePluginParser.py to make it work.

In any case, as discussed on IRC with balloons, I'll split the original MP to isolate this change and not block on it.

review: Needs Fixing
Revision history for this message
Dimitri John Ledkov (xnox) wrote :

On 6 May 2014 19:46, David Planella <email address hidden> wrote:
> Review: Needs Fixing
>
> Note that it's not a matter of just replacing the import, as it seems that the standard module is not a drop-in replacement for lxml.
>
> So further code changes will be required in CMakePluginParser.py to make it work.
>
> In any case, as discussed on IRC with balloons, I'll split the original MP to isolate this change and not block on it.

Yeah, well i've run the tests on the desktop and they all passed....
not sure if CMakePluginParser was actually in fact used or not.

--
Regards,

Dimitri.

Unmerged revisions

71. By Dimitri John Ledkov

Drop lxml dependency

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