fix meta-spec 2, add dynamic_config, remove generated MANIFEST entries#8
fix meta-spec 2, add dynamic_config, remove generated MANIFEST entries#8Grinnz wants to merge 1 commit into
Conversation
|
Also removed upper bound on setting LICENSE since setting it in resources doesn't set the upper-level key in META. |
|
No, I needed the Also the MANIFEST needs to stay: dynamic_config is also not specified on all EUMM versions, so needs to be version guarded. |
It does not complain for me, and this would be useless, since license in
These entries are added by
This is within META_MERGE so is already guarded by that. |
You can read about it here: https://metacpan.org/pod/CPAN::Meta::Spec#dynamic_config Important note is that you should only use it in META_MERGE when also setting meta-spec 2, because meta-spec 1.4 dynamic_config meant something different. |
|
Ok, I see. But I insist on the LICENSE guard, since it fails for me. |
|
I added it back, but this is what it does to the generated metadata: |
|
That's not good. I'll investigate the logic. |
META.json/META.yml are added by make dist and EUMM complains if they're there since they arent in the repo.
Updated META_MERGE contents to meta-spec 2 so they get added to META properly.
and added dynamic_config 0 since Makefile.PL does not do dynamic prereqs.