On Wed, May 01, 2013 at 01:13:55PM +0200, Michael Scherer wrote: > https://bugzilla.redhat.com/show_bug.cgi?id=954187 > > If someone use a custom unit file for the sandbox, the rpm > autodetection fail with a exception. Now, this will show > a error message, asking to use --rpm and use this rpm if > autodetection fail. > --- > bin/virt-sandbox-service | 46 +++++++++++++++++++++++++++++++++++++++++----- > 1 file changed, 41 insertions(+), 5 deletions(-) > @@ -1015,6 +1047,10 @@ def gen_create_args(subparser): > action=CheckUnit, > dest="unitfiles", default=[], > help=_("Systemd Unit file to run within the systemd sandbox container. Commands cannot be specified with unit files.")) > + parser.add_argument("-r", "--rpm", > + action=CheckRpm, > + dest="rpm", default=None, > + help=_("Rpm configuration to use in the container. Default: autodetected from unit files, unless if using a custom unit file.")) Although we currently only integrate with RPM that would change if someone ported it to non-Fedora/RHEL distros. As such we shouldn't expose any command line option named --rpm. Instead we should use a generic term like --package Regards, Daniel -- |: http://berrange.com -o- http://www.flickr.com/photos/dberrange/ :| |: http://libvirt.org -o- http://virt-manager.org :| |: http://autobuild.org -o- http://search.cpan.org/~danberr/ :| |: http://entangle-photo.org -o- http://live.gnome.org/gtk-vnc :| -- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list