search

Perl5-Alien / Alien-Base

Base classes for Alien:: modules (deprecated, see Alien-Build)
Other
35 stars 19 forks source link

New diagnostics #104

Closed plicease closed 9 years ago

plicease commented 9 years ago

These diagnostics would have saved me some time tonight.

The diagnostic about requiring at least one repository is fatal either way, it is just that it catches it earlier with this patch. IF we want to support builds that lack any repositories then we need to restructure the code so that we can do that. I don't see that it is a useful thing to do.

The other diagnostic just gives a hint as to what you need to do. Even though I kind of know what needed to be done, I couldn't remember what the method I had to implement in order to get it to work. If someone else were trying to use it they'd have even more trouble since they hadn't implemented it :)

mohawk2 commented 9 years ago

Looks good. Aye.

zmughal commented 9 years ago

Merge: Aye!

plicease commented 9 years ago

sent this to CPAN as 0.009_02.