GNU bug report logs -
#3860
vc-svn calls "svn resolved" without permission
Previous Next
Reported by: James Y Knight <jknight <at> fuhm.net>
Date: Wed, 15 Jul 2009 20:30:03 UTC
Severity: normal
Fixed in version 31.1
Done: Mauro Aranda <maurooaranda <at> gmail.com>
Bug is archived. No further changes may be made.
Full log
View this message in rfc822 format
> Date: Sat, 29 Mar 2025 11:41:00 -0300
> Cc: 3860 <at> debbugs.gnu.org, larsi <at> gnus.org, jknight <at> fuhm.net, spiegel <at> gnu.org,
> dgutov <at> yandex.ru, Sean Whitton <spwhitton <at> spwhitton.name>
> From: Mauro Aranda <maurooaranda <at> gmail.com>
>
> > void variable vc-resolve-conflicts
> >
> > In my case, the victim was admin/gitmerge.el, which only loads
> > vc-git.el,
>
> Ouch. I mistakenly thought that this could work like the vc-*-switches
> options, but now I see that, for example, vc-git.el autoloads
> vc-switches.
>
> > The various vc-FOO.el files only 'require' vc when compiling, which
> > doesn't solve the run-time problem.
> >
> > So we will need either to make vc-resolve-conflicts autoloaded (which
> > is generally discouraged for defcustom's), or manually 'require' vc in
> > the various functions that reference vc-resolve-conflicts.
> >
> > Are there any other solutions?
>
> I don't know. Can the defcustom be moved to another file where there
> are "generic" vc options, and that it gets loaded at run-time? I see
> some options in vc-hooks.el, but I don't know if (featurep 'vc-hooks)
> evaluates to t in the case you encountered.
vc-hooks is preloaded, so perhaps moving the variable there is the
best option.
Adding Stefan, in case he has better suggestions.
This bug report was last modified 53 days ago.
Previous Next
GNU bug tracking system
Copyright (C) 1999 Darren O. Benham,
1997,2003 nCipher Corporation Ltd,
1994-97 Ian Jackson.