Skip to content

Improvements to Dockerfile, build script, and configuration for Aseprite compilation - #24

Open
loopyd wants to merge 3 commits into
nilsve:masterfrom
loopyd:master
Open

Improvements to Dockerfile, build script, and configuration for Aseprite compilation#24
loopyd wants to merge 3 commits into
nilsve:masterfrom
loopyd:master

Conversation

@loopyd

@loopyd loopyd commented Nov 15, 2025

Copy link
Copy Markdown

This PR makes the following changes:

  • Updated Dockerfile with updated slim trixie python base image and add build arguments/environment variable passthrough to compile.sh w/ more reasonably scoped build context.
  • Updated compile.sh script with enhanced log streaming, no color, verbosity and quiet options, error, trap/break handling and cleanup, as well as get build working again.
  • Fix for vscode: Discard popup, created settings.json to configure SARIF viewer
  • Move license from README.md to LICENSE file for standardized project licensing.
  • Modified Makefile to add clean command to build fresh.
  • Updated docker-compose.yml for better context and arguments.

With this, a working modern build of Aesprite on Linux is possible again.

This PR has been tested for quality assurance (see #24 (comment))

- Introduced Dockerfile with updated base image and environment variables.
- Added compile.sh script with enhanced logging and error handling.
- Created settings.json to configure SARIF viewer.
- Added LICENSE file for project licensing.
- Updated README.md with detailed usage instructions and examples.
- Modified Makefile for improved build commands.
- Updated docker-compose.yml for better context and arguments.
@loopyd loopyd changed the title Add Dockerfile, scripts, and configuration for Aseprite compilation Improvements to Dockerfile, build script, and configuration for Aseprite compilation Nov 15, 2025
@loopyd

loopyd commented Nov 15, 2025

Copy link
Copy Markdown
Author

Compiling Release verison of Aesprite using make ARGS option:

image image

Result (./output/bin/aesprite)

image

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.

1 participant