Created on 2008-09-02.19:21:11 by dleverton, last changed 2010-03-31.13:09:49 by kowey.
msg5884 (view) |
Author: dleverton |
Date: 2008-09-02.19:21:09 |
|
In darcs 2.0.2 and also built from the current darcs repo, "darcs changes
--context --tags=xxx" and "darcs changes --context --to-tag=xxx" don't give
useful output for "non-recent" tags, that is, those that don't appear in the
output of "darcs changes --context | grep TAG". With a recent tag, such as
2.0.2 in darcs's own repository, they produce the expected
Context:
[TAG 2.0.2
David Roundy <droundy@darcs.net>**20080624012041]
however with 2.0.1 I get
Context:
for --tags and
Context:
darcs: Couldn't find a tag matching "tag-name 2.0.1"
for --to-tag
droundy says that --to-tag should definitely work. I'm not sure whether or not
he meant that --tags should work too, so I'm mentioning it just in case. This
may also apply to some of the other "darcs changes" options for selecting
patches, but I'm only interested in --tags and/or --to-tag ;-)
|
msg5886 (view) |
Author: kowey |
Date: 2008-09-02.21:57:18 |
|
So, our diagnosis on #darcs [1] was that changes --context only looks up to the
last checkpoint tag (i.e. --tags, --from-tag will work, but often, there aren't
any non-checkpoint tags since the last checkpoint, and users like dleverton see
no results).
We can reproduce it, and we have a rough idea what is going on.
The only things which are unclear are what to do about it:
- should changes --context match all patches? that doesn't sound very reasonable
- should changes --context match up to the last checkpoint, except when there
are other flags, in which case it matches all tags and does filtering accordingly?
- is it even reasonable for --context to filter patches? I suppose the --from
and --to matcher (both on tags and patches), as well as --index make some sense
Thoughts?
[1] http://irclog.perlgeek.de/darcs/2008-09-01
|
msg5890 (view) |
Author: dleverton |
Date: 2008-09-03.06:14:02 |
|
FWIW, my intention here is to get the equivalent of
darcs get --tag=xxx /my/repo /my/scratch/repo
darcs changes --context --repodir=/my/scratch/repo
but without having to create an extra copy of the repository. I personally only
care about --tag, but it might make sense for --to-match and --to-patch as well.
|
msg5904 (view) |
Author: daveroundy |
Date: 2008-09-04.14:21:54 |
|
On Tue, Sep 2, 2008 at 3:21 PM, David Leverton <bugs@darcs.net> wrote:
> droundy says that --to-tag should definitely work. I'm not sure whether or not
> he meant that --tags should work too, so I'm mentioning it just in case. This
> may also apply to some of the other "darcs changes" options for selecting
> patches, but I'm only interested in --tags and/or --to-tag ;-)
I don't think --tags should be compatible with --context, simply
because --context only makes sense when outputting a complete history,
and --tags doesn't respect dependencies (as e.g. --patches would also
make very little sense with --context).
David
|
msg8221 (view) |
Author: kowey |
Date: 2009-08-17.23:40:11 |
|
Some clarifications here:
1. I think I was wrong when I said this had anything to do with checkpoint tags.
More that it was tags that were deemed safe to move into a separate inventory
(optimize among other things)
2. David Roundy is right that the --tags matcher in particular should not work
with --context. Nothing that ignores dependencies should work with --context.
I've opened a new ticket for this in issue1534
3. David Leverton doesn't want to break dependencies, just to grab changes
beyond the inventory cut-off point, which sounds fair enough.
I would say that under changes --context, maybe the --tag matchers need to work
a little harder?
|
msg8694 (view) |
Author: kowey |
Date: 2009-09-04.17:54:59 |
|
Note that this bug is mutually exclusive with issue1534.
It'd be a good idea to refer to msg1579 from issue428.
The problem is that changes --context is ill-defined with matchers like -p and
--last. One solution (issue1534) would be to refuse to accept matchers.
Another solution would be to define a clear meaning for changes --context -p
(etc) and then implement it.
I'll leave this need-action for somebody to work out a proposal.
(One proposal: the matchers mean 'give me a context which includes at least
these patches')
|
msg10594 (view) |
Author: kowey |
Date: 2010-03-31.13:09:48 |
|
[I just merged in the one message from issue1597]
After last looking at this, I think I have a slightly better
understanding of the situation.
Basically, darcs changes uses matchers without caring about
dependencies. This is useful because dependencies are meaningless in
the context of searching the changelog (eg. --match 'author Eric').
On the other hand, dependencies are very much meaningful for the task of
producing a context file. It seems somewhat awkward to have this sort
of mysterious flip-flopping in matcher semantics depending on whether or
not you pass in the --context flag.
There are three solutions to this bug:
1. here and duplicate issue1597: make matchers behave the right way in
darcs changes --context
2. issue1534: refuse matchers for darcs changes --context
3. issue1258: introduce a new command darcs show context with matchers
that have the right way
Solutions #1 and #2 are mutually exclusive. And so are solutions #1 and
#3, it would seem. Solutions #2 and #3 could be quite complementary.
My personal preferences are for the latter two. I'm going to wont-fix
this tentatively...
|
|
Date |
User |
Action |
Args |
2008-09-02 19:21:11 | dleverton | create | |
2008-09-02 21:57:20 | kowey | set | status: unread -> unknown messages:
+ msg5886 |
2008-09-03 06:14:05 | dleverton | set | messages:
+ msg5890 |
2008-09-04 14:21:56 | daveroundy | set | nosy:
+ daveroundy messages:
+ msg5904 |
2008-09-28 20:50:09 | admin | set | nosy:
+ droundy, simon, thorkilnaur, - daveroundy |
2009-08-06 18:00:52 | admin | set | nosy:
+ markstos, jast, Serware, dmitry.kurochkin, darcs-devel, zooko, mornfall, tommy, beschmi, - droundy, dleverton |
2009-08-06 21:12:18 | admin | set | nosy:
- beschmi |
2009-08-10 21:48:11 | admin | set | nosy:
+ dleverton, - tommy, markstos, darcs-devel, zooko, jast, Serware, mornfall |
2009-08-10 23:44:43 | admin | set | nosy:
- dagit |
2009-08-17 23:40:14 | kowey | set | status: unknown -> needs-implementation nosy:
kowey, simon, thorkilnaur, dmitry.kurochkin, dleverton messages:
+ msg8221 |
2009-08-17 23:50:13 | kowey | set | topic:
+ Matchers nosy:
kowey, simon, thorkilnaur, dmitry.kurochkin, dleverton |
2009-08-25 17:29:40 | admin | set | nosy:
+ darcs-devel, - simon |
2009-08-26 14:28:45 | kowey | set | nosy:
kowey, darcs-devel, thorkilnaur, dmitry.kurochkin, dleverton |
2009-08-27 14:25:47 | admin | set | nosy:
kowey, darcs-devel, thorkilnaur, dmitry.kurochkin, dleverton |
2010-03-31 12:55:26 | admin | set | nosy:
+ jamesdsadler, ahey, tommy messages:
+ msg8694 |
2010-03-31 13:00:47 | kowey | link | issue1597 superseder |
2010-03-31 13:09:49 | kowey | set | status: needs-implementation -> wont-fix priority: bug -> feature title: darcs changes --context doesn't work well with other options -> darcs changes --context with matchers nosy:
tommy, kowey, darcs-devel, jamesdsadler, ahey, thorkilnaur, dmitry.kurochkin, dleverton messages:
+ msg10594 topic:
+ UI |
2010-03-31 13:11:27 | kowey | link | issue428 superseder |
|