Skip to content

Commit 23481bd

Browse files
committed
fix(cordova): obey --source-map for production builds
Fixes #93
1 parent 8c7a597 commit 23481bd

File tree

2 files changed

+13
-2
lines changed

2 files changed

+13
-2
lines changed

builders/cordova-build/index.ts

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -47,6 +47,8 @@ export class CordovaBuildBuilder implements Builder<CordovaBuildBuilderSchema> {
4747
prepareBrowserConfig(options: CordovaBuildBuilderSchema, browserOptions: BrowserBuilderSchema) {
4848
const cordovaBasePath = normalize(options.cordovaBasePath ? options.cordovaBasePath : '.');
4949

50+
browserOptions.sourceMap = options.sourceMap as any;
51+
5052
// We always need to output the build to `www` because it is a hard
5153
// requirement of Cordova.
5254
browserOptions.outputPath = join(cordovaBasePath, normalize('www'));

builders/cordova-serve/index.ts

Lines changed: 11 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -37,9 +37,18 @@ export class CordovaServeBuilder implements Builder<CordovaServeBuilderSchema> {
3737
}
3838

3939
protected _getCordovaBuildConfig(cordovaServeOptions: CordovaServeBuilderSchema): Observable<BuilderConfiguration<CordovaBuildBuilderSchema>> {
40-
const { platform, cordovaBasePath, cordovaAssets, cordovaMock, consolelogs, consolelogsPort } = cordovaServeOptions;
40+
const {
41+
platform,
42+
cordovaBasePath,
43+
cordovaAssets,
44+
cordovaMock,
45+
consolelogs,
46+
consolelogsPort,
47+
sourceMap,
48+
} = cordovaServeOptions;
49+
4150
const [ project, target, configuration ] = cordovaServeOptions.cordovaBuildTarget.split(':');
42-
const cordovaBuildTargetSpec = { project, target, configuration, overrides: { platform, cordovaBasePath, cordovaAssets, cordovaMock, consolelogs, consolelogsPort } };
51+
const cordovaBuildTargetSpec = { project, target, configuration, overrides: { platform, cordovaBasePath, cordovaAssets, cordovaMock, consolelogs, consolelogsPort, sourceMap } };
4352
const cordovaBuildTargetConfig = this.context.architect.getBuilderConfiguration<CordovaBuildBuilderSchema>(cordovaBuildTargetSpec);
4453

4554
return this.context.architect.getBuilderDescription(cordovaBuildTargetConfig).pipe(

0 commit comments

Comments
 (0)