DioxusLabs / dioxus

Fullstack app framework for web, desktop, mobile, and more.
https://dioxuslabs.com
Apache License 2.0
21.69k stars 833 forks source link

Desktop: Error setting the App Icon #2344

Closed Anonyfox closed 1 month ago

Anonyfox commented 7 months ago

Problem

I want to set the app icon on my desktop app (Mac OS). I didn't find something related to that in the docs or the existing issues so far, but stumbled upon the setting in the Dioxus.tml that can be set:

[bundle]
icon = ["icon.icns"]

I tried it with a .png or the .icns file (created properly from that png) and various ways of setting the path. What happens is either a "file not found" bug or

Failed to bundle project: IoError(
    Os {
        code: 63,
        kind: InvalidFilename,
        message: "File name too long",
    },
)

Steps To Reproduce

Steps to reproduce the behavior:

Expected behavior

A clear and concise description of what you expected to happen.

The app should have the correct icon

Environment:

Andrew15-5 commented 4 months ago

Can you try the same thing, but with the latest git version? It works with bd58a9244, but not with v0.5.2 on Linux.

╰─ dx-0.5.2 bundle -r
build desktop done
Adding assets from dist to bundle
thread 'main' panicked at packages/cli/src/cli/bundle.rs:193:13:
Failed to bundle project: BundlerError(
    Error {
        context: "Failed to build data folders and files",
        source: BundlerError(
            Error {
                context: "Failed to copy resource files",
                source: IoError(
                    Os {
                        code: 36,
                        kind: InvalidFilename,
                        message: "File name too long",
                    },
                ),
            },
        ),
    },
)
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
NTL00068 commented 2 months ago

On MacOS, I have observed that deleting target and dist folders, then re-running dx bundle, solves the problem.

jkelleyrtp commented 1 month ago

Should be fixed now that we've upgraded our tauri bundle support