Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(@ngtools/webpack): prevent emitting sourcemaps #4221

Merged
merged 1 commit into from
Feb 2, 2017

Conversation

dobbydog
Copy link
Contributor

In some cases compilation of a component style generates a sourcemap and it's not deleted from assets.
This would cause hangs in Windows and make unnecessary outputs in other platform.

may fix #3019

@filipesilva
Copy link
Contributor

@hansl can you review?

@dobbydog dobbydog force-pushed the fix-emit-sourcemaps branch from e2fd262 to d0ed70f Compare January 25, 2017 13:35
@dobbydog dobbydog changed the title fix(@ngtools/webpack): prevent emitting of sourcemaps fix(@ngtools/webpack): prevent emitting sourcemaps Jan 26, 2017
// If it wasn't there - delete it.
delete this._parentCompilation.assets[outputName];
}
Object.keys(childCompilation.assets).forEach((filename) => {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nit: filename should be fileName, as it is two words.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done.

dobbydog added a commit to dobbydog/angular-cli that referenced this pull request Feb 1, 2017
In some cases compilation of a component style generates a sourcemap and it's not deleted from assets.
This would be cause hangup in Windows.
@dobbydog dobbydog force-pushed the fix-emit-sourcemaps branch from 00e7ee5 to 3c41bcd Compare February 1, 2017 07:03
@hansl hansl merged commit a6b1bdd into angular:master Feb 2, 2017
@dobbydog dobbydog deleted the fix-emit-sourcemaps branch February 6, 2017 02:08
MRHarrison pushed a commit to MRHarrison/angular-cli that referenced this pull request Feb 9, 2017
In some cases compilation of a component style generates a sourcemap and it's not deleted from assets.

This would be cause hangup in Windows.
@angular-automatic-lock-bot
Copy link

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators Sep 11, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Windows ngtools/webpack AoT compilation hangs infinitely at 95% emitting
4 participants