www.digitalmars.com         C & C++   DMDScript  

digitalmars.D.bugs - [Issue 6171] New: rdmd: cache dependency file to improve startup time [patch]

reply d-bugmail puremagic.com writes:
http://d.puremagic.com/issues/show_bug.cgi?id=6171

           Summary: rdmd: cache dependency file to improve startup time
                    [patch]
           Product: D
           Version: unspecified
          Platform: All
        OS/Version: All
            Status: NEW
          Severity: enhancement
          Priority: P2
         Component: DMD
        AssignedTo: nobody puremagic.com
        ReportedBy: edelkind+puremagic gmail.com


--- Comment #0 from ari edelkind <edelkind+puremagic gmail.com> 2011-06-17
09:45:46 PDT ---
Created an attachment (id=1002)
Patch for --cache-deps functionality

The attached patch is quick and dirty, but you get the idea.  Instead of
unconditionally regenerating the deps file, apply this patch and use the
--cache-deps option to rdmd.  Startup time is improved dramatically.

Notes:
  - The deps file is checked to ensure that it's newer than the target.
  - All dependencies (modules) are still checked.  If none of them have
changed, then none of the dependencies would have changed, either.
  - If a dependency changed and the object needs to be recompiled, then the
deps file is regenerated as well.

Patch is against tools.git version 6085e8521653ea031508444759cf37bbc3342245.

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
Jun 17 2011
next sibling parent d-bugmail puremagic.com writes:
http://d.puremagic.com/issues/show_bug.cgi?id=6171



--- Comment #1 from ari edelkind <edelkind+puremagic gmail.com> 2011-06-17
10:29:44 PDT ---
I should note that the temporary-object-file cache directory is probably a more
sensible place for the cached deps file.  This way, the deps file can be cached
even if the source directory is not writable.

Given such improvements, i think that rdmd could become a useful, efficient
substitute for popular scripting languages.

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
Jun 17 2011
prev sibling next sibling parent d-bugmail puremagic.com writes:
http://d.puremagic.com/issues/show_bug.cgi?id=6171



--- Comment #2 from github-bugzilla puremagic.com 2012-02-19 21:51:28 PST ---
Commit pushed to master at https://github.com/D-Programming-Language/tools

https://github.com/D-Programming-Language/tools/commit/89b863d3ad049ccf8dfa1e84f6846783dc6fe37b
fix issue 6171 - rdmd: cache dependency file to improve startup time

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
Feb 19 2012
prev sibling next sibling parent d-bugmail puremagic.com writes:
http://d.puremagic.com/issues/show_bug.cgi?id=6171



--- Comment #3 from Andrei Alexandrescu <andrei metalanguage.com> 2012-02-19
21:56:00 PST ---
Implemented an independent solution. Indeed the speed gain for cached builds is
dramatic - 6x. Currently the deps file is not saved in the temp dir.

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
Feb 19 2012
prev sibling next sibling parent d-bugmail puremagic.com writes:
http://d.puremagic.com/issues/show_bug.cgi?id=6171


Andrej Mitrovic <andrej.mitrovich gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |andrei erdani.com,
                   |                            |andrej.mitrovich gmail.com


--- Comment #4 from Andrej Mitrovic <andrej.mitrovich gmail.com> 2013-02-05
13:21:53 PST ---
(In reply to comment #3)

 Implemented an independent solution. Indeed the speed gain for cached builds is
 dramatic - 6x. Currently the deps file is not saved in the temp dir.


Can we mark this as fixed?

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
Feb 05 2013
prev sibling parent d-bugmail puremagic.com writes:
http://d.puremagic.com/issues/show_bug.cgi?id=6171


Andrei Alexandrescu <andrei erdani.com> changed:

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


--- Comment #5 from Andrei Alexandrescu <andrei erdani.com> 2013-02-05 13:30:20
PST ---
Yes, it's been fixed a while ago.

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
Feb 05 2013