Skip to content

Conversation

qnixsynapse
Copy link
Contributor

@qnixsynapse qnixsynapse commented Aug 22, 2025

Describe Your Changes

The local build script for Linux was failing due to a bundling error. This commit updates the build:tauri:linux script in package.json to be consistent with the CI build pipeline, which resolves the issue.

The updated script now includes:

  • NO_STRIP=1: This environment variable prevents the linuxdeploy utility from stripping debugging symbols, which was a potential cause of the bundling failure.
  • --verbose: This flag provides more detailed output during the build, which can be useful for debugging similar issues in the future.

Fixes Issues

Self Checklist

  • Added relevant comments, esp in complex areas
  • Updated docs (for bug fixes / features)
  • Created issues for follow-up changes or refactoring needed

Important

Update build:tauri:linux script in package.json to fix bundling error by aligning with CI pipeline.

This description was created by Ellipsis for c734982. You can customize this summary. It will automatically update as commits are pushed.

The local build script for Linux was failing due to a bundling error. This commit updates the `build:tauri:linux` script in `package.json` to be consistent with the CI build pipeline, which resolves the issue.

The updated script now includes:
- **`NO_STRIP=1`**: This environment variable prevents the `linuxdeploy` utility from stripping debugging symbols, which was a potential cause of the bundling failure.
- **`--verbose`**: This flag provides more detailed output during the build, which can be useful for debugging similar issues in the future.
Copy link
Contributor

Barecheck - Code coverage report

Total: 36.1%

Your code coverage diff: -0.01% ▾

✅ All code changes are covered

@qnixsynapse qnixsynapse linked an issue Aug 22, 2025 that may be closed by this pull request
@qnixsynapse qnixsynapse merged commit 39e8d3b into release/v0.6.9 Aug 22, 2025
17 checks passed
@qnixsynapse qnixsynapse deleted the fix/6010 branch August 22, 2025 03:52
@github-project-automation github-project-automation bot moved this to QA in Jan Aug 22, 2025
@github-actions github-actions bot added this to the v0.6.9 milestone Aug 22, 2025
Copy link
Contributor

@ellipsis-dev ellipsis-dev bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Important

Looks good to me! 👍

Reviewed everything up to c734982 in 20 minutes and 17 seconds. Click for details.
  • Reviewed 13 lines of code in 1 files
  • Skipped 0 files when reviewing.
  • Skipped posting 2 draft comments. View those below.
  • Modify your settings and rules to customize what types of comments Ellipsis leaves. And don't forget to react with 👍 or 👎 to teach Ellipsis.
1. package.json:25
  • Draft comment:
    The updated 'build:tauri:linux' script now sets NO_STRIP=1 and adds --verbose, aligning it with the CI pipeline. Ensure that shim-linuxdeploy.sh properly handles the --verbose flag.
  • Reason this comment was not posted:
    Confidence changes required: 0% <= threshold 50% None
2. package.json:25
  • Draft comment:
    Verify that NO_STRIP=1 is only needed for shim-linuxdeploy.sh. If buildAppImage.sh (or other steps) requires similar flags, adjust accordingly.
  • Reason this comment was not posted:
    Confidence changes required: 33% <= threshold 50% None

Workflow ID: wflow_HRa6HI0b6I33hkST

You can customize Ellipsis by changing your verbosity settings, reacting with 👍 or 👎, replying to comments, or adding code review rules.

dinhlongviolin1 added a commit that referenced this pull request Aug 26, 2025
* fix: check for env value before setting (#6266)

* fix: check for env value before setting

* Use empty instead of none

* fix: update linux build script to be consistent with CI (#6269)

The local build script for Linux was failing due to a bundling error. This commit updates the `build:tauri:linux` script in `package.json` to be consistent with the CI build pipeline, which resolves the issue.

The updated script now includes:
- **`NO_STRIP=1`**: This environment variable prevents the `linuxdeploy` utility from stripping debugging symbols, which was a potential cause of the bundling failure.
- **`--verbose`**: This flag provides more detailed output during the build, which can be useful for debugging similar issues in the future.

* fix: compatibility imported model

* fix: update copy mmproj setting desc

* fix: toggle vision for remote model

* chore: add tooltip visions

* chore: show model setting only for local provider

* fix/update-ui-info

* chore: update filter hub while searching

* fix: system monitor window permission

* chore: update credit description

---------

Co-authored-by: Akarshan Biswas <[email protected]>
Co-authored-by: Faisal Amir <[email protected]>
Co-authored-by: Minh141120 <[email protected]>
Co-authored-by: Nguyen Ngoc Minh <[email protected]>
dinhlongviolin1 added a commit that referenced this pull request Aug 28, 2025
* fix: check for env value before setting (#6266)

* fix: check for env value before setting

* Use empty instead of none

* fix: update linux build script to be consistent with CI (#6269)

The local build script for Linux was failing due to a bundling error. This commit updates the `build:tauri:linux` script in `package.json` to be consistent with the CI build pipeline, which resolves the issue.

The updated script now includes:
- **`NO_STRIP=1`**: This environment variable prevents the `linuxdeploy` utility from stripping debugging symbols, which was a potential cause of the bundling failure.
- **`--verbose`**: This flag provides more detailed output during the build, which can be useful for debugging similar issues in the future.

* fix: compatibility imported model

* fix: update copy mmproj setting desc

* fix: toggle vision for remote model

* chore: add tooltip visions

* chore: show model setting only for local provider

* fix/update-ui-info

* chore: update filter hub while searching

* fix: system monitor window permission

* chore: update credit description

* chore: bundle license to app

* chore: bundle license to resource mac

* chore: bundle license to linux dist

* chore: relocate LICENSE to resources folder

* fix: handle paste image on linux

* fix: handle copy image from browser in linux

* feat: add regression checklist

* fix: mcp cleanup dropodown tool availabel and sort list

* fix: sort list when add server

* fix: mcp sort list

* fix: handle conditional UI regenerate resp

* fix: code generation more than 300 line

* fix: handle checking compatible gated model

* chore: update test

* chore: remove log

* chore: fix status

* chore: fix status model id

* make validation message infinite (#6318)

---------

Co-authored-by: Akarshan Biswas <[email protected]>
Co-authored-by: Faisal Amir <[email protected]>
Co-authored-by: Minh141120 <[email protected]>
Co-authored-by: Nguyen Ngoc Minh <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: QA
Development

Successfully merging this pull request may close these issues.

devops: standardize local build vs ci build on Linux
3 participants