It looks like things are fixed now (?) so maybe this is moot but I think it was mid- to late- september that I originally put a version check in setCompilers, but it wasn't working correctly so I'd added a line or two, not realizing Matt was already re-hauling it.  Sorry about that.<div>
<br></div><div>-Victor<br clear="all">---<br>Victor L. Minden<br><br>Tufts University<br>School of Engineering<br>Class of 2012<br>
<br><br><div class="gmail_quote">On Fri, Nov 5, 2010 at 3:07 PM, Satish Balay <span dir="ltr"><<a href="mailto:balay@mcs.anl.gov">balay@mcs.anl.gov</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
Matt,<br>
<br>
I don't see self.setCompilers.compilerVersionCUDA in rev history. So I<br>
think this code was broken when you added this stuff [and victor must<br>
have tried to fix it..<br>
<br>
You can revert relavent changes:<br>
BuildSystem<br>
 febbb50e3b78<br>
 0d934bb871e4<br>
 051ab1553c96<br>
<br>
petsc-dev<br>
 5a758fb106fc<br>
<br>
You'll still see no compilerVersionCUDA in  setCompilers.py<br>
<br>
[reverting these, and fixing self.setCompilers.compilerVersionCUDA might work..]<br>
<br>
asterix:/home/balay/tmp/petsc-dev/config/BuildSystem>hg diff<br>
diff -r 3d1e6fbcdc48 config/setCompilers.py<br>
--- a/config/setCompilers.py    Fri Nov 05 09:03:09 2010 -0500<br>
+++ b/config/setCompilers.py    Fri Nov 05 14:07:32 2010 -0500<br>
@@ -513,7 +513,6 @@<br>
<br>
   def checkCUDACompiler(self):<br>
     '''Locate a functional CUDA compiler'''<br>
-    requiredVersion = '3.2'<br>
     if 'with-cudac' in self.framework.argDB and self.framework.argDB['with-cudac'] == '0':<br>
       if 'CUDAC' in self.framework.argDB:<br>
         del self.framework.argDB['CUDAC']<br>
@@ -523,21 +522,7 @@<br>
         if self.getExecutable(compiler, resultName = 'CUDAC'):<br>
           self.checkCompiler('CUDA')<br>
           # Put version info into the log<br>
-          compilerVersion = self.executeShellCommand(self.CUDAC+' --version')<br>
-          compilerVersion = compilerVersion[0]<br>
-          compilerVersion = compilerVersion.split()<br>
-          i = 0<br>
-          for word in compilerVersion:<br>
-            i = i+1<br>
-            if word == 'release':<br>
-              break<br>
-          currentVersion = compilerVersion[i].strip(',')<br>
-          if currentVersion != requiredVersion:<br>
-            #raise RuntimeError('CUDA Error: PETSc currently requires nvcc version ' + requiredVersion)<br>
-              print '==============================================================================='<br>
-              print '** CUDA Error: PETSc requires nvcc version ' + requiredVersion + ' when compiling with --withcuda **'<br>
-              print '==============================================================================='<br>
-              sys.exit(3)<br>
+          self.executeShellCommand(self.CUDAC+' --version')<br>
           break<br>
       except RuntimeError, e:<br>
         self.logPrint('Error testing CUDA compiler: '+str(e))<br>
<div><div></div><div class="h5"><br>
<br>
Satish<br>
<br>
On Fri, 5 Nov 2010, Matthew Knepley wrote:<br>
<br>
> On Fri, Nov 5, 2010 at 1:52 PM, Satish Balay <<a href="mailto:balay@mcs.anl.gov">balay@mcs.anl.gov</a>> wrote:<br>
><br>
> > because it was broken [as there was no<br>
> > self.setCompilers.compilerVersionCUDA] - and I noticed a different<br>
> > version check in setCompilers.py - and assumed thats the proper thing.<br>
> ><br>
> > But looks like vicotr made a mistake by adding in that check in<br>
> > setCompilers.py.<br>
> ><br>
> > Still I don't see where self.setCompilers.compilerVersionCUDA was set<br>
> > - and when that piece of cod removed..<br>
><br>
><br>
> Dammit dammit dammit. Who is fucking with my code. It was in<br>
> setCompilers.py.<br>
> Everyone stop fucking with this while I fix it.<br>
><br>
>     Matt<br>
><br>
><br>
> ><br>
> > Satish<br>
> ><br>
> > On Fri, 5 Nov 2010, Matthew Knepley wrote:<br>
> ><br>
> > > That was the correct one. I took the one out<br>
> > > of setCompilers.py for a reason.<br>
> > ><br>
> > >    Matt<br>
> > ><br>
> > ><br>
> ><br>
> ><br>
><br>
><br>
><br>
<br>
</div></div></blockquote></div><br></div>