js: Remove external source map option

It was merged by accident and the implementation does not look correct.

Fixes #7932
This commit is contained in:
Bjørn Erik Pedersen 2020-11-04 19:09:40 +01:00
parent bf2837a314
commit 944150bafb
No known key found for this signature in database
GPG key ID: 330E6E2BD4859D8F
3 changed files with 3 additions and 11 deletions

View file

@ -78,8 +78,7 @@ func (t *buildTransformation) Transform(ctx *resources.ResourceTransformationCtx
}
opts.sourcefile = ctx.SourcePath
opts.workDir = t.c.rs.WorkingDir
opts.resolveDir = opts.workDir
opts.resolveDir = t.c.rs.WorkingDir
opts.contents = string(src)
opts.mediaType = ctx.InMediaType

View file

@ -93,7 +93,6 @@ type Options struct {
contents string
sourcefile string
resolveDir string
workDir string
tsConfig string
}
@ -365,12 +364,6 @@ func toBuildOptions(opts Options) (buildOptions api.BuildOptions, err error) {
switch opts.SourceMap {
case "inline":
sourceMap = api.SourceMapInline
case "external":
// When doing external sourcemaps we should specify
// out file and no out dir
sourceMap = api.SourceMapExternal
outFile = filepath.Join(opts.workDir, opts.TargetPath)
outDir = ""
case "":
sourceMap = api.SourceMapNone
default:

View file

@ -93,7 +93,7 @@ func TestToBuildOptions(t *testing.T) {
opts, err = toBuildOptions(Options{
Target: "es2018", Format: "cjs", Minify: true, mediaType: media.JavascriptType,
SourceMap: "external"})
SourceMap: "inline"})
c.Assert(err, qt.IsNil)
c.Assert(opts, qt.DeepEquals, api.BuildOptions{
Bundle: true,
@ -102,7 +102,7 @@ func TestToBuildOptions(t *testing.T) {
MinifyIdentifiers: true,
MinifySyntax: true,
MinifyWhitespace: true,
Sourcemap: api.SourceMapExternal,
Sourcemap: api.SourceMapInline,
Stdin: &api.StdinOptions{
Loader: api.LoaderJS,
},