drupal-project icon indicating copy to clipboard operation
drupal-project copied to clipboard

Document how to use a different web-root instead of 'web'

Open weitzman opened this issue 10 years ago • 50 comments

Acquia Cloud requires Drupal in a subdir named 'docroot' but this project uses 'web'. The 'web' name is used in scripts in addition to composer.json. Is there some better solution for changing this than find/replace? Would be good to document this somewhere (I volunteer).

weitzman avatar Oct 24 '15 21:10 weitzman

I put a symlink from web to docroot in the project repository, and then ignored docroot from code indexing in PHPStorm. I'm not sure if Acquia lets you serve from a symlink, but I imagine you could rename web to docroot, and then make web the symlink.

deviantintegral avatar Nov 04 '15 22:11 deviantintegral

I use 'drupal' in pantheon-systems/example-drupal7-circle-composer, and I have also seen htdocs and code.

I'd like to use the same directory name in my project as is used here. It doesn't matter to me very much which one we use; if I'm going to have to rename my root directory anyway, 'docroot' is as good of a choice as any.

If we can agree that it makes sense to standardize on docroot, I'll make a PR here to do it, and convert my project over as well.

greg-1-anderson avatar Nov 26 '15 02:11 greg-1-anderson

I would go with web. Many people use the template already for D7 and D8. web is default in https://github.com/symfony/symfony-standard as well.

webflo avatar Nov 26 '15 09:11 webflo

Okay, I will switch to web to follow the standard here and in symfony.

greg-1-anderson avatar Nov 26 '15 14:11 greg-1-anderson

@weitzman Maybe we can revisit this issue and provide a template without the scaffolding files after #77 went in.

webflo avatar Dec 02 '15 09:12 webflo

~So, it turns out that I have to use code when using Drupal 8 on Pantheon. This limitation should not be around forever; once the restriction is lifted, I'll switch back to web in my fork.~

~When using drupal-composer/drupal-scaffold, it is very easy to change the location of the Drupal root--and is getting easier. It might be nice to make scripts/composer/post-install.sh a function of drupal-scaffold. If we also defined the placeholder folders such as modules/contrib in composer.json, and had drupal-scaffold create those as well, then we could probably remove the web directly altogether.~

greg-1-anderson avatar Dec 12 '15 00:12 greg-1-anderson

Is it possible the directory name to be configurable?

RoSk0 avatar Dec 21 '15 10:12 RoSk0

If we removed the web directory, then the only references to the document root would be in the composer.json file and the post-install script. We might be able to have post-install read composer.json and pick up the location of the document root by checking the location of drupal/core; however, there are multiple references to the document root in the composer.json file. You could maybe factor those out to a single reference by adding variable replacement support to composer/installers, but I don't know if such an enhancement would be welcome or not.

greg-1-anderson avatar Dec 21 '15 23:12 greg-1-anderson

The latest version of #77 makes it pretty easy to change the directory name on the fly. There are still a couple of files where it needs to be changed. Unfortunately, the composer command-line tool will not divulge the installation location of a component. You can do it from a custom installer, but, as I mentioned in #77, I recommended that post-install.sh should be kept a shell script so that it is easier for folks to customize.

greg-1-anderson avatar Dec 22 '15 18:12 greg-1-anderson

#77 / #105 have been merged, so relocating the docroot from web to something else now requires modification of only a few files:

  1. composer.json (several locations)
  2. .gitignore (several locations)
  3. post-install.sh (only DOCROOT variable at top of the script)

We could in principal read DOCROOT from composer.json in post-install.sh, but it would still be necessary to make multiple changes to composer.json and .gitignore, so perhaps this is not worthwhile.

Perhaps we should leave this as a customization to be done by the end user?

greg-1-anderson avatar Feb 21 '16 05:02 greg-1-anderson

This feels like there's enough site-specific components to this that trying to automate more probably isn't worth the effort. Let's just make this a task to document the steps above? So far they're working fine for me.

deviantintegral avatar Mar 07 '16 16:03 deviantintegral

Yes, a docs PR sounds about right. Symlink is simplest solution for many.

weitzman avatar Mar 16 '16 04:03 weitzman

relocating the docroot from web to something else now requires modification of only a few files:

composer.json (several locations)
.gitignore (several locations)
post-install.sh (only DOCROOT variable at top of the script)

After #128 now files are:

  • composer.json (several locations)
  • .gitignore (several locations)
  • scripts/composer/ScriptHandler.php

But as @weitzman is suggesting a symlink is still the simplest solution ("web" -> "any" or "any" -> "web").

gambry avatar Apr 06 '16 16:04 gambry

We could follow symfonys model and add some extra config to our composer.json and use in our ScriptHandler. see https://github.com/symfony/symfony-standard/blob/master/composer.json#L53

webflo avatar Apr 08 '16 14:04 webflo

That's a great idea, but I think it would need to be a feature of Composer Installers. It would be better as a feature of Composer Installers, anyway, as it seems like it would be commonly useful to be able to do replacements from extra config in destination paths -- and Composer Installers already supports replacements in paths.

greg-1-anderson avatar Apr 08 '16 15:04 greg-1-anderson

@greg-1-anderson so far composer Installers supports replacements in paths based on package $name, $vendor, $type. Do you think having project based replacements will be a feature they would take in consideration?

If they won't, I would see it more like a composer script, which ask for docroot folder name, update the composer.json paths and add an extra configuration parameter to be used in ScriptHandler.

gambry avatar Apr 09 '16 09:04 gambry

Just wondering because I am planning on doing this. Could you create a relative symlink from docroot(or whatever you prefer) to web?

joelpittet avatar Feb 13 '17 21:02 joelpittet

@joelpittet Yes, some folks do that; should work in most environments.

greg-1-anderson avatar Feb 13 '17 21:02 greg-1-anderson

Could we use a composer config variable for this? I was just thinking on this and it might be a simple solution.

nalipaz avatar Feb 13 '17 23:02 nalipaz

See my previous comment. The docroot variable exists in a few plain text files. Some fixup is required if the user wants to select something else.

greg-1-anderson avatar Feb 14 '17 00:02 greg-1-anderson

We could in principal read DOCROOT from composer.json in post-install.sh, but it would still be necessary to make multiple changes to composer.json and .gitignore, so perhaps this is not worthwhile.

To that remark, wouldn't it be possible to store the value of the docroot in composer config, maybe composer config documentroot something and then apply it in the composer.json with a "{$documentroot}/modules/contrib/{$name}": ["type:drupal-module"], as an example? I would assume the variable should be able to be obtained in the post-install.sh too. That only leaves the .gitignore which could be edited manually by most or we could go so far as to sed it to the correct value if it is set, or perhaps change to a wildcard for simplicity.

nalipaz avatar Feb 14 '17 00:02 nalipaz

@nalipaz Yeah, that would work for sure. The one wrinkle is that at the moment, we use composer/installers to remap the module install locations, and it does not support a {$documentroot} substitution. At least, in the past it didn't. If you know how to make that work today with a composer config, then a PR to demonstrate that would be most welcome.

greg-1-anderson avatar Feb 14 '17 00:02 greg-1-anderson

@greg-1-anderson I appreciate the details on that. I can't say I know how to make that work and was only speaking theoretically here since it seems the most integrated way. If I can gather a moment I might try my hand at the idea and of course others are welcome to try as well.

nalipaz avatar Feb 14 '17 00:02 nalipaz

I locally changed the following: • composer.json (several locations) • .gitignore (several locations) • scripts/composer/ScriptHandler.php

To say "docroot" instead of "web" but I'm still not getting the root files in docroot like index.php. I know those come from the drupal-scaffold project but I don't have this issue when it's under /web. Is there another hidden config location I cannot find?

fabean avatar Mar 01 '17 18:03 fabean

Okay, I lied... Totally works it must have been a composer cache or something that kept wanting to put things in /web. Sorry.

fabean avatar Mar 03 '17 19:03 fabean

@fabean Please review https://github.com/drupal-composer/drupal-project/pull/255. This removes the requirement to change the ScriptHandler

webflo avatar Mar 22 '17 11:03 webflo

Just tried to deploy to Acquia - docroot symlink did not work for me at all (Pro tier). It kept giving the default Acquia page when no code exists.

Switching to Live Development mode brought it up, but I don't work in that way.

I had to mv web docroot and create a local symlink for web so Docker continued working. That got Acquia up and going.

kevinquillen avatar Feb 12 '19 17:02 kevinquillen

I think this could be resolved now by adding an FAQ entry to the ReadMe like this:

To use a different web root directory, e.g. docroot instead of web:

  1. Create the project without installing it using the --no-install option. e.g.:

composer create-project drupal-composer/drupal-project:8.x-dev my_site_name_dir --no-interaction --no-install

  1. cd into the project directory and edit the 'installer-paths' section in the composer.json file. Replace the web/* paths with the web root you want, e.g. docroot/

  2. Finish the install using the command:

composer install

cgmonroe avatar Sep 20 '19 17:09 cgmonroe

@cgmonroe Good idea, please submit an PR.

webflo avatar Sep 21 '19 07:09 webflo

The post-install script now uses Drupal Finder to locate the web root, so only .gitignore and the composer.json files would need to be modified to use a different location.

greg-1-anderson avatar Sep 21 '19 20:09 greg-1-anderson