darcs

Issue 2513 darcs record test ignores patch dependencies

Title darcs record test ignores patch dependencies
Priority feature Status unknown
Milestone Resolved in
Superseder Nosy List sap
Assigned To
Topics Devel, UI

Created on 2016-10-03.14:16:36 by sap, last changed 2017-08-12.19:13:37 by bf.

Messages
msg19301 (view) Author: sap Date: 2016-10-03.14:16:32
When a new patch is recorded, darcs can optionally run a pre-commit
test. The test environment seems to have all patches of the current
repository, even those on which the committed patch does not depend.
Thus, this test cannot detect if a patch is missing dependencies.

Since correct patch dependencies are crucial for effective darcs use,
the test environment should be minimal, and only include the patches
which are pulled in by the dependencies of the patch under test.
msg19567 (view) Author: bf Date: 2017-08-12.19:13:36
Interesting idea. Needs a bit of discussion though. I guess the proposed
behaviour might not be universally appreciated so perhaps better enable
this with an option?
History
Date User Action Args
2016-10-03 14:16:36sapcreate
2017-08-12 19:13:37bfsetpriority: feature
messages: + msg19567