pused an update to this fix [to next] https://bitbucket.org/petsc/petsc/commits/63bbfdb1562936586445d38d41db54c2b787407e
satish On Mon, 7 Oct 2013, Richard Tran Mills wrote: > Thanks, Matt. It would be good if we could get this into master a little bit > earlier since I'd like to send out instructions to the workshop participants > ahead of time so that they can build PETSc and PFLOTRAN before they get there. > But I guess I could just give instructions to update the particular commit > that has the change if I need to. > > Thanks, > Richard > > On 10/7/13 1:11 PM, Matthew Knepley wrote: > > On Mon, Oct 7, 2013 at 11:58 AM, Satish Balay <[email protected] > > <mailto:[email protected]>> wrote: > > > > perhaps isCygwin() should be changed to isWindows() > > > > > > This has been merged to next. If it passes a two days of builds, we can > > merge it to master. Is that enough time? > > > > Matt > > > > > > satish > > > > On Mon, 7 Oct 2013, Matthew Knepley wrote: > > > > > On Mon, Oct 7, 2013 at 11:51 AM, Satish Balay <[email protected] > > <mailto:[email protected]>> wrote: > > > > > > > Hm self.downloadonWindows is supposed to be for MS compilers only. > > > > > > > > > > if self.download and > > > self.framework.argDB.get('download-'+self.downloadname.lower()) > > and not > > > self.downloadonWindows and self.setCompilers.isCygwin(): > > > raise RuntimeError('External package '+self.name > > <http://self.name>+' does not support > > > --download-'+self.downloadname.lower()+' on Microsoft Windows') > > > > > > Matt > > > > > > > > > > If thats not the case - the bug is somewere else.. > > > > > > > > satish > > > > > > > > On Mon, 7 Oct 2013, Richard Tran Mills wrote: > > > > > > > > > Matt, > > > > > > > > > > I tried this and it works under 32-bit Cygwin. It should also > > work under > > > > > 64-bit Cygwin except that it hits an error when building the > > 'h5ls' > > > > target > > > > > (can overcome this by telling make to ignore errors or by > > making a small > > > > patch > > > > > to the relevant HDF5 code--this is a Cygwin64/HDF5 problem and has > > > > nothing to > > > > > do with BuildSystem). > > > > > > > > > > Can we go ahead and get this change into 'master' ASAP? I am > > not sure > > > > of the > > > > > proper workflow to use for such a small change as this. I'd like > > > > > --download-hdf5 to be working before I leave for Germany in a > > few days > > > > to do a > > > > > short course there. > > > > > > > > > > --Richard > > > > > > > > > > On 10/4/13 5:45 PM, Matthew Knepley wrote: > > > > > > On Fri, Oct 4, 2013 at 4:42 PM, Richard Tran Mills > > <[email protected] <mailto:[email protected]> > > > > > > <mailto:[email protected] <mailto:[email protected]>>> wrote: > > > > > > > > > > > > Hi Satish, > > > > > > > > > > > > I got this message attempting to use the Gnu compilers; > > that's all > > > > > > that I've got on this system. Should I send the > > configure.log to > > > > > > petsc-maint? > > > > > > > > > > > > > > > > > > Want to add > > > > > > > > > > > > self.downloadonWindows = 1 > > > > > > > > > > > > to __init__ of config/BuildSystem/config/packages/hdf5.py > > and see if it > > > > > > work? > > > > > > > > > > > > Thanks, > > > > > > > > > > > > Matt > > > > > > > > > > > > --Richard > > > > > > > > > > > > On 10/4/13 4:34 PM, Satish Balay wrote: > > > > > > > > > > > > On Fri, 4 Oct 2013, Richard Tran Mills wrote: > > > > > > > > > > > > Hi Folks, > > > > > > > > > > > > If I tell configure.py '--download-hdf5=1', I > > get a message > > > > > > that "External > > > > > > package hdf5 does not support --download-hdf5 on > > Microsoft > > > > > > Windows". However, > > > > > > I just built HDF5 1.8.11 from source under > > 64-bit Cygwin > > > > > > and, except for one > > > > > > problem with building the 'h5ls' utility, it > > appears that > > > > > > this ought to work. > > > > > > Can we support this? And what is the procedure > > for changing > > > > > > package > > > > > > configuration info in BuildSystem these days? > > These files > > > > > > show up in the > > > > > > 'petsc' repo but there appears to be a > > 'buildsystem' repo > > > > on > > > > > > BitBucket as > > > > > > well; not sure what the relationship is between > > them. > > > > > > > > > > > > When you say 'Supporting --download-hdf5 in Cygwin' > > and refer > > > > to > > > > > > 64-bit - you > > > > > > mean MS compilers or cygwin gcc/gfortran? > > > > > > > > > > > > The error message should come up only with MS compilers. > > > > > > > > > > > > Presumably --download-hdf5 should work with gnu > > compilers - if > > > > > > it doesn't - it > > > > > > probably needs some tweaking. > > > > > > > > > > > > Wrt repo - we now have a single git repo > > > > > > bitbucket.org/petsc/petsc <http://bitbucket.org/petsc/petsc> > > <http://bitbucket.org/petsc/petsc> > > > > > > where all petsc/buildsystem changes should go to. > > > > > > > > > > > > Satish > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > -- > > > > > > What most experimenters take for granted before they begin their > > > > experiments > > > > > > is infinitely more interesting than any results to which their > > > > experiments > > > > > > lead. > > > > > > -- Norbert Wiener > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > -- > > What most experimenters take for granted before they begin their experiments > > is infinitely more interesting than any results to which their experiments > > lead. > > -- Norbert Wiener > > >
