data files are excluded if they do not exist about hpack HOT 7 CLOSED

Sorry guys for the late reply. This is due to glob expansion. A fix may be somewhat fragile, that's why I was not excited to work on it so far.

I agree that in the case where no globs are used, hpack should not discard files if they don't exist. So I'm labeling this as a bug.

from hpack.

I am still interested in this though I don't currently have the time to commit to do it.
I think that we certainly do want a warning if a literal file does not exist.
I would think that a warning for a non-literal filename with no matches would be appropriate, but I can see cases where it would be superfluous.
I don't see any reason one wouldn't want this behaviour in all the sections you list.

from hpack.

@duog Thanks! We now keep literal files, even if non-existing and warn.

from hpack.

I just ran into this due an over-zealous clean script, and it caused some head-scratching. I think it would be better not to check for file existence here.

from hpack.

Blocked by Deewiant/glob#10.

from hpack.

Now that Deewiant/glob#10 is resolved we are in a better position to change what hpack does. So what exact behavior do we want?

• If a file pattern is a literal file name, we want to keep it regardless of whether that file exists or not. Do we want a warning if the file does not exist?

• If a file pattern is not a literal file name (contains wildcards), we expand it. If it does not match anything it's effectively discarded; currently we emit a warning if that happens. Do we want to keep that warning?

By default, the change would affect:

• data-files
• extra-source-files
• c-sources
• js-sources

from hpack.

@simonmichael @duog Are you still interested in this? If not, I'll close this issue.

from hpack.