Somehow these files were empty in the patch from #2337283: Add a composer.json file to every component.

Without them, its technically not legal at all for somebody to use the components, but that hasn't stopped at least several people from trying:

https://packagist.org/packages/drupal/core-serialization
https://packagist.org/packages/drupal/core-transliteration

We should probably add these, and backport it as far as we can.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Mixologic created an issue. See original summary.

Mixologic’s picture

Status: Active » Needs review
FileSize
36.39 KB
Mile23’s picture

Status: Needs review » Reviewed & tested by the community

Verified that the patch applies to both 8.6.x and 8.5.x.

Verified that after the patch, all components have a LICENSE.txt file.

alexpott’s picture

Status: Reviewed & tested by the community » Needs review
FileSize
1.41 KB
37.8 KB

Let's add a test. The interdiff is the test-only patch - that should fail.

The last submitted patch, 4: 2943860-4.test-only.patch, failed testing. View results

Mile23’s picture

Status: Needs review » Reviewed & tested by the community

+1 on the test.

It'd be great to get this in 8.5.x as well, since it's primarily not a code change, and the only code change is a test improvement.

alexpott’s picture

Status: Reviewed & tested by the community » Fixed

Committed and pushed de1c206038 to 8.6.x and af8e73f8d3 to 8.5.x. Thanks!

Backport to 8.5.x as bugfix that is not disruptive

  • alexpott committed de1c206 on 8.6.x
    Issue #2943860 by Mixologic, alexpott: Add a LICENSE.txt to...

  • alexpott committed af8e73f on 8.5.x
    Issue #2943860 by Mixologic, alexpott: Add a LICENSE.txt to...

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.