http://bugzilla.wpkg.org/show_bug.cgi?id=202
w...@trash-mail.com changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |w...@trash-mail.com
Severity|enhancement |normal
--- Comment #1 from w...@trash-mail.com ---
In a statement like
<download url='https://example/firefox/Firefox Setup %PKGVER%.exe'
target='software\Firefox Setup %PKGVER%.exe' />
the variable is expanded in the "target" attribute, but not in the "url"
attribute. This looks like a bug to me, because it makes the variables
pointless.
In my opinion an "expand" attribute in the "download" element isn't necessary,
since I know of no good reason why the URL should not be expanded. Setup files
do not contain percent signs and even if they would, the package base
maintainer can simply rename them.
Also the ExpandEnvironmentStrings() function only replaces variables that are
actually set, so downloading a file called literally "%setup%.exe" should still
work as long as "%SETUP%" isn't set on the client.
That reduces the patch to an one liner:
url = new ActiveXObject("WScript.Shell").ExpandEnvironmentStrings(url);
--
Configure bugmail: http://bugzilla.wpkg.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.
-------------------------------------------------------------------------
wpkg-users mailing list archives >> http://lists.wpkg.org/pipermail/wpkg-users/
_______________________________________________
wpkg-users mailing list
wpkg-users@lists.wpkg.org
http://lists.wpkg.org/mailman/listinfo/wpkg-users
