Opened 17 years ago

Closed 17 years ago

Last modified 16 years ago

#13046 closed defect (fixed)

UPDATE: Fix openssh for leopard (percent_null errors)

Reported by: macports@… Owned by: skymoo (Adam Mercer)
Priority: Normal Milestone:
Component: ports Version: 1.5.0
Keywords: openssh percent_expand Cc: macports@…, christian@…
Port:

Description

Openssh fails on Leopard with the errors described here:

http://blog.paulbetts.org/index.php/2007/10/27/fixing-percent_expand-null-replacement-ssh-bug-on-os-x-leopard/

To reproduce, simply:

% /opt/local/bin/ssh somehost ls

The attached patch implements the fix described on the above page, which will allow openssh to work on MacPorts until the same fix is made to the upstream sources.

This package marked 'nomaintainer', so I'm taking the liberty of assigning it to jmpp@…, the package's last committer.

Attachments (3)

fix-openssh-4.7p1-for-leopard.patch (1.7 KB) - added by macports@… 17 years ago.
Patch containing Portfile update, and new files/percent_expand_fix.patch file
fix-openssh-4.7p1-for-leopard-1.patch (1.4 KB) - added by callahad@… 17 years ago.
Original uploader's patchfile was reversed. This isn't.
fix-openssh-4.7p1-for-leopard-2.patch (1.5 KB) - added by callahad@… 17 years ago.
Only apply the fix if the host is running leopard. (I presume that otherwise would cause a regression)

Download all attachments as: .zip

Change History (15)

Changed 17 years ago by macports@…

Patch containing Portfile update, and new files/percent_expand_fix.patch file

Changed 17 years ago by callahad@…

Original uploader's patchfile was reversed. This isn't.

Changed 17 years ago by callahad@…

Only apply the fix if the host is running leopard. (I presume that otherwise would cause a regression)

comment:1 Changed 17 years ago by macports@…

Great - thanks for the corrections. I doubt that the patch would cause a regression on Tiger, but it makes sense to be cautious here.

comment:2 Changed 17 years ago by christian@…

How long will it take until the patch makes its way into the repository and onto my machine?

p.s.: I'd like to follow up on this and be on CC: but unfortunately may not edit the ticket details. Anyone cares to put me on cc:?

comment:3 Changed 17 years ago by ryandesign (Ryan Carsten Schmidt)

Cc: christian@… jmpp@… added

comment:4 Changed 17 years ago by ryandesign (Ryan Carsten Schmidt)

According to this page, no patch is necessary. Comments from people running Leopard? Does that work for you?

comment:5 Changed 17 years ago by christian@…

Yes, that does work for me.

However, I feel like adding an option that basically reflects just the default value isn't nice. It should work with the default setup: That doesn't have a ~/.ssh/config file, but might have keys set up.

comment:6 Changed 17 years ago by ryandesign (Ryan Carsten Schmidt)

Cc: jmpp@… removed
Owner: changed from jmpp@… to macports-dev@…

jmpp's last change to this port in r30239 was not anything specific to this port; in fact it was part of a series of changes from r30187 to r30260 that affected thousands of ports. So there's no reason to assign this ticket to him.

comment:7 Changed 17 years ago by macports@…

Apologies for the confused initial assignment to jmpp.

The page describes a workaround for a definite problem in openssh.

OpenSSH should get fixed upstream in due course, and my hope in submitting this patch was that making the tiny fix locally in MacPorts would save the time of many Leopard users, and spare them from looking for answers on the MacPorts mailing list or via the Trac.

comment:8 Changed 17 years ago by macports@…

Update: I found a ticket for the issue upstream in the OpenSSH bugzilla -

https://bugzilla.mindrot.org/show_bug.cgi?id=1377

It includes an enhanced version of the patch included in this ticket.

comment:9 Changed 17 years ago by skymoo (Adam Mercer)

Owner: changed from macports-tickets@… to ram@…
Status: newassigned

comment:10 Changed 17 years ago by skymoo (Adam Mercer)

Resolution: fixed
Status: assignedclosed

fixed in r32161

comment:11 Changed 17 years ago by jmpalacios (Juan Manuel Palacios)

Priority: HighNormal

comment:12 Changed 16 years ago by (none)

Milestone: Port Bugs

Milestone Port Bugs deleted

Note: See TracTickets for help on using tickets.