From 76ff3c2e8dec8d00998c86f6c46440cd60827020 Mon Sep 17 00:00:00 2001
From: Sami Vaarala <sami.vaarala@iki.fi>
Date: Thu, 22 Jun 2017 13:30:48 +0300
Subject: [PATCH 1/2] Fix duk_debug.js source file scanning

---
 debugger/duk_debug.js | 34 ++++++++++++++++++++++++----------
 1 file changed, 24 insertions(+), 10 deletions(-)

diff --git a/debugger/duk_debug.js b/debugger/duk_debug.js
index 8dddca87..55c96241 100755
--- a/debugger/duk_debug.js
+++ b/debugger/duk_debug.js
@@ -432,27 +432,36 @@ RateLimited.prototype.trigger = function () {
 function SourceFileManager(directories) {
     this.directories = directories;
     this.extensions = { '.js': true, '.jsm': true };
-    this.files;
+    this.fileMap = {};  // filename as seen by debug target -> file path
+    this.files = [];    // filenames as seen by debug target
 }
 
 SourceFileManager.prototype.scan = function () {
     var _this = this;
-    var fileMap = {};   // absFn -> true
+    var fileMap = {};   // relative path -> file path
     var files;
 
     this.directories.forEach(function (dir) {
         console.log('Scanning source files: ' + dir);
         try {
             recursiveReadSync(dir).forEach(function (fn) {
-                var absFn = path.normalize(path.join(dir, fn));   // './foo/bar.js' -> 'foo/bar.js'
-                var ent;
-
-                if (fs.existsSync(absFn) &&
-                    fs.lstatSync(absFn).isFile() &&
-                    _this.extensions[path.extname(fn)]) {
+                // Example: dir     ../../tests
+                //          normFn  ../../tests/foo/bar.js
+                //          relFn   foo/bar.js
+                var normDir = path.normalize(dir);
+                var normFn = path.normalize(fn);
+                var relFn = path.relative(normDir, normFn);
+
+                if (fs.existsSync(normFn) && fs.lstatSync(normFn).isFile() &&
+                    _this.extensions[path.extname(normFn)]) {
                     // We want the fileMap to contain the filename relative to
-                    // the search dir root.
-                    fileMap[fn] = true;
+                    // the search dir root.  First directory containing a
+                    // certail relFn wins.
+                    if (relFn in fileMap) {
+                        console.log('Found', relFn, 'multiple times, first match wins');
+                    } else {
+                        fileMap[relFn] = normFn;
+                    }
                 }
             });
         } catch (e) {
@@ -464,6 +473,8 @@ SourceFileManager.prototype.scan = function () {
     files.sort();
     this.files = files;
 
+    this.fileMap = fileMap;
+
     console.log('Found ' + files.length + ' source files in ' + this.directories.length + ' search directories');
 };
 
@@ -480,6 +491,9 @@ SourceFileManager.prototype.search = function (fileName) {
     // assigned by selecting a file from a dropdown populated by scanning
     // the filesystem for available sources and there's no way of knowing
     // if the debug target uses the exact same name.
+    //
+    // We intentionally allow any files from the search paths, not just
+    // those scanned to this.fileMap.
 
     function tryLookup() {
         var i, fn, data;

From 26b7341f1f51e2d197662bef31be6537709a9f11 Mon Sep 17 00:00:00 2001
From: Sami Vaarala <sami.vaarala@iki.fi>
Date: Sat, 1 Jul 2017 01:29:02 +0300
Subject: [PATCH 2/2] Releases: duk_debug.js source scan

---
 RELEASES.rst | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/RELEASES.rst b/RELEASES.rst
index a96eaa8f..f114c2c4 100644
--- a/RELEASES.rst
+++ b/RELEASES.rst
@@ -2936,6 +2936,9 @@ Planned
 
 * Fix MSVC cast warning in error augmentation code (GH-1511)
 
+* Fix duk_debug.js --source-dirs scanning for file dropdown; the dropdown was
+  empty (GH-1580)
+
 * Improve support for old MSVC versions without __pragma(), long long, and
   LL/ULL constants (GH-1559, GH-1562)