Looks promising. I hope this improves and makes it into a future release.

Just grokking the patch, it seems like it does not handle the case
where a previously ignored patch is required by other patches that
need to be pushed or pulled. Ideally, it should ask whether or not to
continue the push or pull if a dependency is marked as ignored. If the
user decides to continue, it should remove the ignored patch from the
ignore list.

At least, that's how I think it should work in that particular case.
Though this patch is preliminary, it works well for the simple
use-case where a pushed or pulled patch does not depend on a
previously ignored patch.

On 12/19/05, Zachary P. Landau <[EMAIL PROTECTED]> wrote:
> Hello,
>
> Attached is an attempt at adding patch ignoring support.  As
> previously discussed, the 'i' option has been added when selecting
> patches which will add the patch to an ignore list.  It is still
> preliminary.  The issues I currently know of are:
>
> * There is not yet any --dont-ignore-any-patches option for commands.
> read_repo is called
>    in many places and it will take a while to add support for passing
> in DarcsFlags.
>
> * Git Repositories are not supported.  I don't know enough about Git
> to attempt this.
>
> * It needs more testing.  It seems to work for push, pull, and send
> but I haven't tested all
>   commands yet.
>
> --
> Zachary P. Landau <[EMAIL PROTECTED]>
>
>
> _______________________________________________
> darcs-users mailing list
> [email protected]
> http://www.abridgegame.org/mailman/listinfo/darcs-users
>
>
>
>



--
_nimrod_a_abing_

"I reject your reality and substitute my own." -- Adam Savage

_______________________________________________
darcs-users mailing list
[email protected]
http://www.abridgegame.org/mailman/listinfo/darcs-users

Reply via email to