WordPress.org

Make WordPress Core

Opened 4 weeks ago

Last modified 4 weeks ago

#44263 new defect (bug)

`grunt build` removes `wp-config.php` file.

Reported by: desrosj Owned by:
Milestone: Awaiting Review Priority: normal
Severity: normal Version: trunk
Component: Build/Test Tools Keywords: dev-feedback
Focuses: Cc:

Description

Every time I run grunt build locally, I am presented with an install screen when running WordPress from the build directory.

Before r43309, the wp-config.php file was created in the src directory. When grunt build runs, it removes everything in the build directory and copies everything in the src directory into build. But now that you can't run WordPress from the src directory, the wp-config.php file is never created in src to be copied to `build.

Change History (5)

#1 @desrosj
4 weeks ago

Forgot to specify, I am using VVV.

This ticket was mentioned in Slack in #core by azaozz. View the logs.


4 weeks ago

#3 @jeremyfelt
4 weeks ago

Poor VVV planning on my part when core moved to the develop.svn workflow . :)

The wp-config-sample.php and wp-config-tests-sample.php files are already in the correct location (project root). VVV (and others') installations via WP-CLI can be adjusted to match. The make/core installing from svn docs already do a pretty good job of referencing it in its correct location.

I don't think there's anything we need to do in core for this one, though having the ticket here will help with awareness.

#4 @azaozz
4 weeks ago

having the ticket here will help with awareness.

Right. The wp-config.php file should be one level up, in the WP checkout dir, the parent directory of /src and /build. It may need to be moved there after running the install.

The only thing we can possibly do to help from core would be to check the location of wp-config-sample.php and wp-config-tests-sample.php and create wp-config.php next to them. (That looks like a pretty simple patch imho).

Last edited 4 weeks ago by azaozz (previous) (diff)

This ticket was mentioned in Slack in #core by jeremyfelt. View the logs.


4 weeks ago

Note: See TracTickets for help on using tickets.