Remove dependency on CTools

Unsure how to best approach this. Removing requirements on drupal modules is really awkward, your site is broken if this was the only reason you had ctools.

So I'm unsure about dropping it without also doing a major new version or something.

Idea: We remove it from .info.yml but keep it in composer, then the code won't vanish on existing sites and if you feel strongly about not having the code you can do a replace definition in composer.json. And then we can remove it some time later, maybe as an eventual 2.x release...

Note: I've included this in #3266631: Drupal 10 compatibility, require Drupal 9.3 to test against D10, will likely still commit here with the suggested approach above later on.

@Berdir I think the idea in #4 is reasonable. Could the ctools dependency even be moved to suggest? I'm not sure if that would work the same way or not...

A suggest would still require manual intervention, that's something I want to avoid in a minor update. Rerolled the patch and removed the composer.info snippet.

If you really don't want ctools in your project code base, then you could add this to your composer.info.yml:

   "replace": {
        "drupal/ctools": "*"
    },

But I think most projects won't care about that.

Committed this!

Aw, I forgot about the core version requirement that comes with this. Reverted until we require Drupal 9.3. I might just merge this together with #3266631: Drupal 10 compatibility, require Drupal 9.3 after all.

Thanks for the work on this issue. Just to confirm, once either 31d8bb1 is re-committed under this issue, or committed under #3266631: Drupal 10 compatibility, require Drupal 9.3 and Pathauto 8.x-1.10 is tagged, after I update my codebase to 8.x-1.10 I can simply uninstall ctools, correct?

Uninstalling ctools does not seem to be possible on my installation. It still says there is a dependency on pathauto. I am running Drupal 9.3.12 and 8.x-1.10.

The release notes need to be read as preparation for "Drupal 10 and removal of ctools dependency". It's neither drupal 10 compatible nor ctools-free *yet*.

Ok, that was not clear to me. It looked to me like "preparation for Drupal 10" + "removal of ctools dependency".

Updated patch to require Drupal 9.3.

Committed.

Any plans for a release with this feature?

This was not really fixed

@KlemenDEV, this was fixed in commit 0bb2e94 on 8.x-1.x-dev, pending a 8.x-1.11 tagged release.

Thanks for the heads-up, did not notice the merge :)

The dependency is still present in the custom composer.json as a requirement:
https://git.drupalcode.org/project/pathauto/-/blob/8.x-1.x/composer.json...

Yes. the release notes explain why that is there.