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

Bug #268722 reported by Christopher Armstrong on 2008-09-10
4
Affects Status Importance Assigned to Milestone
landscape-client (Ubuntu)
Undecided
Christopher Armstrong
smart (Ubuntu)
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
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
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

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.

Christopher Armstrong (radix) wrote :

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

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
Christopher Armstrong (radix) wrote :

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

Christopher Armstrong (radix) wrote :

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

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  Edit
Everyone can see this information.

Other bug subscribers