Skip to content

Commit

Permalink
fix: Build assets only for downloaded app (frappe#802)
Browse files Browse the repository at this point in the history
  • Loading branch information
netchampfaris authored May 15, 2019
1 parent 3374cbe commit 75b0901
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 3 deletions.
2 changes: 1 addition & 1 deletion bench/app.py
Original file line number Diff line number Diff line change
Expand Up @@ -137,7 +137,7 @@ def get_app(git_url, branch=None, bench_path='.', build_asset_files=True, verbos
if postprocess:

if build_asset_files:
build_assets(bench_path=bench_path)
build_assets(bench_path=bench_path, app=app_name)
conf = get_config(bench_path=bench_path)

if conf.get('restart_supervisor_on_update'):
Expand Down
7 changes: 5 additions & 2 deletions bench/utils.py
Original file line number Diff line number Diff line change
Expand Up @@ -194,13 +194,16 @@ def patch_sites(bench_path='.'):
except subprocess.CalledProcessError:
raise PatchError

def build_assets(bench_path='.'):
def build_assets(bench_path='.', app=None):
bench.set_frappe_version(bench_path=bench_path)

if bench.FRAPPE_VERSION == 4:
exec_cmd("{frappe} --build".format(frappe=get_frappe(bench_path=bench_path)), cwd=os.path.join(bench_path, 'sites'))
else:
run_frappe_cmd('build', bench_path=bench_path)
command = 'bench build'
if app:
command += ' --app {}'.format(app)
exec_cmd(command, cwd=bench_path)

def get_sites(bench_path='.'):
sites_dir = os.path.join(bench_path, "sites")
Expand Down

0 comments on commit 75b0901

Please sign in to comment.