Nunjucks-render won't build on windows

[jimboobrien]

Hello,

I have a project that uses your npm package and it runs fine on linux, but it hangs on Windows. Both Windows 10 (my local machine) and our Jenkins servers (Windows Server 2013).

Windows 10: node -v | v10.16.0 npm -v | 6.9.0

I'm using gulp and here is my nunjucksrender() function:

.pipe( nunjucksRender( { path: './demo/nunjucks/', // Relative path to templates ext: '.html', //Extension for compiled templates data: { //Data passed to template cdc_path: 'https://cdc.gov/', }, inheritExtension: false, //If true, uses same extension that is used for template envOptions: { //These are options provided for nunjucks Environment. watch: false }, manageEnv: manageEnvironment, //Hook for managing environment before compilation. loaders: null //If provided, uses that as first parameter to Environment constructor. } ) ) //nunjucks render

I've tried removing all options, just adding in the path, etc. etc.

I've confirmed on multiple machines both linux and windows the same behavior. It works on linux and mac but not on windows.

It literally just sits there in gitbash and hangs, taking up CPU usage without ever finishing.

Just trying to give you the right information to help troubleshoot. I've been researching for days, please tell me if I'm missing something.

Thanks

• Jim

[jimboobrien]

I ended up writing my on npm module and copying a lot of the source code from this repo. Thank you.

In gulpfile:

const nunjucks_test = require( './nunjucks' )

function nunjucks() {
	return src( './demo/**/*.+(html|nunjucks|njk)' )
		.pipe( nunjucks_test( {
			path: './demo/nunjucks/', // Relative path to templates
			ext: '.html', //Extension for compiled templates
			data: { //Data passed to template
				sites: JSON.parse( fs.readFileSync( './demo/json/nav.json' ) )
			},
			inheritExtension: false, //If true, uses same extension that is used for template
			envOptions: { //These are options provided for nunjucks Environment.
				watch: false
			},
			manageEnv: manageEnvironment, //Hook for managing environment before compilation.
			loaders: null //If provided, uses that as first parameter to Environment constructor.
		} ) )
		.pipe( dest( './dist/' ) )
}

nunjucks.js file:

use strict';
const through = require( 'through2' );
const nunjucks = require( 'nunjucks' );
const PluginError = require( 'plugin-error' );

//https://github.com/carlitoplatanito/gulp-nunjucks-render/blob/master/index.js

function compile( options ) {
	//set base dir
	nunjucks.configure( options.path );

	//nunjucks setup
	var env = new nunjucks.FileSystemLoader( options.path );

	//nunjucks env
	var compile_test = new nunjucks.Environment( env, {
		watch: false
	} );

	//add environment variables
	options.manageEnv.call( null, compile_test );

	/*
	 * file = file
	 * enc = encoding - utf-8
	 * cb   = callback function
	 */
	return through.obj( function( file, enc, cb ) {
		//spread data
		var data = { ...options.data };

		//if data is null exit
		if ( file.isNull() ) {
			this.push( file );
			return cb();
		}

		//spread even more if necessary
		if ( file.data ) {
			data = { ...file.data, ...data };
		}

		//if stream exit
		if ( file.isStream() ) {
			this.emit( 'error', new PluginError( 'namespace-nunjucks', 'Streaming not supported' ) );
			return cb();
		}

		//get content for this
		var _this = this;
		var filePath = file.path;

		//try to compile nunjucks
		try {
			//take nunjucks env and run renderString on it
			compile_test.renderString( file.contents.toString(), data, function( err, result ) {
				if ( err ) {
					_this.emit( 'error', new PluginError( 'namespace-nunjucks', err, {fileName: filePath} ) );
					return cb();
				}

				//take the compiled contents and add it to the file as a buffer
				file.contents = Buffer.from( result );

				//force extention if necessary
				//file.path = replaceExtension( filePath, '.html' );

				//this is needed to finally push the file
				_this.push( file );
				cb();
			} );
		} catch ( err ) {
			_this.emit( 'error', new PluginError( 'namespace-nunjucks', err, {fileName: filePath} ) );
			cb();
		}
	} );

}

module.exports = compile;

[jimboobrien]

My Implementation now works on Windows 10, I think it was an issue with lodash or the replaceExtension library maybe.

[jimboobrien]

Also had to change the buffer calls to the new syntax.