Include Node.js version in notice file (#20133)

Signed-off-by: Tyler Smalley <tyler.smalley@elastic.co>
This commit is contained in:
Tyler Smalley 2018-06-22 07:55:16 -07:00 committed by GitHub
parent 2c1fcf9604
commit 3b983e9c22
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
5 changed files with 20 additions and 6 deletions

View file

@ -35,6 +35,7 @@ export function getNodeDownloadInfo(config, platform) {
url, url,
downloadName, downloadName,
downloadPath, downloadPath,
extractDir extractDir,
version,
}; };
} }

View file

@ -49,11 +49,16 @@ export const CreateNoticeFileTask = {
log.info('Generating build notice'); log.info('Generating build notice');
const { extractDir: nodeDir } = getNodeDownloadInfo(config, config.getLinuxPlatform()); const {
extractDir: nodeDir,
version: nodeVersion,
} = getNodeDownloadInfo(config, config.getLinuxPlatform());
const notice = await generateBuildNoticeText({ const notice = await generateBuildNoticeText({
noticeFromSource, noticeFromSource,
packages, packages,
nodeDir, nodeDir,
nodeVersion,
}); });

View file

@ -33,7 +33,7 @@ import { generateNodeNoticeText } from './generate_node_notice_text';
* @return {undefined} * @return {undefined}
*/ */
export async function generateBuildNoticeText(options = {}) { export async function generateBuildNoticeText(options = {}) {
const { packages, nodeDir, noticeFromSource } = options; const { packages, nodeDir, nodeVersion, noticeFromSource } = options;
const packageNotices = await Promise.all( const packageNotices = await Promise.all(
packages.map(generatePackageNoticeText) packages.map(generatePackageNoticeText)
@ -42,6 +42,6 @@ export async function generateBuildNoticeText(options = {}) {
return [ return [
noticeFromSource, noticeFromSource,
...packageNotices, ...packageNotices,
generateNodeNoticeText(nodeDir), generateNodeNoticeText(nodeDir, nodeVersion),
].join('\n---\n'); ].join('\n---\n');
} }

View file

@ -24,6 +24,7 @@ import { generateBuildNoticeText } from './generate_build_notice_text';
const NODE_MODULES = resolve(__dirname, '__fixtures__/node_modules'); const NODE_MODULES = resolve(__dirname, '__fixtures__/node_modules');
const NODE_DIR = resolve(process.execPath, '../..'); const NODE_DIR = resolve(process.execPath, '../..');
const NODE_VERSION = '8.11.3';
const PACKAGES = [ const PACKAGES = [
{ {
name: 'bar', name: 'bar',
@ -49,6 +50,7 @@ describe('src/dev/build/tasks/notice_file/generate_notice', () => {
(notice = await generateBuildNoticeText({ (notice = await generateBuildNoticeText({
packages: PACKAGES, packages: PACKAGES,
nodeDir: NODE_DIR, nodeDir: NODE_DIR,
nodeVersion: NODE_VERSION,
noticeFromSource: 'NOTICE_FROM_SOURCE\n' noticeFromSource: 'NOTICE_FROM_SOURCE\n'
})) }))
); );
@ -78,6 +80,12 @@ describe('src/dev/build/tasks/notice_file/generate_notice', () => {
)); ));
}); });
it('includes node version', () => {
expect(notice).toEqual(expect.stringContaining(
'This product bundles Node.js v8.11.3'
));
});
it('includes the noticeFromSource', () => { it('includes the noticeFromSource', () => {
expect(notice).toEqual(expect.stringContaining( expect(notice).toEqual(expect.stringContaining(
'NOTICE_FROM_SOURCE' 'NOTICE_FROM_SOURCE'

View file

@ -20,8 +20,8 @@
import { resolve } from 'path'; import { resolve } from 'path';
import { readFileSync } from 'fs'; import { readFileSync } from 'fs';
export function generateNodeNoticeText(nodeDir) { export function generateNodeNoticeText(nodeDir, nodeVersion) {
const licensePath = resolve(nodeDir, 'LICENSE'); const licensePath = resolve(nodeDir, 'LICENSE');
const license = readFileSync(licensePath, 'utf8'); const license = readFileSync(licensePath, 'utf8');
return `This product bundles Node.js.\n\n${license}`; return `This product bundles Node.js v${nodeVersion}.\n\n${license}`;
} }