You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@flex.apache.org by ah...@apache.org on 2014/06/13 21:40:16 UTC

[4/5] git commit: [flex-utilities] [refs/heads/develop] - om says we don't need timestamping here

om says we don't need timestamping here


Project: http://git-wip-us.apache.org/repos/asf/flex-utilities/repo
Commit: http://git-wip-us.apache.org/repos/asf/flex-utilities/commit/b1b5d057
Tree: http://git-wip-us.apache.org/repos/asf/flex-utilities/tree/b1b5d057
Diff: http://git-wip-us.apache.org/repos/asf/flex-utilities/diff/b1b5d057

Branch: refs/heads/develop
Commit: b1b5d057ae1911fc93131d409accb57767201e11
Parents: 5d9a11f
Author: Alex Harui <ah...@apache.org>
Authored: Thu Jun 12 14:38:56 2014 -0700
Committer: Alex Harui <ah...@apache.org>
Committed: Thu Jun 12 14:38:56 2014 -0700

----------------------------------------------------------------------
 installer/src/InstallApacheFlex.mxml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/flex-utilities/blob/b1b5d057/installer/src/InstallApacheFlex.mxml
----------------------------------------------------------------------
diff --git a/installer/src/InstallApacheFlex.mxml b/installer/src/InstallApacheFlex.mxml
index 8794b45..1590872 100644
--- a/installer/src/InstallApacheFlex.mxml
+++ b/installer/src/InstallApacheFlex.mxml
@@ -609,7 +609,7 @@ variables are not required because the locations of these pieces are known.
 			} else {
 				var d:Date = new Date();
 				var ts:String = d.getTime().toString();
-				request = new URLRequest(Constants.APACHE_FLEX_URL + Constants.CONFIG_XML_NAME + "?ts=" + ts);
+				request = new URLRequest(Constants.APACHE_FLEX_URL + Constants.CONFIG_XML_NAME);
 				//request = new URLRequest(Constants.CONFIG_XML_NAME);
 			}
 			


Re: [4/5] git commit: [flex-utilities] [refs/heads/develop] - om says we don't need timestamping here

Posted by OmPrakash Muppirala <bi...@gmail.com>.
On Fri, Jun 13, 2014 at 12:40 PM, <ah...@apache.org> wrote:

> om says we don't need timestamping here
>
>
> Project: http://git-wip-us.apache.org/repos/asf/flex-utilities/repo
> Commit:
> http://git-wip-us.apache.org/repos/asf/flex-utilities/commit/b1b5d057
> Tree: http://git-wip-us.apache.org/repos/asf/flex-utilities/tree/b1b5d057
> Diff: http://git-wip-us.apache.org/repos/asf/flex-utilities/diff/b1b5d057
>
> Branch: refs/heads/develop
> Commit: b1b5d057ae1911fc93131d409accb57767201e11
> Parents: 5d9a11f
> Author: Alex Harui <ah...@apache.org>
> Authored: Thu Jun 12 14:38:56 2014 -0700
> Committer: Alex Harui <ah...@apache.org>
> Committed: Thu Jun 12 14:38:56 2014 -0700
>
> ----------------------------------------------------------------------
>  installer/src/InstallApacheFlex.mxml | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> ----------------------------------------------------------------------
>
>
>
> http://git-wip-us.apache.org/repos/asf/flex-utilities/blob/b1b5d057/installer/src/InstallApacheFlex.mxml
> ----------------------------------------------------------------------
> diff --git a/installer/src/InstallApacheFlex.mxml
> b/installer/src/InstallApacheFlex.mxml
> index 8794b45..1590872 100644
> --- a/installer/src/InstallApacheFlex.mxml
> +++ b/installer/src/InstallApacheFlex.mxml
> @@ -609,7 +609,7 @@ variables are not required because the locations of
> these pieces are known.
>                         } else {
>                                 var d:Date = new Date();
>                                 var ts:String = d.getTime().toString();
> -                               request = new
> URLRequest(Constants.APACHE_FLEX_URL + Constants.CONFIG_XML_NAME + "?ts=" +
> ts);
> +                               request = new
> URLRequest(Constants.APACHE_FLEX_URL + Constants.CONFIG_XML_NAME);
>                                 //request = new
> URLRequest(Constants.CONFIG_XML_NAME);
>                         }
>
>
>

Probably want to get rid of the variables d and ts?  No need to make a new
RC for this change.

Thanks,
Om