Mailing-List: contact dev-help@flex.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@flex.apache.org
MIME-Version: 1.0
Sender: omuppi1@gmail.com
In-Reply-To: <CACK5iZcx46MZaB72PHdkG-PghUEO9qNTaUUpOhYBjk5u7Lghdg@mail.gmail.com>
References: <f18b91e842804418914cdc6df6c75557@git.apache.org>
 <CACK5iZc9_BVLj3-J6CURjKmcOZc7NTdmzPWreBv41LSfEquDbw@mail.gmail.com>
 <CADZSWG53D8z7hhM8cN6scRnmQnCUKqzFy7vYEHS4MgME2id3vw@mail.gmail.com>
 <CACK5iZc5fD1cT-Me=Ox7CX7S_mTHEF55qDWyAe9ZhL1-Xe0vVg@mail.gmail.com>
 <CADZSWG7vqXOtiLRu=t8GYO1DV4ynPA7ZRhPCYPmsttzR5i72Vw@mail.gmail.com> <CACK5iZcx46MZaB72PHdkG-PghUEO9qNTaUUpOhYBjk5u7Lghdg@mail.gmail.com>
From: OmPrakash Muppirala <bigosmallm@gmail.com>
Date: Wed, 17 Aug 2016 13:10:56 -0700
Message-ID: <CACK5iZf7ONFMeDdMKA=NbmNs7Z5BAo=WM+sv_8wtzjuh2FuxRA@mail.gmail.com>
Subject: Re: git commit: [flex-utilities] [refs/heads/develop] - npm-flexjs:
 changes line ending of scripts to LF because CRLF does not work on Mac,
 causing bad interpreter: No such file or directory error
To: "dev@flex.apache.org" <dev@flex.apache.org>
Content-Type: multipart/alternative; boundary=001a114a9fe4f29661053a4a1110
archived-at: Wed, 17 Aug 2016 20:11:38 -0000

--001a114a9fe4f29661053a4a1110
Content-Type: text/plain; charset=UTF-8

No, it does not work on cmd or powershell.  If I remember correctly, this
only worked on Cygwin for me.  I was hoping to add cmd/powershell support
at a later point.

On Wed, Aug 17, 2016 at 12:39 PM, OmPrakash Muppirala <bigosmallm@gmail.com>
wrote:

> On Wed, Aug 17, 2016 at 12:37 PM, Josh Tynjala <joshtynjala@gmail.com>
> wrote:
>
>> So these scripts work correctly in PowerShell and cmd?
>>
>>
> <Runs to go check>
>
>
>> - Josh
>>
>> On Wed, Aug 17, 2016 at 12:28 PM, OmPrakash Muppirala <
>> bigosmallm@gmail.com>
>> wrote:
>>
>> > I don't think we will be adding bat files here.  The idea is to be able
>> to
>> > run the same file on either platform so that they can be integrated into
>> > other flows.  At least, that is the current thinking.
>> >
>> > Thanks,
>> > Om
>> >
>> > On Wed, Aug 17, 2016 at 12:21 PM, Josh Tynjala <joshtynjala@gmail.com>
>> > wrote:
>> >
>> > > I assume that we plan to add .bat files for Windows at some point, and
>> > > those probably need CRLF. We could check the file extension. Either
>> skip
>> > > .bat files or only fix files without an extension. I'm not sure which
>> is
>> > > better.
>> > >
>> > > - Josh
>> > >
>> > > On Wed, Aug 17, 2016 at 12:01 PM, OmPrakash Muppirala <
>> > > bigosmallm@gmail.com>
>> > > wrote:
>> > >
>> > > > On Wed, Aug 17, 2016 at 10:28 AM, <joshtynjala@apache.org> wrote:
>> > > >
>> > > > > Repository: flex-utilities
>> > > > > Updated Branches:
>> > > > >   refs/heads/develop e5c3d36e0 -> 59664053c
>> > > > >
>> > > > >
>> > > > > npm-flexjs: changes line ending of scripts to LF because CRLF does
>> > not
>> > > > > work on Mac, causing bad interpreter: No such file or directory
>> error
>> > > > >
>> > > > >
>> > > > > Project: http://git-wip-us.apache.org/r
>> epos/asf/flex-utilities/repo
>> > > > > Commit: http://git-wip-us.apache.org/repos/asf/flex-utilities/
>> > > > > commit/59664053
>> > > > > Tree: http://git-wip-us.apache.org/repos/asf/flex-utilities/tree/
>> > > > 59664053
>> > > > > Diff: http://git-wip-us.apache.org/repos/asf/flex-utilities/diff/
>> > > > 59664053
>> > > > >
>> > > > > Branch: refs/heads/develop
>> > > > > Commit: 59664053cea8436c2d4a6f0b7df10c619f040826
>> > > > > Parents: e5c3d36
>> > > > > Author: Josh Tynjala <joshtynjala@apache.org>
>> > > > > Authored: Wed Aug 17 10:28:19 2016 -0700
>> > > > > Committer: Josh Tynjala <joshtynjala@apache.org>
>> > > > > Committed: Wed Aug 17 10:28:19 2016 -0700
>> > > > >
>> > > > > ------------------------------------------------------------
>> > ----------
>> > > > >  .../dependencies/download_dependencies.js       | 29
>> > > > ++++++++++++++++++++
>> > > > >  npm-flexjs/package.json                         |  1 +
>> > > > >  2 files changed, 30 insertions(+)
>> > > > > ------------------------------------------------------------
>> > ----------
>> > > > >
>> > > > >
>> > > > > http://git-wip-us.apache.org/repos/asf/flex-utilities/blob/
>> > > > > 59664053/npm-flexjs/dependencies/download_dependencies.js
>> > > > > ------------------------------------------------------------
>> > ----------
>> > > > > diff --git a/npm-flexjs/dependencies/download_dependencies.js
>> > > > > b/npm-flexjs/dependencies/download_dependencies.js
>> > > > > index e8fa370..3858049 100644
>> > > > > --- a/npm-flexjs/dependencies/download_dependencies.js
>> > > > > +++ b/npm-flexjs/dependencies/download_dependencies.js
>> > > > > @@ -21,6 +21,7 @@
>> > > > >
>> > > > >  var fs = require('fs');
>> > > > >  var mkdirp = require('mkdirp');
>> > > > > +var eol = require('eol');
>> > > > >  var constants = require('./Constants');
>> > > > >  var adobeair = require('./AdobeAIR');
>> > > > >  var flashplayerglobal = require('./FlashPlayerGlobal');
>> > > > > @@ -30,6 +31,7 @@ var swfObject = require('./SWFObject');
>> > > > >  var flatUI = require('./FlatUI');
>> > > > >
>> > > > >  var installSteps = [
>> > > > > +    updateScriptEOL,
>> > > > >      createDownloadsDirectory,
>> > > > >      installFlatUI,
>> > > > >      installFlashPlayerGlobal,
>> > > > > @@ -45,6 +47,33 @@ function start()
>> > > > >      installSteps[0].call();
>> > > > >  }
>> > > > >
>> > > > > +function updateScriptEOL()
>> > > > > +{
>> > > > > +    try
>> > > > > +    {
>> > > > > +        var files =
>> > > > > +        [
>> > > > > +            'js/bin/asjscnpm',
>> > > > > +            'js/bin/asjscompcnpm',
>> > > > > +            'js/bin/mxmlcnpm'
>> > > > > +        ];
>> > > > >
>> > > >
>> > > > Since we are running this first, can we safely assume that all the
>> > files
>> > > > that are under js/bin need to be fixed?  That way, any new files
>> added
>> > > > later will not require a code change here.
>> > > >
>> > > > Thanks,
>> > > > Om
>> > > >
>> > > >
>> > > >
>> > > > > +        do
>> > > > > +        {
>> > > > > +            var file = files.shift();
>> > > > > +            var data = fs.readFileSync(file, {encoding: 'utf8'});
>> > > > > +            data = eol.lf(data);
>> > > > > +            fs.writeFileSync(file, data, {encoding: 'utf8', mode:
>> > > > 0o755});
>> > > > > +        }
>> > > > > +        while(files.length > 0)
>> > > > > +    }
>> > > > > +    catch(e)
>> > > > > +    {
>> > > > > +        handleAbort();
>> > > > > +        return;
>> > > > > +    }
>> > > > > +    handleInstallStepComplete();
>> > > > > +}
>> > > > > +
>> > > > >  function createDownloadsDirectory()
>> > > > >  {
>> > > > >      //Create downloads directory if it does not exist already
>> > > > >
>> > > > > http://git-wip-us.apache.org/repos/asf/flex-utilities/blob/
>> > > > > 59664053/npm-flexjs/package.json
>> > > > > ------------------------------------------------------------
>> > ----------
>> > > > > diff --git a/npm-flexjs/package.json b/npm-flexjs/package.json
>> > > > > index 6181aea..0fd08b3 100644
>> > > > > --- a/npm-flexjs/package.json
>> > > > > +++ b/npm-flexjs/package.json
>> > > > > @@ -16,6 +16,7 @@
>> > > > >    "author": "OmPrakash Muppirala <bigosmallm@apache.org>",
>> > > > >    "license": "Apache-2.0",
>> > > > >    "dependencies": {
>> > > > > +    "eol": "^0.2.0",
>> > > > >      "fs-extra": "^0.26.3",
>> > > > >      "merge-dirs": "^0.2.1",
>> > > > >      "mkdirp": "^0.5.1",
>> > > > >
>> > > > >
>> > > >
>> > >
>> >
>>
>
>

--001a114a9fe4f29661053a4a1110--