fix: Optionally remove DISABLE_WAYLAND for snaps

[aripollak]

Fixes #6888

[changeset-bot]

🦋 Changeset detected

Latest commit: 41c5fa5

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 8 packages

Name	Type
app-builder-lib	Patch
dmg-builder	Patch
electron-builder-squirrel-windows	Patch
electron-builder	Patch
electron-forge-maker-appimage	Patch
electron-forge-maker-nsis-web	Patch
electron-forge-maker-nsis	Patch
electron-forge-maker-snap	Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[netlify]

✅ Deploy Preview for car-park-attendant-cleat-11576 ready!

Name	Link
🔨 Latest commit	41c5fa5
🔍 Latest deploy log	https://app.netlify.com/sites/car-park-attendant-cleat-11576/deploys/62c26f62b745a100080f6736
😎 Deploy Preview	https://deploy-preview-6961--car-park-attendant-cleat-11576.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify site settings.

[mmaietta]

This feels like a breaking change for electron-builder. I'm wondering if we can make this a feature instead by adding allowNativeWayland as a property in SnapOptions.

allowNativeWayland: boolean | null = null

Then DISABLE_WAYLAND: Number(!allowNativeWayland)

What are your thoughts?

[aripollak]

Unfortunately I'm not sure if this would be a breaking change, but I'm happy to add an option to enable it in the meantime. Maybe in the future it could be enabled by default if a minimum known-working electron version is used.

[aripollak]

I'll have to use some different logic for setting DISABLE_WAYLAND because this expects it to be unset/"" for false, or a non-empty value for true.

[aripollak]

Okay, I added that option. I wasn't able to update the docs locally because I haven't been able to get my local environment set up to compile this properly (pnpm install is complaining about yarn exiting with status 1).