Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fixes and refactors linux build scripts #8269

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

rubienr
Copy link

@rubienr rubienr commented Feb 1, 2025

Description

As a new OrcaSlicer user i found myself not being able to build from source on Ubuntu 24.10.

Details

  • build_linux.sh
    • fixes wrong AppImage build folder
    • refactors script
    • enhances help text
  • harmonizes names to comply with other build_*.(sh|bat) scripts
    • BuildLinux.sh and BuildLinuxImage.sh
  • fixes file permissions
    • cmake inherits *.in-file permissions for target files
    • removes necessity for chmod 755 in scripts
  • linux.d/debian
    • removes false positive error message
  • updates documentation
    • makes help text more understandable

Notes

  • does not introduce new features
  • Dockerfile depends on PR changes but fix should not break it
  • no UI changes

Tests

  • tested manually:
    1. checked out git repo
    2. installed system deps: ./build_linux.sh -u
    3. built deps: ./build_linux.sh -d
    4. built slicer: ./build_linux.sh -s
    5. built AppImage: ./build_linux.sh -i
    6. started: build/OrcaSlicer_Linux_V2.3.0-dev.AppImage; started successfully
    7. started: build/package/bin/orca-slicer; started successfully

@rubienr rubienr force-pushed the ubuntu-build-fixes branch 2 times, most recently from f825563 to 3617f4a Compare February 3, 2025 22:07
@rubienr rubienr closed this Feb 3, 2025
@rubienr rubienr reopened this Feb 3, 2025
@rubienr rubienr force-pushed the ubuntu-build-fixes branch from 3617f4a to 6eaab5e Compare February 4, 2025 00:57
@rubienr
Copy link
Author

rubienr commented Feb 4, 2025

Dear maintainer|s,

(@Noisyfox)

I would like to prepare the next PR (on top of this) where build_linux.sh can take an opt. arg to disable precompiled headers (SLIC3R_PCH=OFF) so that compiling with ccache can be broken down to about one minute.

Is there something i can do to help and make this PR review easier?

Kind regards,
RR

@rubienr rubienr force-pushed the ubuntu-build-fixes branch from 6eaab5e to acc5212 Compare February 6, 2025 17:14
@rubienr rubienr force-pushed the ubuntu-build-fixes branch 3 times, most recently from 5e37ea8 to c661915 Compare February 9, 2025 16:14
 build_linux.sh: fixes wrong AppImage build folder; refactors script; enhances help text
 harmonizes names: BuildLinux.sh and BuildLinuxImage.sh
 fixes file permissions: cmake inherits .in-file permisson; removes chmod 755 in scripts
 linux.d/debian: removes false positive error message
 updates documentation
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.

Post build AppRun script references wrong file path
2 participants