Windows fixes:
ClosedPublic

Authored by arrowd on Feb 8 2017, 12:49 PM.

Details

Reviewers
mludwig
Group Reviewers
Kile
Commits
R468:9b2b4478263f: Windows fixes:
Summary
  • Don't exit application if KDBusService has failed.
  • Don't confuse Windows\system32\convert.exe utility with ImageMagick's one.
Test Plan

"System Check" now works on Windows and all-green with MikTeX installed.

Diff Detail

Repository
R468 Kile
Branch
windows_fixes (branched from master)
Lint
No Linters Available
Unit
No Unit Test Coverage
arrowd updated this revision to Diff 11060.Feb 8 2017, 12:49 PM
arrowd retitled this revision from to Windows fixes:.
arrowd updated this object.
arrowd edited the test plan for this revision. (Show Details)
arrowd added a reviewer: Kile.
Restricted Application added a project: Kile. · View Herald TranscriptFeb 8 2017, 12:49 PM
mludwig added inline comments.
src/configtester.cpp
132

I find it a little surprising that a distinction is necessary here.

What location is returned for QStandardPaths::DataLocation?

Can you try to use to use QStandardPaths::AppDataLocation?

src/main.cpp
147

Do you know what the status of dbus for Windows is?

Running two instances of Kile is actually quite risky when it comes to configuration changes.

arrowd added inline comments.Feb 12 2017, 6:18 PM
src/configtester.cpp
132

From the official docs:

On Linux:
DataLocation "~/.local/share/<APPNAME>", "/usr/local/share/<APPNAME>", "/usr/share/<APPNAME>"

On Windows:
"C:/Users/<USER>/AppData/Local/<APPNAME>", "C:/ProgramData/<APPNAME>", "<APPDIR>", "<APPDIR>/data"

That "test" dir ends up in "data/kile" because ECM sets KDE_INSTALL_DIR or whatever that variable is called to "bin/data/<APPNAME>" on Windows. Not sure who is wrong there, but many other apps are working somehow.

Using AppDataLocation wouldn't help there, as it differs only in "Local" part.

src/main.cpp
147

Hum, well, I'm just not packaging DBus on Windows at all, so I need a way to start an appliction without DBus.

KDevelop, for instance, use QSingleApplication to ensure only one process is running.

mludwig accepted this revision.Feb 13 2017, 8:06 PM
mludwig added a reviewer: mludwig.
mludwig added inline comments.
src/configtester.cpp
132

Thanks for your explanations!

According to the docs, DataLocation is deprecated - I guess we should change it to AppDataLocation.

src/main.cpp
147

Ok, Kile will also have to use QtSingleApplication on Windows / Mac then.

This revision is now accepted and ready to land.Feb 13 2017, 8:06 PM
arrowd closed this revision.Jun 5 2017, 10:02 AM
arrowd marked 2 inline comments as done.