[vlc-devel] [vlc-commits] package/macosx: remove .rar from list of default file types

[David Fuhrmann]

Am 31.03.2014 um 20:59 schrieb Felix Paul Kühne <fkuehne at videolan.org>:

> Hey David,
> 
> On 31 Mar 2014, at 20:48, David Fuhrmann <david.fuhrmann at gmail.com> wrote:
> 
>> 
>> Am 31.03.2014 um 20:35 schrieb Felix Paul Kühne <fkuehne at videolan.org>:
>> 
>>> Hey David,
>>> 
>>> On 27.03.2014, at 22:46, David Fuhrmann <git at videolan.org> wrote:
>>> 
>>>> vlc | branch: master | David Fuhrmann <dfuhrmann at videolan.org> | Thu Mar 27 22:45:27 2014 +0100| [7c2652cd8b4bbb4be81aa2a928f8cafb86351b6a] | committer: David Fuhrmann
>>>> 
>>>> package/macosx: remove .rar from list of default file types
>> 
>>> This breaks double-clicks on rar files
>> 
>> Yes, this is the whole purpose of this patch. IMO, we cannot assume that a rar file will contain video data and the user wants to play them with VLC. Rar is quite universal. :-)
> 
> Come on. Who else uses rar for packaging than back-up sites for self-produced contents? Seriously, there are way better formats and they are being used a lot. Personally speaking, I never received a rar archive without media contents to date.
> 
>>> and also dragging of rar files on the VLC icon the dock to open.
>> 
>> I also noticed that. But the files can be still dragged to the window itself.
> 
> Excellent. My mom will never find out.
> 
>>> If users don't have an archive utility installed to decrypt such files, it is expected that VLC becomes the default app.
>> 
>> Is VLC for Windows also registering rar as a VLC file type _by default_?
> 
> Yep.
> 
>> And what happens if a user first installs VLC, but then an Unarchiver?
> 
> On Windows, this depends on what’s selected in the Unarchiver installation wizard. On OS X, VLC would remain the default app. However, this can’t be that bad because there were 0 user reports about this ever.

Ok, you convinced me. :-)

I just saw that osx (or the archiver app) is even smart enough to automatically change the file association to the unarchiver, once you install such an app. Then, this patch should be reverted, indeed.

With best regards,
David