Bug#1029670: node-browser-pack: puts relative path to prelude into source map

[Yadd]

On 1/26/23 13:04, Thorsten Glaser wrote:

Package: node-browser-pack
Version: 6.1.0+ds+~6.1.1-2
Severity: normal
User: reproducible-bui...@lists.alioth.debian.org
Usertags: buildpath toolchain
X-Debbugs-Cc: t...@mirbsd.de
Control: affects -1 src:dygraphs

     var preludePath = opts.preludePath ||
         path.relative(basedir, defaultPreludePath).replace(/\\/g, '/');

… leads to reproducible-builds differences:

  ····"sources":·[
-········"../../../../usr/share/nodejs/browser-pack/_prelude.js",
+········"../../../../../usr/share/nodejs/browser-pack/_prelude.js",

I think for Debian it should just always write the absolute path in.

Hi,

could you try attached patch ? It keeps related paths where needed.

Cheers,
Yadd
Description: replace ../../../usr by /usr in maps
Author: Yadd <y...@debian.org>
Bug-Debian: https://bugs.debian.org/1029670
Forwarded: not-needed
Last-Update: 2023-01-26

--- a/index.js
+++ b/index.js
@@ -35,7 +35,7 @@
     var basedir = defined(opts.basedir, process.cwd());
     var prelude = opts.prelude || defaultPrelude;
     var preludePath = opts.preludePath ||
-        path.relative(basedir, defaultPreludePath).replace(/\\/g, '/');
+        path.relative(basedir, defaultPreludePath).replace(/\\/g, '/').replace(/^(\.\.\/)*usr\/share/, '/usr/share');
     
     var lineno = 1 + newlinesIn(prelude);
     var sourcemap;