On Sat, 16 Jul 2011, Andreas Beckmann wrote:
> OK, patch attached.
Thanks!
> That fixes most regressions I could produce in above patch. There is one
> left when --force is used, I'm not sure if I did this correctly or I
> found another bug.
In fact set_choice() did not support passing supplementary parameters, I
fixed that.
> +ok(-f "$bindir/slave2", "forced switching w/o slave keeps real files
> installed as slave links");
This test was reversed, with --force it must remove the file.
> +system("rm $bindir/slave2");
Thus this is useless.
Otherwise it was fine. I committed the fix and your tests.
Cheers,
--
Raphaël Hertzog ◈ Debian Developer
Follow my Debian News ▶ http://RaphaelHertzog.com (English)
▶ http://RaphaelHertzog.fr (Français)
--
To UNSUBSCRIBE, email to [email protected]
with a subject of "unsubscribe". Trouble? Contact [email protected]
