#2700 Change DownloadUrl[] constant to current URL

Branch_+_Trunk
closed-fixed
nobody
None
5
2008-09-30
2008-09-29
Tim Gerundt
No

This patch change the DownloadUrl[] constant to the current URL:

Index: Constants.h

--- Constants.h (Revision 5987)
+++ Constants.h (Arbeitskopie)
@@ -26,7 +26,7 @@
const TCHAR LicenceUrl[] = _T("http://www.gnu.org/licenses/gpl-2.0.html");

/** @brief WinMerge download page URL. */
-const TCHAR DownloadUrl[] = _T("http://winmerge.org/downloads.php");
+const TCHAR DownloadUrl[] = _T("http://winmerge.org/downloads/");

/** @brief Relative (to WinMerge executable ) path to local help file. */
const TCHAR DocsPath[] = _T("\\Docs\\WinMerge.chm");

Since our webserver redirect the old URL to the new one, the problem is not so big.

Greetings,
Tim

Discussion

  • Kimmo Varis

    Kimmo Varis - 2008-09-30

    I'm wondering about value of this in 2.10 branch. If we have redirect then user don't see the old URL, right? So by changing the URL we save a bit of work from web server. And stable releases generate a lot more traffic than experimental/beta releases. So looks worthwhile change for 2.10 branch.

     
  • Tim Gerundt

    Tim Gerundt - 2008-09-30
    • milestone: --> Branch_+_Trunk
     
  • Tim Gerundt

    Tim Gerundt - 2008-09-30

    > If we have redirect then user don't see the old URL, right?

    Only if he is offline or have a really slow internet connection. But normally it only takes not even a second to redirect to the new page.

    > So looks worthwhile change for 2.10 branch.

    I think it too.

     
  • Tim Gerundt

    Tim Gerundt - 2008-09-30

    Committed to SVN:

    * In Revision 5989 (Trunk)
    * In Revision 5990 (Branch R2_10)

     
  • Tim Gerundt

    Tim Gerundt - 2008-09-30
    • status: open --> closed-fixed
     

Log in to post a comment.