[devel] [PATCH for apt] Fix copying release information from cdrom

[Aleksei Nikiforov]

06.12.2019 15:12, Dmitry V. Levin пишет:
> On Fri, Dec 06, 2019 at 03:05:38PM +0300, Dmitry V. Levin wrote:
>> On Fri, Dec 06, 2019 at 02:15:54PM +0300, Aleksei Nikiforov wrote:
>>> ---
>>>   apt/cmdline/rpmindexcopy.cc | 4 ++--
>>>   1 file changed, 2 insertions(+), 2 deletions(-)
>>>
>>> diff --git a/apt/cmdline/rpmindexcopy.cc b/apt/cmdline/rpmindexcopy.cc
>>> index 6d5a414..06b391a 100644
>>> --- a/apt/cmdline/rpmindexcopy.cc
>>> +++ b/apt/cmdline/rpmindexcopy.cc
>>> @@ -180,10 +180,10 @@ bool RPMIndexCopy::CopyPackages(const string &CDROM,const string &Name,vector<st
>>>   		     release.c_str());
>>>   	    string TargetF = _config->FindDir("Dir::State::lists") + "partial/";
>>>   	    TargetF += URItoFileName(S);
>>> -	    if (FileExists(RipDirectory(*I) + release) == true)
>>> +	    if (FileExists(RipDirectory(*I) + "/" + release) == true)
>>>   	    {
>>>   	       FileFd Target(TargetF,FileFd::WriteEmpty);
>>> -	       FileFd Rel(RipDirectory(*I) + release,FileFd::ReadOnly);
>>> +	       FileFd Rel(RipDirectory(*I) + "/" + release,FileFd::ReadOnly);
>>>   	       if (_error->PendingError() == true)
>>>   		  return false;
>>
>> LGTM.
>>
>> I wonder is there any more bugs in constructing path names in the code.
> 
> I'd also add something like "Fixes: https://bugzilla.altlinux.org/37531".
> 

Ok, I'll add this text to commit message.