Skip to content

Commit

Permalink
fix(app-webpack): CORDOVA mode > build -> correctly reference quasarC…
Browse files Browse the repository at this point in the history
…onf #17138
  • Loading branch information
rstoenescu committed May 6, 2024
1 parent e673e95 commit e42e7c7
Show file tree
Hide file tree
Showing 2 changed files with 15 additions and 15 deletions.
2 changes: 1 addition & 1 deletion app-webpack/lib/cmd/dev.js
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@ if (process.env.NODE_ENV === void 0) {

const parseArgs = require('minimist')

const { log, warn, fatal } = require('../helpers/logger')
const { log, fatal } = require('../helpers/logger')

const argv = parseArgs(process.argv.slice(2), {
alias: {
Expand Down
28 changes: 14 additions & 14 deletions app-webpack/lib/cordova/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -46,8 +46,8 @@ class CordovaRunner {
}

async run (quasarConfFile, argv) {
const cfg = quasarConfFile.quasarConf
const url = cfg.build.APP_URL
const { quasarConf } = quasarConfFile
const url = quasarConf.build.APP_URL

if (this.url === url) {
return
Expand All @@ -61,11 +61,11 @@ class CordovaRunner {

if (argv.ide) {
await this.__runCordovaCommand(
cfg,
quasarConf,
[ 'prepare', this.target ].concat(argv._)
)

await openIde('cordova', cfg.bin, this.target, true)
await openIde('cordova', quasarConf.bin, this.target, true)
return
}

Expand All @@ -76,21 +76,21 @@ class CordovaRunner {
}

await this.__runCordovaCommand(
cfg,
quasarConf,
args.concat(argv._)
)
}

async build (quasarConfFile, argv) {
const cfg = quasarConfFile.quasarConf
const { quasarConf } = quasarConfFile

const cordovaContext = {
debug: this.ctx.debug === true,
target: this.target
}

const outputTargetList = (
ensureArray(this.quasarConf.cordova.getCordovaBuildOutputFolder?.(cordovaContext))
ensureArray(quasarConf.cordova.getCordovaBuildOutputFolder?.(cordovaContext))
|| cordovaOutputFolders[ this.target ]
)

Expand All @@ -104,12 +104,12 @@ class CordovaRunner {
const args = argv[ 'skip-pkg' ] || argv.ide
? [ 'prepare', this.target ]
: (
this.quasarConf.cordova.getCordovaBuildParams?.(cordovaContext)
quasarConf.cordova.getCordovaBuildParams?.(cordovaContext)
|| [ 'build', this.ctx.debug ? '--debug' : '--release', '--device', this.target ]
)

await this.__runCordovaCommand(
cfg,
quasarConf,
args.concat(argv._)
)

Expand All @@ -118,11 +118,11 @@ class CordovaRunner {
}

if (argv.ide) {
await openIde('cordova', cfg.bin, this.target)
await openIde('cordova', quasarConf.bin, this.target)
process.exit(0)
}

const targetFolder = cfg.build.packagedDistDir
const targetFolder = quasarConf.build.packagedDistDir

for (const folder of outputTargetList) {
const outputFolder = appPaths.resolve.cordova(folder)
Expand Down Expand Up @@ -150,10 +150,10 @@ class CordovaRunner {
this.__cleanup()
}

__runCordovaCommand (cfg, args) {
this.cordovaConfig.prepare(cfg)
__runCordovaCommand (quasarConf, args) {
this.cordovaConfig.prepare(quasarConf)

if (this.target === 'ios' && cfg.cordova.noIosLegacyBuildFlag !== true) {
if (this.target === 'ios' && quasarConf.cordova.noIosLegacyBuildFlag !== true) {
args.push('--buildFlag=-UseModernBuildSystem=0')
}

Expand Down

0 comments on commit e42e7c7

Please sign in to comment.