[petsc-dev] Supporting --download-hdf5 in Cygwin?

[Satish Balay]

ran a couple of tests and pushed to master.

satish

On Tue, 8 Oct 2013, Satish Balay wrote:

> 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 <balay at mcs.anl.gov
> > > <mailto:balay at mcs.anl.gov>> 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 <balay at mcs.anl.gov
> > >     <mailto:balay at mcs.anl.gov>> 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
> > >     <rtm at eecs.utk.edu <mailto:rtm at eecs.utk.edu>
> > >     > > > > <mailto:rtm at eecs.utk.edu <mailto:rtm at eecs.utk.edu>>> 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
> > 
> > 
> > 
> 
>