Skip to content

Conversation

@keith-packard
Copy link

I'm packaging this for debian and have a few minor changes that I've made:

  1. Use QSettings(org, app) instead of QSettings(filename). This lets Qt pick how to store settings, rather than storing them in the same directory as the application binary (which doesn't work on a multi-user system).
  2. Use default settings if none were stored. I just used the code in frmsettings.cpp.
  3. Allow any filename on the command line. My gcode files have a lot of random extensions.

Thanks for a really useful app; I'm no longer dependent on a Windows VM to drive my laser and router.

Apply target-specific Qt code about how to store application
preference values by specifying the organization and application name
rather than a file name

Signed-off-by: Keith Packard <[email protected]>
If there are no application settings available, then
apply the default values.

Signed-off-by: Keith Packard <[email protected]>
Don't filter based upon assumptions about gcode file extensions

Signed-off-by: Keith Packard <[email protected]>
@Hostaler
Copy link

Hostaler commented Jun 13, 2025

Hello, I have version 1.2b, I don't know anything about computers, but I've read that you're creating some patches or you already have them created .Will your patches also work for version 1.2b?

@keith-packard
Copy link
Author

I don't know; I didn't try applying them to the development version.

@github-actions
Copy link

This PR has been automatically marked as stale due to inactivity.

@github-actions github-actions bot added the Stale label Aug 22, 2025
@MarcFontaine
Copy link

I'm updating the NixOS package for Candle.
Storing the application settings in the usual place is crucial for packaging Candle.

MarcFontaine added a commit to MarcFontaine/nixpkgs that referenced this pull request Sep 10, 2025
* Update candle to 1.1.8.
* Apply Denvi/Candle#658.
  PR 658 enables the user to save application setting.
MarcFontaine added a commit to MarcFontaine/nixpkgs that referenced this pull request Sep 11, 2025
* Update candle to 1.1.8.
* Add a patch that applies Denvi/Candle#658.
  PR 658 enables the user to save application setting.
@github-actions github-actions bot removed the Stale label Sep 16, 2025
MarcFontaine added a commit to MarcFontaine/nixpkgs that referenced this pull request Sep 16, 2025
* Update candle to 1.1.8.
* Add a patch that applies Denvi/Candle#658.
  PR 658 enables the user to save application setting.
MarcFontaine added a commit to MarcFontaine/nixpkgs that referenced this pull request Sep 16, 2025
* Update candle to 1.1.8.
* Add a patch that applies Denvi/Candle#658.
  PR 658 enables the user to save application setting.
@github-actions
Copy link

github-actions bot commented Nov 4, 2025

This PR has been automatically marked as stale due to inactivity.

@github-actions github-actions bot added the Stale label Nov 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants