On Wed, Aug 17, 2016 at 10:28 AM, <joshtynj...@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/repos/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 <joshtynj...@apache.org>
> Authored: Wed Aug 17 10:28:19 2016 -0700
> Committer: Josh Tynjala <joshtynj...@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 <bigosma...@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",
>
>
