Return-Path: X-Original-To: apmail-incubator-flex-dev-archive@minotaur.apache.org Delivered-To: apmail-incubator-flex-dev-archive@minotaur.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id DCC2E955E for ; Fri, 24 Feb 2012 22:26:12 +0000 (UTC) Received: (qmail 11076 invoked by uid 500); 24 Feb 2012 22:26:12 -0000 Delivered-To: apmail-incubator-flex-dev-archive@incubator.apache.org Received: (qmail 11031 invoked by uid 500); 24 Feb 2012 22:26:12 -0000 Mailing-List: contact flex-dev-help@incubator.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: flex-dev@incubator.apache.org Delivered-To: mailing list flex-dev@incubator.apache.org Received: (qmail 11023 invoked by uid 99); 24 Feb 2012 22:26:12 -0000 Received: from nike.apache.org (HELO nike.apache.org) (192.87.106.230) by apache.org (qpsmtpd/0.29) with ESMTP; Fri, 24 Feb 2012 22:26:12 +0000 X-ASF-Spam-Status: No, hits=-2.3 required=5.0 tests=RCVD_IN_DNSWL_MED,SPF_PASS X-Spam-Check-By: apache.org Received-SPF: pass (nike.apache.org: domain of cframpto@adobe.com designates 64.18.1.181 as permitted sender) Received: from [64.18.1.181] (HELO exprod6og101.obsmtp.com) (64.18.1.181) by apache.org (qpsmtpd/0.29) with ESMTP; Fri, 24 Feb 2012 22:26:05 +0000 Received: from outbound-smtp-2.corp.adobe.com ([193.104.215.16]) by exprod6ob101.postini.com ([64.18.5.12]) with SMTP ID DSNKT0gOZ3eejz+MA3E8HjSWYGWzrXnACduD@postini.com; Fri, 24 Feb 2012 14:25:44 PST Received: from inner-relay-4.eur.adobe.com (inner-relay-4b [10.128.4.237]) by outbound-smtp-2.corp.adobe.com (8.12.10/8.12.10) with ESMTP id q1OMPgN3025918 for ; Fri, 24 Feb 2012 14:25:42 -0800 (PST) Received: from nahub01.corp.adobe.com (nahub01.corp.adobe.com [10.8.189.97]) by inner-relay-4.eur.adobe.com (8.12.10/8.12.9) with ESMTP id q1OMPfPl012867 for ; Fri, 24 Feb 2012 14:25:41 -0800 (PST) Received: from nambx09.corp.adobe.com ([10.8.189.47]) by nahub01.corp.adobe.com ([10.8.189.97]) with mapi; Fri, 24 Feb 2012 14:25:40 -0800 From: Carol Frampton To: "flex-dev@incubator.apache.org" Date: Fri, 24 Feb 2012 14:25:38 -0800 Subject: Re: [CODE] Short cleanup Thread-Topic: [CODE] Short cleanup Thread-Index: AczzQz02fKE+DELeTwyr+E9sXF7f+A== Message-ID: In-Reply-To: Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: user-agent: Microsoft-MacOutlook/14.14.0.111121 acceptlanguage: en-US Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 X-Virus-Checked: Checked by ClamAV on apache.org I don't like this approach. The replaced tag will surely get submitted. I really don't understand why so much attention has been devoted to this today. There are so many other things to do. Carol On 2/24/12 5 :19PM, "Om" wrote: >I like this approach. This is how we bake in application version numbers, >build timestamp, etc. while compiling release builds of Flex apps we ship. > > > >> Minor issue with this is that now the Version.as files are out of sync >> with SVN, could accidentally be checked back in with wrong version >>numbers >> or the like. >> >> I guess there's the more philosophical question of do we really want >>ANT >> scripts that modify the source code? >> > >We can revert Version.as to its original state after compiling or just not >check in the changed Version.as. A carefully constructed build script >should do the trick. This way we don't really care if Ant is making >changes to the source code. > >Om > >On Fri, Feb 24, 2012 at 2:11 PM, Justin Mclean >wrote: > >> Hi, >> >> ANT can do this quite easily. >> >> If we change all Version.as files to be like so: >> mx_internal static const VERSION:String =3D "@release@"; >> >> Then this target will inject the version number into all Version.as >>files: >> >> >token=3D"@release@" >> value=3D"${release.version}.${build.number}" /> >> >> >> We have these currently in the build file: >> >> >> >> So that would replace every occurrence of "@release@" with "4.6.0.0". >> >> Minor issue with this is that now the Version.as files are out of sync >> with SVN, could accidentally be checked back in with wrong version >>numbers >> or the like. >> >> I guess there's the more philosophical question of do we really want >>ANT >> scripts that modify the source code? >> >> Justin >> >> >> >>