darcs

Issue 1179 make disttest re-runs configure

Title make disttest re-runs configure
Priority bug Status wont-fix
Milestone Resolved in
Superseder Nosy List darcs-devel, dmitry.kurochkin, kowey, thorkilnaur, twb
Assigned To
Topics Devel

Created on 2008-10-28.03:09:52 by dagit, last changed 2009-08-27.14:23:44 by admin.

Messages
msg6491 (view) Author: dagit Date: 2008-10-28.03:09:49
I was doing some refactoring for ghc-6.6 (not my default compiler),
and when I was done I thought I'd just run, 'make disttest' so I could
see if I pass the tests before taking the time to record everything.

disttest makes darcs check out your current working directory to a
temp location and then run configure on it.  This made the tests use
the wrong version of ghc (6.6 is not the version of ghc that is in my
path).

I'm filing this bug because Trent asked me to make note of it so he
can look at it later.

Thanks!
Jason
msg6558 (view) Author: droundy Date: 2008-11-02.23:59:54
Running configure is the point of disttest, which doesn't check out a clean copy
of the source before running.  If you wanted to just run the tests, you could
have just done that.
msg6559 (view) Author: droundy Date: 2008-11-03.00:00:21
That said, some docs or maybe a better name would help?
msg6560 (view) Author: dagit Date: 2008-11-03.00:02:04
What target should be run to do all the tests, witnesses, unit, the shell tests?
msg6563 (view) Author: droundy Date: 2008-11-03.01:30:03
On Mon, Nov 03, 2008 at 12:02:07AM -0000, Jason Dagit wrote:
> What target should be run to do all the tests, witnesses, unit, the shell tests?

We don't have a target to do that, but you can do it with

make test_unit disttest

and yes it will rerun configure, because that's the only way to do the
witnesses test (and because disttest is designed to test, among other
things, that the configure script works).
-- 
David Roundy
http://www.darcs.net
msg6564 (view) Author: dagit Date: 2008-11-03.02:36:22
Trent,

Do you have the information now to look into this?  What we need is a way to run
the same things as disttest minus the run of configure that changes options? 
For example, we shouldn't need to re-run configure, just flip the switch so that
GADT_WITNESSES is defined and we make clean.  And David obviously wants to keep
disttest as a way of doing the things it currently does so we'd need a new name
I suppose.

I'm assigning it to you for now, but if you change your mind about wanting to
work on it go ahead on and mark it unassigned.

Jason
msg8120 (view) Author: kowey Date: 2009-08-12.15:05:53
Now moot due to the Cabal switchover
History
Date User Action Args
2008-10-28 03:09:52dagitcreate
2008-11-02 23:59:56droundysetpriority: not-our-bug
nosy: + droundy
status: unread -> unknown
messages: + msg6558
2008-11-03 00:00:22droundysetnosy: droundy, kowey, dagit, simon, thorkilnaur, dmitry.kurochkin
messages: + msg6559
2008-11-03 00:02:07dagitsetnosy: droundy, kowey, dagit, simon, thorkilnaur, dmitry.kurochkin
messages: + msg6560
2008-11-03 01:30:08droundysetnosy: droundy, kowey, dagit, simon, thorkilnaur, dmitry.kurochkin
messages: + msg6563
2008-11-03 02:36:25dagitsetpriority: not-our-bug -> bug
nosy: + twb
messages: + msg6564
assignedto: twb
2008-11-03 02:45:55twbsetnosy: droundy, kowey, dagit, simon, twb, thorkilnaur, dmitry.kurochkin
2009-04-18 04:30:19twbsetassignedto: twb ->
nosy: droundy, kowey, dagit, simon, twb, thorkilnaur, dmitry.kurochkin
2009-08-06 18:01:42adminsetnosy: + markstos, jast, Serware, darcs-devel, zooko, mornfall, tommy, beschmi, - droundy, twb
2009-08-06 21:13:48adminsetnosy: - beschmi
2009-08-10 21:50:20adminsetnosy: + twb, - tommy, markstos, darcs-devel, zooko, jast, Serware, mornfall
2009-08-10 23:49:40adminsetnosy: - dagit
2009-08-12 15:05:59koweysetstatus: unknown -> wont-fix
nosy: kowey, simon, twb, thorkilnaur, dmitry.kurochkin
topic: + Devel
messages: + msg8120
2009-08-25 17:32:51adminsetnosy: + darcs-devel, - simon
2009-08-27 14:23:44adminsetnosy: kowey, darcs-devel, twb, thorkilnaur, dmitry.kurochkin