Skip to content

Commit

Permalink
fix: frappe app can exist without public folder
Browse files Browse the repository at this point in the history
other change:
* sync apps.txt manually after dep apps are also setup; right before
  build step
gavindsouza committed Nov 15, 2021
1 parent 5170048 commit 13ed9be
Showing 2 changed files with 3 additions and 2 deletions.
3 changes: 2 additions & 1 deletion bench/app.py
Original file line number Diff line number Diff line change
@@ -355,14 +355,15 @@ def install_app(
exec_cmd("yarn install", cwd=app_path)

bench = Bench(bench_path)
bench.apps.sync()

conf = bench.conf
if conf.get("developer_mode"):
from bench.utils.bench import install_python_dev_dependencies

install_python_dev_dependencies(apps=app)

bench.apps.sync()

if not skip_assets:
build_assets(bench_path=bench_path, app=app)

2 changes: 1 addition & 1 deletion bench/utils/__init__.py
Original file line number Diff line number Diff line change
@@ -19,7 +19,7 @@

logger = logging.getLogger(PROJECT_NAME)
bench_cache_file = ".bench.cmd"
paths_in_app = ("hooks.py", "modules.txt", "patches.txt", "public")
paths_in_app = ("hooks.py", "modules.txt", "patches.txt")
paths_in_bench = ("apps", "sites", "config", "logs", "config/pids")
sudoers_file = "/etc/sudoers.d/frappe"

0 comments on commit 13ed9be

Please sign in to comment.