Bug#486577: 486577 is actually serious since the app doesn't work (without some hacking)

[Eddy Petrișor]

Package: universalindentgui
# useless unless you think to hack the TMPDIR env variable before
starting the thing
severity 486577 serious
tags 486577 patch
thanks

Hello,

Indeed the problem was in the wrong temp handling. It seems that there
was some confusion about the default presence of a final '/' in the
TMPDIR environment variable on Linux and Windows.



I have prepared a fix for this bug (pretty trivial) and I prepared an
updated version ready for upload.

With this fix the output goes like this:

0 [EMAIL PROTECTED] ~/tmp/_eraseme $ universalindentgui
Using directories:
settings =  "/home/eddy/.universalindentgui"
globalFiles =  "/usr/share/universalindentgui"
indenterDirctoryStr =  "/usr/share/universalindentgui/indenters"
tempDirctoryStr =  "/tmp/UniversalIndentGUI"



Initially it was:
0 [EMAIL PROTECTED] ~/tmp/_eraseme $ universalindentgui
Using directories:
settings =  "/home/eddy/.universalindentgui"
globalFiles =  "/usr/share/universalindentgui"
indenterDirctoryStr =  "/usr/share/universalindentgui/indenters"
tempDirctoryStr =  "/tmpUniversalIndentGUI"




Please upload the package to unstable since the change is so small it
can't be wrong :-)


You can grab the package from

http://users.alioth.debian.org/~eddyp-guest/upload/universalindentgui-0.8.1-1.1/:

dget from: 
http://users.alioth.debian.org/~eddyp-guest/upload/universalindentgui-0.8.1-1.1/universalindentgui_0.8.1-1.1.dsc



Or you can get it from the attachments (orig, deb anc changes were
skipped, grab the orig from the archive).

-- 
Regards,
EddyP
=============================================
"Imagination is more important than knowledge" A.Einstein

diff --git a/universalindentgui-0.8.1/debian/changelog b/universalindentgui-0.8.1/debian/changelog
index 38cb52f..a328415 100644
--- a/universalindentgui-0.8.1/debian/changelog
+++ b/universalindentgui-0.8.1/debian/changelog
@@ -1,3 +1,13 @@
+universalindentgui (0.8.1-1.1) unstable; urgency=high
+
+  * Non-maintainer upload.
+  * urgency high since universalindentgui is actually useless by default
+    (can be hacked to work by setting TMPDIR='/tmp/a')
+  * fixed temporary path asamblation so indents can work
+    (Closes: 486577)
+
+ -- Eddy Petrișor <[EMAIL PROTECTED]>  Wed, 05 Nov 2008 03:51:48 +0200
+
 universalindentgui (0.8.1-1) unstable; urgency=low
 
   * New upstream release
diff --git a/universalindentgui-0.8.1/src/mainwindow.cpp b/universalindentgui-0.8.1/src/mainwindow.cpp
index 287662b..fdb22c4 100644
--- a/universalindentgui-0.8.1/src/mainwindow.cpp
+++ b/universalindentgui-0.8.1/src/mainwindow.cpp
@@ -87,11 +87,7 @@ MainWindow::MainWindow(QWidget *parent) : QMainWindow(parent) {
             QFile::copy( globalFilesDirectoryStr+"/config/UiGuiSyntaxHighlightConfig.ini", settingsDirctoryStr+"/UiGuiSyntaxHighlightConfig.ini" );
         }
         indenterDirctoryStr = globalFilesDirectoryStr + "/indenters";
-#ifdef Q_OS_WIN
         tempDirctoryStr = QDir::tempPath() + "/UniversalIndentGUI";
-#else
-        tempDirctoryStr = QDir::tempPath() + "UniversalIndentGUI";
-#endif
         dirCreator.mkpath( tempDirctoryStr );
     }
 

universalindentgui_0.8.1-1.1.diff.gz
Description: GNU Zip compressed data

universalindentgui_0.8.1-1.1.dsc
Description: Binary data