1668798 - Let glean_parser fail on absent files

Status: RESOLVED FIXED

(Data Platform and Tools :: Glean: SDK, enhancement, P1)

Description

[Chris H-C :chutten]

glean_parser doesn't fail when asked to process a file that doesn't exist. This is apparently due to Gradle Reasons, so it's a good feature to have... but can we sequester the feature to just Gradle's use?

Maybe we add a --dont-fail-on-absent-files option and get the Gradle Plugin to always use that, and then change the default behaviour to fail on absent files (with, as always, a helpful message)?

Comment 1

[Michael Droettboom [:mdroettboom]]

Makes sense.

🚲🏠 Would --allow-missing-files be better?

Comment 2

[Chris H-C :chutten]

(In reply to Michael Droettboom [:mdroettboom] from comment #1)

Makes sense.

🚲🏠 Would --allow-missing-files be better?

I'm not fussy on the name, just that it exist. Gradle's the only one who needs to use it : )

Comment 3

[GitHub Bugzilla PR Linker]

Attachment: Link to GitHub pull-request: https://github.com/mozilla/glean_parser/pull/236