- Issue created by @phenaproxima
- ๐บ๐ธUnited States phenaproxima Massachusetts
Needs sign-off from @pameeela and @tim.plunkett.
- ๐บ๐ธUnited States phenaproxima Massachusetts
I propose we fully remove the launcher from Drupal CMS 1.1.0, and in the next 1.0.x release, we keep the launcher there but update the README.md file.
- ๐บ๐ธUnited States Kristen Pol Santa Cruz, CA, USA
I agree that โless is moreโ and we shouldnโt maintain stuff thatโs not adding enough value. Those of us that use DDEV or the command line are comfortable with a few extra commands. Those who arenโt comfortable with the command line probably wonโt be testing it this way anyway.
- ๐ฌ๐งUnited Kingdom catch
I think this is technically impossible to update for sites because it's in the composer template. And while we would not expect people to ever run it more than once, more or less immediately after downloading, not having code sitting around that can never be updated is good.
- ๐บ๐ธUnited States phenaproxima Massachusetts
That's a great point, and certainly another nail in the launcher's coffin.
- ๐บ๐ธUnited States phenaproxima Massachusetts
Just needs final sign-off from @pameeela.
- ๐ฆ๐บAustralia pameeela
Tweaked the wording to try to make it super clear that DDEV is just one option. It's not *recommended* per se, it is just the one we are supporting.
But otherwise looks good to me!
- ๐บ๐ธUnited States Kristen Pol Santa Cruz, CA, USA
Once this lands, this page needs updating:
https://new.drupal.org/docs/drupal-cms/get-started/install-drupal-cms/in...
- ๐บ๐ธUnited States phenaproxima Massachusetts
@kristen pol: Only when we release 1.1.0. That's why we're keeping the script in the 1.0.x branch, although updating the README in both branches.
-
phenaproxima โ
committed 0ebbc6f5 on 1.x
Issue #3501605 by phenaproxima, kristen pol, catch, tim.plunkett,...
-
phenaproxima โ
committed 0ebbc6f5 on 1.x
-
phenaproxima โ
committed f87754ec on 1.0.x
Issue #3501605 by phenaproxima, kristen pol, catch, tim.plunkett,...
-
phenaproxima โ
committed f87754ec on 1.0.x
- ๐บ๐ธUnited States rfay Palisade, CO, USA
I guess I would have removed the zipball entirely and just done the `ddev composer create`. What's the value of the zipball here?
- ๐บ๐ธUnited States Amber Himes Matz Portland, OR USA
@eojthebrave or I will take care of updating the documentation in the Drupal CMS User Guide at the appropriate time. We'll check in with @pameeela about when it should be updated.
-
phenaproxima โ
committed 8e424cca on 1.x
Issue #3501605 follow-up by phenaproxima: Remove unit test of launcher...
-
phenaproxima โ
committed 8e424cca on 1.x
- ๐บ๐ธUnited States phenaproxima Massachusetts
@amber himes matz: It should be updated when we release Drupal CMS 1.1.0, since that's the first version that will not have the launcher. :)
- ๐บ๐ธUnited States phenaproxima Massachusetts
@rfay: We can't really "remove" the zipball because it's needed when we're generating release nodes on drupal.org.
- ๐บ๐ธUnited States rfay Palisade, CO, USA
Well, you don't have to *use* the zipball :) Or tell anybody it exists!
- ๐บ๐ธUnited States phenaproxima Massachusetts
That would be one to bring to the DA, I think; the download link on https://new.drupal.org/download is not under my control. :)
- ๐ฌ๐งUnited Kingdom dunx
You don't even need
ddev config --project-type=drupal11 --docroot=web
becauseddev config --project-type=drupal --docroot=web
will work as ddev recently changed thedrupal
alias fromdrupal10
todrupal11
. We are assuming some ddev defaults here that ddev might change. I don't know what oversight or warning we might get of those.I think
--create-docroot
is also now a default and I don't think it was before.Related to https://www.drupal.org/project/drupal_cms/issues/3500075 ๐ Test (and then possibly document) installing on shared hosting Active , but what about PHP version? A shared hosting provider might only offer PHP8.3 and Drupal Core 11.1 supports PHP8.4. What happens when PHP8.3 is no longer supported (by Drupal or PHP).
No action required, just a reminder that what works now might easily change. I'm not suggesting we re-enable that installation script, but a more intelligent installation script might be exactly what we need in the future to help more people install Drupal CMS more easily.
Automatically closed - issue fixed for 2 weeks with no activity.
- ๐บ๐ธUnited States phenaproxima Massachusetts
This is a breaking change in a minor release and therefore definitely needs a change record. We also need to update the documentation pages on drupal.org that refer to the launcher script, which is a release-blocking task.
- ๐บ๐ธUnited States phenaproxima Massachusetts
Created the change record, it just needs details filled in. https://git.drupalcode.org/project/drupal_cms/-/wikis/Committer-Info/Cha... can be used as a template.
- Status changed to Fixed
1 day ago 9:49pm 1 April 2025 Automatically closed - issue fixed for 2 weeks with no activity.