Bug 874

Summary: cvs-import.sh needs to be updated to support xz and lzma compression
Product: Infrastructure Reporter: Nicolas Chauvet <kwizart>
Component: CVSAssignee: Xavier Lamien <lxtnow>
Status: VERIFIED FIXED    
Severity: normal CC: fedora, leigh123linux
Priority: P5    
Version: NA   
Hardware: All   
OS: GNU/Linux   
namespace:
Attachments: Patch to upload xz and lzma compressed content to the cache directory

Description Nicolas Chauvet 2009-10-19 21:51:32 CEST
Created attachment 307 [details]
Patch to upload xz and lzma compressed content to the cache directory

see:
Comment 1 Nicolas Chauvet 2009-11-01 23:48:11 CET
Xavier, we are waiting for a quick review/commit for this patch.
Please don't let other think your are still wondering about what it does!?

This is aimed to fix the cache upload for lzma files on cvs-import.sh

Comment 2 Xavier Lamien 2009-11-02 12:25:07 CET
(In reply to comment #1)
> Xavier, we are waiting for a quick review/commit for this patch.
> Please don't let other think your are still wondering about what it does!?
> 

Huh ?
Do i really look like care what people think about me ?


> This is aimed to fix the cache upload for lzma files on cvs-import.sh
> 

FYI, I _do_ know what this little patch does.

Hey, it's not that way to come to other and ask for something, really.
Instead of come with such comment, ask why this patch has been not applied yet.

There're a couple of critical tasks which need to be fixed at first and this is why your patch is still in a pending state.

You're getting make me feel that you don't understand how things work around a _project_.
Comment 3 Michael Schwendt 2009-11-30 18:59:04 CET
Why don't you simply say why it takes time to apply this or whether anything is wrong with the patch?
Comment 4 Xavier Lamien 2009-12-01 12:48:12 CET
That's what i did in my previous comment (specifically for the "why?").

Anyway, We have a scheduled CVS outage in the pipe and i will make sure to apply this patch at the same time.
We'll send a notification to the -devel list as usual.
Comment 5 Xavier Lamien 2009-12-14 11:55:56 CET
Your patch has been applied.

Thanks.