| Summary: | kile-2.1.2 (kde-4.8.90) regression: cannot open project with "kile file.kilepr" | ||
|---|---|---|---|
| Product: | [Applications] kile | Reporter: | Andreas K. Huettel <dilfridge> |
| Component: | general | Assignee: | Michel Ludwig <michel.ludwig> |
| Status: | RESOLVED FIXED | ||
| Severity: | normal | CC: | rdieter |
| Priority: | NOR | ||
| Version: | unspecified | ||
| Target Milestone: | --- | ||
| Platform: | Gentoo Packages | ||
| OS: | Linux | ||
| Latest Commit: | http://commits.kde.org/kile/0f44079d5ef7493587b35fb9edbca555de41539e | Version Fixed In: | |
| Sentry Crash Report: | |||
|
Description
Andreas K. Huettel
2012-06-13 11:26:53 UTC
Which version of Qt are you using? I'm using Qt 4.8.2 here and 'QDir::currentPath()' seems to return the wrong path... (In reply to comment #1) > Which version of Qt are you using? > > I'm using Qt 4.8.2 here and 'QDir::currentPath()' seems to return the wrong > path... Qt 4.8.2 here too... Git commit f0c365b46be65093d5b33864553df500a985cbfe by Michel Ludwig. Committed on 18/06/2012 at 22:29. Pushed by mludwig into branch '2.1'. Ensure that the current working directory isn't changed from within Kile Otherwise, projects that are to be opened and which are given as relative path arguments to Kile cannot be opened correctly. Also, remove the warning about running sessions which originates from Konsole. M +16 -12 src/main.cpp M +0 -2 src/widgets/konsolewidget.cpp http://commits.kde.org/kile/f0c365b46be65093d5b33864553df500a985cbfe Git commit 0f44079d5ef7493587b35fb9edbca555de41539e by Michel Ludwig. Committed on 18/06/2012 at 22:29. Pushed by mludwig into branch 'master'. Ensure that the current working directory isn't changed from within Kile Otherwise, projects that are to be opened and which are given as relative path arguments to Kile cannot be opened correctly. Also, remove the warning about running sessions which originates from Konsole. M +16 -12 src/main.cpp M +0 -2 src/widgets/konsolewidget.cpp http://commits.kde.org/kile/0f44079d5ef7493587b35fb9edbca555de41539e Just for the record, the problem was located within Kile (and due to a slightly wrong usage of the Konsole widget). 'QDir::currentPath()' was working fine. Awesome, thanks a lot. I've tried to backport this to our Gentoo kile-2.1.2, only thing I had to modify was the last chunk of the patch, see below for how it looks now. Seems to work fine, projects are loaded correctly.
diff -ruN kile-2.1.2.orig/src/widgets/konsolewidget.cpp kile-2.1.2/src/widgets/konsolewidget.cpp
--- kile-2.1.2.orig/src/widgets/konsolewidget.cpp 2012-04-22 18:54:51.000000000 +0200
+++ kile-2.1.2/src/widgets/konsolewidget.cpp 2012-06-19 00:12:18.451297729 +0200
@@ -78,8 +78,6 @@
layout()->addWidget(m_part->widget());
setFocusProxy(m_part->widget());
connect(m_part, SIGNAL(destroyed()), this, SLOT(slotDestroyed()));
-
- m_term->showShellInDir(QString());
}
Yes, seems to look fine. Git commit 98333619440d955ce55db8b48e6d3bd09ddc92ab by Michel Ludwig. Committed on 19/06/2012 at 22:24. Pushed by mludwig into branch '2.1'. Use 'TerminalInterface::showShellInDir' again to avoid problems with older Konsole versions (4.5) M +3 -0 src/widgets/konsolewidget.cpp http://commits.kde.org/kile/98333619440d955ce55db8b48e6d3bd09ddc92ab Git commit 5b69812b8030084f5844812fa1eed71ccc4b9ac9 by Michel Ludwig. Committed on 19/06/2012 at 22:24. Pushed by mludwig into branch 'master'. Use 'TerminalInterface::showShellInDir' again to avoid problems with older Konsole versions (4.5) M +3 -0 src/widgets/konsolewidget.cpp http://commits.kde.org/kile/5b69812b8030084f5844812fa1eed71ccc4b9ac9 |