search

ZedThree / fort_depend.py

A python script to generate dependencies for Fortran projects
MIT License
41 stars 21 forks source link

Add build directory option. #4

Closed everythingfunctional closed 8 years ago

everythingfunctional commented 8 years ago

Added an option to include the name of the build directory, which is just prepended to all of the file names. Also put a comment at the top of the output file that indicates it is an automatically generated file.

ZedThree commented 8 years ago

Hi @bprichar, this looks great, thanks! Do you think you could use os.path.join to append build (here and here)? This ensures the correct path separators are included, in case the user forgets them on the command line.

everythingfunctional commented 8 years ago

Absolutely. Didn't remember that function at the time. Much safer option, thanks.

ZedThree commented 8 years ago

I think fort_depend.py:177 should be

build = args.build[0] if args.build else ''

rather than

build = args.build[0] if args.build else None

otherwise I get an error from os.path.join.

everythingfunctional commented 8 years ago

Ok, I'll fix that right quick.