Skip to content

Fix icon build issues and update author information#3

Merged
lpolish merged 1 commit intomainfrom
staging
Jun 10, 2025
Merged

Fix icon build issues and update author information#3
lpolish merged 1 commit intomainfrom
staging

Conversation

@lpolish
Copy link
Owner

@lpolish lpolish commented Jun 10, 2025

Summary

This PR fixes critical build issues that were preventing successful creation of distributable packages across all platforms.

🔧 Issues Fixed

Icon Format Error

  • Problem: The original assets/icon.png was a placeholder text file, not an actual image
  • Error: image /path/to/icon.png shas unknown format during AppImage build
  • Solution: Replaced with actual PNG image from project root

Missing Platform-Specific Icons

  • Problem: Missing .ico and .icns files for Windows and macOS builds
  • Solution: Generated proper icon files for all platforms

📝 Changes Made

Icon Files

  • assets/icon.png - Valid PNG for Linux (AppImage/DEB packages)
  • assets/icon.ico - Windows ICO format for NSIS installer
  • assets/icon.icns - macOS ICNS format for DMG packages

Package Configuration

  • ✅ Updated author information to Luis Pulido Diaz luis@luispulido.com
  • ✅ Maintained proper electron-builder configuration for all platforms

🧪 Testing

  • ✅ Linux build tested locally - both AppImage and DEB packages generated successfully
  • ✅ No more 'unknown format' errors
  • ✅ All icon references properly resolved

🎯 Impact

This resolves the build failures in CI/CD pipelines and enables successful creation of:

  • Windows NSIS installers (.exe)
  • macOS DMG packages (.dmg)
  • Linux AppImage (.AppImage)
  • Linux DEB packages (.deb)

📋 Verification

Before merge, the GitHub Actions workflows should complete successfully for all platform builds.


Type: Bug Fix
Priority: High
Platforms: Windows, macOS, Linux

- Replace placeholder icon.png with actual image file
- Add proper icon.ico for Windows builds
- Add proper icon.icns for macOS builds
- Update author to Luis Pulido Diaz <luis@luispulido.com>
- Resolves electron-builder 'unknown format' error during Linux builds
@lpolish lpolish merged commit 726fe3f into main Jun 10, 2025
6 checks passed
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