From aafdf1239e447157eaf836324730a49a2376ab5c Mon Sep 17 00:00:00 2001
From: Feng Yu <F3n67u@outlook.com>
Date: Sun, 29 May 2022 12:19:04 +0800
Subject: [PATCH] tools: refactor `tools/license2rtf` to ESM
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

This reverts commit 331088f4a450e29f3ea8a28a9f98ccc9f8951386.

PR-URL: https://github.com/nodejs/node/pull/43232
Refs: https://github.com/nodejs/node/issues/43213
Reviewed-By: Darshan Sen <raisinten@gmail.com>
Reviewed-By: LiviaMedeiros <livia@cirno.name>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
---
 Makefile                                  |  2 +-
 tools/{license2rtf.js => license2rtf.mjs} | 34 +++++++++++------------
 vcbuild.bat                               | 10 +++----
 3 files changed, 22 insertions(+), 24 deletions(-)
 rename tools/{license2rtf.js => license2rtf.mjs} (94%)

diff --git a/Makefile b/Makefile
index dc90da2fd1e438..c75e1c74056cc9 100644
--- a/Makefile
+++ b/Makefile
@@ -1096,7 +1096,7 @@ endif
 		$(MACOSOUTDIR)/dist/npm/usr/local/lib/node_modules
 	unlink $(MACOSOUTDIR)/dist/node/usr/local/bin/npm
 	unlink $(MACOSOUTDIR)/dist/node/usr/local/bin/npx
-	$(NODE) tools/license2rtf.js < LICENSE > \
+	$(NODE) tools/license2rtf.mjs < LICENSE > \
 		$(MACOSOUTDIR)/installer/productbuild/Resources/license.rtf
 	cp doc/osx_installer_logo.png $(MACOSOUTDIR)/installer/productbuild/Resources
 	pkgbuild --version $(FULLVERSION) \
diff --git a/tools/license2rtf.js b/tools/license2rtf.mjs
similarity index 94%
rename from tools/license2rtf.js
rename to tools/license2rtf.mjs
index 817da81d7a6ada..d982e679c3e653 100644
--- a/tools/license2rtf.js
+++ b/tools/license2rtf.mjs
@@ -1,8 +1,7 @@
-'use strict';
-
-const assert = require('assert');
-const Stream = require('stream');
-
+import assert from 'node:assert';
+import Stream from 'node:stream';
+import { pipeline } from 'node:stream/promises';
+import { stdin, stdout } from 'node:process';
 
 /*
  * This filter consumes a stream of characters and emits one string per line.
@@ -28,6 +27,7 @@ class LineSplitter extends Stream {
     if (this.buffer) {
       this.emit('data', this.buffer);
     }
+    this.writable = false;
     this.emit('end');
   }
 }
@@ -53,6 +53,7 @@ class ParagraphParser extends Stream {
     if (data)
       this.parseLine(data + '');
     this.flushParagraph();
+    this.writable = false;
     this.emit('end');
   }
 
@@ -212,6 +213,7 @@ class Unwrapper extends Stream {
   end(data) {
     if (data)
       this.write(data);
+    this.writable = false;
     this.emit('end');
   }
 }
@@ -273,6 +275,7 @@ class RtfGenerator extends Stream {
       this.write(data);
     if (this.didWriteAnything)
       this.emitFooter();
+    this.writable = false;
     this.emit('end');
   }
 
@@ -287,19 +290,14 @@ class RtfGenerator extends Stream {
   }
 }
 
-
-const stdin = process.stdin;
-const stdout = process.stdout;
-const lineSplitter = new LineSplitter();
-const paragraphParser = new ParagraphParser();
-const unwrapper = new Unwrapper();
-const rtfGenerator = new RtfGenerator();
-
 stdin.setEncoding('utf-8');
 stdin.resume();
 
-stdin.pipe(lineSplitter);
-lineSplitter.pipe(paragraphParser);
-paragraphParser.pipe(unwrapper);
-unwrapper.pipe(rtfGenerator);
-rtfGenerator.pipe(stdout);
+await pipeline(
+  stdin,
+  new LineSplitter(),
+  new ParagraphParser(),
+  new Unwrapper(),
+  new RtfGenerator(),
+  stdout,
+);
diff --git a/vcbuild.bat b/vcbuild.bat
index 1f200a6818eb6a..eb2fc28628ed47 100644
--- a/vcbuild.bat
+++ b/vcbuild.bat
@@ -405,7 +405,7 @@ if errorlevel 1 echo "Could not create junction to 'out\%config%'." & exit /B
 if not defined sign goto licensertf
 
 call tools\sign.bat Release\node.exe
-if errorlevel 1 echo Failed to sign exe&goto exit
+if errorlevel 1 echo Failed to sign exe, got error code %errorlevel%&goto exit
 
 :licensertf
 @rem Skip license.rtf generation if not requested.
@@ -426,12 +426,12 @@ if "%use_x64_node_exe%"=="true" (
     set exit_code=1
     goto exit
   )
-  %x64_node_exe% tools\license2rtf.js < LICENSE > %config%\license.rtf
+  %x64_node_exe% tools\license2rtf.mjs < LICENSE > %config%\license.rtf
 ) else (
-  %node_exe% tools\license2rtf.js < LICENSE > %config%\license.rtf
+  %node_exe% tools\license2rtf.mjs < LICENSE > %config%\license.rtf
 )
 
-if errorlevel 1 echo Failed to generate license.rtf&goto exit
+if errorlevel 1 echo Failed to generate license.rtf, got error code %errorlevel%&goto exit
 
 :stage_package
 if not defined stage_package goto install-doctools
@@ -538,7 +538,7 @@ if errorlevel 1 goto exit
 
 if not defined sign goto upload
 call tools\sign.bat node-v%FULLVERSION%-%target_arch%.msi
-if errorlevel 1 echo Failed to sign msi&goto exit
+if errorlevel 1 echo Failed to sign msi, got error code %errorlevel%&goto exit
 
 :upload
 @rem Skip upload if not requested