Problem/Motivation
To install migrate_plus:^6, the dependencies of this module should be more loose.
Your requirements could not be resolved to an installable set of packages.
Problem 1
- drupal/media_migration 1.0.0-alpha1 requires drupal/core ^8 -> found drupal/core[8.0.0-beta6, ..., 8.9.20] but it conflicts with your root composer.json require (^9.2@alpha).
- drupal/media_migration[1.0.0-alpha2, ..., 1.0.0-alpha14] require drupal/migrate_plus ^4.2 || ^5 -> found drupal/migrate_plus[dev-4.x, dev-5.x, 4.2.0, 4.x-dev (alias of dev-4.x), 5.0.0-rc1, ..., 5.x-dev (alias of dev-5.x)] but it conflicts with your root composer.json require (^6@dev).
- Root composer.json requires drupal/media_migration ^1.0@alpha -> satisfiable by drupal/media_migration[1.0.0-alpha1, ..., 1.0.0-alpha14].
Steps to reproduce
Proposed resolution
Remaining tasks
API changes
| Comment | File | Size | Author |
|---|---|---|---|
| #18 | media_migration-3269743-16-18-interdiff.txt | 394 bytes | rosk0 |
| #18 | media_migration-3269743-18.patch | 640 bytes | rosk0 |
| #16 | media_migration-n3269743-16.patch | 579 bytes | damienmckenna |
Issue fork media_migration-3269743
Show commands
Start within a Git clone of the project using the version control instructions.
Or, if you do not have SSH keys set up on git.drupalcode.org:
Comments
Comment #3
jonathan_hunt commentedComment #4
xurizaemonAttaching a copy of MR !6 as of 014103e5 as Gitlab MR URLs are subject to unexpected changes (ref #3204538).
Comment #5
markie commentedPatch applies cleanly, but the composer.lock does not update and then when you try to update the migration suites, it does not allow. So if you want to use this before the next release, you will have to do the following:
1. `composer remove drupal/media_migration`
2. Update the repositories section of your composer.json to the following:
3. `composer require drupal/media_migration:dev-3269743-support-drupal-10`
4. `composer require drupal/migrate_plus drupal/migrate_tools drupal/migrate_upgrade -W`
make sure you remove the patch..
@see https://www.drupal.org/docs/develop/git/using-git-to-contribute-to-drupa...
Comment #6
huzookaThank you all!
I'm postponing this on #3263827: Make Migrate Magician compatible with Drupal core 10.x.
Right after it gets solved, I will merge this PR.
Comment #7
jastraat commentedPinging here since it looks like the issue in comment 6 was solved and part of a new Migrate Magician release.
Comment #8
huzookaThis isn't postponed anymore.
Comment #9
Christopher Riley commentedAny word on getting a commit done on this?
Comment #10
xurizaemonBack to needs review with the proposed change.
Comment #11
proteo commentedI've tested the change and, while the patch applies cleanly and everything seems to work, I couldn't do the actual upgrade as Composer still sees the previous version constrain in the
composer.lockfile.I added the patch to
composer.json(using cweagans/composer-patches), then removed and required the project again with no luck (probably because of some caching?).Anyway, I had to delete the lock file and run
composer installfor Composer to be able to finally see the changes and upgrade to Migrate Plus 6.0Comment #12
huzookaTest dependencies also should be Drupal 10 compatible.
Comment #13
damienmckennaThen please refocus this on compatibility with Migrate Plus 6 rather than blocking D9 sites from upgrading Migrate Plus and Migrate Tools.
Comment #14
damienmckennaGiven #3297617 exists for actual Drupal 10 compatibility, let's focus this on just Migrate Plus 6 support.
Comment #15
damienmckennaComment #16
damienmckennaUpdated migrate_tools too.
Comment #17
pianomansam commentedCould we get some movement on this? I am unable to install this module with the patch method, and I cannot use the composer repository method as the issue fork is quite a bit behind dev.
Comment #18
rosk0Based the latest test run , it looks to me that
migrate_upgradeconstraint should be updated as well.At the moment Drupal CI doesn't allow us to test changes to
composer.json. From https://www.drupal.org/docs/creating-custom-modules/add-a-composerjson-f...So I'm bumping priority in this regard - these changes should be merged ASAP. There is no big changes that could break client websites, also we are still in alpha here , so that is kinda allowed ;) .
Comment #19
pianomansam commentedFor what it's worth, I rebased the issue branch off 8.x-1.x-dev so it has the latest code. This allows me to use the composer repo method.
Comment #20
drupalninja99 commentedI am not sure how to test this? Can we get composer instructions or can we go ahead and merge?
Comment #21
damienmckennaThe merge request needs work to follow what's on the latest patch.
Comment #22
pianomansam commented@DamienMcKenna the pull request has been updated with the updates in the latest patch.
@drupalninja99 if you would like to test the work in the feature branch, you can follow the instructions in #5
Comment #23
chrissnyderWhat else needs to be done to get this work into a new release?
Comment #24
heddnWorking on test failures.
Comment #25
heddnI futzed around a long time with the RDF and Color modules test coverage. There isn't any much of a path forward for those 2 modules, they are making tests hard and in Drupal 10.0 they were moved out of core. For the sake of expediency in tests, I've removed test coverage for them. It is just easier to do that for something that has very little future life.
That said, tests are now green again on 9.4 and 10.0/10.1. I found a small bug in Migrate Upgrade 4.x while working on the test coverage here (fixed already and new tag released). All that is left here is for someone to review and someone to commit the code.
Comment #26
heddnI think that merging this means that this can be fixed.