New Ticket     Wiki     Browse Source     Timeline     Roadmap     Ticket Reports     Search

Ticket #16491 (closed defect: fixed)

Opened 3 years ago

Last modified 2 years ago

base/src/port/port.tcl should quote homepage when opening it

Reported by: blb@… Owned by: macports-tickets@…
Priority: Normal Milestone: MacPorts 1.7.0
Component: base Version: 1.7.0
Keywords: port command homepage gohome Cc: raimue@…
Port:

Description

In the code for handling the gohome command, the $homepage variable is not quoted when passed to the shell, which causes issues when a character handled by the shell is present in the homepage. To see this, try port gohome xdvipdfmx and note that the page loaded has everything from the & to the end stripped. To see a more catastrophic error, try port gohome libksba (at least until that port's homepage is corrected).

Attachments

base_src_port_port.tcl.diff Download (0.6 KB) - added by blb@… 3 years ago.
diff to base/src/port/port.tcl (on trunk)

Change History

Changed 3 years ago by raimue@…

  • cc raimue@… added

Shouldn't that be single quotes? With double quotes, there is still replacement for $HOME or other environment variables happening. Unlikely that it appears in an homepage URL, but let's fix it completely.

Changed 3 years ago by blb@…

diff to base/src/port/port.tcl (on trunk)

Changed 3 years ago by blb@…

Hmm, yeah, not sure why I went with double, even with singles not having to be escaped. Patch updated.

Changed 3 years ago by raimue@…

  • status changed from new to closed
  • resolution set to fixed

Committed your patch in r40414.

Changed 3 years ago by toby@…

  • milestone changed from MacPorts base bugs to MacPorts Future

Milestone MacPorts base bugs deleted

Changed 2 years ago by jmr@…

  • milestone changed from MacPorts Future to MacPorts 1.7.0
Note: See TracTickets for help on using tickets.