https://bugs.kde.org/show_bug.cgi?id=457951

--- Comment #14 from Friedrich W. H. Kossebau <kosse...@kde.org> ---
Thanks again for testing.

(In reply to Duncan from comment #10)
> (In reply to Friedrich W. H. Kossebau from comment #9)
> > https://invent.kde.org/games/libkdegames/-/merge_requests/20.diff
> 
> Good and bad news:
> Good:   The relative style ksudokurc theme entry works with that patch.

Okay :)

> Bad: The full-path style does not.
> 
> If there hasn't been a release with the now-existing code that creates the
> full-path entry I imagine the bad news can be ignored (unless we want to
> give people hand-editing their ksudokurc files the choice of doing full
> paths, too).

No release of ksudoko using the new code yet, though sadly 22.08.0 release is
currently prepared for tomorrow release and would be the first version of
ksudoko with the new code path. libkdegames has seen a quite some releases with
the id screwing logic (was added in 2014).

I would agree that the most simple thing would be to just fallback to the
default theme if the config holds a theme id that cannot be resolved.

Most places also already do that, so those games can at least be used for
gaming. Though ksudoku needs a fix-up for handling that situation, blame on me
here, no idea why I failed to consider bad config entries. Local patch already
prepared and tested, landing next.

Will see to have the prepared 22.08.0 tarballs updated in time for tomorrow
now.

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to