Skip to content

Conversation

@WSH032
Copy link
Member

@WSH032 WSH032 commented Sep 13, 2025

This was discovered in #21: https://github.com/pytauri/create-pytauri-app/actions/runs/17697765693/job/50299377278?pr=21#step:15:119

Unlike Vue and React, Svelte doesn't follow Vite's build.outDir configuration. Its output directory is specified by adapter-static, which defaults to build instead of dist. This caused build failures because build.frontendDist in tauri.conf.json was set to ../dist.

In fact, create-tauri-app does specifically set ../build as frontendDist for Svelte. However, for the existing architecture of create-pytauri-app, I don't want to create a special case for svelte in templates/./{{ project_name }}. So I chose to change the output directory in svelte.config.js to dist.

CC @ISOR3X, since you are the author of the Svelte template.

@WSH032 WSH032 requested a review from ISOR3X September 13, 2025 15:22
@ISOR3X
Copy link
Collaborator

ISOR3X commented Sep 15, 2025

Looks good to me, nice catch 👍

@ISOR3X ISOR3X closed this Sep 15, 2025
@ISOR3X ISOR3X reopened this Sep 15, 2025
@ISOR3X ISOR3X added this pull request to the merge queue Sep 15, 2025
Merged via the queue into main with commit 59570c8 Sep 15, 2025
24 checks passed
@ISOR3X ISOR3X deleted the fix/svelte branch September 15, 2025 07:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants