Merge lp:~sylvain-pineau/checkbox/build_dep_80211_resource into lp:~checkbox-dev/checkbox/checkbox-packaging

Proposed by Sylvain Pineau
Status: Superseded
Proposed branch: lp:~sylvain-pineau/checkbox/build_dep_80211_resource
Merge into: lp:~checkbox-dev/checkbox/checkbox-packaging
Prerequisite: lp:~sylvain-pineau/checkbox/80211_resource
Diff against target: 22 lines (+3/-2)
1 file modified
debian/control (+3/-2)
To merge this branch: bzr merge lp:~sylvain-pineau/checkbox/build_dep_80211_resource
Reviewer Review Type Date Requested Status
Sylvain Pineau (community) Approve
Daniel Manrique (community) Approve
Review via email: mp+203365@code.launchpad.net

This proposal supersedes a proposal from 2014-01-27.

This proposal has been superseded by a proposal from 2014-01-28.

Description of the change

Add the libnl-genl-3-dev build dependency (required to build 80211_resource)

To post a comment you must log in.
Revision history for this message
Daniel Manrique (roadmr) wrote :

Does libnl3-genl-dev also depend on libnl3-dev? if not, maybe it's worth adding it as a separate dependency.

Also, maybe we need the run-time dependency on the binary package, even if it's installed by default on Ubuntu I'd prefer to explicitly state that we need it.

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

Yes it depends on libnl-3-dev: http://packages.ubuntu.com/fr/trusty/libnl-genl-3-dev

I'll add the runtime dependency, good idea. thanks

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

Added a runtime dep on libnl-genl-3-200 (itself depending on libnl-3-200, see http://packages.ubuntu.com/fr/trusty/libnl-genl-3-200)

review: Needs Resubmitting
Revision history for this message
Daniel Manrique (roadmr) wrote :

Thanks so much! Approving now.

review: Approve
Revision history for this message
Daniel Manrique (roadmr) wrote :
Revision history for this message
Daniel Manrique (roadmr) wrote :

whaat ;)

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

approved, run tarmac, run

review: Approve
Revision history for this message
Zygmunt Krynicki (zyga) wrote :

"run luke, run" ;-)

Revision history for this message
Daniel Manrique (roadmr) wrote :
Revision history for this message
Daniel Manrique (roadmr) wrote :

OHHH, so you can't depend on a branch if the merge target for that branch is not the same as this one :(

Could you please supersede this MR with another one removing the prerequisite? And please self-approve that to expedite things. The prerequisite is already merged.

Unmerged revisions

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'debian/control'
2--- debian/control 2014-01-07 14:03:28 +0000
3+++ debian/control 2014-01-27 19:18:44 +0000
4@@ -7,6 +7,7 @@
5 XS-Testsuite: autopkgtest
6 Build-Depends: debhelper (>= 9.2),
7 dh-translations,
8+ libnl-genl-3-dev,
9 libqt4-dev,
10 libqt5qml-graphicaleffects,
11 libqt5sql5-sqlite,
12@@ -169,9 +170,9 @@
13 This package contains the CheckBox support python3 library needed by some jobs.
14
15 Package: plainbox-provider-resource-generic
16-Architecture: all
17+Architecture: any
18 Depends: python3 (>= 3.2), python3-checkbox-support, ${misc:Depends}
19-Recommends: dmidecode, dpkg (>= 1.13), lsb-release, wodim
20+Recommends: dmidecode, dpkg (>= 1.13), libnl-genl-3-200, lsb-release, wodim
21 Description: CheckBox generic resource jobs provider
22 This package provides the generic resource jobs.
23

Subscribers

People subscribed via source and target branches