smart reports "error: Unsupported scheme: ssh" if ssh method used in APT source list

Bug #841661 reported by Eric Williams
16
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Smart Package Manager
Fix Released
Low
Unassigned
smart (Ubuntu)
Triaged
Medium
Unassigned

Bug Description

## Issue

- smart reports "error: Unsupported scheme: ssh" if ssh method used in APT source list

## Environment

- smartpm

- apt repository URI starting with "ssh"

## Resolution

- (workaround) Assign "SCPHandler" as handler for "ssh" in fetcher.py:

--- /usr/share/pyshared/smart/fetcher.py~ 2010-02-08 12:49:08.000000000 +0000
+++ /usr/share/pyshared/smart/fetcher.py 2011-09-02 14:45:36.996798493 +0000
@@ -1838,5 +1838,6 @@ class SCPHandler(FetcherHandler):
         self.changeActiveDownloads(-1)

 Fetcher.setHandler("scp", SCPHandler)
+Fetcher.setHandler("ssh", SCPHandler)

 # vim:ts=4:sw=4:et

This is untested, as I didn't have access to an ssh-based repo at the time it was reported.

## Background

APT supports the following URI methods: cdrom, file, http, ftp, copy, ssh, rsh. Smart doesn't accept "ssh" as a valid method.

ProblemType: Bug
DistroRelease: Ubuntu 10.04
Package: smartpm 1.2-5
ProcVersionSignature: Ubuntu 2.6.38-8.42-genusername 2.6.38.2
Uname: Linux 2.6.38-8-generic x86_64
NonfreeKernelModules: appletalk ipx p8023 veth macvlan usb_storage uas binfmt_misc iscsi_trgt i8k ip6table_filter ip6_tables ipt_MASQUERADE iptable_nat nf_nat nf_conntrack_ipv4 nf_defrag_ipv4 xt_state nf_conntrack ipt_REJECT xt_CHECKSUM iptable_mangle xt_tcpudp iptable_filter ip_tables x_tables kvm_intel kvm parport_pc ppdev dm_crypt autofs4 vesafb nvidia snd_hda_codec_hdmi snd_hda_codec_idt snd_hda_intel snd_hda_codec usblp snd_usb_audio snd_hwdep nfsd snd_usbmidi_lib snd_pcm snd_seq_midi exportfs nfs snd_rawmidi snd_seq_midi_event snd_seq snd_timer lockd snd_seq_device snd fscache nfs_acl i5000_edac auth_rpcgss edac_core coretemp soundcore gspca_ov534 sunrpc bridge snd_page_alloc stp gspca_main lp i5k_amb parport videodev shpchp serio_raw v4l2_compat_ioctl32 dcdbas raid10 raid456 async_pq async_xor xor async_memcpy async_raid6_recov raid6_pq async_tx raid1 raid0 multipath linear usbhid hid tg3
Architecture: amd64
Date: Mon Sep 5 09:14:08 2011
PackageArchitecture: all
ProcEnviron:
 LANG=en_GB.UTF-8
 SHELL=/bin/bash
SourcePackage: smart

Revision history for this message
Eric Williams (eric-canonical) wrote :
Changed in smart (Ubuntu):
status: New → Confirmed
Revision history for this message
Jonathan Davies (jpds) wrote :

I've tested and have confirmed that Eric's proposed fix fixes the issue.

Changed in smart (Ubuntu):
status: Confirmed → Triaged
importance: Undecided → Medium
Revision history for this message
Anders F Björklund (afb) wrote :

Alternatively one could "translate" it (ssh -> scp), in the aptchannelsync plugin ?

Changed in smart:
importance: Undecided → Low
Revision history for this message
Jonathan Davies (jpds) wrote :

APT already knows it as ssh:

$ ls -l /usr/lib/apt/methods/ssh
lrwxrwxrwx 1 root root 3 2011-07-14 13:12 /usr/lib/apt/methods/ssh -> rsh*

So it would probably be best to be things consistent and change smart, rather than have people change their sources.list?

Revision history for this message
Anders F Björklund (afb) wrote :

It doesn't matter to the sources.list (remains as-is), just avoids smart having to add rsh/ssh alias to the more sane "scp".

Changed in smart:
milestone: none → 1.4.1
Revision history for this message
Anders F Björklund (afb) wrote :
Revision history for this message
Ubuntu Foundations Team Bug Bot (crichton) wrote :

The attachment "smart-aptchannelsync-ssh.diff" of this bug report has been identified as being a patch. The ubuntu-reviewers team has been subscribed to the bug report so that they can review the patch. In the event that this is in fact not a patch you can resolve this situation by removing the tag 'patch' from the bug report and editing the attachment so that it is not flagged as a patch. Additionally, if you are member of the ubuntu-sponsors please also unsubscribe the team from this bug report.

[This is an automated message performed by a Launchpad user owned by Brian Murray. Please contact him regarding any issues with the action taken in this bug report.]

tags: added: patch
Revision history for this message
Anders F Björklund (afb) wrote :

Of course it's a patch, that's why it's marked "patch". Not sure what the redundant keyword is all about...

Changed in smart:
status: New → Fix Committed
Changed in smart:
status: Fix Committed → 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.