[m-rev.] for post-commit review: minor cleanups of code related to library installs

Zoltan Somogyi zoltan.somogyi at runbox.com
Wed Aug 14 22:05:38 AEST 2024


For review by anyone.

My planned next two steps are

- moving the part of make.program_target.m that deals with
  library installs to a new module, named make.install_library
  (unless someone can propose a better name),

- adding a module comment to the new module that describes
  what files are put into what directories by the moved code.

The latter is needed because (a) the rules for what files are put where
seem to be different for *installed* libraries than for the workspaces
that the libraries are installed *from*, which is itself probably a mistake,
and (b) the rules for installed libraries do not seems to be documented
anywhere (yet). I think I need that documentation before I can fix
the code installing .module_dep files *properly*.

Zoltan.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: DIFF.gsl1
Type: application/octet-stream
Size: 17433 bytes
Desc: not available
URL: <http://lists.mercurylang.org/archives/reviews/attachments/20240814/a1f45653/attachment-0004.obj>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: Log.gsl1
Type: application/octet-stream
Size: 561 bytes
Desc: not available
URL: <http://lists.mercurylang.org/archives/reviews/attachments/20240814/a1f45653/attachment-0005.obj>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: Log.gmdi
Type: application/octet-stream
Size: 418 bytes
Desc: not available
URL: <http://lists.mercurylang.org/archives/reviews/attachments/20240814/a1f45653/attachment-0006.obj>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: DIFF.gmdi
Type: application/octet-stream
Size: 9765 bytes
Desc: not available
URL: <http://lists.mercurylang.org/archives/reviews/attachments/20240814/a1f45653/attachment-0007.obj>


More information about the reviews mailing list