Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added a CLI flag that lets you change the default pages path from ./pages to something else #391

Closed
wants to merge 2 commits into from
Closed
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
109 changes: 70 additions & 39 deletions lib/bin/cli.js
Original file line number Diff line number Diff line change
Expand Up @@ -4,76 +4,107 @@ const fs = require('fs-extra')
const path = require('path')
import _ from 'lodash'

// Copy our load-context function to root of site in a dot file.
const gatsbyFile = `${__dirname}/../utils/load-context.js`
const siteDirectory = path.resolve('.')
const fileName = `${siteDirectory}/.gatsby-context.js`
fs.copy(gatsbyFile, fileName)

const defaultHost = process.platform === 'win32'
? 'localhost'
: '0.0.0.0'

const directory = path.resolve('.')

const HOST_OPTION_ARGS = [
'-H, --host <hostname>',
`Hostname at which to start the server. Defaults to ${defaultHost}.`,
defaultHost,
]
const PAGES_PATH_OPTION_ARGS = [
'-P, --pagesPath <pagesPath>',
'Path to a collection of page source files. ' +
'Defaults to `./pages`.',
'./pages',
]
const PORT_OPTION_ARGS = [
'-p, --port <port>',
'Port to start the server on. Defaults to 8000.',
8000,
]
const OPEN_OPTION_ARGS = [
'-o, --open',
'Open a browser window once the server has started.',
false,
]

function writeGatsbyContextFile (pagesPath) {
// Copy our load-context function to root of site in a dot file.
const getLoadContext = require(
path.resolve(__dirname, '..', 'utils', 'get-load-context.js'),
)
fs.writeFile(
path.resolve('.', '.gatsby-context.js'),
getLoadContext(path.relative(directory, pagesPath)),
)
}

function prepareOptions (command) {
const options = {
...command,
directory,
}
if (command.pagesPath) {
options.pagesPath = path.resolve(directory, command.pagesPath)
}
return options
}

function getCommandHandler (fn) {
return function (command) {
const options = prepareOptions(command)
writeGatsbyContextFile(options.pagesPath)
fn(options)
}
}

program
.version(packageJson.version)
.usage('[command] [options]')

program.command('develop')
.description('Start development server. Watches files and rebuilds and hot reloads if something changes') // eslint-disable-line max-len
.option('-H, --host <url>',
`Set host. Defaults to ${defaultHost}`,
defaultHost
)
.option('-p, --port <port>', 'Set port. Defaults to 8000', '8000')
.option('-o, --open', 'Open the site in your browser for you.')
.action((command) => {
.option(...HOST_OPTION_ARGS)
.option(...OPEN_OPTION_ARGS)
.option(...PORT_OPTION_ARGS)
.option(...PAGES_PATH_OPTION_ARGS)
.action(getCommandHandler(options => {
const develop = require('../utils/develop')
const p = {
...command,
directory,
}
develop(p)
})
develop(options)
}))

program.command('build')
.description('Build a Gatsby project.')
.option(...PAGES_PATH_OPTION_ARGS)
.option('--prefix-links', 'Build site with links prefixed (set prefix in your config).')
.action((command) => {
.action(getCommandHandler(options => {
// Set NODE_ENV to 'production'
process.env.NODE_ENV = 'production'

const build = require('../utils/build')
const p = {
...command,
directory,
}
build(p, (err) => {
build(options, (err) => {
if (err) {
throw err
} else {
console.log('Done')
}
})
})
}))

program.command('serve-build')
.description('Serve built site.')
.option('-H, --host <url>',
`Set host. Defaults to ${defaultHost}`,
defaultHost
)
.option('-p, --port <port>', 'Set port. Defaults to 8000', '8000')
.option('-o, --open', 'Open the site in your browser for you.')
.action((command) => {
.option(...HOST_OPTION_ARGS)
.option(...OPEN_OPTION_ARGS)
.option(...PAGES_PATH_OPTION_ARGS)
.option(...PORT_OPTION_ARGS)
.action(getCommandHandler(options => {
const serve = require('../utils/serve-build')
const p = {
...command,
directory,
}
serve(p)
})
serve(options)
}))

program
.command('new [rootPath] [starter]')
Expand Down
3 changes: 2 additions & 1 deletion lib/isomorphic/create-routes.js
Original file line number Diff line number Diff line change
Expand Up @@ -10,9 +10,10 @@ module.exports = (files, pagesReq) => {
file.file.name === '_template'
)

const rootTemplate = templates.find(t => t.templatePath === '/')
const routes = {
path: prefixLink('/'),
component: require('pages/_template'),
component: rootTemplate ? pagesReq(`./${rootTemplate.requirePath}`) : null,
childRoutes: [],
indexRoute: {},
pages,
Expand Down
7 changes: 4 additions & 3 deletions lib/loaders/config-loader/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -7,13 +7,14 @@ const globPages = require('../../utils/glob-pages')
module.exports = function (source) {
this.cacheable()
const callback = this.async()
const directory = loaderUtils.parseQuery(this.query).directory
const pagesPath = loaderUtils.parseQuery(this.query).pagesPath
const config = toml.parse(source)

const value = {}
value.config = config
value.relativePath = path.relative('.', directory)
globPages(directory, (err, pagesData) => {
value.relativePath = path.relative('.', pagesPath)

globPages(pagesPath, (err, pagesData) => {
value.pages = pagesData
return callback(null, `module.exports = ${JSON.stringify(value, void 0, '\t')}`)
})
Expand Down
7 changes: 5 additions & 2 deletions lib/utils/build-html.js
Original file line number Diff line number Diff line change
Expand Up @@ -7,9 +7,12 @@ import fs from 'fs'
const debug = require('debug')('gatsby:html')

module.exports = (program, callback) => {
const { directory } = program
const {
directory,
pagesPath,
} = program

globPages(directory, (err, pages) => {
globPages(pagesPath, (err, pages) => {
debug('generating static HTML')
const routes = pages.filter((page) => page.path).map((page) => page.path)

Expand Down
5 changes: 2 additions & 3 deletions lib/utils/build-page/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -4,10 +4,9 @@ import objectAssign from 'object-assign'
import pathResolver from './path-resolver'
import loadFrontmatter from './load-frontmatter'

export default function buildPage (directory: string, page: string) {
export default function buildPage (pagesPath: string, page: string) {
const pageData = loadFrontmatter(page)

const relativePath: string = path.relative(path.join(directory, 'pages'), page)
const relativePath: string = path.relative(pagesPath, page)
const pathData = pathResolver(relativePath, pageData)

return objectAssign({}, pathData, { data: pageData })
Expand Down
7 changes: 3 additions & 4 deletions lib/utils/build-page/path-resolver.js
Original file line number Diff line number Diff line change
@@ -1,9 +1,9 @@
/* @flow */
import slash from 'slash'
import parsePath from 'parse-filepath'
import path from 'path'
import slash from 'slash'
import urlResolver from './url-resolver'


export default function pathResolver (relativePath: string, pageData: {} = {}) {
const data = {}

Expand All @@ -23,9 +23,8 @@ export default function pathResolver (relativePath: string, pageData: {} = {}) {

// Set the "template path"
if (data.file.name === '_template') {
data.templatePath = `/${data.file.dirname}/`
data.templatePath = path.join(path.sep, data.file.dirname, path.sep)
}

return data
}

2 changes: 1 addition & 1 deletion lib/utils/build.js
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@ function customPost (program, callback) {
if (customPostBuild) {
console.log('Performing custom post-build steps')

return globPages(directory, (globError, pages) =>
return globPages(program.pagesPath, (globError, pages) =>
customPostBuild(pages, (error) => {
if (error) {
console.log('customPostBuild function failed')
Expand Down
6 changes: 3 additions & 3 deletions lib/utils/develop.js
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@ function startServer (program, launchPort) {
const serverPort = launchPort || program.port

// Load pages for the site.
return globPages(directory, (err, pages) => {
return globPages(program.pagesPath, (err, pages) => {
const compilerConfig = webpackConfig(program, directory, 'develop', program.port)

const compiler = webpack(compilerConfig.resolve())
Expand Down Expand Up @@ -90,7 +90,7 @@ function startServer (program, launchPort) {
path: '/{path*}',
handler: {
directory: {
path: `${program.directory}/pages`,
path: program.pagesPath,
listing: false,
index: false,
},
Expand All @@ -104,7 +104,7 @@ function startServer (program, launchPort) {
const parsed = parsePath(request.path)
const page = _.find(pages, (p) => p.path === (`${parsed.dirname}/`))

let absolutePath = `${program.directory}/pages`
let absolutePath = program.pagesPath
let path
if (page) {
path = `/${parsePath(page.requirePath).dirname}/${parsed.basename}`
Expand Down
26 changes: 26 additions & 0 deletions lib/utils/get-load-context.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,26 @@
function getLoadContext (relativePagesPath) {
return `
/* @flow weak */
// This file is auto-written and used by Gatsby to require
// files from your \`--pagesPath\` directory.
module.exports = function (callback) {
let context = require.context(
'./${relativePagesPath}',
true,
/(coffee|cjsx|jsx|js|markdown|md|html|json|yaml|toml)$/,
)
if (module.hot) {
module.hot.accept(context.id, () => {
context = require.context(
'./${relativePagesPath}',
true,
/(coffee|cjsx|jsx|js|markdown|md|html|json|yaml|toml)$/
)
return callback(context)
})
}
return callback(context)
}
`.trim()
}
module.exports = getLoadContext
14 changes: 7 additions & 7 deletions lib/utils/glob-pages.js
Original file line number Diff line number Diff line change
Expand Up @@ -5,21 +5,21 @@ const debug = require('debug')('gatsby:glob')

import pageFileTypes from './page-file-types'

function globQuery (directory) {
function globQuery (pagesPath) {
const fileGlobQuery = pageFileTypes.map((type) => `*.${type}`)
const joinedFileQuery = fileGlobQuery.join('|')
return `${directory}/pages/**/?(${joinedFileQuery})`
return `${pagesPath}/**/?(${joinedFileQuery})`
}

let globCache
function createCache (directory, callback) {
function createCache (pagesPath, callback) {
const pagesData = []

glob(globQuery(directory), null, (err, pages) => {
glob(globQuery(pagesPath), null, (err, pages) => {
if (err) { return callback(err) }

pages.forEach((page) => {
pagesData.push(buildPage(directory, page))
pagesData.push(buildPage(pagesPath, page))
})

debug(`globbed ${pagesData.length} pages`)
Expand All @@ -28,9 +28,9 @@ function createCache (directory, callback) {
})
}

module.exports = function globPages (directory, callback) {
module.exports = function globPages (pagesPath, callback) {
if (typeof globCache === 'undefined') {
createCache(directory, callback)
createCache(pagesPath, callback)
} else {
callback(null, globCache)
}
Expand Down
13 changes: 0 additions & 13 deletions lib/utils/load-context.js

This file was deleted.

6 changes: 3 additions & 3 deletions lib/utils/post-build.js
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ import globPages from './glob-pages'
module.exports = (program, cb) => {
const directory = program.directory

return globPages(directory, (err, pages) => {
return globPages(program.pagesPath, (err, pages) => {
debug('copying files')
// Async callback to copy each file.
const copy = function copyFile (file, callback) {
Expand All @@ -21,7 +21,7 @@ module.exports = (program, cb) => {
// rewritten to my-sweet-blog-post, we find that path rewrite so
// our asset gets copied to the right directory.
const parsed = parsePath(file)
const relativePath = path.relative(`${directory}/pages`, file)
const relativePath = path.relative(program.pagesPath, file)
let oldDirectory = parsePath(relativePath).dirname
let newPath = ''

Expand Down Expand Up @@ -57,7 +57,7 @@ module.exports = (program, cb) => {

// Copy static assets to public folder.
const assetTypes = '*.jpg|*.jpeg|*.png|*.pdf|*.gif|*.ico|*.svg|*.pdf|*.txt|CNAME'
const globString = `${directory}/pages/**/?(${assetTypes})`
const globString = `${program.pagesPath}/**/?(${assetTypes})`
return glob(globString, null, (e, files) =>
async.map(files, copy, (error, results) =>
cb(error, results)
Expand Down
5 changes: 2 additions & 3 deletions lib/utils/webpack.config.js
Original file line number Diff line number Diff line change
Expand Up @@ -227,7 +227,7 @@ module.exports = (program, directory, suppliedStage, webpackPort = 1500, routes
test: /config\.toml/,
loader: 'config',
query: {
directory,
pagesPath: program.pagesPath,
},
})
// Image loaders.
Expand Down Expand Up @@ -417,9 +417,8 @@ module.exports = (program, directory, suppliedStage, webpackPort = 1500, routes
}

const config = new Config()

config.merge({
context: `${directory}/pages`,
context: program.pagesPath,
node: {
__filename: true,
},
Expand Down