This is the mail archive of the cygwin-apps mailing list for the Cygwin project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Proposed patch for cygport to fix homepage command


Hi,

The "cygport <PN.cygport> homepage" command fails in two situations.

1. The default install location for Firefox 64 in WIn7 64 is

    "C:\Program Files\Mozilla Firefox\firefox.exe"

To use this from cygwin, it must be convert with "cygpath -u" and
results in a path name which includes spaces, and thus require
quoting. Cygport will use the BROWSER environment variable, when set,
but it does not quote the value everywhere it is used.


2. If the package does not include a HOMEPAGE variable, cygport
defaults to searching google for the program name, e.g.:

    "http://www.google.com/search?q=${ORIG_PN:-${PN}}";

however it does not quote the value everywhere it is used.


The attached patch quotes the path to the browser executable, and the
homepage URL at all locations where they are used.


HTH,

Doug

-- 
Doug Henderson, Calgary, Alberta, Canada - from gmail.com

Attachment: cygport-0.24.1-2.src.patch
Description: Binary data


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]