[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: svn merge revert commit adds files to the repo that aremissing in the wc

From: Daniel Shahaf <d.s_at_daniel.shahaf.name>
Date: Sun, 5 Oct 2008 14:38:27 +0300 (Jerusalem Daylight Time)

Ping? There's a second patch[1] (later in the thread; sorry, don't have
the mail here) awaiting response.

Jojakim: please send patches as MIME attachments, or inline (but watch for
whitespace-eating mailers, in the latter case). You can see below (or at [1])
how they look to most of us. Many would not bother decoding the "attachment"
manually.

Thanks,

Daniel

[1] http://thread.gmane.org/gmane.comp.version-control.subversion.devel/104483/focus=104834

Jojakim Stahl wrote on Fri, 19 Sep 2008 at 11:52 +0200:
> Hello Julian,
> I'm no expert in writing Subversion tests and didn't it ever, but sometime
> is first time, so attached you will find a patch for merge_tests.py.
> Hope this helps,
> Jojakim
>
> "Julian Foad" <julianfoad_at_btopenworld.com> schrieb im Newsbeitrag
> news:1221777871.29026.35.camel_at_edith.foad.me.uk...
> > On Thu, 2008-09-18 at 12:53 +0000, Jojakim Stahl wrote:
> >> Hello all,
> >>
> >> I multiple times now had the problem, that I had files in the repo that
> >> were
> >> not in the wc any more. I now found the reason:
> >
> > Hi Jojakim.
> >
> > Thank you for this bug report. If you can spare a bit more time, please
> > could you see if there is already a test for this problem in our test
> > suite. Issue #1962 has a shell script attached, and it mentions recently
> > something about "all but 1 tests passing", but that change was reverted
> > and I don't know if there are actually any tests. If not, could you
> > convert your recipe into a test? That would really help.
> >
> >>From reading some of issue #1962, it looks like this is quite a
> > difficult problem. It would be really good to get it fixed.
> > Unfortunately there don't seem to be any volunteers working on it at the
> > moment. Do you know anybody who could help?
> >
> > Thanks.
> >
> > - Julian
> >
> >
> >> 1.
> >> Merge a changeset X from p1 to p2, where X was
> >> A dir1
> >> A dir1/file1.txt
> >> A dir1/file2.txt
> >>
> >> svn merge -c X p1 p2
> >>
> >> 2.
> >> Merges marks file1.txt as added, but for some reason in p2 I do not need
> >> file1.txt any more (due to some restructuring or something else).
> >> Therefore
> >> revert file1.txt
> >>
> >> svn revert p2/dir1/file1.txt
> >>
> >> 3.
> >> Commit p2
> >>
> >> svn ci p2 -m "Merged dir1 from p1"
> >>
> >> 4.
> >> svn status reports file1.txt as unversioned now. Therefore delete
> >> file1.txt
> >>
> >> del p2/dir1/file1.txt
> >>
> >> 5.
> >> svn status reports nothing now, also svn status --show-updates doesn't
> >> show
> >> anything. Aswell, svn up reports that the wc is up to date. However, svn
> >> ls
> >> p2/dir1 reports file1.txt to be in the repo.
> >>
> >> I struggled may times over this problem now. Especially because you do
> >> not see
> >> that the repo has someting that is not in the wc. And if you take p2 as
> >> base
> >> for other merges now, you get the file1.txt duplicated to pX.
> >>
> >> I search the bug database and found #1962 to be probably the reason of
> >> the
> >> problem. But #1962 is scheduled consider-1.6. I'm wondering whether such
> >> a
> >> fundamental bug in merging shouldn't be fixed as soon as possible (next
> >> 1.5
> >> fix?). Aren't others dealing with such problems, too?
> >>
> >> The problem in reality is: Your wc is compiling well and you commit.
> >> Others
> >> update there wc and your commit breaks their wc compiling. This happens
> >> one
> >> time, two time, and then your boss will ask you some questions...
> >>
> >> Thanks,
> >>
> >> Jojakim Stahl
> >>
> >>
> >>
> >> ---------------------------------------------------------------------
> >> To unsubscribe, e-mail: dev-unsubscribe_at_subversion.tigris.org
> >> For additional commands, e-mail: dev-help_at_subversion.tigris.org
> >>
>
>
> begin 666 merge_tests.py.patch
> M26YD97_at_Z(&UE<F=E7W1E<W1S+G!Y#0H]/3T]/3T]/3T]/3T]/3T]/3T]/3T]
> M/3T]/3T]/3T]/3T]/3T]/3T]/3T]/3T]/3T]/3T]/3T]/3T]/3T]/3T]/3T]
> M#0HM+2T@;65R9V5?=&5S=',N<'D)*%)E=FES:6]N(#,S,3<W*0T**RLK(&UE
> M<F=E7W1E<W1S+G!Y"2A!<F)E:71S:V]P:64I#0I 0" M,3,Y,S0L,3 @*S$S
> M.3,T+#0X($! #0H@(" @(" @(" @(" @(" @97AP96-T961?<W1A='5S+ T*
> M(" @(" @(" @(" @(" @(&5X<&5C=&5D7W-K:7 L#0H@(" @(" @(" @(" @
> M("D_at_72P@1F%L<V4I#0HK(RTM+2TM+2TM+2TM+2TM+2TM+2TM+2TM+2TM+2TM
> M+2TM+2TM+2TM+2TM+2TM+2TM+2TM+2TM+2TM+2TM+2TM+2TM+2TM+2T-"B -
> M"BLC($-O;6UI=&EN9R!A(&UE<F=E('=I=&@@<F5V97)T960_at_861D960@9FEL
> M92!S:&]U;&0@;F]T(&%D9"!T:&4_at_9FEL92 -"BLC('1O('1H92!R97!O#0H@
> M#0HK9&5F(&1O;G1?861D7W)E=F5R=&5D7V9I;&4H<V)O>"DZ#0HK(" B9&]N
> M)W0_at_861D(&$@<F5V97)T960_at_02L@<W1A=&4_at_9FEL92!T;R!R97!O(@T*( T*
> M*R @<V)O>"YB=6EL9"@I#0HK("!W8U]D:7(@/2!S8F]X+G=C7V1I<@T**R @
> M<W9N7V-O;6UI="YR97!O7W)E=B ](#$-"B -"BL@(%$Q7W!A=&@@/2!O<RYP
> M871H+FIO:6XH=V-?9&ER+" G43$G*0T**R @43)?<&%T:" ](&]S+G!A=&@N
> M:F]I;BAW8U]D:7(L("=1,B<I#0HK("!27W!A=&@@/2!O<RYP871H+FIO:6XH
> M43%?<&%T:"P@)U(G*0T**R @9FEL93%?43%?<&%T:" ](&]S+G!A=&@N:F]I
> M;BA1,5]P871H+" G4B<L("=F:6QE,2<I#0HK("!F:6QE,E]1,5]P871H(#T@
> M;W,N<&%T:"YJ;VEN*%$Q7W!A=&@L("=2)RP@)V9I;&4R)RD-"BL@(&9I;&4Q
> M7U$R7W!A=&@@/2!O<RYP871H+FIO:6XH43)?<&%T:"P@)U(G+" G9FEL93$G
> M*0T**R @#0HK("!S=FYT97-T+FUA:6XN<G5N7W-V;BA.;VYE+" G;6MD:7(G
> M+"!1,5]P871H*0T**R @<W9N=&5S="YM86EN+G)U;E]S=FXH3F]N92P@)VUK
> M9&ER)RP_at_43)?<&%T:"D-"BL@('-V;E]C;VUM:70H=V-?9&ER*0T**R @<W9N
> M=&5S="YM86EN+G)U;E]S=FXH3F]N92P@)W5P9&%T92<L('=C7V1I<BD-"BL@
> M( T**R @;W,N;6MD:7(H4E]P871H*0T**R @9FEL;%]F:6QE7W=I=&A?;&EN
> M97,H9FEL93%?43%?<&%T:"P@,2D-"BL@(&9I;&Q?9FEL95]W:71H7VQI;F5S
> M*&9I;&4R7U$Q7W!A=&@L(#$I#0HK("!S=FYT97-T+FUA:6XN<G5N7W-V;BA.
> M;VYE+" G861D)RP_at_4E]P871H*0T**R @<U]R979?861D(#T@<W9N7V-O;6UI
> M="AW8U]D:7(I#0HK#0HK("!S=FY?;65R9V4H<U]R979?861D+"!1,5]P871H
> M+"!1,E]P871H*0T**R @<W9N=&5S="YM86EN+G)U;E]S=FXH3F]N92P@)W)E
> M=F5R="<L(&9I;&4Q7U$R7W!A=&@I#0HK("!S=FY?8V]M;6ET*'=C7V1I<BD-
> M"BL@(&5X<&5C=&5D7V]U='!U=" ]('-V;G1E<W0N=F5R:69Y+E5N;W)D97)E
> M9$]U='!U="A;(E(O7&XB+" B4B]F:6QE,EQN(ETI#0HK("!S=FYT97-T+F%C
> M=&EO;G,N<G5N7V%N9%]V97)I9GE?<W9N*$YO;F4L(&5X<&5C=&5D7V]U='!U
> M="P_at_6UTL("=L<R<L("<M4B<L#0HK(" @(" @(" @(" @(" @(" @(" @(" @
> M(" @(" @(" @(" @(%$R7W!A=&@I#0HK(" -"BL-"BL-"BL-"BL-"BL-"B C
> M(R,C(R,C(R,C(R,C(R,C(R,C(R,C(R,C(R,C(R,C(R,C(R,C(R,C(R,C(R,C
> M(R,C(R,C(R,C(R,C(R,C(R,C(R,C(R,C(R,-"B C(%)U;B!T:&4@=&5S=',-
> M"B -"D! ("TQ-#$S-RPV("LQ-#$W-2PW($! #0H@(" @(" @(" @(" @("!T
> M<F5E7V-O;F9L:6-T<U]O;E]M97)G95]N;U]L;V-A;%]C:5\U7S$L#0H@(" @
> M(" @(" @(" @("!T<F5E7V-O;F9L:6-T<U]O;E]M97)G95]N;U]L;V-A;%]C
> M:5\U7S(L#0H@(" @(" @(" @(" @("!T<F5E7V-O;F9L:6-T<U]O;E]M97)G
> M95]N;U]L;V-A;%]C:5\V+ T**R @(" @(" @(" @(" @9&]N=%]A9&1?<F5V
> M97)T961?9FEL92P-"B @(" @(" @(" @(" @70T*( T*(&EF(%]?;F%M95]?
> 1(#T]("=?7VUA:6Y?7R<Z#0H`
> `
> end

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe_at_subversion.tigris.org
For additional commands, e-mail: dev-help_at_subversion.tigris.org
Received on 2008-10-05 17:48:47 CEST

This is an archived mail posted to the Subversion Dev mailing list.