Bug 86941

Summary: $VARIABLE include directories should be in paranthesis
Product: [Applications] kdevelop Reporter: Roland Krause <rokrau>
Component: Build tools: QMakeAssignee: Andreas Pakulat <apaku>
Status: RESOLVED FIXED    
Severity: normal    
Priority: NOR    
Version First Reported In: 3.0.3   
Target Milestone: ---   
Platform: RedHat Enterprise Linux   
OS: Linux   
Latest Commit: Version Fixed/Implemented In:
Sentry Crash Report:

Description Roland Krause 2004-08-10 23:51:40 UTC 
Version:           3.0.3 (using KDE 3.2.3-6.5.rh90.kde, Red Hat Linux release 9 (Shrike))
Compiler:          gcc version 3.2.2 20030222 (Red Hat Linux 3.2.2-5)
OS:                Linux (i686) release 2.4.20-31.9smp

When using the qmake manager and trying to add an include directory that is given through a global shell variable (e.g. $HOME) the line edit autocompletes entries that begin with the $ sign. The problem with this behavior is that, if the user accepts the choice, the variable is added to the Makefile as $HOME but it must be $(HOME). 

The paranthesis must be added. I bet this can be done very quickly.
Comment 1 Andreas Pakulat 2006-11-22 21:16:56 UTC 
SVN commit 607017 by apaku:

Really fix replacing Shell variables, we need to add () around not {}

BUG: 86941


 M  +2 -2      scope.cpp  


--- branches/kdevelop/3.4/buildtools/qmake/scope.cpp #607016:607017
@@ -700,8 +700,8 @@
             }else if ( origValues.isEmpty() )
                 origValues.append(" ");
             QString newval = *it;
-            QRegExp re("\\$([^$\\{\\) /]*)( |\\)|/)");
-            newval.replace(re, "${\\1}\\2");
+            QRegExp re("\\$([^$\\(\\)\\{\\} /]*)( |\\)|/)");
+            newval.replace(re, "$(\\1)\\2");
             if( (newval).contains(" ") || (newval).contains("\t") || (newval).contains("\n") )
                 origValues.append( "\""+newval+"\"" );
             else