[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#647255: RFA: curl -- A library and a commandline client for HTTP and a number of protocols



On 11/15/2011 07:12 PM, Alessandro Ghedini wrote:
> On Tue, Nov 15, 2011 at 06:35:48PM +0100, Luk Claes wrote:
>> On 11/15/2011 05:40 PM, Alessandro Ghedini wrote:
>>> On Tue, Nov 15, 2011 at 04:17:32PM +0100, Luk Claes wrote:
>>>> On 11/13/2011 10:05 PM, Alessandro Ghedini wrote:
>>>>> On Sun, Nov 13, 2011 at 06:25:37PM +0100, Luk Claes wrote:
>>>>>> On 11/11/2011 08:56 PM, Alessandro Ghedini wrote:
>>>>>>> On Fri, Nov 04, 2011 at 08:45:08PM +0100, Alessandro Ghedini wrote:
>>>>>>>> Apart from that the package looks quite complete, could someone please have
>>>>>>>> an "in depth" look at it?
>>>>>>>
>>>>>>> Did anyone have a look at the new version yet? We don't need to forward 
>>>>>>> upstream all the patches immediately, and IMHO we can go on with the upload.
>>>>>>
>>>>>> Git commits look fine, but can you please point us to a source package
>>>>>> (on mentors.d.n or elsewhere)?
>>>>>
>>>>> Sure. Package just built and uploaded to mentors
>>>>> http://mentors.debian.net/debian/pool/main/c/curl/curl_7.22.0-1.dsc
>>>>
>>>> Thanks. When reviewing the diff I found that the dependencies of the
>>>> -dev packages are wrong: the gnutls and nss variants need other
>>>> dependencies than the openssl one. Can you please upload -2 fixing that
>>>> to mentors, so we can upload, TIA?
>>>
>>> Oops, it was a copy-paste error, sorry (there was also another typo)
>>>
>>> I've uploaded the -2 version, I hope it's all fine now :)
>>> http://mentors.debian.net/debian/pool/main/c/curl/curl_7.22.0-2.dsc
>>
>> I'm afraid that it's not: libssh2-1-dev should only be included for the
>> openssl variant (due to licensing issues).
> 
> Uh. Sorry again, doing things in a hurry leads to this kind of stupid 
> errors. I've re-fixed it, by using the git-diff of before I put my hands on 
> it. I just noticed that I had also added librtmp-dev to *nss-dev, but given
> #640260 I think it is better now (I've also updated the changelog with this
> change).
> 
> http://mentors.debian.net/debian/pool/main/c/curl/curl_7.22.0-3.dsc

Uploaded.

Apparently upstream just released 7.23.0. I guess we should let 7.22.0
migrate to testing first, but are you ready for the next round?

Cheers

Luk



Reply to: