Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Currently core allows the installation of "paragonie/random_compat" with the following constraints: "^1.0|^2.0".
If these get updated to "^1.0|^2.0|^9.99.99", then the NOOP version would be installable as well.
This NOOP version is only installable on php 7.0+ and does not populate the autoloader like other versions. Which provides a microscopic performance gain.
Comment | File | Size | Author |
---|---|---|---|
#12 | 2994315-12.patch | 482 bytes | alexpott |
#8 | version-constraint.patch | 464 bytes | BackEndTea |
#2 | paragonie.patch | 444 bytes | BackEndTea |
Comments
Comment #2
BackEndTea CreditAttribution: BackEndTea commentedComment #3
BackEndTea CreditAttribution: BackEndTea commentedComment #4
longwavehttps://github.com/paragonie/random_compat/ suggests using ">=1" as the version constraint.
However, is this even worth changing, as this package will presumably be removed in 8.7.x or 8.8.x when we drop PHP 5 support?
Comment #5
BackEndTea CreditAttribution: BackEndTea commentedThis version constraint is more in line with how other version constraints are handled. It would also allow the decision of a possible 3.* version to be made at that point.
8.7 and 8.8 are still a bit away, and the update has no negative impact on the upgrade to 8.7/8.8
Comment #6
longwaveComment #8
BackEndTea CreditAttribution: BackEndTea commentedI have added the correct patch
Comment #9
BackEndTea CreditAttribution: BackEndTea commentedComment #11
amateescu CreditAttribution: amateescu for Pfizer, Inc. commentedLooks good to me. Note that we're removing
paragonie/random_compat
for 8.8.x in #3058116: Remove paragonie/random_compat as a top-level dependency from 8.8.x, so this is only a 8.7.x patch.Comment #12
alexpottrerolled the patch
Comment #13
alexpottCommitted db53fd7 and pushed to 8.7.x. Thanks!