Re: [PATCH] better error message with unknown component-dependency

[kooda]

felix.winkelmann@bevuta.com wrote:
> Reported by wasamasa.
> 

Resurecting this bug report.

As reference, here is an example egg file that shows the error (from wasamasa):

((synopsis "Broken")
 (components (program test (component-dependencies test-files))
             (data test-files (files "test.txt"))))

Unknown component dependenciy errors are already handled in filter-deps, the 
problem here is trying to generate build commands for data files.

The original proposed patch displays an error message even though the egg file 
is valid, it’s just a no-op to have a data component as a dependency, because 
the data is always there.
I think having this kind of error message (“unknown dependency: test-files” for 
the above example) would be very confusing.

Here is a different patch for this issue: it just adds the three missing cases 
for static data components, mapping them to an empty build command list.
For good measure there’s also a fallback case that prints an error message 
indicating that there is a bug in chicken-install if we someday add a new 
component type and forget to add its build commands.

txtRaXaN9FlC4.txt
Description: Text document