Missing dependencies do not raise an error

[dschwoerer]

I have in my project.info graphs explicitly enabled, yet if graphviz is not installed, the graphs are not build and no error is raised. An info is printed, but I normally don't check CI logs.

While it might be nice for quick testing, it is rather annoying for CI.

Does it make sense to add an option to error on missing, optional dependencies? Should this always be an error, but the error message includes an option to disable generating graphs?

[cmacmackin]

I certainly wouldn't want FORD to produce and uncaught exception in this case and am inclined to think it would be best if it still runs to completion. However, perhaps a non-zero return code would make sense, or at least having the option to produce one.

On Wed, 17 Nov 2021, 11:15 dschwoerer, @.***> wrote:

I have in my project.info graphs explicitly enabled, yet if graphviz is not installed, the graphs are not build and no error is raised. An info is printed, but I normally don't check CI logs.

While it might be nice for quick testing, it is rather annoying for CI.

Does it make sense to add an option to error on missing, optional dependencies? Should this always be an error, but the error message includes an option to disable generating graphs?

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/Fortran-FOSS-Programmers/ford/issues/368, or unsubscribe https://github.com/notifications/unsubscribe-auth/AB6ESPN7PTUZTOOJGF6CXJTUMOFELANCNFSM5IGVDDWQ . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.