Thread overview
[Issue 21844] makedeps option adds spurious/incorrect dependency
Mar 04, 2022
Tobias Pankrath
Mar 04, 2022
Dlang Bot
Mar 04, 2022
Dlang Bot
Mar 04, 2022
Luís Ferreira
Mar 05, 2022
Dlang Bot
Mar 08, 2022
Dlang Bot
March 04, 2022
https://issues.dlang.org/show_bug.cgi?id=21844

Tobias Pankrath <tobias@pankrath.net> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |tobias@pankrath.net

--- Comment #1 from Tobias Pankrath <tobias@pankrath.net> ---
mir-core has the same problem, which also boils down to __traits(compiles, import <module>), adding a file to the makedeps.

One could argue, that it is correct to add files to the makedeps, but given multiple import paths it's hard to do it correctly.

--
March 04, 2022
https://issues.dlang.org/show_bug.cgi?id=21844

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

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

--- Comment #2 from Dlang Bot <dlang-bot@dlang.rocks> ---
@Panke created dlang/dmd pull request #13755 "Fix Issue 21844: makedeps contain non-existing files" fixing this issue:

- Fix Issue 21844: makedeps contain non-existing files

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

--
March 04, 2022
https://issues.dlang.org/show_bug.cgi?id=21844

--- Comment #3 from Dlang Bot <dlang-bot@dlang.rocks> ---
@Panke updated dlang/dmd pull request #13756 "Fix issue 21844: makedeps contain non-existing files" fixing this issue:

- Fix Issue 21844: makedeps contain non-existing files

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

--
March 04, 2022
https://issues.dlang.org/show_bug.cgi?id=21844

Luís Ferreira <contact@lsferreira.net> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |contact@lsferreira.net
           Severity|minor                       |major

--
March 05, 2022
https://issues.dlang.org/show_bug.cgi?id=21844

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

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

--- Comment #4 from Dlang Bot <dlang-bot@dlang.rocks> ---
dlang/dmd pull request #13755 "Fix Issue 21844: makedeps contain non-existing files" was merged into stable:

- 315fdaf3040cfc37969549bc95c38ff81a4f4aef by Tobias Pankrath:
  Fix Issue 21844 - makedeps contain non-existing files

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

--
March 08, 2022
https://issues.dlang.org/show_bug.cgi?id=21844

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

- 24aa1e71af36f857c3a0863b4266225061436989 by Tobias Pankrath:
  Fix Issue 21844 - makedeps contain non-existing files

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

--