Problem/Motivation

In #3255353-6: Update dependencies for Drupal 10.0.x before alpha1, @longwave said:

edit: and local testing suggests we can remove paragonie/random_compat and symfony/polyfill-php70 as nothing depends on those any more anyway - maybe let's spin off a separate issue for this

Steps to reproduce

Proposed resolution

See if d.o. testing agrees with the above statement.

Remaining tasks

User interface changes

API changes

Data model changes

Release notes snippet

CommentFileSizeAuthor
#6 3255623-6.patch380 bytesspokje
#4 3255623-4-10_0_x.patch861 bytesspokje
#3 3255623-3-9_4_x.patch861 bytesspokje
#2 3255623-2.patch380 bytesspokje

Comments

Spokje created an issue. See original summary.

spokje’s picture

spokje
Primary language Dutch
commented
Status: Active » Needs work
StatusFileSize
new 3255623-2.patch380 bytes
spokje’s picture

spokje
Primary language Dutch
commented
StatusFileSize
new 3255623-3-9_4_x.patch861 bytes
spokje’s picture

spokje
Primary language Dutch
commented
StatusFileSize
new 3255623-4-10_0_x.patch861 bytes
elber’s picture

elber
Primary language Portuguese, Brazil
Location Brazil
commented
Assigned: Unassigned » elber
spokje’s picture

spokje
Primary language Dutch
commented
StatusFileSize
new 3255623-6.patch380 bytes

So with the test failure caused by #2675006: Write UnitTest for MigrationConfigurationTrait and fix Exception Handling now resolved, it looks like D10 is a bit more picky about the composer.lock hash than D9.

I think the below patch (which is the patch from #2) will do for D9.3/9.4

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
commented
Title: Remove paragonie/random_compat and symfony/polyfill-php70 » Remove composer replace statements for paragonie/random_compat and symfony/polyfill-php70
Status: Needs work » Reviewed & tested by the community

This is not removing paragonie/random_compat and symfony/polyfill-php70 :) They have not been installed for a while - this is removing the replace statements which have been stopping them being installed.

The question is are they still needed. To test this I ran composer update --prefer-lowest on Drupal 9.3 on PHP 7.3 with the patch applied. These dependencies where not brought in. So this is good to go.

elber’s picture

elber
Primary language Portuguese, Brazil
Location Brazil
commented
Assigned: elber » Unassigned

  • catch committed fce76cc on 10.0.x 
    Issue #3255623 by Spokje, alexpott: Remove composer replace statements...

  • catch committed 4f4f390 on 9.4.x 
    Issue #3255623 by Spokje, alexpott: Remove composer replace statements...
catch’s picture

catch
he/him
Primary language English
commented
Status: Reviewed & tested by the community » Closed (outdated)

Committed/pushed to 10.0.x and 9.4.x, thanks! Not seeing a compelling reason to commit this to 9.3.x so holding of that for now.