Closed (won't fix)
Project:
Drupal core
Version:
main
Component:
documentation
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
15 Aug 2018 at 19:49 UTC
Updated:
13 Mar 2026 at 03:22 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
grasmash commentedComment #3
grasmash commentedUpdated patch does not leave behind tarball.
Comment #4
geerlingguy commentedTrying these commands, I get the following:
It seems like those commands might require 8.6.x which is not yet released, so _technically_ this can't be fully tested.
One last note: I remember some discussion over using
tarvsunzipin the issue which generated thequick-startcommand. I'll try to see if there were any caveats to switching.Comment #5
geerlingguy commentedAdding related issue.
And regarding unzip vs tar... this was because of Windows; a couple comments mentioned it's more likely to be available than tar in a Windows environment—most notably:
From: #2911319-121: Provide a single command to install & run Drupal.
Is there a 'latest zipped archive' release URL as well?
Comment #6
grasmash commented@geerlingguy Yes
As far as zip vs tar, the unzip command doesn't have a corollary for --strip-components.
Executing:
Results in extraction to
drupal/drupal-8.5.6/, which makes it impossible to provide a static set of commands to follow. IMO that's a bad user experience.Should we provide a set of commands for Linux & MacOS and a separate set for Windows?
Comment #7
grasmash commentedComment #8
grasmash commentedI've added separate command sets for MacOS/Linux and Windows.
The MacOS/Linux commands use the tar.gz. This these commands do not require manual string replacement and will download the smaller tar.gz archive faster. IMO, this justifies listing two sets of commands.
The Windows commands use the zip file. This does require manual string replacement and will take longer to download the archive.
If we want to simplify this, I'd look into have drupal.org generate an archive that has a consistent (not version dependent) directory structure.
Comment #9
seangambierross commentedI've picked this up at #uoe-d8-contribution
Comment #10
seangambierross commentedI've applied and tested this patch.
https://www.drupal.org/files/issues/2018-08-16/2992879-quick-start-docs-...
Install.txt has been updated with the appropriate MacOS/Linux and Windows commands.
Reviewed and and happy with this patch.
Comment #11
seangambierross commentedComment #12
geerlingguy commentedNot 100% sure if this should be RTBC yet; I think the main concern I have is this splits off the directions between Mac/Linux users and Windows users... which reverses course on the initial purpose of "one set of instructions no matter who you are".
It's nice to be optimized for the best use case, but this would be yet another example where those using Windows will end up being 2nd class citizens, in that their instructions are later in the README, and their instructions are not the same as the instructions the non-Windows users have (meaning fewer people would be using that set of instructions, it's less likely that someone who could quickly make an impact on fixing things would notice that the Windows download link isn't working vs. the one for Mac/Linux, etc.).
I'm not fully opposed to this patch... but I do think it's something we should get more input on, since we ended up spending significant energy (and helping a few first-time Windows-using contributors get their feet wet at the last DrupalCon) making sure the instructions worked on all major platforms, and not just the POSIX-centric ones.
Comment #19
needs-review-queue-bot commentedThe Needs Review Queue Bot tested this issue. It either no longer applies to Drupal core, or fails the Drupal core commit checks. Therefore, this issue status is now "Needs work".
Apart from a re-roll or rebase, this issue may need more work to address feedback in the issue or MR comments. To progress an issue, incorporate this feedback as part of the process of updating the issue. This helps other contributors to know what is outstanding.
Consult the Drupal Contributor Guide to find step-by-step guides for working with issues.
Comment #21
quietone commentedComment #22
quietone commentedDownloading via a tarball is being removed. So, the instructions should be changed to using composer.
Comment #24
quietone commentedComment #25
quietone commentedSo, this hasn't been a response to the questions in #12 for 7 years. And @geerlingguy makes a valid point that having separate instructions for *nix family and Windows isn't ideal. However, we now only promoting installs via composer so this is now a won't fix. The wget commands are being removing in #3086655: Add composer instructions to INSTALL.txt and that is eligible for backport.
Therefor, I have updated credit and am closing this one.