[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: [bug] check-mime-type.pl: Windows Support

From: Daniel Shahaf <d.s_at_daniel.shahaf.name>
Date: Thu, 12 Jan 2017 15:34:45 +0000

[moving to dev@]

Matthias Buecher / Germany wrote on Sun, Jan 08, 2017 at 01:57:08 +0100:
> https://svn.apache.org/viewvc/subversion/trunk/contrib/hook-scripts/check-mime-type.pl
>
> I'm using the attached patch for several years now without any issues
> (also directly in this mail below).
>
> I want to file this as an issue (patch) and want to make sure that
> everything is fine with this and my patch.

Please do file an issue.

However, note that contrib/ is no longer released; it's not officially
maintained/supported any more. The reason for an issue is so that when
somebody adopts check-mime-type.pl and starts releasing it, they will be
able to take the backlogged issues from our issue tracker.

I also took a quick look at the patch. My most important comment is
that the warning about "may not work" should become a croak(). Also,
the hand-rolled escaping doesn't handle the case of a backslash present
in the input. (And the best way to fix both of these would be to use
a library that abstracts away the platform differences)

Cheers,

Daniel
Received on 2017-01-12 16:38:31 CET

This is an archived mail posted to the Subversion Dev mailing list.