Smart library should be split into a separate package from smart command line tool

Bug #268722 reported by Christopher Armstrong
4
Affects Status Importance Assigned to Milestone
landscape-client (Ubuntu)
Fix Released
Undecided
Christopher Armstrong
smart (Ubuntu)
Fix Released
Undecided
Dustin Kirkland 

Bug Description

This would help packages that want to depend on the library without forcing the smart binary to be installed.

Changed in smart:
assignee: nobody → kirkland
status: New → In Progress
Revision history for this message
Christopher Armstrong (radix) wrote :

landscape-client needs to be updated in two ways:

 * It should depend on python-smartpm instead of smartpm-core, so it doesn't bring in the binary
 * It should invoke /usr/share/smart/smart.py instead of /usr/bin/smart in its cron job.

Changed in landscape-client:
assignee: nobody → radix
status: New → Confirmed
Revision history for this message
Dustin Kirkland  (kirkland) wrote :

Patch attached for smartpm, splitting the python library part into a new package, python-smartpm (which could perhaps be considered for Ubuntu main), and changing smartpm-core to simply contain a symlink for /usr/bin/smart -> /usr/share/smart/smart to actually expose the smart tool/interface.

:-Dustin

Revision history for this message
Christopher Armstrong (radix) wrote :

I've linked a branch ( lp:~radix/landscape-client/intrepid-python-smartpm ) which updates landscape-client to depend on the new python-smartpm package.

As it happens, landscape-client doesn't currently invoke smart in a cron job, so the only thing this needs to update is the dependency.

Revision history for this message
Christopher Armstrong (radix) wrote :

For convenience, here's a debdiff of the changes in that landscape-client branch.

Revision history for this message
Dustin Kirkland  (kirkland) wrote :

Updated smartpm diff, with feedback from mvo. Notable changes:
 * added a Replaces for python-smartpm
 * removed an artifact changelog.dch file
 * updated descriptions to remove the "beta" verbage.

:-Dustin

Changed in smart:
status: In Progress → Fix Released
Revision history for this message
Christopher Armstrong (radix) wrote :

Oops, here's a new landscape-client change that has my correct email address.

Revision history for this message
Christopher Armstrong (radix) wrote :

oops, here's *another* landscape-client change that includes the Fixes: for the LP bug.

Revision history for this message
Christopher Armstrong (radix) wrote :

The landscape-client part of this bug was fixed by Mathias Gug's upload of landscape-client:

https://launchpad.net/ubuntu/+source/landscape-client/1.0.18-0ubuntu2

Thanks Mathias.

Changed in landscape-client:
status: Confirmed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.