https://issues.dlang.org/show_bug.cgi?id=23177

Dlang Bot <dlang-bot@dlang.rocks> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |pull

--- Comment #1 from Dlang Bot <dlang-bot@dlang.rocks> ---
@rikkimax updated dlang/dmd pull request #14200 "Fix Issue 23177 - ModuleInfo is not exported on Windows" fixing this issue:

- Fix issue 23177

https://github.com/dlang/dmd/pull/14200

--

https://issues.dlang.org/show_bug.cgi?id=23177

--- Comment #2 from Richard Cattermole <alphaglosined@gmail.com> ---
It was indeed as simple as adding:

```d
objmod.export_symbol(m.csym, 0);
```

to ``genModuleInfo``.

However I ran into a lot of issues in the test suite (mostly fixable, since it just needed to know about the fact that import + export libraries were being generated).

For the test it was as simple as adding a new module ``test/dshell/extra-files/dll/issue23177.d`` and importing it to ``mydll.d`` and ``testdll.d``.

I managed to get it down to only one test failing. Unicode symbols are part of it ``runnable/testmodule.d``. I have no idea how to fix this particular issue, so unfortunately my fix is on hold until that can be resolved.

--

https://issues.dlang.org/show_bug.cgi?id=23177

Dlang Bot <dlang-bot@dlang.rocks> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|---                         |FIXED

--- Comment #3 from Dlang Bot <dlang-bot@dlang.rocks> ---
dlang/dmd pull request #14200 "Fix Issue 23177 - ModuleInfo is not exported on Windows" was merged into stable:

- 6b93e00b535fc020794c50010e1b9040fa6e2476 by richard andrew cattermole:
  Fix Issue 23177 - ModuleInfo is not exported on Windows

https://github.com/dlang/dmd/pull/14200

--

https://issues.dlang.org/show_bug.cgi?id=23177

Richard Cattermole <alphaglosined@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|RESOLVED                    |REOPENED
         Resolution|FIXED                       |---

--- Comment #4 from Richard Cattermole <alphaglosined@gmail.com> ---
Unfortunately, my fix is nowhere near good enough.

Here is a code snippet that will fail, it will need to be added to dshell/dll test in its main function (in some form).

```d
void main() {
    import std.stdio;
    foreach (m; ModuleInfo) {
        writeln(m.name);
        if (auto i = m.importedModules()) {
            writeln(i);
            stdout.flush;
            foreach (j; i) {
                writeln("  - ", j.name.ptr);
                stdout.flush;
            }
        }
    }
}

```

--

https://issues.dlang.org/show_bug.cgi?id=23177

--- Comment #5 from Dlang Bot <dlang-bot@dlang.rocks> ---
dlang/dmd pull request #14206 "Revert "Fix Issue 23177 - ModuleInfo is not exported on Windows"" was merged into stable:

- aa10936019c4b68ffef39fff6af605e067f0e9e2 by Martin Kinkelin:
  Revert "Fix Issue 23177 - ModuleInfo is not exported on Windows"

  This reverts commit 2aabe864da7c95ea8eb8abbca7c4f0b1dcfe55ae.

https://github.com/dlang/dmd/pull/14206

--

https://issues.dlang.org/show_bug.cgi?id=23177

--- Comment #6 from Dlang Bot <dlang-bot@dlang.rocks> ---
@BorisCarvajal updated dlang/dmd pull request #14226 "Fix Issue 23172 - [REG2.100] Wrong cast inserted for ternary operator and non-int enums" fixing this issue:

- Fix Issue 23177 - ModuleInfo is not exported on Windows

- Revert "Fix Issue 23177 - ModuleInfo is not exported on Windows"

  This reverts commit 2aabe864da7c95ea8eb8abbca7c4f0b1dcfe55ae.

https://github.com/dlang/dmd/pull/14226

--

https://issues.dlang.org/show_bug.cgi?id=23177

--- Comment #7 from Dlang Bot <dlang-bot@dlang.rocks> ---
@ibuclaw created dlang/dmd pull request #14280 "merge stable" fixing this issue:

- Fix Issue 23177 - ModuleInfo is not exported on Windows

- Revert "Fix Issue 23177 - ModuleInfo is not exported on Windows"

  This reverts commit 2aabe864da7c95ea8eb8abbca7c4f0b1dcfe55ae.

https://github.com/dlang/dmd/pull/14280

--

https://issues.dlang.org/show_bug.cgi?id=23177

Dlang Bot <dlang-bot@dlang.rocks> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|REOPENED                    |RESOLVED
         Resolution|---                         |FIXED

--- Comment #8 from Dlang Bot <dlang-bot@dlang.rocks> ---
dlang/dmd pull request #14280 "merge stable" was merged into master:

- 2aabe864da7c95ea8eb8abbca7c4f0b1dcfe55ae by richard andrew cattermole:
  Fix Issue 23177 - ModuleInfo is not exported on Windows

- 4ee3ac43f5235d3798c0c0cd9f317f5e1aa07ce4 by Martin Kinkelin:
  Revert "Fix Issue 23177 - ModuleInfo is not exported on Windows"

  This reverts commit 2aabe864da7c95ea8eb8abbca7c4f0b1dcfe55ae.

https://github.com/dlang/dmd/pull/14280

--

https://issues.dlang.org/show_bug.cgi?id=23177

Richard Cattermole <alphaglosined@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|RESOLVED                    |REOPENED
         Resolution|FIXED                       |---

--

https://issues.dlang.org/show_bug.cgi?id=23177

--- Comment #9 from Dlang Bot <dlang-bot@dlang.rocks> ---
@rikkimax created dlang/dmd pull request #14647 "Fix issue 23177 - ModuleInfo is not exported on Windows" fixing this issue:

- Fix issue 23177 - ModuleInfo is not exported on Windows

https://github.com/dlang/dmd/pull/14647

--