New Ticket     Tickets     Wiki     Browse Source     Timeline     Roadmap     Ticket Reports     Search

Ticket #13046 (closed defect: fixed)

Opened 6 years ago

Last modified 4 years ago

UPDATE: Fix openssh for leopard (percent_null errors)

Reported by: macports@… Owned by: ram@…
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

fix-openssh-4.7p1-for-leopard.patch (1.7 KB) - added by macports@… 6 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@… 6 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@… 6 years ago.
Only apply the fix if the host is running leopard. (I presume that otherwise would cause a regression)

Change History

Changed 6 years ago by macports@…

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

Changed 6 years ago by callahad@…

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

Changed 6 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 6 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 6 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 6 years ago by ryandesign@…

  • Cc christian@…, jmpp@… added

comment:4 Changed 6 years ago by ryandesign@…

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

comment:5 Changed 6 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 6 years ago by ryandesign@…

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

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 6 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 6 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 5 years ago by ram@…

  • Owner changed from macports-tickets@… to ram@…
  • Status changed from new to assigned

comment:10 Changed 5 years ago by ram@…

  • Status changed from assigned to closed
  • Resolution set to fixed

fixed in r32161

comment:11 Changed 5 years ago by jmpp@…

  • Priority changed from High to Normal

comment:12 Changed 4 years ago by anonymous

  • Milestone Port Bugs deleted

Milestone Port Bugs deleted

Note: See TracTickets for help on using tickets.