Skip to content

Commit

Permalink
fix: add content to launcher binary
Browse files Browse the repository at this point in the history
  • Loading branch information
remarkablemark committed Jun 10, 2024
1 parent 57aeebb commit 650294e
Show file tree
Hide file tree
Showing 4 changed files with 16 additions and 6 deletions.
12 changes: 11 additions & 1 deletion .github/workflows/build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -86,6 +86,11 @@ jobs:
command -v renpy-cli
command -v renpy-launcher
- name: Show launcher
run: |
ls -alt $(command -v renpy-launcher)
cat $(command -v renpy-launcher)
- name: Check version
if: matrix.os != 'windows-latest'
run: |
Expand All @@ -106,4 +111,9 @@ jobs:
run: renpy-cli ${{ steps.renpy.outputs.launcher }} set_project .

- name: Check web support
run: ls ${{ steps.renpy.outputs.launcher}}/../web/index.html
run: ls -alt ${{ steps.renpy.outputs.launcher}}/../web/index.html

- name: Build web
run: |
renpy-launcher web_build ${{ steps.renpy.outputs.launcher }}/../the_question --destination dist
ls -alt dist dist.zip
2 changes: 1 addition & 1 deletion dist/index.js

Large diffs are not rendered by default.

4 changes: 2 additions & 2 deletions src/index.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -72,8 +72,8 @@ describe.each([
]);

expect(mockedExec.exec).toHaveBeenCalledWith('echo', [
expect.stringMatching(/.+cli-name(.exe)? .+launcher .+[$@].+/),
'>',
expect.stringMatching(/^'.+cli-name(.exe)? .+launcher .+[$@].+'$/),
'>>',
expect.stringContaining(launcherName),
]);

Expand Down
4 changes: 2 additions & 2 deletions src/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -60,8 +60,8 @@ export async function run() {
const launcherPath = getLauncherPath(binaryDirectory, launcherName);
await exec('touch', [launcherPath]);
await exec('echo', [
`${binaryPath} ${launcherDirectory} "$@"`,
'>',
`'${binaryPath} ${launcherDirectory} "$@"'`,
'>>',
launcherPath,
]);
await exec('chmod', ['+x', launcherPath]);
Expand Down

0 comments on commit 650294e

Please sign in to comment.