[Issue 7016] local import does not create -deps dependency
https://issues.dlang.org/show_bug.cgi?id=7016 --- Comment #35 from Martin Nowak --- Nobody thought about incremental rebuilds with other build tools like make, cmake, or ninja, despite the argument being made in comment 5. Incremental rebuilds in those tools are now slower as the fix implies running semantic3 on all imports when collecting -deps, even though only shallow dependencies are needed. Furthermore the deps options isn't (and apparently never has been) used by rdmd. So leaving only a recursive dependency option in the compiler hasn't served any purpose. Furthermore the "fix" is broken since inception, see Issue 17898. --
[Issue 7016] local import does not create -deps dependency
https://issues.dlang.org/show_bug.cgi?id=7016 --- Comment #34 from github-bugzi...@puremagic.com --- Commit pushed to dmd-cxx at https://github.com/dlang/phobos https://github.com/dlang/phobos/commit/29273f261c94e1bbe1042ec58a362d70cb344188 remove .deps file generation --
[Issue 7016] local import does not create -deps dependency
https://issues.dlang.org/show_bug.cgi?id=7016 --- Comment #33 from github-bugzi...@puremagic.com --- Commits pushed to newCTFE at https://github.com/dlang/dmd https://github.com/dlang/dmd/commit/6be32270b411d13f49ffccaa055b2cb13060c495 Fix Issue 7016 https://github.com/dlang/dmd/commit/afebe0c2ba89594b434d676fbe0c050389a5b48c Merge pull request #6748 from RazvanN7/Fix_Issue_7016 --
[Issue 7016] local import does not create -deps dependency
https://issues.dlang.org/show_bug.cgi?id=7016 --- Comment #32 from github-bugzi...@puremagic.com --- Commit pushed to stable at https://github.com/dlang/phobos https://github.com/dlang/phobos/commit/29273f261c94e1bbe1042ec58a362d70cb344188 remove .deps file generation --
[Issue 7016] local import does not create -deps dependency
https://issues.dlang.org/show_bug.cgi?id=7016 --- Comment #31 from github-bugzi...@puremagic.com --- Commits pushed to stable at https://github.com/dlang/dmd https://github.com/dlang/dmd/commit/6be32270b411d13f49ffccaa055b2cb13060c495 Fix Issue 7016 https://github.com/dlang/dmd/commit/afebe0c2ba89594b434d676fbe0c050389a5b48c Merge pull request #6748 from RazvanN7/Fix_Issue_7016 --
[Issue 7016] local import does not create -deps dependency
https://issues.dlang.org/show_bug.cgi?id=7016 github-bugzi...@puremagic.com changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution|--- |FIXED --
[Issue 7016] local import does not create -deps dependency
https://issues.dlang.org/show_bug.cgi?id=7016 --- Comment #30 from github-bugzi...@puremagic.com --- Commits pushed to master at https://github.com/dlang/dmd https://github.com/dlang/dmd/commit/6be32270b411d13f49ffccaa055b2cb13060c495 Fix Issue 7016 https://github.com/dlang/dmd/commit/afebe0c2ba89594b434d676fbe0c050389a5b48c Merge pull request #6748 from RazvanN7/Fix_Issue_7016 Fix Issue 7016 - local import does not create -deps dependency merged-on-behalf-of: Andrei Alexandrescu --
[Issue 7016] local import does not create -deps dependency
https://issues.dlang.org/show_bug.cgi?id=7016 --- Comment #29 from github-bugzi...@puremagic.com --- Commit pushed to master at https://github.com/dlang/phobos https://github.com/dlang/phobos/commit/29273f261c94e1bbe1042ec58a362d70cb344188 remove .deps file generation - for development, people can use the much faster std/algorithm.test targets - for batch unittest recompiling isn't of much interest as all the testers just start from a clean slate anyhow - furthermore the .deps generation is broken anyhow, see Issue 7016 --
[Issue 7016] local import does not create -deps dependency
https://issues.dlang.org/show_bug.cgi?id=7016 --- Comment #28 from Vladimir Panteleev --- (In reply to RazvanN from comment #27) > -> add another compiler flag: "-rdeps" (or maybe a more inspired name) which >will recurs through all the dependencies, excluding druntime or phobos > imports. >The rationale behind this is that both druntime and phobos have circular > imports >which will cause the recursive implementation to fail; also having just 1 > phobos >import will lead to a cascade of imports which will be hard to follow. Hard-coding blacklists for druntime and phobos into the compiler sounds wrong. (People may need to build D against other libraries with the same issues, or build against Phobos from source - i.e. without linking to libphobos2.) If this is done, there would probably need to be ways to add to or remove from that list. Either way, it sounds like this will also result is a huge slowdown... > Any better ideas? Would it be a far stretch from this to add a switch to just do code generation whenever an import is done, whether it's in semantic3 or earlier? Meaning, move rdmd into dmd itself. I think all we need is a switch to enable this behavior (e.g. -r, for recursive compilation), and switches to add or remove from the list of packages that will be excluded from compilation (default being std.*, etc.* and core.*). This will make rdmd much faster, too, since it will no longer need to ask the compiler to perform semantics just to get a list of modules to compile and do it all over again during the real compilation. --
[Issue 7016] local import does not create -deps dependency
https://issues.dlang.org/show_bug.cgi?id=7016 --- Comment #27 from RazvanN --- Running semantic3 recursively on dependencies will end up in an infinite loop when there are circular imports. So, I propose the following solution: -> the current -deps implementation should remain untouched and this will print all the direct dependencies. -> add another compiler flag: "-rdeps" (or maybe a more inspired name) which will recurs through all the dependencies, excluding druntime or phobos imports. The rationale behind this is that both druntime and phobos have circular imports which will cause the recursive implementation to fail; also having just 1 phobos import will lead to a cascade of imports which will be hard to follow. Any better ideas? RazvanN --
[Issue 7016] local import does not create -deps dependency
https://issues.dlang.org/show_bug.cgi?id=7016 Timothee Cour changed: What|Removed |Added Severity|normal |blocker --
[Issue 7016] local import does not create -deps dependency
https://issues.dlang.org/show_bug.cgi?id=7016
--- Comment #26 from Timothee Cour ---
I confirm this is still broken with dmd HEAD DMD64 D Compiler
v2.073.0-devel-a2b772f
RazvanN, are you sure you're not using rdmd ?
Please re-run using -v and show your entire log.
rdmd -deps=a.deps --force -c a.d && cat a.deps
would indeed produce a line "c (c.d) : private : object"
dmd -deps=a.deps --force -c a.d && cat a.deps
would currently NOT produce this line, which is what this bug is about.
the reason rdmd produces the correct -deps here is because a.d contains top
level "import b" so it'll also compile b.d and hence show c.d, but that won't
work with more complex cases (eg if c.d contains: void test2(){import c2;}, it
won't list c2.d)
This is the biggest bug for d's build system.
--
[Issue 7016] local import does not create -deps dependency
https://issues.dlang.org/show_bug.cgi?id=7016 Timothee Cour changed: What|Removed |Added Priority|P2 |P1 --
[Issue 7016] local import does not create -deps dependency
https://issues.dlang.org/show_bug.cgi?id=7016
--- Comment #25 from Vladimir Panteleev ---
(In reply to RazvanN from comment #17)
> Cannot reproduce on Ubuntu 16.04, latest version of compiler.
I can reproduce this on a brand new Ubuntu 16.04 amd64 installation. Please see
the log below. Something on your system must be causing you to get a different
result, please try in a new chroot or virtual machine.
--
~/tmp/2016-12-30b $ sudo debootstrap --arch amd64 xenial xenial
http://archive.ubuntu.com/ubuntu/
[...]
sudo debootstrap --arch amd64 xenial xenial http://archive.ubuntu.com/ubuntu/
24.10s user 3.41s system 35% cpu 1:17.96 total
~/tmp/2016-12-30b $ sudo systemd-nspawn -D xenial
Spawning container xenial on /home/vladimir/tmp/2016-12-30b/xenial.
Press ^] three times within 1s to kill container.
root@xenial:~# apt install curl xz-utils
[...]
root@xenial:~# curl -fsS https://dlang.org/install.sh | bash -s dmd
100.0%
The latest version of this script was installed as ~/dlang/install.sh.
It can be used it to install further D compilers.
Run `~/dlang/install.sh --help` for usage information.
Downloading and unpacking
http://downloads.dlang.org/releases/2.x/2.072.1/dmd.2.072.1.linux.tar.xz
100.0%
Downloading https://dlang.org/d-keyring.gpg
100.0%
Downloading and unpacking
http://code.dlang.org/files/dub-1.1.1-linux-x86_64.tar.gz
100.0%
Run `source ~/dlang/dmd-2.072.1/activate` in your shell to use dmd-2.072.1.
This will setup PATH, LIBRARY_PATH, LD_LIBRARY_PATH, DMD, DC, and PS1.
Run `deactivate` later on to restore your environment.
root@xenial:~# source ~/dlang/dmd-2.072.1/activate
(dmd-2.072.1)root@xenial:~# echo 'import b; void main() { f(); }' > a.d
(dmd-2.072.1)root@xenial:~# echo 'int f() { import c; return i; }' > b.d
(dmd-2.072.1)root@xenial:~# echo 'int i = 42;' > c.d
(dmd-2.072.1)root@xenial:~# dmd -deps=a.deps -o- a.d
(dmd-2.072.1)root@xenial:~# cat a.deps
a (a.d) : private : object
(/root/dlang/dmd-2.072.1/linux/bin64/../../src/druntime/import/object.d)
b (b.d) : private : object
(/root/dlang/dmd-2.072.1/linux/bin64/../../src/druntime/import/object.d)
a (a.d) : private : b (b.d)
(dmd-2.072.1)root@xenial:~#
--
[Issue 7016] local import does not create -deps dependency
https://issues.dlang.org/show_bug.cgi?id=7016 --- Comment #24 from Alexey G --- same result on nightly DMD32 D Compiler v2.073.0-master-cb7f8fe --
[Issue 7016] local import does not create -deps dependency
https://issues.dlang.org/show_bug.cgi?id=7016 --- Comment #23 from Alexey G --- Sorry, loaded latest >dmd --version DMD32 D Compiler v2.072.1 Copyright (c) 1999-2016 by Digital Mars written by Walter Bright Windows 7 32 bit exact example from first post produce a.deps: a (a.d) : private : object (R:\\dmd2\\windows\\bin\\..\\..\\src\\druntime\\import\\object.d) b (b.d) : private : object (R:\\dmd2\\windows\\bin\\..\\..\\src\\druntime\\import\\object.d) a (a.d) : private : b (b.d) --
[Issue 7016] local import does not create -deps dependency
https://issues.dlang.org/show_bug.cgi?id=7016 --- Comment #22 from Alexey G --- >dmd --version DMD32 D Compiler v2.072.0-master-386ef6d Copyright (c) 1999-2016 by Digital Mars written by Walter Bright Windows 7 32 bit exact example from first post produce a.deps: a (a.d) : private : object (R:\\dmd2\\windows\\bin\\..\\..\\src\\druntime\\import\\object.d) b (b.d) : private : object (R:\\dmd2\\windows\\bin\\..\\..\\src\\druntime\\import\\object.d) a (a.d) : private : b (b.d) --
[Issue 7016] local import does not create -deps dependency
https://issues.dlang.org/show_bug.cgi?id=7016 --- Comment #21 from RazvanN --- (In reply to Vladimir Panteleev from comment #20) > (In reply to RazvanN from comment #19) > > I ran the exact example as the one in 2011 with the exact command and the > > output is: > > > > (a.d) : private : object (..) > > > > (b.d) : private : object (..) > > (a.d) : private : b (b.d) > > (c.d) : private : object (..) > > (a.d) : private : c (c.d) > > > > It seems to me that it prints all the dependencies of a.d > > Strange... I can't get that result with DMD 2.072.1 or latest git master (on > Linux). What compiler version did you use, exactly? DMD64 D Compiler v2.073.0-devel-878b882 --
[Issue 7016] local import does not create -deps dependency
https://issues.dlang.org/show_bug.cgi?id=7016 --- Comment #20 from Vladimir Panteleev --- (In reply to RazvanN from comment #19) > I ran the exact example as the one in 2011 with the exact command and the > output is: > > (a.d) : private : object (..) > > (b.d) : private : object (..) > (a.d) : private : b (b.d) > (c.d) : private : object (..) > (a.d) : private : c (c.d) > > It seems to me that it prints all the dependencies of a.d Strange... I can't get that result with DMD 2.072.1 or latest git master (on Linux). What compiler version did you use, exactly? --
[Issue 7016] local import does not create -deps dependency
https://issues.dlang.org/show_bug.cgi?id=7016 --- Comment #19 from RazvanN --- I ran the exact example as the one in 2011 with the exact command and the output is: (a.d) : private : object (..) (b.d) : private : object (..) (a.d) : private : b (b.d) (c.d) : private : object (..) (a.d) : private : c (c.d) It seems to me that it prints all the dependencies of a.d --
[Issue 7016] local import does not create -deps dependency
https://issues.dlang.org/show_bug.cgi?id=7016 --- Comment #18 from Vladimir Panteleev --- (In reply to RazvanN from comment #17) > Cannot reproduce on Ubuntu 16.04, latest version of compiler. Hopefully I > will try on a windows machine when I get the chance What have you tried? I can reproduce it exactly as per the original description from 2011. --
[Issue 7016] local import does not create -deps dependency
https://issues.dlang.org/show_bug.cgi?id=7016 --- Comment #17 from RazvanN --- Cannot reproduce on Ubuntu 16.04, latest version of compiler. Hopefully I will try on a windows machine when I get the chance --
[Issue 7016] local import does not create -deps dependency
https://issues.dlang.org/show_bug.cgi?id=7016 Andrei Alexandrescu changed: What|Removed |Added Status|NEW |ASSIGNED Assignee|nob...@puremagic.com|razvan.nitu1...@gmail.com --
[Issue 7016] local import does not create -deps dependency
https://issues.dlang.org/show_bug.cgi?id=7016 --- Comment #16 from Andrei Alexandrescu --- Thanks. Razvan will be on this. --
[Issue 7016] local import does not create -deps dependency
https://issues.dlang.org/show_bug.cgi?id=7016 Timothee Cour changed: What|Removed |Added CC||timothee.co...@gmail.com Hardware|All |Other OS|All |Windows --- Comment #15 from Timothee Cour --- all platforms. This is the #1 problem with rdmd --
[Issue 7016] local import does not create -deps dependency
https://issues.dlang.org/show_bug.cgi?id=7016 Vladimir Panteleev changed: What|Removed |Added Hardware|Other |All OS|Windows |All --
[Issue 7016] local import does not create -deps dependency
https://issues.dlang.org/show_bug.cgi?id=7016 Mathias Lang changed: What|Removed |Added CC||mathias.l...@sociomantic.co ||m Hardware|Other |All OS|Windows |All --- Comment #14 from Mathias Lang --- It happens on all platforms. --
[Issue 7016] local import does not create -deps dependency
https://issues.dlang.org/show_bug.cgi?id=7016 --- Comment #13 from Andrei Alexandrescu --- So is this on Windows only? Just confirming. --
[Issue 7016] local import does not create -deps dependency
https://issues.dlang.org/show_bug.cgi?id=7016 Andrei Alexandrescu changed: What|Removed |Added Keywords||bootcamp CC||and...@erdani.com --
[Issue 7016] local import does not create -deps dependency
https://issues.dlang.org/show_bug.cgi?id=7016 Alexey G changed: What|Removed |Added CC||golovanov_ale...@mail.ru --- Comment #12 from Alexey G --- DMD 2.070.0-b1 Windows, problem still here :( RDMD affected by this bug. --
[Issue 7016] local import does not create -deps dependency
https://issues.dlang.org/show_bug.cgi?id=7016 Vladimir Panteleev changed: What|Removed |Added CC||ket...@ketmar.no-ip.org --- Comment #11 from Vladimir Panteleev --- *** Issue 14118 has been marked as a duplicate of this issue. *** --
[Issue 7016] local import does not create -deps dependency
https://issues.dlang.org/show_bug.cgi?id=7016 Vladimir Panteleev changed: What|Removed |Added Keywords||bounty --
[Issue 7016] local import does not create -deps dependency
https://issues.dlang.org/show_bug.cgi?id=7016 --- Comment #10 from Vladimir Panteleev --- Added $25 bounty: https://www.bountysource.com/issues/3785375-local-import-does-not-create-deps-dependency --
[Issue 7016] local import does not create -deps dependency
https://d.puremagic.com/issues/show_bug.cgi?id=7016
Dmitry Olshansky changed:
What|Removed |Added
CC||dmitry.o...@gmail.com
--- Comment #9 from Dmitry Olshansky 2013-12-23
02:29:42 PST ---
(In reply to comment #0)
> a.d : import b; void main() { f(); }
> b.d : int f() { import c; return i; }
> c.d : int i = 42;
>
> The command
> $ dmd -deps=a.deps -o- a.d
> creates:
> a (a.d) : public : object
> (C:\\Soft\\dmd2d\\windows\\bin\\..\\..\\import\\druntime\\object.di)
> b (b.d) : public : object
> (C:\\Soft\\dmd2d\\windows\\bin\\..\\..\\import\\druntime\\object.di)
> a (a.d) : public : b (b.d)
Problem is still here, with 2.065alpha.
The output now looks like:
a (a.d) : private : object
(/home/dmitry/dmd2/linux/bin64/../../src/druntime/import/object.di)
b (b.d) : private : object
(/home/dmitry/dmd2/linux/bin64/../../src/druntime/import/object.di)
a (a.d) : private : b (b.d)
--
Configure issuemail: https://d.puremagic.com/issues/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
[Issue 7016] local import does not create -deps dependency
http://d.puremagic.com/issues/show_bug.cgi?id=7016 --- Comment #8 from thelastmamm...@gmail.com 2013-07-11 23:26:15 PDT --- see http://forum.dlang.org/post/mailman.1984.1373610213.13711.digitalmar...@puremagic.com [fix for enumerating local import dependencies, resolving all rdmd link errors] -- Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email --- You are receiving this mail because: ---
[Issue 7016] local import does not create -deps dependency
http://d.puremagic.com/issues/show_bug.cgi?id=7016 --- Comment #7 from jfanati...@gmx.at 2013-04-17 06:10:03 PDT --- (In reply to comment #6) > jfanatiker, the compiler is supposed to print the dependencies between all > modules when given just the starting point. This is the purpose of the -deps > switch, it is not intended to be run once for every module in the program. The > same goes for the verbose output (-v). I see. You are right of course, I viewed it from an incremental build tool perspective, where it would not matter, but for rdmd this bug is of course a problem. -- Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email --- You are receiving this mail because: ---
[Issue 7016] local import does not create -deps dependency
http://d.puremagic.com/issues/show_bug.cgi?id=7016 --- Comment #6 from Vladimir Panteleev 2013-04-17 15:53:52 EEST --- jfanatiker, the compiler is supposed to print the dependencies between all modules when given just the starting point. This is the purpose of the -deps switch, it is not intended to be run once for every module in the program. The same goes for the verbose output (-v). -- Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email --- You are receiving this mail because: ---
[Issue 7016] local import does not create -deps dependency
http://d.puremagic.com/issues/show_bug.cgi?id=7016 jfanati...@gmx.at changed: What|Removed |Added CC||jfanati...@gmx.at --- Comment #5 from jfanati...@gmx.at 2013-04-17 05:46:19 PDT --- I am not sure this really needs to be fixed: b.d either needs to be built at some point, then the dependency is printed and can be gathered by tools or it does not get built, in this case it does not matter either, because any use of the function would need to be inlined, causing the dependency to be printed. If the function is not used at all, then the dependency also does not matter. -- Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email --- You are receiving this mail because: ---
[Issue 7016] local import does not create -deps dependency
http://d.puremagic.com/issues/show_bug.cgi?id=7016 Martin Nowak changed: What|Removed |Added CC||c...@dawg.eu --- Comment #4 from Martin Nowak 2013-04-07 11:50:45 PDT --- Right, semantic3 for imported functions is only run during the inline pass. -- Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email --- You are receiving this mail because: ---
[Issue 7016] local import does not create -deps dependency
http://d.puremagic.com/issues/show_bug.cgi?id=7016 Andrej Mitrovic changed: What|Removed |Added CC||andrej.mitrov...@gmail.com --- Comment #3 from Andrej Mitrovic 2013-04-07 11:13:38 PDT --- Apparently it's because semantic isn't run on the dependencies. It's in Import::semantic where the import is printed. You can verify this with that -rb feature. -- Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email --- You are receiving this mail because: ---
[Issue 7016] local import does not create -deps dependency
http://d.puremagic.com/issues/show_bug.cgi?id=7016 --- Comment #2 from Andrej Mitrovic 2013-01-11 19:15:13 PST --- *** Issue 8856 has been marked as a duplicate of this issue. *** -- Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email --- You are receiving this mail because: ---
[Issue 7016] local import does not create -deps dependency
http://d.puremagic.com/issues/show_bug.cgi?id=7016 Andrej Mitrovic changed: What|Removed |Added CC||thelastmamm...@gmail.com --- Comment #1 from Andrej Mitrovic 2013-01-11 19:14:58 PST --- *** Issue 8858 has been marked as a duplicate of this issue. *** -- Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email --- You are receiving this mail because: ---
