"Bert Huijben" <b...@qqmail.nl> writes:

>> -----Original Message-----
>> From: Noorul Islam K M [mailto:noo...@collab.net]
>> Sent: maandag 30 mei 2011 12:17
>> To: Subversion
>> Subject: Bug in exporting externals?
>> 
>> 
>> While working on issue 3799, in one of the review comments julianf said
>> the following about number 10 test in external_tests.py.
>> 
>> <quote>
>> A/B/gamma is only an external: it does not appear in the WC until
>> Subversion processes the external definitions.
>> 
>> It looks to me like that failure was showing us a bug.  If I run the
>> test, without your patch, in verbose mode, I see:
>> 
>> CMD: svn export svn-test-work/working_copies/externals_tests-10
>> svn-test-work/working_copies/externals_tests-10.export [...]
>> A    svn-test-work/working_copies/externals_tests-10.export/A
>> A    svn-test-work/working_copies/externals_tests-10.export/A/B
>> A    svn-test-work/working_copies/externals_tests-10.export/A/B/lambda
>> A    svn-test-work/working_copies/externals_tests-10.export/A/B/gamma
>> [...]
>> A    svn-test-work/working_copies/externals_tests-10.export/A/B/gamma
>> [...]
>> CMD: svn export --ignore-externals
>> svn-test-work/working_copies/externals_tests-10
>> svn-test-work/working_copies/externals_tests-10.export [...]
>> A    svn-test-work/working_copies/externals_tests-10.export/A
>> A    svn-test-work/working_copies/externals_tests-10.export/A/B
>> A    svn-test-work/working_copies/externals_tests-10.export/A/B/lambda
>> A    svn-test-work/working_copies/externals_tests-10.export/A/B/gamma
>> [...]
>> 
>> There is a comment in the test about --ignore-externals not ignoring
>> A/B/gamma.  That's a bug.  And the first export (without
>> --ignore-externals) is also buggy.  It shouldn't export A/B/gamma twice.
>> 
>> We shouldn't just quietly tweak the test to hide the bug.  We should
>> write a new test specifically to check for that bug, or fix the bug, or
>> file an issue, or write to the dev@ list about it.  Something.
>> 
>> - Julian
>> </quote>
>> 
>> I tried to re-create this out of test environment and I came up with a
>> reproduction script which is attached.
>> 
>> Here I have a repository with the following files
>> 
>> A/B/lambda
>> A/D/gamma
>> 
>> Along with this I have an external set on A/B as
>> 
>> "^/A/D/gamma gamma"
>> 
>> Now while exporting the working copy. A/B/gamma is exported twice which
>> looks like a bug and while exporting the WC with --ignore-externals
>> A/B/gamma is exported once which also seems to be a bug since A/B/gamma
>> is an external.
>> 
>> I can go ahead and create an issue for this if people can confirm this.
>
> A/B/gamma is a file external. And as such it is also threated as part of the
> working copy itself.
>
> We export it again as an external later, but the original problem is that it
> is just a versioned file in a versioned directory.
>

Is this the expected behaviour? 

Thanks and Regards
Noorul

Reply via email to