Yes we do include the generated files and autogen.sh should only be used when adding new files or changing the .am files. This is because (among other reasons) we need to build on system where the automake tools do not exist and can't install them. So having files like configure pre-generated solves the situation.

While is does break the standard this is how we have chosen to do it, hopefully it also makes it easier for most people since they don't have to run autogen.sh.

Danny

On 04/14/14 10:51, Ralph Castain wrote:

On Apr 14, 2014, at 9:40 AM, Michael Jennings <[email protected]> wrote:

On Mon, Apr 14, 2014 at 9:18 AM, E V <[email protected]> wrote:

I noticed you excluded the configure script and other stuff needed to build the 
package. I guess I'll add it back in on my fork. If you don't want it in your 
repo I'll just put it on a build branch.
"configure" and other auto-generated files should *never* be in SCM
trees such as GitHub.  Use the "autogen.sh" script to generate them.
"configure.ac" is the source file, and it's in the original
slurm-drmaa tree.

While I agree with the sentiment...have you looked at the Slurm repo?


Michael

--
Michael Jennings <[email protected]>
Senior HPC Systems Engineer
High-Performance Computing Services
Lawrence Berkeley National Laboratory
Bldg 50B-3209E        W: 510-495-2687
MS 050B-3209          F: 510-486-8615

Reply via email to