Use standardized SPDX license identifiers

Created on 3 April 2024, 10 days ago
Updated 9 April 2024, 3 days ago

Problem/Motivation

The standardized SPDX (System Package Data Exchange) license identifiers are used by many other projects in various places, such as package.json and composer.json files, and I'd say Drupal should use these standard identifiers too. See https://spdx.org/licenses/

Drupal uses GNU-GPL-2.0-or-later, this should be shortened to simply GPL-2.0-or-later

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

Release notes snippet

๐Ÿ› Bug report
Status

Fixed

Version

10.3 โœจ

Component
Asset libraryย  โ†’

Last updated 1 day ago

No maintainer
Created by

๐Ÿ‡บ๐Ÿ‡ธUnited States mfb San Francisco

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Merge Requests

Comments & Activities

  • Issue created by @mfb
  • Pipeline finished with Success
    10 days ago
    Total: 691s
    #136013
  • Status changed to Needs review 10 days ago
  • ๐Ÿ‡บ๐Ÿ‡ธUnited States mfb San Francisco
  • Status changed to RTBC 5 days ago
  • ๐Ÿ‡บ๐Ÿ‡ธUnited States smustgrave

    Make sense. Applied MR and searched for GNU-GPL-2.0-or-later and seems everything has been replaced.

    Not sure this needs any kind of sign off or not but going to mark.

  • Status changed to Needs work 5 days ago
  • ๐Ÿ‡ฌ๐Ÿ‡งUnited Kingdom longwave UK

    I think adopting this standard is a good idea, and we could document this in LibraryDiscoveryParser for the name field:

       * - license: If the remote property is set, the license information is
       *   required. It has 3 properties:
       *   - name: The human-readable name of the license.
       *   - url: The URL of the license file/information for the version of the
       *     library used.
       *   - gpl-compatible: A Boolean for whether this library is GPL compatible.
    
  • ๐Ÿ‡ฎ๐Ÿ‡ณIndia pradhumanjainOSL

    pradhumanjain2311 โ†’ made their first commit to this issueโ€™s fork.

  • Pipeline finished with Success
    4 days ago
    Total: 638s
    #141003
  • Status changed to Needs review 4 days ago
  • ๐Ÿ‡บ๐Ÿ‡ธUnited States mfb San Francisco

    Added documentation to resolve #5

  • Pipeline finished with Success
    4 days ago
    Total: 588s
    #141149
  • Status changed to RTBC 4 days ago
  • ๐Ÿ‡บ๐Ÿ‡ธUnited States smustgrave

    Description added by @mfb reads well.

  • First commit to issue fork.
  • Pipeline finished with Failed
    4 days ago
    Total: 990s
    #141783
  • Status changed to Needs work 4 days ago
  • ๐Ÿ‡ฌ๐Ÿ‡งUnited Kingdom catch

    Rebased to check if this fails the new performance asset size coverage and it does - we're sending less bytes because the license information is included in the CSS aggregates, and it's now shorter - just needs the assertions updating.

  • Pipeline finished with Failed
    3 days ago
    Total: 1018s
    #142025
  • Pipeline finished with Failed
    3 days ago
    Total: 1051s
    #142057
  • Pipeline finished with Success
    3 days ago
    Total: 989s
    #142081
  • Status changed to Needs review 3 days ago
  • ๐Ÿ‡บ๐Ÿ‡ธUnited States mfb San Francisco

    Tests are passing

  • Status changed to RTBC 3 days ago
  • ๐Ÿ‡ฌ๐Ÿ‡งUnited Kingdom longwave UK

    Thanks!

    • catch โ†’ committed 135e3ba1 on 11.x
      Issue #3437783 by mfb, pradhumanjain2311: Use standardized SPDX license...
  • Status changed to Downport 3 days ago
  • ๐Ÿ‡ฌ๐Ÿ‡งUnited Kingdom catch

    Committed/pushed to 11.x, thanks!

    This doesn't cherry-pick cleanly to 10.3.x so will need a backport MR.

  • Pipeline finished with Success
    3 days ago
    Total: 631s
    #142153
  • Status changed to Needs review 3 days ago
  • ๐Ÿ‡บ๐Ÿ‡ธUnited States mfb San Francisco

    Backport ready for review

  • Status changed to RTBC 3 days ago
  • ๐Ÿ‡ฌ๐Ÿ‡งUnited Kingdom longwave UK

    Backport looks good.

  • ๐Ÿ‡ฌ๐Ÿ‡งUnited Kingdom alexpott ๐Ÿ‡ช๐Ÿ‡บ๐ŸŒ

    Committed e3da4cc and pushed to 10.3.x. Thanks!

  • Status changed to Fixed 3 days ago
Production build https://api.contrib.social 0.62.1