Review Request 22246: New parser for Matlab .mat files

2014-06-04 Thread Ann Burgess
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22246/ --- Review request for tika and Chris Mattmann. Repository: tika Description

Re: Review Request 22246: New parser for Matlab .mat files

2014-06-04 Thread Ann Burgess
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22246/ --- (Updated June 4, 2014, 10:23 p.m.) Review request for tika and Chris Mattmann.

Re: Review Request 22246: New parser for Matlab .mat files

2014-06-04 Thread Matthias Krueger
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22246/#review44773 --- The Matlab MIME types used seem to be application/x-matlab-data or

Re: Review Request 22246: New parser for Matlab .mat files

2014-06-04 Thread Chris Mattmann
> On June 4, 2014, 11:25 p.m., Matthias Krueger wrote: > > The Matlab MIME types used seem to be application/x-matlab-data or > > application/matlab-mat. > > > > Would it make sense to add them to the mime XML for detection? > > > > > > MATLAB data file > > > > > > > > > >

Re: Review Request 22246: New parser for Matlab .mat files

2014-06-05 Thread Nick Burch
> On June 4, 2014, 11:25 p.m., Matthias Krueger wrote: > > The Matlab MIME types used seem to be application/x-matlab-data or > > application/matlab-mat. > > > > Would it make sense to add them to the mime XML for detection? > > > > > > MATLAB data file > > > > > > > > > >

Re: Review Request 22246: New parser for Matlab .mat files

2014-06-05 Thread Ann Burgess
> On June 4, 2014, 11:25 p.m., Matthias Krueger wrote: > > The Matlab MIME types used seem to be application/x-matlab-data or > > application/matlab-mat. > > > > Would it make sense to add them to the mime XML for detection? > > > > > > MATLAB data file > > > > > > > > > >

Re: Review Request 22246: New parser for Matlab .mat files

2014-06-05 Thread Matthias Krueger
> On June 4, 2014, 11:25 p.m., Matthias Krueger wrote: > > The Matlab MIME types used seem to be application/x-matlab-data or > > application/matlab-mat. > > > > Would it make sense to add them to the mime XML for detection? > > > > > > MATLAB data file > > > > > > > > > >

Re: Review Request 22246: New parser for Matlab .mat files

2014-06-05 Thread Ann Burgess
> On June 4, 2014, 11:25 p.m., Matthias Krueger wrote: > > The Matlab MIME types used seem to be application/x-matlab-data or > > application/matlab-mat. > > > > Would it make sense to add them to the mime XML for detection? > > > > > > MATLAB data file > > > > > > > > > >

Re: Review Request 22246: New parser for Matlab .mat files

2014-06-05 Thread Mattmann, Chris A (3980)
ryant, Ann C (398J-Affiliate)" , Nick Burch Subject: Re: Review Request 22246: New parser for Matlab .mat files > > >> On June 4, 2014, 11:25 p.m., Matthias Krueger wrote: >> > The Matlab MIME types used seem to be application/x-matlab-data or >>application/matlab-mat. &g

Re: Review Request 22246: New parser for Matlab .mat files

2014-06-06 Thread Ann Burgess
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22246/ --- (Updated June 6, 2014, 11:06 p.m.) Review request for tika and Chris Mattmann.

Re: Review Request 22246: New parser for Matlab .mat files

2014-06-06 Thread Ann Burgess
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22246/ --- (Updated June 6, 2014, 11:07 p.m.) Review request for tika and Chris Mattmann.

Re: Review Request 22246: New parser for Matlab .mat files

2014-06-06 Thread Ann Burgess
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22246/#review44980 --- File Attachment: TIKA-1327.aburgess.140606.patch.txt - TIKA-1327.a

Re: Review Request 22246: New parser for Matlab .mat files

2014-06-07 Thread Tyler Palsulich
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22246/#review45019 --- One more step for making the diff -- you have to run `svn add [file]

Re: Review Request 22246: New parser for Matlab .mat files

2014-06-07 Thread Mattmann, Chris A (3980)
(398J-Affiliate)" , Tyler Palsulich Subject: Re: Review Request 22246: New parser for Matlab .mat files > >--- >This is an automatically generated e-mail. To reply, visit: >https://reviews

Re: Review Request 22246: New parser for Matlab .mat files

2014-06-09 Thread Ann Burgess
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22246/ --- (Updated June 9, 2014, 8:11 p.m.) Review request for tika and Chris Mattmann.

Re: Review Request 22246: New parser for Matlab .mat files

2014-06-09 Thread Tyler Palsulich
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22246/#review45126 --- trunk/tika-parsers/pom.xml

Re: Review Request 22246: New parser for Matlab .mat files

2014-06-09 Thread Chris Mattmann
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22246/#review45154 --- Ship it! Ship It! trunk/tika-parsers/pom.xml

Re: Review Request 22246: New parser for Matlab .mat files

2014-06-09 Thread Chris Mattmann
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22246/#review45156 --- Annie, this looks great. I will just take out those 2 parts about th

Re: Review Request 22246: New parser for Matlab .mat files

2014-06-09 Thread Ann Burgess
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22246/ --- (Updated June 10, 2014, 3:21 a.m.) Review request for tika and Chris Mattmann.

Re: Review Request 22246: New parser for Matlab .mat files

2014-06-10 Thread Ann Burgess
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22246/ --- (Updated June 10, 2014, 6:17 p.m.) Review request for tika and Chris Mattmann.

Re: Review Request 22246: New parser for Matlab .mat files

2014-06-12 Thread Ann Burgess
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22246/ --- (Updated June 13, 2014, 1:23 a.m.) Review request for tika and Chris Mattmann.

Re: Review Request 22246: New parser for Matlab .mat files

2014-06-12 Thread Ann Burgess
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22246/ --- (Updated June 13, 2014, 1:24 a.m.) Review request for tika and Chris Mattmann.