mingw-cross-env-list
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Mingw-cross-env-list] including hdf5


From: Luis Saavedra
Subject: Re: [Mingw-cross-env-list] including hdf5
Date: Sun, 11 Mar 2012 12:45:32 -0400

2012/3/6 Mark Brand <address@hidden>
Alois Schlögl wrote:

Please include the attached file in the repo.

Hi Alois,

Thanks for this contribution. I have a few comments about it.

1) Instead of using sed to change Makefile.in, I think it would be better to supply a patch for Makefile.am, and then regenerate Makefile.in. This way it's easier to see whether patches continue to apply as expected when the package is upgraded. See gnutls.mk for an example of how to do this. Perhaps hdf5 comes with an 'autogen.sh' or similar script.

2) It would also be nice to have your change in the form of a patch that applies to the mercurial repo. You can make it using 'hg export -g'. This will contain your email address as the contributor.

3) Also please consider whether CHOST can appear at the end of the configure invocation. This is the style generally used in mingw-cross-env.

regards,

Mark



Hi Mark,

I've had such problems with pango, in this case a Makefile.in is patched and MSYS regenerate it from Makefile.am, I attached a patch for this,

regards,
Luis

Attachment: pango.patch
Description: Binary data


reply via email to

[Prev in Thread] Current Thread [Next in Thread]