Checkbox

Merge lp://qastaging/~sylvain-pineau/checkbox/checkbox-ng-packaging-fixes into lp://qastaging/~checkbox-dev/checkbox/checkbox-packaging

  1. checkbox-ng-packaging-fixes
  2. Merge into checkbox-packaging
Proposed by Sylvain Pineau
Status: Merged
Approved by: Sylvain Pineau
Approved revision: 27
Merged at revision: 27
Proposed branch: lp://qastaging/~sylvain-pineau/checkbox/checkbox-ng-packaging-fixes
Merge into: lp://qastaging/~checkbox-dev/checkbox/checkbox-packaging
Diff against target: 24 lines (+8/-0)
1 file modified
debian/rules (+8/-0)
To merge this branch: bzr merge lp://qastaging/~sylvain-pineau/checkbox/checkbox-ng-packaging-fixes
Reviewer Review Type Date Requested Status
Sylvain Pineau (community) Approve
Zygmunt Krynicki (community) Needs Information
Review via email: mp+193210@code.qastaging.launchpad.net

Description of the change

checkbox-ng packaging is broken because it needs an installed plainbox module.

We can satisfy this dependency by using the develop mode and an .egg-link file in checkbox-ng.

To post a comment you must log in.
Revision history for this message
Zygmunt Krynicki (zyga) wrote :

22 + cd $(CURDIR)/plainbox && ./setup.py develop -md $(CURDIR)/checkbox-ng

Why -m ? That's "--multi-version (-m) make apps have to require() a version"

review: Needs Information
Revision history for this message
Sylvain Pineau (sylvain-pineau) wrote :

copy/paste error, fixed now

review: Needs Resubmitting
lp://qastaging/~sylvain-pineau/checkbox/checkbox-ng-packaging-fixes updated
27. By Sylvain Pineau

debian:rules: Install a plainbox egg-link to allow checkbox-ng builds

As checkbox-ng depends on plainbox actually.

Revision history for this message
Sylvain Pineau (sylvain-pineau) wrote :

Added a missing PYTHONPATH to avoid:

You are attempting to install a package to a directory that is not
on PYTHONPATH and which Python does not read ".pth" files from. The
installation directory you specified (via --install-dir, --prefix, or
the distutils default setting) was:

    /tmp/buildd/checkbox-0.16.14/checkbox-ng

Tested with pbuilder on a raring system.

review: Approve

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